Disable video uploads for successful Cypress tests #13747
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## master #13747 +/- ##
==========================================
+ Coverage 83.36% 83.42% +0.05%
==========================================
Files 372 372
Lines 29397 29397
Branches 2143 2142 -1
==========================================
+ Hits 24508 24523 +15
+ Misses 2746 2732 -14
+ Partials 2143 2142 -1
Continue to review full report at Codecov.
|
nemanjaglumac
changed the title
ariya
approved these changes
Nov 11, 2020
nemanjaglumac
added a commit
that referenced
this pull request
Nov 26, 2020
nemanjaglumac
added a commit
that referenced
this pull request
Nov 26, 2020
* Improve Cypress helper functions with log messages (#13749) [ci skip] - Adds helpful log message to various helper functions in `frontend/test/__support__/cypress.js`. - `--- Logging in as ${user} ---` to `signIn()` - `--- Signing out ---` to `signOut()` and - `--- Restore Data Set ---` to `restore()` * Disable video uploads for successful Cypress tests (#13747) * https://docs.cypress.io/guides/references/configuration.html#Videos * Add Cypress file download testing (#13430) * Add `read-excel-file` library * Add "file download" config to `cypress-plugins.js` * Switch to SheetJS for xlsx/csv testing and upgrade Cypress because of cypress-io/cypress#7382 Co-authored-by: Damon P. Cortesi <d.lifehacker@gmail.com> * Quarantine `field.cy.spec.js` (#13896) Full quarantine list in #13682. * Fix smoke tests (#13897) Failure happened because of the obsolete references to the now updated actions/drill-through popover menu.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Status
PENDING CI
What does this PR accomplish?
Additional info:
let's see how long will this CI run take(unfortunately, parallelization is currently disabled in Cypress Dashboard so there wasn't much point in trying to measure and/or compare this run to a random previous one)