Page MenuHomeSoftware Heritage

Fix snapshot page title
AbandonedPublic

Authored by KShivendu on Mar 1 2021, 5:37 AM.

Details

Reviewers
vlorentz
Group Reviewers
Reviewers
Maniphest Tasks
T2801: Wrong <title> on snapshot pages
Summary

Fixes T2801

Diff Detail

Repository
rDWAPPS Web applications
Branch
fix-snapshot-title
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 19547
Build 30329: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 30328: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D5164 (id=18470)

Rebasing onto d91cbf3780...

Current branch diff-target is up to date.
Changes applied before test
commit 51c7de9c4bd0613cbd18daf96e76b47a04790e60
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Mon Mar 1 10:05:53 2021 +0530

    Fix snapshot page title

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

vlorentz requested changes to this revision.Mar 1 2021, 2:02 PM
vlorentz added a subscriber: vlorentz.

Hi, thanks again.

Could you add a test?

swh/web/browse/snapshot_context.py
840

no space before a colon

843

same here

844

Shouldn't the branch remain before the path?

This revision now requires changes to proceed.Mar 1 2021, 2:02 PM

Hi @vlorentz. The test should be from the frontend or backend?

@KShivendu , I am not sure that change is needed, see T2801#59654.

Otherwise, tests should be written for the backend to check expected title string is well present in the produced HTML.

I do not think we added tests to check HTML title pages so far so that diff could evolve to add those.