Page MenuHomeSoftware Heritage

tasks: Add tests around indexer tasks
ClosedPublic

Authored by ardumont on Aug 6 2020, 2:30 PM.

Details

Summary

Preparatory work to rename current "content"-range task to
"content"-partition (as that is what is happening now).

Test Plan

tox

Diff Detail

Repository
rDCIDX Metadata indexer
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D3724 (id=13119)

Rebasing onto 3aedf90c9c...

Current branch diff-target is up to date.
Changes applied before test
commit 89472cdaa61a2330f1b6b7a32e31a80bed7e9f9e
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Aug 6 14:28:35 2020 +0200

    tasks: Add tests around indexer tasks

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

vlorentz added a subscriber: vlorentz.

I think {"status": "eventful"} is misleading, indexer tasks' returned value doesn't look anywhere near that, does it?

swh/indexer/tests/conftest.py
96

use something other than "loader"

This revision is now accepted and ready to land.Aug 6 2020, 3:00 PM

I think {"status": "eventful"} is misleading, indexer tasks' returned value doesn't look anywhere near that, does it?

nah, they do, well at least initially they did (and some are still returning such values [1]).

[1] https://forge.softwareheritage.org/source/swh-indexer/browse/master/swh/indexer/indexer.py$333

swh/indexer/tests/conftest.py
96

damn, I forgot to update it. thanks.

Fix "loader.yml" to "indexer.yml"

Build is green

Patch application report for D3724 (id=13121)

Rebasing onto 3aedf90c9c...

Current branch diff-target is up to date.
Changes applied before test
commit 08adca9457a7d208f47bf7907ce47070d521d76f
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Aug 6 14:28:35 2020 +0200

    tasks: Add tests around indexer tasks

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

This revision was automatically updated to reflect the committed changes.