Skip to content

Commit a34ca55

Browse files
committed
fix(test-scripts): pin cargo run --bin perry so multi-bin workspace doesn't break parity
The styling-matrix binary added in v0.5.300 (#185 Phase A) created a second workspace binary alongside `perry`. Without `--bin perry`, `cargo run --quiet --` errors with "could not determine which binary to run", which the parity runner reports as "compile error" for every single test. All open PRs have been red on parity since v0.5.300 hit origin/main. Same fix applied to run_tests.sh.
1 parent b043be4 commit a34ca55

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

run_parity_tests.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,7 @@ for test_file in "$TEST_DIR"/*.ts; do
215215
echo "$node_output" > "$node_output_file"
216216

217217
# Compile with Perry
218-
compile_output=$(cargo run --quiet -- $BACKEND_FLAG "$test_file" -o "$perry_binary" 2>&1)
218+
compile_output=$(cargo run --quiet --bin perry -- $BACKEND_FLAG "$test_file" -o "$perry_binary" 2>&1)
219219
compile_exit=$?
220220

221221
if [[ $compile_exit -ne 0 ]]; then

run_tests.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,9 +68,9 @@ for test_file in "$TEST_DIR"/*.ts; do
6868
fi
6969

7070
# Compile the test (suppress warnings)
71-
if ! cargo run --quiet -- "$test_file" -o "$output_file" 2>/dev/null; then
71+
if ! cargo run --quiet --bin perry -- "$test_file" -o "$output_file" 2>/dev/null; then
7272
# Try again to get error message
73-
compile_output=$(cargo run --quiet -- "$test_file" -o "$output_file" 2>&1 | grep -i "error" | head -3)
73+
compile_output=$(cargo run --quiet --bin perry -- "$test_file" -o "$output_file" 2>&1 | grep -i "error" | head -3)
7474
echo -e "${RED}FAIL${NC} $test_name (compile error)"
7575
if [[ -n "$compile_output" ]]; then
7676
echo " $compile_output"

0 commit comments

Comments
 (0)