-
Notifications
You must be signed in to change notification settings - Fork 41
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
Cypress component test #1257
Conversation
Codecov Report
@@ 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 |
f97e70a
to
2cfc599
Compare
18b355e
to
eb8b99a
Compare
…-components into cypress_component_test
…-components into cypress_component_test
…-components into cypress_component_test
There was a problem hiding this 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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing newline
There was a problem hiding this comment.
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
c035d0e
to
9c97b25
Compare
…-components into cypress_component_test
8070a8b
to
9330d2c
Compare
e16d1b8
to
624605c
Compare
No description provided.