Skip to content

Commit

Permalink
Merge bitcoin#29304: fuzz: Exit and log stderr for parse_test_list er…
Browse files Browse the repository at this point in the history
…rors

9d09c87 fuzz: Exit and log stderr for parse_test_list errors (dergoegge)

Pull request description:

  We should log all errors that occur when attempting to print the harness list in the fuzz test runner.

ACKs for top commit:
  maflcko:
    lgtm ACK 9d09c87

Tree-SHA512: 50471b732c8cbe287dacba14487e7c8a5826f146432d93aa3bb55d063a8ba158d01641d6cb1360241dd4cd54ef5e045b0412f9cc34d06c181134921d1f1ceced
  • Loading branch information
fanquake committed Jan 24, 2024
2 parents e69796c + 9d09c87 commit ea4ddd8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/fuzz/test_runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -372,8 +372,8 @@ def parse_test_list(*, fuzz_bin):
'PRINT_ALL_FUZZ_TARGETS_AND_ABORT': ''
},
stdout=subprocess.PIPE,
stderr=subprocess.DEVNULL,
text=True,
check=True,
).stdout.splitlines()
return test_list_all

Expand Down

0 comments on commit ea4ddd8

Please sign in to comment.