ReadAlongs / Studio-Web

Suite of web packages for creating interactive ReadAlongs
https://readalong-studio.mothertongues.org/
Other
11 stars 9 forks source link

fix(ci): wait on web-component needs sleep, URL is not reliable #373

Closed joanise closed 1 hour ago

joanise commented 5 days ago

PR Goal?

Stabilize the flaky web-component test in CI

Fixes?

The Web-Component cypress test fails every now and again, randomly, and it turns out it's because server hasn't finished starting yet when Cypress starts to do the tests.

The problem is that wait-on: "http://localhost:3333/build/web-component.esm.js" is not sufficient: that tiny page is available quickly, but the other .js file it points to are not, but their name contains a hash so we can't wait on them.

Doing a sleep 15 is kinda ugly, but it's the best I could find. The runs I checked took 9 seconds.

Feedback sought?

rubber stamping, really, unless you have a better solution.

Priority?

medium

Tests added?

n/a

How to test?

Just see that CI is more reliable, but it was a sporadic problem, so it's only over time we'll see the difference.

Confidence?

pretty high

Version change?

no

semanticdiff-com[bot] commented 5 days ago

Review changes with  SemanticDiff

Changed Files
| File | Status | | :--- | :--- | | [](https://app.semanticdiff.com/gh/ReadAlongs/Studio-Web/pull/373/changes#package-lock.json)  [package\-lock\.json](https://app.semanticdiff.com/gh/ReadAlongs/Studio-Web/pull/373/changes#package-lock.json) | [](https://app.semanticdiff.com/gh/ReadAlongs/Studio-Web/pull/373/changes#package-lock.json)  71% smaller | | [](https://app.semanticdiff.com/gh/ReadAlongs/Studio-Web/pull/373/changes#.github/workflows/end-to-end-tests.yml)  [\.github/workflows/end\-to\-end\-tests\.yml](https://app.semanticdiff.com/gh/ReadAlongs/Studio-Web/pull/373/changes#.github/workflows/end-to-end-tests.yml) | Unsupported file format | | [](https://app.semanticdiff.com/gh/ReadAlongs/Studio-Web/pull/373/changes#.github/workflows/windows-tests.yml)  [\.github/workflows/windows\-tests\.yml](https://app.semanticdiff.com/gh/ReadAlongs/Studio-Web/pull/373/changes#.github/workflows/windows-tests.yml) | Unsupported file format |
github-actions[bot] commented 5 days ago

PR Preview Action v1.4.8 :---: Preview removed because the pull request was closed. 2024-11-20 14:03 UTC