Skip to content

Commit

Permalink
cr fix
Browse files Browse the repository at this point in the history
  • Loading branch information
OlimpiaZurek committed Oct 30, 2024
1 parent ef3d927 commit c25e3c8
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion contributingGuides/REASSURE_PERFORMANCE_TEST.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ We use Reassure for monitoring performance regression. It helps us check if our
## Performance Testing Strategy (`measureRenders`)

- Before adding new tests, check if the proposed scenario or component is already covered in existing tests. Duplicate tests can slow down the CI suite, making it harder to spot meaningful regressions.
- Review the current test suite to ensure your test scenario or component isn’t already being tested elsewhere.
- Test only scenarios that cover new or unique interactions. Avoid testing repetitive user actions that could be captured within a single, comprehensive scenario.
- Where applicable, use utility functions and helper methods to consolidate common actions (e.g., data mocking, scenario setup) across tests. This reduces redundancy and allows tests to be more focused and reusable.
- The primary focus is on testing business cases rather than small, reusable parts that typically don't introduce regressions, although some tests in that area are still necessary.
Expand Down

0 comments on commit c25e3c8

Please sign in to comment.