-
-
Notifications
You must be signed in to change notification settings - Fork 929
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
test: rewrite random #490
test: rewrite random #490
Conversation
Codecov Report
@@ Coverage Diff @@
## main #490 +/- ##
==========================================
- Coverage 99.34% 99.34% -0.01%
==========================================
Files 1919 1919
Lines 176173 176174 +1
Branches 904 901 -3
==========================================
- Hits 175019 175016 -3
- Misses 1098 1102 +4
Partials 56 56
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where are the seeded tests? This test class looks a lot different from the other ones.
Should we really structure this class differently?
Maybe if I have time and will, I can add seeded tests in next few days. |
Im fine if we add the seed later :) |
No description provided.