Skip to content

Commit f0c56c5

Browse files
donoghuclogstashmachine
authored andcommitted
Fix acceptance test assertions for updated plugin remove (#17122)
This commit updates the acceptance tests to expect messages in the updated format for removing plugins. See #17030 for change. (cherry picked from commit e094054)
1 parent de68d4b commit f0c56c5

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

qa/acceptance/spec/shared_examples/cli/logstash-plugin/integration_plugin.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@
7575
context "trying to uninstall an inner plugin" do
7676
it "fails to uninstall it" do
7777
result = logstash.run_sudo_command_in_path("bin/logstash-plugin uninstall logstash-input-rabbitmq")
78-
expect(result.stderr).to match(/is already provided by/)
78+
expect(result.stderr).to include("The plugin `logstash-input-rabbitmq` is provided by 'logstash-integration-rabbitmq' so it can't be removed individually")
7979
end
8080
end
8181
end

qa/acceptance/spec/shared_examples/cli/logstash-plugin/uninstall.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
context "when the plugin isn't installed" do
3434
it "fails to uninstall it" do
3535
result = logstash.run_sudo_command_in_path("bin/logstash-plugin uninstall logstash-filter-qatest")
36-
expect(result.stderr).to match(/This plugin has not been previously installed/)
36+
expect(result.stderr).to include("The plugin `logstash-filter-qatest` has not been previously installed")
3737
end
3838
end
3939

0 commit comments

Comments
 (0)