browse: Fix pagination links display issue in branches / releases views

When a snapshot contains a branch or a release alias, the display of pagination links in the branches / releases view could be missing due to an invalid test in the branches/releases view implementation.

See https://archive.softwareheritage.org/browse/origin/https://github.com/torvalds/linux/releases/ showcasing the issue fixed in that diff.

---

Migrated from D2920 (view on Phabricator)

Build is green See https://jenkins.softwareheritage.org/job/DWAPPS/job/tox/1042/ for more details.

Build is green See https://jenkins.softwareheritage.org/job/DWAPPS/job/cypress-diff/655/ for more details.

Could you add a short docstring to the test, explaining the same thing as the diff's description?

Merge request was accepted

approved this merge request

Update: Add new test description in docstring

Build is green See https://jenkins.softwareheritage.org/job/DWAPPS/job/cypress-diff/656/ for more details.

Build has FAILED

Patch application report for D2920 (id=10378)

Rebasing onto d4446bca...

Current branch diff-target is up to date.

Changes applied before test

commit 504a0e534f051102905e555440418bf1b4ed947c
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Fri Mar 27 18:49:25 2020 +0100

    browse: Fix pagination links display issue in branches / releases views
    
    When a snapshot contains a branch or a release alias, the display of pagination
    links in the branches / releases view could be missing due to an invalid test
    in the branches/releases view implementation.

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/9/ See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/9/console

Build has FAILED

Patch application report for D2920 (id=10378)

Rebasing onto d4446bca...

Current branch diff-target is up to date.

Changes applied before test

commit 504a0e534f051102905e555440418bf1b4ed947c
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Fri Mar 27 18:49:25 2020 +0100

    browse: Fix pagination links display issue in branches / releases views
    
    When a snapshot contains a branch or a release alias, the display of pagination
    links in the branches / releases view could be missing due to an invalid test
    in the branches/releases view implementation.

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/10/ See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/10/console

Build is green

Patch application report for D2920 (id=10378)

Rebasing onto d4446bca...

Current branch diff-target is up to date.

Changes applied before test

commit 504a0e534f051102905e555440418bf1b4ed947c
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Fri Mar 27 18:49:25 2020 +0100

    browse: Fix pagination links display issue in branches / releases views
    
    When a snapshot contains a branch or a release alias, the display of pagination
    links in the branches / releases view could be missing due to an invalid test
    in the branches/releases view implementation.

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

Rebase

Build has FAILED

Patch application report for D2920 (id=10413)

Rebasing onto f8ac17e1...

Current branch diff-target is up to date.

Changes applied before test

commit f1c805be3ab6e71af650e5e51a9a9e36fec5f6c5
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Fri Mar 27 18:49:25 2020 +0100

    browse: Fix pagination links display issue in branches / releases views
    
    When a snapshot contains a branch or a release alias, the display of pagination
    links in the branches / releases view could be missing due to an invalid test
    in the branches/releases view implementation.

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/23/ See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/23/console

Rebase + add commit upgrading yarn dependencies (notably cypress)

Build is green

Patch application report for D2920 (id=10419)

Rebasing onto fc6dc351...

Current branch diff-target is up to date.

Changes applied before test

commit 717159e4594e480d97c0adccd1cebab7ec58722c
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Tue Mar 31 12:18:28 2020 +0200

    package.json: Upgrade dependencies

commit c0baf47fb30e73803531d613c5e29baf42d003e7
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Fri Mar 27 18:49:25 2020 +0100

    browse: Fix pagination links display issue in branches / releases views
    
    When a snapshot contains a branch or a release alias, the display of pagination
    links in the branches / releases view could be missing due to an invalid test
    in the branches/releases view implementation.

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

Merge request was merged

closed