Page MenuHomeSoftware Heritage

cypress: Use non github origin for save code now tests
AbandonedPublic

Authored by anlambert on May 2 2022, 5:25 PM.

Details

Reviewers
None
Group Reviewers
Reviewers
Summary

Cypress tests for save code now was using https://github.com/memononen/libtess2
as input origin for most of these tests. But using a github origin will trigger
an HTTP request to https://api.github.com/repos/ client-side to get the canonical
URL of the repository.

That request was not intercepted by cypress so real HTTP requests on the internet
were issued during the tests which slowdown them and can make them fail under
certain conditions.

So prefer to use a sample git origin URL not related to github to avoid these kind
of issues.

Diff Detail

Repository
rDWAPPS Web applications
Branch
cypress-fix-flaky-tests-on-jenkins
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 29014
Build 45357: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 45356: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D7729 (id=27942)

Rebasing onto 25bf6ca843...

Current branch diff-target is up to date.
Changes applied before test
commit 70cf0f13af36ffd7b99d5ded1a04f951a6ecaa94
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Mon May 2 17:18:05 2022 +0200

    cypress: Use non github origin for save code now tests
    
    Cypress tests for save code now was using https://github.com/memononen/libtess2
    as input origin for most of these tests. But using a github origin will trigger
    an HTTP request to https://api.github.com/repos/ client-side to get the canonical
    URL of the repository.
    
    That request was not intercepted by cypress so real HTTP requests on the internet
    were issued during the tests which slowdown them and can make them fail under
    certain conditions.
    
    So prefer to use a sample git origin URL not related to github to avoid these kind
    of issues.

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