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

fix: v100.3.1 #2907

Merged
merged 10 commits into from
Nov 8, 2023
Merged

fix: v100.3.1 #2907

merged 10 commits into from
Nov 8, 2023

Conversation

janhenrikoverland
Copy link
Collaborator

See logs for list of fixes.

martinkrulltott and others added 10 commits October 23, 2023 08:21
To follow cypress best practice, consolidate assertions into single tests when
they represent a single logical user journey

Positive side effects (in addition to those listed in the best practice page):
* less tests contributing to our monthly cypress subscription total
* less skewed impression of coverage

With these changes the number of tests has gone down from 637 to 569
Note: no assertions were harmed during the making of this PR.

What has changed: Every describe under the top level describe
has been changed to an it.

All the its within the former describe have been removed and their title
converted to a comment or a cy.log.

Comments were converted to cy.log to make it easier to locate eventual
errors in the cypress dashboard output.
add BRANCH_NAME and PR_TITLE as RP attributes
dhis2/workflows#46  merged [CI_BUILD_ID]
dhis2/workflows#47 merged [BRANCH_NAME and PR_TITLE]
@dhis2-bot
Copy link
Contributor

@dhis2-bot dhis2-bot temporarily deployed to netlify November 8, 2023 15:04 Inactive
Copy link

cypress bot commented Nov 8, 2023

1 flaky test on run #2529 ↗︎

0 450 1 0 Flakiness 1

Details:

Merge branch 'master' into dev
Project: Data Visualizer App Commit: 0e08ec9cd3
Status: Passed Duration: 08:39 💡
Started: Nov 8, 2023 3:27 PM Ended: Nov 8, 2023 3:35 PM
Flakiness  cypress/integration/new.cy.js • 1 flaky test • e2e-chrome-parallel-2.40

View
Output

Test Artifacts
creating a new AO > create AO of type Column Screenshots

Review all test suite changes for PR #2907 ↗︎

Copy link

cypress bot commented Nov 8, 2023

Passing run #2530 ↗︎

0 452 1 0 Flakiness 0

Details:

Merge 0e08ec9 into fea8f06...
Project: Data Visualizer App Commit: eaee0fe1e7 ℹ️
Status: Passed Duration: 07:04 💡
Started: Nov 8, 2023 3:51 PM Ended: Nov 8, 2023 3:58 PM

Review all test suite changes for PR #2907 ↗︎

@janhenrikoverland janhenrikoverland merged commit ba1fcf5 into master Nov 8, 2023
25 checks passed
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.3.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants