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

Cypress component test #1257

Merged
merged 23 commits into from
Nov 22, 2022
Merged

Conversation

avanti-em
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Oct 19, 2022

Codecov Report

Merging #1257 (86a1d81) into master (4c46de6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1257   +/-   ##
=======================================
  Coverage   36.45%   36.45%           
=======================================
  Files         149      149           
  Lines        8163     8163           
  Branches     2188     2188           
=======================================
  Hits         2976     2976           
  Misses       5153     5153           
  Partials       34       34           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@avanti-em avanti-em force-pushed the cypress_component_test branch from f97e70a to 2cfc599 Compare October 27, 2022 08:00
@avanti-em avanti-em force-pushed the cypress_component_test branch from 18b355e to eb8b99a Compare October 28, 2022 10:10
Copy link
Collaborator

@anders-kiaer anders-kiaer left a comment

Choose a reason for hiding this comment

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

A couple of small questions, otherwise looks great - nice work. 🙂

As discussed earlier, it is hard to test this fully prior to merge as it is about the CI pipeline itself, with some complex features.

run: |
mkdir clone
cd clone
git clone --branch cypress-snaps --depth 1 --filter=blob:none https://github.com/avanti-em/webviz-subsurface-components
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this be changed to official repo before merge?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops! I forgot to change it. Thanks for catching.

@@ -155,7 +181,7 @@ jobs:
run: npm run build-storybook --prefix ./react

- name: 📚 Update Storybook and GitHub demo application
if: github.event_name == 'push' && (github.ref == 'refs/heads/master' || contains(github.event.head_commit.message, '[deploy test]')) && matrix.python-version == '3.8'
if: github.event_name == 'push' && (github.ref == 'refs/heads/master' || contains(github.event.head_commit.message, '[deploy test]')) && matrix.python-version == '3.8' && failure() && matrix.e2e-tests
Copy link
Collaborator

Choose a reason for hiding this comment

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

For my understanding: Why do we want to update official storybook only on failure?

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 think I did some kind of mistake while I was doing some "search and replace" stuff. Its a complete mistake, I will undo it.

body: This is an auto-generated PR to update cypress snapshots.
base: cypress-snaps
reviewers: ${{ github.event.sender.login }}
delete-branch: true
Copy link
Collaborator

Choose a reason for hiding this comment

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

Missing newline

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it, will fix it

@avanti-em avanti-em force-pushed the cypress_component_test branch 3 times, most recently from c035d0e to 9c97b25 Compare November 15, 2022 13:36
@avanti-em avanti-em force-pushed the cypress_component_test branch 2 times, most recently from 8070a8b to 9330d2c Compare November 16, 2022 09:28
@avanti-em avanti-em force-pushed the cypress_component_test branch from e16d1b8 to 624605c Compare November 17, 2022 06:38
@anders-kiaer anders-kiaer self-requested a review November 21, 2022 10:21
@avanti-em avanti-em merged commit c2a2917 into equinor:master Nov 22, 2022
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