That's to demonstrate [1]
Details
- Reviewers
olasd - Group Reviewers
Reviewers - Commits
- rDLDHGbf39050051ec: test: Explicit that 2 visits without change ends up with no snapshot
tox
Diff Detail
- Repository
- rDLDHG Mercurial loader
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 23605 Build 36833: Phabricator diff pipeline on jenkins Jenkins console · Jenkins Build 36832: arc lint + arc unit
Event Timeline
Build is green
Patch application report for D6262 (id=22671)
Rebasing onto 9be124af21...
Current branch diff-target is up to date.
Changes applied before test
commit b0433072af5b6e0ae4077ef3ce1d30171f66fb0f Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org> Date: Wed Sep 15 10:24:48 2021 +0200 test: Explicit that 2 visits without change ends up with no snapshot
See https://jenkins.softwareheritage.org/job/DLDHG/job/tests-on-diff/289/ for more details.
swh/loader/mercurial/tests/test_from_disk.py | ||
---|---|---|
187–188 | If there was no filtering whatsoever, we would expect the result of the second load to be exactly the same result as the first one (i.e. the same snapshot, with the exact same underlying objects). In effect, the counters should not change between the first and the second load (except for one more OriginVisit). The issue that I think you had pointed out, was that the final OriginVisitStatus of the second OriginVisit didn't get a snapshot entry. So you need to add a check for that. |
Build is green
Patch application report for D6262 (id=22672)
Rebasing onto 9be124af21...
Current branch diff-target is up to date.
Changes applied before test
commit 139061bcc38bd5652a266b5eb0645a1492e7a5aa Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org> Date: Wed Sep 15 10:24:48 2021 +0200 test: Explicit that 2 visits without change ends up with no snapshot
See https://jenkins.softwareheritage.org/job/DLDHG/job/tests-on-diff/290/ for more details.
swh/loader/mercurial/tests/test_from_disk.py | ||
---|---|---|
187–188 | Totally agreed! Thanks. I've updated the diff to that effect. |
It's a bit weird to accept a diff with an explicit reference to a bug. :P
I've suggested a tiny change.
swh/loader/mercurial/tests/test_from_disk.py | ||
---|---|---|
184–186 |
Build is green
Patch application report for D6262 (id=22714)
Rebasing onto 9be124af21...
Current branch diff-target is up to date.
Changes applied before test
commit bf39050051ec5c7b4c6cc7b4801079ea591cd549 Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org> Date: Wed Sep 15 10:24:48 2021 +0200 test: Explicit that 2 visits without change ends up with no snapshot
See https://jenkins.softwareheritage.org/job/DLDHG/job/tests-on-diff/298/ for more details.