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.

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

contracts/tests: Fix flaky SIWE and gas cost #444

Merged
merged 2 commits into from
Oct 24, 2024

Conversation

matevz
Copy link
Member

@matevz matevz commented Oct 23, 2024

Fixes #443 by waiting for the block with sufficiently new timestamp instead of fixed time.
Another workaround for #337.

Copy link

netlify bot commented Oct 23, 2024

Deploy Preview for oasisprotocol-sapphire-paratime canceled.

Name Link
🔨 Latest commit 6e142c6
🔍 Latest deploy log https://app.netlify.com/sites/oasisprotocol-sapphire-paratime/deploys/6718de6a08b25200086b9bdd

@matevz matevz force-pushed the matevz/fix/flaky-siwe-gascost branch from c16165d to 6e142c6 Compare October 23, 2024 11:30
@matevz matevz requested review from CedarMist and aefhm October 23, 2024 13:44
contracts/test/auth.ts Show resolved Hide resolved
@CedarMist CedarMist merged commit a665ff3 into main Oct 24, 2024
11 checks passed
@CedarMist CedarMist deleted the matevz/fix/flaky-siwe-gascost branch October 24, 2024 07:36
@CedarMist CedarMist added contracts Pull requests that update sapphire-contracts bug labels Oct 24, 2024
github-actions bot added a commit that referenced this pull request Oct 24, 2024
…atevz/fix/flaky-siwe-gascost

contracts/tests: Fix flaky SIWE and gas cost a665ff3
github-actions bot added a commit that referenced this pull request Oct 24, 2024
…tevz/fix/flaky-siwe-gascost

contracts/tests: Fix flaky SIWE and gas cost a665ff3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug contracts Pull requests that update sapphire-contracts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix SIWE intermittent test issue
3 participants