Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow Modifying Transaction in Dry Run and Other Improvements #9

Closed
wants to merge 39 commits into from
Closed

Allow Modifying Transaction in Dry Run and Other Improvements #9

Show file tree
Hide file tree
Changes from 29 commits
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
fed6c64
separate out the algod liveness test
Apr 6, 2022
1d81276
separate out the algod liveness test
Apr 6, 2022
b5482d6
add lastMessage assertion and discover a problem with factorizer lsig
Apr 6, 2022
1697a10
flake8
Apr 6, 2022
45bd201
add hypothesis dependency - probly temporary
Apr 7, 2022
480d5c8
default message
Apr 7, 2022
65d5023
use upper case consts
Apr 7, 2022
fec8eda
rename fuzz-less for now
Apr 7, 2022
a76c347
versionize the factorizer games
Apr 7, 2022
b207aba
refactoring oh boy
Apr 8, 2022
269cbb1
passing tests
Apr 8, 2022
e9dbbda
lint
Apr 8, 2022
4020d72
pre-commit-check
Apr 8, 2022
72650be
add transactions into the CSV report
Apr 8, 2022
cfb9603
multi-proc the integration test
Apr 8, 2022
00a3b0c
graviton/py.typed
Apr 8, 2022
14016b0
mypy for graviton
Apr 8, 2022
ffd632a
make py 3.10 compatible
Apr 8, 2022
143f2b1
default txn_params in execute_one_dryrun()
Apr 9, 2022
3b4bcbc
mypy
Apr 10, 2022
1b2bbd4
Juypter notebook
Apr 12, 2022
6d55ace
wip
Apr 12, 2022
7228676
it still works
Apr 14, 2022
78bfa53
images
Apr 14, 2022
14d17ff
Update IMAGES.md
tzaffi Apr 14, 2022
8be5719
lint
Apr 14, 2022
248fa5a
missing comma
Apr 14, 2022
cf301fb
try pytest/nbmake to test Jupyter as well
Apr 14, 2022
e8998d9
distribute the notebook test and check that can pass with an intentio…
Apr 14, 2022
d80e98d
clarify comments
Apr 15, 2022
ef52891
declare much of the old code deprecated and break out into seperate m…
Apr 15, 2022
5416bb9
refactor deprecations
Apr 19, 2022
7465ae5
wrong import and dry_run --> dryrun
Apr 19, 2022
fc499fe
better docs and easier make commands
Apr 20, 2022
dc2c8be
proofreading
Apr 20, 2022
897bf72
space
Apr 20, 2022
9366e27
better explanation of passing with an erroring cell
Apr 20, 2022
155f213
trigger build
tzaffi Apr 21, 2022
4e9af8d
Trigger build 9:30 central
tzaffi Apr 21, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 4 additions & 2 deletions .github/workflows/build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,9 @@ jobs:
uses: lucasvanmol/algorand-sandbox-action@v1
with:
config: ${{ env.SANDBOX_CONFIG }}
- name: Setup integration test environment
- name: Setup integration test environment (and run a smoke unit test)
run: make pip-development unit-test
- name: Run integration tests
- name: Setup Jupyter notebooks environment
run: make pip-notebooks
- name: Run integration tests and Jupyter notebooks
run: make gh-blackbox
3 changes: 3 additions & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,9 @@ sandbox
# IDE
.idea

# Mac OS
.DS_Store

##### BEGIN: github recommends for Python #####

# Byte-compiled / optimized / DLL files
Expand Down
17 changes: 14 additions & 3 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,19 @@ pip:
pip-development: pip
pip install -e.[development]

pip-notebooks: pip-development
pip install -e.[notebooks]

flake8:
flake8 graviton tests

black:
black --check .

lint: black flake8
mypy:
mypy graviton tests

lint: flake8 black mypy

unit-test:
pytest -sv tests/unit
Expand All @@ -26,9 +32,14 @@ blackbox-smoke-prefix:
ls -l sandbox
cd sandbox && docker-compose ps

NUM_PROCS = auto
integration-test:
pytest -sv tests/integration
pytest -n $(NUM_PROCS) --durations=10 -sv tests/integration

notebooks-test:
pytest --nbmake -n $(NUM_PROCS) notebooks

pre-commit-check: lint unit-test integration-test

###### Local Only ######

Expand Down Expand Up @@ -57,4 +68,4 @@ gh-sandbox-test:

gh-blackbox-smoke: blackbox-smoke-prefix gh-sandbox-test

gh-blackbox: gh-blackbox-smoke integration-test
gh-blackbox: gh-blackbox-smoke integration-test notebooks-test
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

testing jupyter notebooks as well