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.

Sign up for GitHub

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

Jump to bottom

Add CI infrastructure to run UI tests #24

Closed
3 of 4 tasks
lbianchi-lbl opened this issue Jul 14, 2022 · 0 comments
Closed
3 of 4 tasks

Add CI infrastructure to run UI tests #24

lbianchi-lbl opened this issue Jul 14, 2022 · 0 comments
Assignees
@lbianchi-lbl
Labels
Priority:Normal Normal Priority Issue or PR

Comments

@lbianchi-lbl
Copy link
Contributor

lbianchi-lbl commented Jul 14, 2022 •
edited by MichaelPesce
Loading

In no particular order:
@ksbeattie ksbeattie added the Priority:Normal Normal Priority Issue or PR label Jul 14, 2022
@MichaelPesce MichaelPesce mentioned this issue Jul 15, 2022
Merged
lbianchi-lbl added a commit that referenced this issue Jul 19, 2022
@MichaelPesce @lbianchi-lbl
Add CI infrastructure to run UI tests - Issue #24 (#25)
4184947 
* added ability to choose from past runs and view output

* added comparison tab and table

* added download ability for comparison table

* removed debug print statement

* right aligned value difference column; cleaned up some code

* added main.yml for component and e2e testing

* added main.yml for component and e2e testing

* removed node.js.yml

* added additional step for failures

* cleaned up spacing and naming of steps

* updated cypress config to include videos

* added additional screenshots to cypress test

* added a short sleep to (hopefully) avoid flaky test

* Update .github/workflows/main.yml

Co-authored-by: Ludovico Bianchi <[email protected]>

* Update backend/app/internal/flowsheet/flowsheet.py

Co-authored-by: Ludovico Bianchi <[email protected]>

* Update .github/workflows/main.yml

Co-authored-by: Ludovico Bianchi <[email protected]>

* updates from Ludovicos comments

* additional updates from comments

Co-authored-by: Ludovico Bianchi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lbianchi-lbl lbianchi-lbl

Labels
Priority:Normal Normal Priority Issue or PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ksbeattie @MichaelPesce @lbianchi-lbl