Skip to content

Commit

Permalink
Fix acceptance test assertions for updated plugin remove (elastic#1…
Browse files Browse the repository at this point in the history
…7122)

This commit updates the acceptance tests to expect messages in the updated
format for removing plugins. See elastic#17030
for change.
  • Loading branch information
donoghuc committed Feb 20, 2025
1 parent 4e1664a commit f22e3b0
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_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_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 f22e3b0

Please sign in to comment.