Skip to content

test(e2e): remove AVAX→USDC swap from smoke suite#945

Merged
blockqa merged 1 commit into
mainfrom
remove-avax-usdc-smoke
Jun 3, 2026
Merged

test(e2e): remove AVAX→USDC swap from smoke suite#945
blockqa merged 1 commit into
mainfrom
remove-avax-usdc-smoke

Conversation

@blockqa

@blockqa blockqa commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The AVAX→USDC live swap (SWP-016) was the only swap test tagged @smoke, which meant the smoke suite (run via --grep "@smoke") executed a real on-chain transaction that costs real fees and can be flaky.
  • This drops the @smoke tag so all live swap pairs are gated behind @swap-live only (manual workflow_dispatch / full regression).

AVAX_USDC was the only live swap pair tagged @smoke, gating the smoke
suite on a real on-chain transaction. Drop @smoke so all live pairs are
gated behind @swap-live (manual workflow dispatch) for now.
@blockqa blockqa requested a review from a team as a code owner June 3, 2026 16:00
Copilot AI review requested due to automatic review settings June 3, 2026 16:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the @smoke tag from the AVAX→USDC live swap so the smoke suite no longer triggers real on-chain swap transactions (fees + flakiness), while keeping the live swap coverage available under @swap-live.

Changes:

  • Dropped @smoke tagging from the live swap pair loop so all live pairs run only under @swap-live.
  • Updated the header comment to reflect running the suite via --grep @swap-live.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@blockqa blockqa merged commit dc9ee92 into main Jun 3, 2026
11 checks passed
@blockqa blockqa deleted the remove-avax-usdc-smoke branch June 3, 2026 16:16
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.

4 participants