Skip to content
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.

Sign up for GitHub

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

Jump to bottom

update: session counter test to use new atomic counter #2

Merged
merged 1 commit into from
Sep 17, 2023
Merged

update: session counter test to use new atomic counter #2

merged 1 commit into from
Sep 17, 2023

Conversation

nrednav
Copy link
Owner

@nrednav nrednav commented Sep 17, 2023 •
edited
Loading

What's changed

  • Updated session counter test to use new atomic counter

Why

  • It was still using the old method

Test Results

2023/09/17 16:38:25 Sample Cuids: [tjdqtzang308udsi7aown959 g9wyn37urujqwf34ok4in9ox vulhpbdw81gexfww4xlgtbry bhypcl561xlcx7q5mqcoez6x zwq967miwu10hfr7l9et0m77 ja1qu149umd9qqmvw3h7jq88 r2vvxcue2xjw27ufs6w9r1kz ezuc35tw3rm16qm2axc6kfb9 fn44wpv4426w95m9oys491of vzsawovzuff52bny3mj8l73b]
2023/09/17 16:38:25 Histogram: [83200 82245 83230 82994 82846 82788 83553 82852 83257 82835 83510 82886 82917 82320 81578 80887 81357 80625 80615 80591]
2023/09/17 16:38:25 Expected bin size: 82354
2023/09/17 16:38:25 Min bin size: 78236
2023/09/17 16:38:25 Maximum bin size: 86472
2023/09/17 16:38:25 Validating all 11529602 Cuids...
PASS
ok      github.com/nrednav/cuid2        22.327s

@nrednav nrednav added the enhancement New feature or request label Sep 17, 2023
@nrednav nrednav self-assigned this Sep 17, 2023
@nrednav nrednav merged commit 7823905 into main Sep 17, 2023
@nrednav nrednav deleted the update/test/session-counter branch September 17, 2023 15:41
@nrednav nrednav mentioned this pull request Sep 17, 2023
Open
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nrednav nrednav

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nrednav