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

Split serialization tests into staging and regression #25

Merged
merged 3 commits into from
Oct 10, 2024

Conversation

rmgaray
Copy link
Collaborator

@rmgaray rmgaray commented Oct 10, 2024

Depends on #24 .

Description from latest commit:

Manage staging and regression tests separately

This commit refactors the serialization tests
to handle staging and regression tests separately.

TXs in the staging folder that are also available
in the regression folder are automatically
filtered out, since they will be tested anyway
when the regression suite is run.

In practice, this means that one should run the
tests twice after adding new TXs to staging
to get a stable output report.

This commit also improves the report to include
failure messages.

To get new transactions from the blockchain tip, run npm run get-transactions. This will automatically delete all TXs saved under staging and replace them with new ones.

This commit refactors the serialization tests
to handle staging and regression tests separately.

TXs in the staging folder that are also available
in the regression folder are automatically
filtered out, since they will be tested anyway
when the regression suite is run.

In practice, this means that one should run the
tests twice after adding new TXs to staging
to get a stable output report.

This commit also improves the report to include
failure messages.
@rmgaray rmgaray force-pushed the rmgaray/split-serialization-test branch from 8fd438a to d84c27d Compare October 10, 2024 16:56
@itsfarseen itsfarseen merged commit f24b5b4 into master Oct 10, 2024
1 check 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.

2 participants