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

Feil: Alle integrasjonstester med snapshots feiler #1104

Closed
piofinn opened this issue Jul 7, 2020 · 0 comments · Fixed by #1246
Closed

Feil: Alle integrasjonstester med snapshots feiler #1104

piofinn opened this issue Jul 7, 2020 · 0 comments · Fixed by #1246
Labels
🐝 Bug Noe er feil eller ikke som det skal

Comments

@piofinn
Copy link
Contributor

piofinn commented Jul 7, 2020

Feilbeskrivelse
Alle integrasjonstester via Cypress, som bruker snapshots, feiler på grunn av en bug i cypress-plugin-snapshots.

For å reprodusere
Se på en nylig kjørt integrasjonstest på GitHub (f.eks.: https://github.com/fremtind/jokul/pull/1099/checks?check_run_id=844920677). Feilen gjør seg gjeldende med følgende feilmelding:

Cannot find module 'js-base64/base64.min.js'

Forventet oppførsel
Integrasjonstestene bør ikke feile på grunn av feilen over

Mer info
Følgende PR vil fikse feilen i plugin'en: meinaart/cypress-plugin-snapshots#134
Med mindre det tar veldig lang tid for PRen å blir dratt inn foreslår jeg å vente med å gjøre noe, og ignorere den feilende testen i mellomtiden.

@piofinn piofinn added the 🐝 Bug Noe er feil eller ikke som det skal label Jul 7, 2020
lfbergee pushed a commit that referenced this issue Jul 30, 2020
piofinn added a commit that referenced this issue Aug 20, 2020
lfbergee pushed a commit that referenced this issue Aug 20, 2020
test(integration): re-enable cypress snapshots

ISSUES CLOSED: #1104

chore(cypress): enable screenshot tests

affects: @fremtind/jkl-accordion-react, @fremtind/jkl-button-react

fix(cypress): add snapshots to git

chore(cypress): add all files from workflow

chore(cypress): add test mode to portal

affects: @fremtind/portal
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐝 Bug Noe er feil eller ikke som det skal
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant