github-mirrors/logstash
1
0
Fork 0
mirror of https://github.com/elastic/logstash.git synced 2025-04-24 22:57:16 -04:00
Code Issues Projects Releases Packages Wiki Activity

dont mutate SETTINGS object in keystore specs

Browse source 
Fixes #11270
This commit is contained in:
Joao Duarte 2019-10-25 09:28:18 +01:00 • committed by João Duarte
parent 382f6d76dd
commit 1f14cba3e3
1 changed files with 10 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

19
logstash-core/spec/logstash/util/secretstore_spec.rb
View file

@ -5,31 +5,32 @@ java_import "org.logstash.secret.store.SecretStoreExt"
describe SecretStoreExt do describe SecretStoreExt do
subject {SecretStoreExt} subject {SecretStoreExt}
let(:settings) { LogStash::SETTINGS.clone }
describe "with missing keystore" do describe "with missing keystore" do
before :each do before :each do
LogStash::SETTINGS.set("keystore.file", File.join(File.dirname(__FILE__), "nothing_here")) settings.set("keystore.file", File.join(File.dirname(__FILE__), "nothing_here"))
end end
it "should be not exist" do it "should be not exist" do
expect(subject.exists(LogStash::SETTINGS.get_setting("keystore.file").value, LogStash::SETTINGS.get_setting("keystore.classname").value)).to be_falsey expect(subject.exists(settings.get_setting("keystore.file").value, settings.get_setting("keystore.classname").value)).to be_falsey
expect(subject.getIfExists(LogStash::SETTINGS.get_setting("keystore.file").value, LogStash::SETTINGS.get_setting("keystore.classname").value)).to be_nil expect(subject.getIfExists(settings.get_setting("keystore.file").value, settings.get_setting("keystore.classname").value)).to be_nil
end end
end end
describe "with implicit password keystore" do describe "with implicit password keystore" do
before :each do before :each do
LogStash::SETTINGS.set("keystore.file", File.join(File.dirname(__FILE__), "../../../src/test/resources/logstash.keystore.with.default.pass")) settings.set("keystore.file", File.join(File.dirname(__FILE__), "../../../src/test/resources/logstash.keystore.with.default.pass"))
end end
it "should be readable" do it "should be readable" do
expect(subject.getIfExists(LogStash::SETTINGS.get_setting("keystore.file").value, LogStash::SETTINGS.get_setting("keystore.classname").value).list).to include(subject.get_store_id("keystore.seed")) expect(subject.getIfExists(settings.get_setting("keystore.file").value, settings.get_setting("keystore.classname").value).list).to include(subject.get_store_id("keystore.seed"))
end end
end end
describe "with explicit password keystore" do describe "with explicit password keystore" do
before :each do before :each do
LogStash::SETTINGS.set("keystore.file", File.join(File.dirname(__FILE__), "../../../src/test/resources/logstash.keystore.with.defined.pass")) settings.set("keystore.file", File.join(File.dirname(__FILE__), "../../../src/test/resources/logstash.keystore.with.defined.pass"))
end end
describe "and correct password" do describe "and correct password" do
@ -42,7 +43,7 @@ describe SecretStoreExt do
end end
it "should be readable" do it "should be readable" do
expect(subject.getIfExists(LogStash::SETTINGS.get_setting("keystore.file").value, LogStash::SETTINGS.get_setting("keystore.classname").value).list).to include(subject.get_store_id("keystore.seed")) expect(subject.getIfExists(settings.get_setting("keystore.file").value, settings.get_setting("keystore.classname").value).list).to include(subject.get_store_id("keystore.seed"))
end end
end end
@ -56,13 +57,13 @@ describe SecretStoreExt do
end end
it "should be not readable" do it "should be not readable" do
expect {subject.getIfExists(LogStash::SETTINGS.get_setting("keystore.file").value, LogStash::SETTINGS.get_setting("keystore.classname").value)}.to raise_error.with_message(/Can not access Logstash keystore/) expect {subject.getIfExists(settings.get_setting("keystore.file").value, settings.get_setting("keystore.classname").value)}.to raise_error.with_message(/Can not access Logstash keystore/)
end end
end end
describe "and missing password" do describe "and missing password" do
it "should be not readable" do it "should be not readable" do
expect {subject.getIfExists(LogStash::SETTINGS.get_setting("keystore.file").value, LogStash::SETTINGS.get_setting("keystore.classname").value)}.to raise_error.with_message(/Could not determine keystore password/) expect {subject.getIfExists(settings.get_setting("keystore.file").value, settings.get_setting("keystore.classname").value)}.to raise_error.with_message(/Could not determine keystore password/)
end end
end end
end end