Skip to content

Commit

Permalink
Fixup
Browse files Browse the repository at this point in the history
  • Loading branch information
bigspider committed Dec 17, 2024
1 parent c126634 commit d776184
Show file tree
Hide file tree
Showing 6 changed files with 14 additions and 40 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci-workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -158,8 +158,8 @@ jobs:

- name: Run tests
run: |
cd tests_mainnet
pip install -r requirements.txt
cd tests_mainnet
PYTHONPATH=$PYTHONPATH:/speculos pytest --headless --model=${{ matrix.model }} --timeout=300
job_test_python_lib_legacyapp:
Expand All @@ -184,8 +184,8 @@ jobs:

- name: Run tests
run: |
cd bitcoin_client/tests
pip install -r requirements.txt
cd bitcoin_client/tests
PYTHONPATH=$PYTHONPATH:/speculos pytest --headless --timeout=300 --model=nanos
job_perftests:
Expand Down Expand Up @@ -217,8 +217,8 @@ jobs:

- name: Run tests
run: |
cd tests_perf
pip install -r requirements.txt
cd tests_perf
PYTHONPATH=$PYTHONPATH:/speculos pytest --headless --model=nanosp --benchmark-json=benchmarks.json
- name: Upload benchmarks summary
uses: actions/upload-artifact@v4
Expand Down
13 changes: 8 additions & 5 deletions bitcoin_client/tests/requirements.txt → requirements.txt
Original file line number Diff line number Diff line change
@@ -1,10 +1,13 @@
bip32>=3.4,<4.0
# requirements for the various test suites

pytest>=6.1.1,<7.0.0
pytest-timeout>=2.1.0,<3.0.0
ledgercomm>=1.1.0,<1.2.0
ecdsa>=0.16.1,<0.17.0
typing-extensions>=3.7,<4.0
embit>=0.7.0,<0.8.0
mnemonic==0.20
pytest>=6.1.1,<7.0.0
pytest-timeout>=2.1.0,<3.0.0
bip32>=3.4,<4.0
speculos>=0.12.0,<0.13.0
typing-extensions>=3.7,<4.0
-e ../
ragger[speculos, ledgerwallet]>=1.6.0
-e ./bitcoin_client # path relative to the current working directory; assume it's the root of the repo
2 changes: 0 additions & 2 deletions test_utils/requirements.txt

This file was deleted.

14 changes: 3 additions & 11 deletions tests/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,11 +1,3 @@
pytest>=6.1.1,<7.0.0
pytest-timeout>=2.1.0,<3.0.0
ledgercomm>=1.1.0,<1.2.0
ecdsa>=0.16.1,<0.17.0
typing-extensions>=3.7,<4.0
embit>=0.7.0,<0.8.0
mnemonic==0.20
bip32>=3.4,<4.0
speculos>=0.12.0,<0.13.0
-e ../bitcoin_client
ragger[speculos, ledgerwallet]>=1.6.0
# The reusable ragger workflow expects a requirements.txt file in the tests directory, but we want to just use the one in the repository's root.
# This works because the `pip install command` in the workflow is run from the repo's root.
-r ./requirements.txt
10 changes: 0 additions & 10 deletions tests_mainnet/requirements.txt

This file was deleted.

9 changes: 0 additions & 9 deletions tests_perf/requirements.txt

This file was deleted.

0 comments on commit d776184

Please sign in to comment.