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.

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

Restructure test folder #825

Merged
merged 4 commits into from
Apr 22, 2021
Merged

Restructure test folder #825

merged 4 commits into from
Apr 22, 2021

Conversation

rmi7
Copy link
Contributor

@rmi7 rmi7 commented Apr 12, 2021

fixes #611

Tests are now performed on four solidity versions (0.4.25, 0.5.16, 0.6.11, 0.7.6).
Some detectors are not tested on all versions due to either:

  • the bug being specific to certain solidity versions
  • the bug not being present in later solc versions due to the solidity compiler not allowing the flaw anymore, i.e. compiler error if you try.

The tests/test_detectors.py file now deduces the location of a test file from the detector name and the specified solc version.

The new test folder structure is:
tests/detectors/<detector>/<solc version>/<test.sol>

@rmi7 rmi7 requested a review from montyly April 12, 2021 18:11
@rmi7 rmi7 changed the title WIP Restructure test folder Restructure test folder Apr 16, 2021
@montyly montyly merged commit eabdd9e into dev Apr 22, 2021
@montyly montyly deleted the restructure-test-folder branch April 22, 2021 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants