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

Refactor tests #34

Merged
merged 9 commits into from
Mar 29, 2024
Merged

Refactor tests #34

merged 9 commits into from
Mar 29, 2024

Conversation

tuturu-tech
Copy link
Collaborator

@tuturu-tech tuturu-tech commented Mar 28, 2024

This PR aims to improve testing, it is based off PR #27 and related to issue #8

Documentation updates

  • Added simple testing guidance to CONTRIBUTING.md

Tests

  • Refactored type tests to reduce code duplication by separating common logic into a helper function
  • Added fixture to run tests in temporary directory
  • Added targeted bytes and string parsing/decoding tests
  • Added remapping detection tests

Bug fixes

  • Echidna bytes/string decoding was updated. Tests are now generated with unicode literals
  • Remapping detection when a remappings.txt file does not exist was fixed

@tuturu-tech tuturu-tech marked this pull request as ready for review March 29, 2024 11:12
@tuturu-tech tuturu-tech merged commit 99f6809 into main Mar 29, 2024
11 checks passed
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.

1 participant