github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-05-23 13:01:07 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

DEV: Fix command error message formatting (#31491)

Browse Source 
a follow up to 51b0903f514c3c22569401e74ebd1215c317d787
This commit is contained in:
Jarek Radosz
2025-02-25 02:28:01 +01:00
committed by GitHub
parent 51b0903f51
commit d4e5d63d7e
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/discourse.rb
View File

@ -169,7 +169,7 @@ module Discourse
stdout, stderr, status = Open3.capture3(*args, chdir: chdir)
if !status.exited? || !success_status_codes.include?(status.exitstatus)
message = [command, failure_message, stderr].filter(&:present?).join("\n")
message = [command.join(" "), failure_message, stderr].filter(&:present?).join("\n")
raise CommandError.new(message, stdout: stdout, stderr: stderr, status: status)
end

4
spec/lib/discourse_spec.rb
View File

@ -553,8 +553,8 @@ RSpec.describe Discourse do
it "includes the command in the error message" do
expect do
Discourse::Utils.execute_command("false", failure_message: "oops")
end.to raise_error(RuntimeError, "false\noops")
Discourse::Utils.execute_command("false", "'foo'", failure_message: "oops")
end.to raise_error(RuntimeError, "false 'foo'\noops")
end
end