Skip to content

Commit b2ddc3a

Browse files
committed
Resolve comments
1 parent db76fd3 commit b2ddc3a

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

recipes/automated-testing.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -216,8 +216,6 @@ deployment process.
216216
#### Antipatterns
217217
- Avoid trying to cover all use cases or edge cases in E2E tests; these are
218218
better suited for unit or integration tests.
219-
- Don’t bypass or disable flaky tests without fixing them, as they undermine the
220-
test suite’s reliability.
221219

222220
### Performance Tests
223221

@@ -269,7 +267,7 @@ inputs and dataset sizes.
269267
### Visual Tests
270268

271269
The type of test where test runner navigates to browser page, takes snapshot and
272-
then compares the snapshots with the reference snapshot.
270+
then compares the snapshot with the reference snapshot.
273271

274272
Visual tests allow you to quickly cover large portions of the application,
275273
ensuring that changes in the UI are detected without writing complex test cases.
@@ -287,8 +285,9 @@ development.
287285
- To test a specific user scenario.
288286

289287
#### Best practices
290-
- Ensure the UI consistently renders the same output by controlling randomness
291-
(e.g., setting seeds for random data or controlling API responses).
288+
- Ensure the UI consistently renders the same output by eliminating randomness
289+
(e.g., by always using same seeds data or controlling API responses to always
290+
return same values).
292291
- Add as many pages as possible but keep the tests simple.
293292
- Consider running visual tests at the component level to isolate and detect
294293
issues earlier.

0 commit comments

Comments
 (0)