Skip to content

Commit

Permalink
Fix acceptance test assertions for updated plugin remove (#17122)
Browse files Browse the repository at this point in the history
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)
  • Loading branch information
donoghuc authored and logstashmachine committed Feb 19, 2025
1 parent de68d4b commit f0c56c5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@
context "trying to uninstall an inner plugin" do
it "fails to uninstall it" do
result = logstash.run_sudo_command_in_path("bin/logstash-plugin uninstall logstash-input-rabbitmq")
expect(result.stderr).to match(/is already provided by/)
expect(result.stderr).to include("The plugin `logstash-input-rabbitmq` is provided by 'logstash-integration-rabbitmq' so it can't be removed individually")
end
end
end
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
context "when the plugin isn't installed" do
it "fails to uninstall it" do
result = logstash.run_sudo_command_in_path("bin/logstash-plugin uninstall logstash-filter-qatest")
expect(result.stderr).to match(/This plugin has not been previously installed/)
expect(result.stderr).to include("The plugin `logstash-filter-qatest` has not been previously installed")
end
end

Expand Down

0 comments on commit f0c56c5

Please sign in to comment.