Skip to content
This repository has been archived by the owner on Dec 15, 2023. It is now read-only.

Parametrize block number test #238

Merged
merged 6 commits into from
Aug 30, 2022
Merged

Parametrize block number test #238

merged 6 commits into from
Aug 30, 2022

Conversation

mikiw
Copy link
Contributor

@mikiw mikiw commented Aug 29, 2022

Usage related changes

Now we can run only one parameterized test which is test_block_number_incremented (resolves #234).

Development related changes

There is a new conftest.py file with fixture_run_devnet_in_background that was moved one directory up.

Checklist:

  • Applied formatting - ./scripts/format.sh
  • No linter errors - ./scripts/lint.sh
  • Performed code self-review
  • Rebased to the last commit of the target branch (or merged it into my branch)
  • Documented the changes
  • Linked the issues which this PR resolves
  • Updated the tests
  • All tests are passing

mikiw added 2 commits August 29, 2022 12:07
This version seems to work but further refactoring is needed. Function fixture_run_devnet_in_background should be referenced and tests should check if --lite-mode works and hashes are 0x0+.

[skip ci]
@FabijanC FabijanC self-requested a review August 29, 2022 13:40
@FabijanC FabijanC changed the title Test block modification Parametrize block number test Aug 30, 2022
@mikiw mikiw merged commit f150787 into master Aug 30, 2022
@mikiw mikiw deleted the test-block-modification branch August 30, 2022 11:42
internnos pushed a commit to internnos/starknet-devnet that referenced this pull request Aug 30, 2022
@mikiw mikiw mentioned this pull request Sep 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Apply pytest parametrization where possible
2 participants