Skip to content

Conversation

tr-io-global
Copy link
Contributor

@tr-io-global tr-io-global commented Oct 6, 2025

Checklist

  • JIRA - LW-13684
  • Proper tests implemented
  • Screenshots added.

Proposed solution

A previously passing test started failing with the error AssertionError: expected 'https://midnighttge.io/' to include 'https://www.midnight.gd' as it appears the Midnight app redirects from https://midnighttge.io/ to https://www.midnight.gd. This PR solves the problem by waiting for the redirect to complete before asserting the final URL.

Testing

Run the test and verify that it passes with no new regressions. ENV=preprod TEST_DAPP_URL=https://ljagiela.github.io/cardano-wallet-connector/ WALLET_1_PASSWORD=*** npx wdio run wdio.conf.chrome.ts --spec ./src/features/MidnightBanner.feature --cucumberOpts.tags='@LW-13628'

Screenshots

https://github.com/input-output-hk/lace/actions/runs/18295692889/job/52093921874#step:7:93
Screenshot 2025-10-06 at 6 19 06 PM

@lace-bot
Copy link
Collaborator

lace-bot commented Oct 6, 2025

Allure Report

allure-report-publisher generated test report!

processReports: ✅ test report for e64e46a3

passed failed skipped flaky total result
Total 33 0 4 0 37


async assertSeeMidnightURL() {
const EXPECTED_MIDNIGHT_URL = 'https://www.midnight.gd';
await browser.url(EXPECTED_MIDNIGHT_URL);
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think it is the best solution to fix this failing assertion

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm open to suggestions! We can chat about this in our sync later today.

Copy link

sonarqubecloud bot commented Oct 7, 2025

@tr-io-global tr-io-global enabled auto-merge (squash) October 7, 2025 13:38
@tr-io-global tr-io-global merged commit d7ac2d0 into main Oct 7, 2025
52 of 56 checks passed
@tr-io-global tr-io-global deleted the test/maintenance-LW-13684-update-midnight-url branch October 7, 2025 14:22
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.

3 participants