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

AST-1949 chore: restructure tests #30

Merged
merged 2 commits into from
Oct 18, 2023
Merged

Conversation

0xgregthedev
Copy link
Contributor

No description provided.

@0xgregthedev 0xgregthedev changed the title chore: restructure tests AST-1920 chore: restructure tests Oct 16, 2023
@0xgregthedev 0xgregthedev changed the title AST-1920 chore: restructure tests AST-1949 chore: restructure tests Oct 16, 2023
@linear
Copy link

linear bot commented Oct 16, 2023

AST-1949 Restructure tests

utils/ (utilities to aid in testing)
unit-testing/ (contains same directory structure as source)
fuzz-testing/differential
fuzz-testing/general
invariant-testing/(helper and invariants)

@AstariaXYZ AstariaXYZ deleted a comment from linear bot Oct 16, 2023
Copy link
Contributor

@androolloyd androolloyd left a comment

Choose a reason for hiding this comment

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

so the issue with these changes is that it'll break for anything that includes starport as a module,

this is why we stopped using src/ and moved to starport-core and starport-test

foundry assumes src and test are for the top level dirs from the directory that you execute the cmd in

@androolloyd androolloyd merged commit 8a78323 into main Oct 18, 2023
1 check passed
@justingreenberg justingreenberg deleted the chore/restructure-tests branch November 20, 2023 20:33
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.

2 participants