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

🎁 [Feature Request]: Regression tests for bug patches #351

Open
3 tasks done
4meta5 opened this issue Oct 28, 2024 · 0 comments
Open
3 tasks done

🎁 [Feature Request]: Regression tests for bug patches #351

4meta5 opened this issue Oct 28, 2024 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers needs triage Needs to be assigned the appropriate labels. tests integration and unit tests

Comments

@4meta5
Copy link
Contributor

4meta5 commented Oct 28, 2024

templates

  • Generic Runtime Template
  • Evm Template

What is the feature you would like to see?

There have been a lot of bug patches recently. While updating the pallet abstraction macros in OpenZeppelin/openzeppelin-pallet-abstractions#1 to include the patches, it was noticed that templates/main lacks regression tests that would fail when the patches are reverted and the bugs are reintroduced.

We need to

  1. Require all bug patching PRs to include regression tests that fail before the PR's changes.
  2. Add regression tests for all previous bug patches which forgot to include them

The priority for this issue is increased now that we are extracting pallet configs into the macros.

(1) can be addressed by

  • updating the PR template to include a check for regression tests if it is a bug fix
  • requesting regression tests during bug fix reviews (manually)

(2) requires

  • make a list of all previously merged bug fixes which lacked regression tests
  • add regression tests for all of the bug fixes on that list

Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines
@4meta5 4meta5 added enhancement New feature or request good first issue Good for newcomers tests integration and unit tests needs triage Needs to be assigned the appropriate labels. labels Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers needs triage Needs to be assigned the appropriate labels. tests integration and unit tests
Projects
Status: 🗒 Backlog
Development

No branches or pull requests

1 participant