Skip to content

Commit

Permalink
Merge pull request #212 from near/relative_path
Browse files Browse the repository at this point in the history
Use absolute paths in run-tests-zkasm.js
  • Loading branch information
aborg-dev authored Feb 8, 2024
2 parents 2a62f95 + 283aeb1 commit b9c037c
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion ci/test-all-zkasm.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ set -eux
ALL_ARGS=$@

function run_tests() {
./ci/test-zkasm.sh $1 $ALL_ARGS
./ci/test-zkasm.sh $(pwd)/$1 $ALL_ARGS
}

run_tests "cranelift/zkasm_data"
Expand Down
2 changes: 1 addition & 1 deletion ci/test-zkasm.sh
Original file line number Diff line number Diff line change
Expand Up @@ -14,5 +14,5 @@ ALL_ARGS=$@
# The first argument is expected to be a path to a folder with tests.
TEST_PATH=$1
TEST_RESULTS_PATH=$(mktemp)
npm test --prefix tests/zkasm "../../${TEST_PATH}/generated" $TEST_RESULTS_PATH
npm test --prefix tests/zkasm "${TEST_PATH}/generated" $TEST_RESULTS_PATH
python3 ci/zkasm-result.py $ALL_ARGS < $TEST_RESULTS_PATH
3 changes: 1 addition & 2 deletions tests/zkasm/run-tests-zkasm.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,7 @@ async function main() {
const cmPols = await compilePil();

// Get all zkasm files
const pathZkasm = path.join(process.cwd(), process.argv[2]);
const files = await getTestFiles(pathZkasm);
const files = await getTestFiles(process.argv[2]);

// Run all zkasm files
let testResults = [];
Expand Down

0 comments on commit b9c037c

Please sign in to comment.