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

screenshot tests: add more screens, blackout time, enable legacy #4145

Merged
merged 3 commits into from
Aug 26, 2023

Conversation

himdel
Copy link
Collaborator

@himdel himdel commented Aug 26, 2023

Change screenshot folder from cypress/screenshots/test_name.js/ to cypress/screenshots/test_suite_name/test_name.js/ (test_suite meaning all the cypress/e2e/ subdirs used in test matrix).

Wrap our DateComponent in a <time> element, following https://microformats.org/wiki/datetime-design-pattern,
blackout time elements in non-failure cypress screenshots - we can test more screens.

Enable legacy roles for the screenshots run, allowing us to test those screens.

@github-actions github-actions bot added the backport-4.7 This PR should be backported to stable-4.7 (2.4) label Aug 26, 2023
@himdel himdel added the tests tests PR label Aug 26, 2023
@himdel
Copy link
Collaborator Author

himdel commented Aug 26, 2023

(merging and cleaning the cache, so the cron action can recreate the new cache for #4143)

@himdel himdel merged commit 9b0eca6 into ansible:master Aug 26, 2023
@himdel himdel deleted the screenshots branch August 26, 2023 16:54
@patchback
Copy link

patchback bot commented Aug 26, 2023

Backport to stable-4.7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4.7/9b0eca617aa75ecc134f5dc96b366b20f219d0d7/pr-4145

Backported as #4146

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Aug 26, 2023
* screenshot tests: add more screens, blackout time, enable legacy

No-Issue

* lint

* rename screenshots.js to screenshots-main when cached, fix paths for extra suite name

(cherry picked from commit 9b0eca6)
himdel added a commit that referenced this pull request Aug 27, 2023
…) (#4146)

* screenshot tests: add more screens, blackout time, enable legacy

No-Issue

* lint

* rename screenshots.js to screenshots-main when cached, fix paths for extra suite name

(cherry picked from commit 9b0eca6)

Co-authored-by: Martin Hradil <mhradil@redhat.com>
@github-actions github-actions bot added the backported-4.7 This PR has been backported to stable-4.7 (2.4) label Aug 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-4.7 This PR should be backported to stable-4.7 (2.4) backported-4.7 This PR has been backported to stable-4.7 (2.4) tests tests PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant