Skip to content

test: use crypto.randomUUID for resource names in e2e tests - #8091

Merged
trivikr merged 2 commits into
mainfrom
test-e2e-randomUUID
Jun 9, 2026
Merged

test: use crypto.randomUUID for resource names in e2e tests#8091
trivikr merged 2 commits into
mainfrom
test-e2e-randomUUID

Conversation

@trivikr

@trivikr trivikr commented Jun 9, 2026

Copy link
Copy Markdown
Member

Issue

#8088

Description

Replace Math.random() and Date.now() based ID generation with
crypto.randomUUID() across all e2e spec files to guarantee uniqueness
and avoid resource name collisions in concurrent test runs.

Testing

CI

Checklist

  • If the PR is a feature, add integration tests (*.integ.spec.ts) or E2E tests.
    • It's not a feature.
  • My E2E tests are resilient to concurrent i/o.
    • I didn't write any E2E tests.
  • I added access level annotations e.g. @public, @internal tags and enabled doc generation on the package. Remember that access level annotations go below the description, not above.
    • I didn't add any public functions.
  • Streams - how do they work?? My WebStream readers/locks are properly lifecycled. Node.js stream backpressure is handled. Error handling.
    • No streams here.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@trivikr
trivikr requested a review from a team as a code owner June 9, 2026 16:47
@trivikr
trivikr merged commit e4ef6c5 into main Jun 9, 2026
7 checks passed
@trivikr
trivikr deleted the test-e2e-randomUUID branch June 9, 2026 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants