Skip to content

Commit

Permalink
fix: empty/success effect should be deser, not treated as None
Browse files Browse the repository at this point in the history
  • Loading branch information
0x0ece committed May 28, 2024
1 parent 772c9ea commit bfb25b8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/test_suite/multiprocessing_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -356,7 +356,7 @@ def build_test_results(results: dict[str, str | None]) -> tuple[int, dict | None
for target, result in results.items():
# Create a Protobuf struct to compare and output, if applicable
instruction_effects = None
if result:
if result is not None:
# Turn bytes into human readable fields
instruction_effects = globals.harness_ctx.effects_type()
instruction_effects.ParseFromString(result)
Expand Down

0 comments on commit bfb25b8

Please sign in to comment.