Skip to content

fix test failures#602

Merged
jrhea merged 2 commits intomainfrom
test-fails
Sep 29, 2023
Merged

fix test failures#602
jrhea merged 2 commits intomainfrom
test-fails

Conversation

@jrhea
Copy link
Copy Markdown
Contributor

@jrhea jrhea commented Sep 29, 2023

resolves #592

Added these failure cases to both tests. Details are in the issue.

@jrhea jrhea changed the title add edge cases to tests fix test failures Sep 29, 2023
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 29, 2023

Hyperdrive Gas Benchmark

Benchmark suite Current: ab5b930 Previous: dc1c956 Deviation Status
addLiquidity: min 785 gas 785 gas 0% 🟰
addLiquidity: avg 57119 gas 55489 gas 2.9375% 🚨
addLiquidity: max 98947 gas 98947 gas 0% 🟰
checkpoint: min 514 gas 514 gas 0% 🟰
checkpoint: avg 29519 gas 29522 gas -0.0102%
checkpoint: max 69038 gas 69038 gas 0% 🟰
closeLong: min 852 gas 852 gas 0% 🟰
closeLong: avg 23197 gas 23123 gas 0.3200% 🚨
closeLong: max 126367 gas 126367 gas 0% 🟰
closeShort: min 765 gas 765 gas 0% 🟰
closeShort: avg 23991 gas 23885 gas 0.4438% 🚨
closeShort: max 104901 gas 104901 gas 0% 🟰
initialize: min 803 gas 803 gas 0% 🟰
initialize: avg 159824 gas 159788 gas 0.0225% 🚨
initialize: max 233284 gas 233284 gas 0% 🟰
openLong: min 740 gas 740 gas 0% 🟰
openLong: avg 59393 gas 59284 gas 0.1839% 🚨
openLong: max 229269 gas 229269 gas 0% 🟰
openShort: min 782 gas 782 gas 0% 🟰
openShort: avg 61184 gas 61183 gas 0.0016% 🚨
openShort: max 244012 gas 244012 gas 0% 🟰
removeLiquidity: min 762 gas 762 gas 0% 🟰
removeLiquidity: avg 78621 gas 77051 gas 2.0376% 🚨
removeLiquidity: max 171012 gas 171012 gas 0% 🟰

This comment was automatically generated by workflow using github-action-benchmark.

@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Sep 29, 2023

Coverage Status

coverage: 97.056% (-0.1%) from 97.157% when pulling ab5b930 on test-fails into dc1c956 on main.

@jrhea jrhea requested a review from jalextowle September 29, 2023 20:42
@jrhea jrhea enabled auto-merge (squash) September 29, 2023 20:43
@jrhea jrhea merged commit c1f4e87 into main Sep 29, 2023
@jrhea jrhea deleted the test-fails branch September 29, 2023 20:53
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.

Intermittent Test Failure

3 participants