cypress: Override type command to remove key press delay
ClosedPublic
Actions

Authored by anlambert on Jun 16 2022, 1:56 PM.

Details

Reviewers

Group Reviewers

Commits

rDWAPPSd5f020c02bb4: cypress: Override type command to remove key press delay
rDWAPPSf8c7ca101e6a: cypress/persistent-identifiers: Workaround tests hang issue

Summary

Cypress use a default key press delay of 10ms which significantly
slow downs e2e tests when typing a long string in an input element.

So override the type command to remove that delay.

Locally, overall cypress tests execution is reduced by one minute
(from 6m36 to 5m37).

Diff Detail

Repository

rDWAPPS Web applications

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 29901
Build 46741: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 46740: arc lint + arc unit

Event Timeline

anlambert created this revision.Jun 16 2022, 1:56 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptJun 16 2022, 1:56 PM

anlambert added inline comments.Jun 16 2022, 1:59 PM

cypress/e2e/deposit-admin.cy.js
191	Modal was not closing anymore after the type command overriding, click on close button instead which is equivalent.
cypress/e2e/persistent-identifiers.cy.js
34	I do not know why those tests start hanging locally but this is not related to the type command overriding. I put that change in a separate commit.

Build is green

Patch application report for D7997 (id=28815)

Rebasing onto 662ce6eafc...

Current branch diff-target is up to date.

Changes applied before test

commit d5f020c02bb47bf3dab5d9e0e9928d38fba4c5be
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Thu Jun 16 13:52:23 2022 +0200

    cypress: Override type command to remove key press delay
    
    Cypress use a default key press delay of 10ms which significantly
    slow downs e2e tests when typing a long string in an input element.
    
    So override the type command to remove that delay.
    
    Locally, overall cypress tests execution is reduced by one minute
    (from 6m36 to 5m37).

commit f8c7ca101e6a089b470c39924688f2625b0c05c4
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Thu Jun 16 13:51:27 2022 +0200

    cypress/persistent-identifiers: Workaround tests hang issue

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1903/ for more details.

Harbormaster completed remote builds in B29901: Diff 28815.Jun 16 2022, 2:18 PM

anlambert requested review of this revision.Jun 16 2022, 2:18 PM

On Jenkins , we gain more than one minute in overall cypress tests execution, see build log before that change and build log after that change.