Skip to content

Commit

Permalink
use uv run in coverage script
Browse files Browse the repository at this point in the history
  • Loading branch information
winsvega committed Sep 7, 2024
1 parent a0ae617 commit b4edd64
Showing 1 changed file with 5 additions and 11 deletions.
16 changes: 5 additions & 11 deletions .github/workflows/coverage.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -158,9 +158,6 @@ jobs:
source $GITHUB_ENV
files=$(echo "$NEW_TESTS" | tr ',' '\n')
python3 -m venv ./venv/
source ./venv/bin/activate
# fill new tests
# using `|| true` here because if no tests found, pyspec fill returns error code
mkdir -p fixtures/state_tests
Expand All @@ -169,8 +166,8 @@ jobs:
# Use a while loop with a here-string to avoid subshell issues
while IFS= read -r file; do
echo "Fill: $file"
fill "$file" --until=Cancun --evm-bin evmone-t8n || true >> filloutput.log 2>&1
(fill "$file" --fork=CancunEIP7692 --evm-bin evmone-t8n -k eof_test || true) > >(tee -a filloutput.log filloutputEOF.log) 2>&1
uv run fill "$file" --until=Cancun --evm-bin evmone-t8n || true >> filloutput.log 2>&1
(uv run fill "$file" --fork=CancunEIP7692 --evm-bin evmone-t8n -k eof_test || true) > >(tee -a filloutput.log filloutputEOF.log) 2>&1
done <<< "$files"
if grep -q "FAILURES" filloutput.log; then
Expand All @@ -194,7 +191,7 @@ jobs:
# Include basic evm operations into coverage by default
# As when we translate from yul/solidity some dup/push opcodes could become untouched
fill tests/homestead/coverage/test_coverage.py --until=Cancun --evm-bin evmone-t8n
uv run fill tests/homestead/coverage/test_coverage.py --until=Cancun --evm-bin evmone-t8n
PATCH_TEST_PATH=${{ github.workspace }}/evmtest_coverage/coverage/PATCH_TESTS
mkdir -p $PATCH_TEST_PATH
Expand All @@ -215,9 +212,6 @@ jobs:
PREV_COMMIT=$(git rev-parse HEAD)
echo "Checkout head $PREV_COMMIT"
python3 -m venv ./venv/
source ./venv/bin/activate
rm -r fixtures
rm filloutput.log
rm filloutputEOF.log
Expand All @@ -227,8 +221,8 @@ jobs:
# Use a while loop with a here-string to avoid subshell issues
while IFS= read -r file; do
echo "Fill: $file"
fill "$file" --until=Cancun --evm-bin evmone-t8n || true >> filloutput.log 2>&1
(fill "$file" --fork=CancunEIP7692 --evm-bin evmone-t8n -k eof_test || true) > >(tee -a filloutput.log filloutputEOF.log) 2>&1
uv run fill "$file" --until=Cancun --evm-bin evmone-t8n || true >> filloutput.log 2>&1
(uv run fill "$file" --fork=CancunEIP7692 --evm-bin evmone-t8n -k eof_test || true) > >(tee -a filloutput.log filloutputEOF.log) 2>&1
done <<< "$files"
if grep -q "FAILURES" filloutput.log; then
Expand Down

0 comments on commit b4edd64

Please sign in to comment.