Page MenuHomeSoftware Heritage
Differential D2122

npm.lister: Add integration test which checks the scheduled tasks
ClosedPublic
Actions

Authored by ardumont on Oct 10 2019, 10:15 PM.

Details

Reviewers
anlambert
douardda
vlorentz
Group Reviewers
Reviewers
Commits
rDLS678b7ea5bd51: npm.lister: Add integration test which checks the scheduled tasks
Summary

This checks only the basic listing implementation.

Related T2032

Test Plan

tox

Diff Detail

Repository
rDLS Listers
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 8331
Build 12054: tox-on-jenkinsJenkins
Build 12053: arc lint + arc unit

Event Timeline

ardumont created this revision.Oct 10 2019, 10:15 PM
Harbormaster failed remote builds in B8311: Diff 7109!Oct 10 2019, 10:15 PM
swh-public-ci added a comment.Oct 10 2019, 10:15 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tox/411/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tox/411/console

ardumont edited the summary of this revision. (Show Details)Oct 10 2019, 10:16 PM
ardumont added a parent revision: D2120: cran.lister: Fix cran lister and add proper integration test.
swh-public-ci added a comment.Oct 10 2019, 10:17 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tox/412/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tox/412/console

ardumont edited the summary of this revision. (Show Details)Oct 10 2019, 10:19 PM
ardumont added a parent revision: D2121: pypi.lister: Add integration test which checks the scheduled tasks.
swh-public-ci added a comment.Oct 10 2019, 10:19 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tox/413/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tox/413/console

ardumont removed a parent revision: D2120: cran.lister: Fix cran lister and add proper integration test.Oct 10 2019, 10:20 PM
ardumont edited the summary of this revision. (Show Details)
swh-public-ci added a comment.Oct 10 2019, 10:21 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tox/414/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tox/414/console

ardumont added a comment.Oct 10 2019, 10:23 PM

Build has FAILED

well, i don't get it yet.
tomorrow.

vlorentz accepted this revision.Oct 11 2019, 1:29 PM
This revision is now accepted and ready to land.Oct 11 2019, 1:29 PM
ardumont updated this revision to Diff 7128.Oct 11 2019, 2:01 PM
ardumont edited the summary of this revision. (Show Details)

Rebase to latest master

Harbormaster failed remote builds in B8330: Diff 7128!Oct 11 2019, 2:01 PM
swh-public-ci added a comment.Oct 11 2019, 2:01 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tox/424/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tox/424/console

ardumont removed a parent revision: D2121: pypi.lister: Add integration test which checks the scheduled tasks.Oct 11 2019, 2:02 PM
ardumont edited the summary of this revision. (Show Details)
swh-public-ci added a comment.Oct 11 2019, 2:03 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tox/425/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tox/425/console

ardumont added a comment.Oct 11 2019, 2:08 PM

Build has FAILED

It might be related to symbolic i use to avoid duplication.

Checking patch swh/lister/npm/tests/test_npm_lister.py...
Checking patch swh/lister/npm/tests/api_response.json => swh/lister/npm/tests/data/replicate.npmjs.com/_all_docs,startkey=%22%22,limit=1001...
Checking patch dev/null => swh/lister/npm/tests/api_response.json...
error: dev/null: does not exist in index
ardumont updated this revision to Diff 7129.Oct 11 2019, 2:09 PM

Moving tests data within the tests/data/ folder.

And try to fix the failing build

swh-public-ci added a comment.Oct 11 2019, 2:12 PM

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

Harbormaster completed remote builds in B8331: Diff 7129.Oct 11 2019, 2:12 PM
Closed by commit rDLS678b7ea5bd51: npm.lister: Add integration test which checks the scheduled tasks (authored by ardumont). · Explain WhyOct 11 2019, 2:13 PM
This revision was automatically updated to reflect the committed changes.
ardumont added a commit: rDLS678b7ea5bd51: npm.lister: Add integration test which checks the scheduled tasks.

Revision Contents
Changeset List

Diff 7129

View Options

swh/lister/npm/tests/api_empty_response.json

Loading...
View Options

swh/lister/npm/tests/api_inc_empty_response.json

Loading...
View Options

swh/lister/npm/tests/api_inc_response.json

Loading...
View Options

swh/lister/npm/tests/api_response.json

Loading...
View Options

swh/lister/npm/tests/data/api_empty_response.json

Loading...
View Options

swh/lister/npm/tests/data/api_inc_empty_response.json

Loading...
View Options

swh/lister/npm/tests/data/api_inc_response.json

Loading...
View Options

swh/lister/npm/tests/data/replicate.npmjs.com/_all_docs,startkey=%22%22,limit=1001

Loading...
View Options

swh/lister/npm/tests/data/replicate.npmjs.com/api_response.json

Loading...
View Options

swh/lister/npm/tests/test_lister.py

Loading...
View Options

swh/lister/npm/tests/test_npm_lister.py

Loading...
About · Developer information · Legal