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

E2e folder page #1742

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

E2e folder page #1742

wants to merge 29 commits into from

Conversation

MaPoKen
Copy link
Contributor

@MaPoKen MaPoKen commented Feb 6, 2024

Legger til e2e tester på folders page men mangler en gjennomgang av tester på toolbar buttons, kan potensielt bare sjekke at knappene eksisterer og at de åpner riktig modal?

@MaPoKen MaPoKen marked this pull request as ready for review February 6, 2024 09:36
@MaPoKen MaPoKen requested review from a team February 6, 2024 09:39
Copy link
Contributor

@rauboti rauboti left a comment

Choose a reason for hiding this comment

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

Kjørte noen runder igjennom:

  • Alle tester kjører som normalt når jeg kun kjører 1 og 1
  • Dersom jeg kjører MyNDLAFolder.spec.ts fila samla, fullfører alle tester som de skal
  • Dersom jeg kjører hele authenticated-mappa, feiler alltid 1, men det er forskjellig hver gang og det er alltid på første linja som venter på og prøver finne Mine mapper-headingen.

Er det slik at beforeEach kjører 1 gang for hver test (og derfor åpner sida 8 ganger); finnes det kanskje en måte å åpne den kun en gang på slik at ikke hver test starter med å vente på sida? 🤔

e2e/specs/authenticated/MyNDLAFolder.spec.ts Outdated Show resolved Hide resolved
e2e/specs/authenticated/MyNDLAFolder.spec.ts Outdated Show resolved Hide resolved
@rauboti
Copy link
Contributor

rauboti commented Feb 13, 2024

@MaPoKen Har testet litt videre på dette for deg. Som nevnt over, feilet 1-2 random queries på await expect(page.getByRole("heading").getByText("Mine mapper")).toBeInViewport() når jeg kjørte hele authenticated-mappa. Når jeg prøvde å kjøre flere ganger, fikk jeg det av og til te å skje om jeg kjørte hele MyNDLAFolder.spec.ts også, selv om observasjonen over var at det gikk fint. En og en test går fortsatt helt fint. Jeg testa to ting:

  • Flytte den linja som feilet opp til beforeEach. Det hadde ingen merkbar effekt.
  • Sette testid på elementet, og skrive linja om til await expect(page.getByTestId("folder-Mine mapper")).toBeInViewport(). Da feilet steget aldri for meg lenger (jeg har kjørt authenticated-mappa 5-6 ganger nå uten å få det til å skje).

Et par steder kan det fortsatt feile etter den endringa når jeg kjører igjennom, men på lignende som den som feila sist; f.eks. folderList konstanten i "can drag and drop folders"-testen; const folderList = page.getByRole("list").filter({ has: page.locator('[data-type="list"]') });
Jeg veit ikke om getByRole i seg selv er problemet, eller om problemet er kombinasjoner av selectorer / filters. Men kanskje, så lenge poenget kun er å verifisere at elementer er der, at test id er hakket bedre for å unngå flaky tester.

Copy link
Contributor

@rauboti rauboti left a comment

Choose a reason for hiding this comment

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

Kjørt på nytt noen ganger, med workers satt i playwright.config.ts, og da fungerer alt.

@Jonas-C
Copy link
Contributor

Jonas-C commented Apr 16, 2024

Flaky!

@MaPoKen MaPoKen requested a review from rauboti April 16, 2024 15:34
@rauboti
Copy link
Contributor

rauboti commented Apr 17, 2024

Klarer ikke reprodusere det lokalt, men ser ut til å være 3 flaky tester når integration tests kjører

3 flaky
[MyNdla specs] › specs/authenticated/MyNDLAFolder.spec.ts:119:1 › can share and unshare folder ─
[MyNdla specs] › specs/authenticated/MyNDLAFolder.spec.ts:170:1 › can go to shared folder page ─
[MyNdla specs] › specs/authenticated/MyNDLAFolder.spec.ts:191:1 › can edit folder name on list item

Copy link
Contributor

@rauboti rauboti left a comment

Choose a reason for hiding this comment

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

Kjører fint både lokalt og ved push, ser det ut til.

@MaPoKen MaPoKen force-pushed the e2e-folder-page branch 6 times, most recently from 2f51a68 to 2f03f70 Compare May 30, 2024 11:14
@MaPoKen MaPoKen force-pushed the e2e-folder-page branch from 2f03f70 to 4bb34d8 Compare May 30, 2024 11:33
@MaPoKen MaPoKen force-pushed the e2e-folder-page branch 6 times, most recently from 5ae3001 to e6ae4ef Compare May 30, 2024 13:22
@MaPoKen MaPoKen force-pushed the e2e-folder-page branch from e6ae4ef to 5eab664 Compare May 30, 2024 13:28
@MaPoKen MaPoKen force-pushed the e2e-folder-page branch from c575855 to 856a89d Compare May 31, 2024 07:43
@gunnarvelle
Copy link
Member

Eller denne? Skal det ferdigstilles eller lukkes?

@MaPoKen
Copy link
Contributor Author

MaPoKen commented Aug 23, 2024

Tenker jeg ferdig stiller men skal vente på at nytt design er inne

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