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

feat: stamp issuer utilization rate test #4398

Merged
merged 1 commit into from
Oct 12, 2023
Merged

feat: stamp issuer utilization rate test #4398

merged 1 commit into from
Oct 12, 2023

Conversation

istae
Copy link
Member

@istae istae commented Oct 12, 2023 •
edited
Loading

Checklist

  • I have read the coding guide.
  • My change requires a documentation update, and I have done it.
  • I have added tests to cover my changes.
  • I have filled out the description and linked the related issues.

Description

Minimum depth is now 22.

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

@istae istae requested a review from mrekucci October 12, 2023 14:22
mrekucci
mrekucci approved these changes Oct 12, 2023
View reviewed changes
pkg/postage/stampissuer_test.go Outdated Show resolved Hide resolved
pkg/postage/stampissuer_test.go Outdated Show resolved Hide resolved
pkg/postage/stampissuer_test.go Outdated Show resolved Hide resolved
@istae istae linked an issue Oct 12, 2023 that may be closed by this pull request
create a new unit test to measure utilization rates of the stamp issuer at different depths ethersphere/bee-backlog#66
Closed
@istae istae mentioned this pull request Oct 12, 2023
Closed
@istae
Copy link
Member Author

istae commented Oct 12, 2023 

go test  -run ^TestUtilization$ github.com/ethersphere/bee/pkg/postage -count=1 -v
=== RUN   TestUtilization
    stampissuer_test.go:236: depth: 17, actual utilization: 0.037651
    stampissuer_test.go:236: depth: 18, actual utilization: 0.102993
    stampissuer_test.go:236: depth: 19, actual utilization: 0.223402
    stampissuer_test.go:236: depth: 20, actual utilization: 0.349894
    stampissuer_test.go:236: depth: 21, actual utilization: 0.491061
    stampissuer_test.go:236: depth: 22, actual utilization: 0.610335
    stampissuer_test.go:236: depth: 23, actual utilization: 0.672418
    stampissuer_test.go:236: depth: 24, actual utilization: 0.757001

@istae istae requested a review from mrekucci October 12, 2023 15:23
@istae istae merged commit 847a902 into master Oct 12, 2023
12 checks passed
@istae istae deleted the utiliation-test branch October 12, 2023 16:43
@istae istae mentioned this pull request Oct 13, 2023
Closed
@istae istae self-assigned this Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mrekucci mrekucci mrekucci approved these changes

Assignees

@istae istae

Labels
pull-request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

create a new unit test to measure utilization rates of the stamp issuer at different depths
2 participants
@istae @mrekucci