-
Notifications
You must be signed in to change notification settings - Fork 13k
test: Added uuid to createUnit default name to prevent name collisions #35738
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
Conversation
|
Looks like this PR is ready to merge! 🎉 |
|
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #35738 +/- ##
===========================================
+ Coverage 61.15% 61.21% +0.05%
===========================================
Files 3013 3011 -2
Lines 71589 71506 -83
Branches 16397 16346 -51
===========================================
- Hits 43779 43771 -8
+ Misses 24841 24767 -74
+ Partials 2969 2968 -1
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
ee86a88 to
a702949
Compare
Proposed changes (including videos or screenshots)
This PR appends uuid to the default name used by the createUnit util function in our e2e tests. This aims to prevent name collisions due to repeated names generated by faker.
Issue(s)
ARCH-1534
Steps to test or reproduce
N/A
Further comments