Skip to content
Snippets Groups Projects
Commit 8785fc1a authored by David Douard's avatar David Douard
Browse files

cgit: fix cgit's task module and tests

forgot some `url_prefix` there.
parent 87cec2f5
No related branches found
No related tags found
No related merge requests found
......@@ -7,17 +7,9 @@ from swh.scheduler.celery_backend.config import app
from .lister import CGitLister
def new_lister(url='https://git.kernel.org/',
url_prefix=None,
instance='kernal', **kw):
return CGitLister(url=url, instance=instance, url_prefix=url_prefix,
**kw)
@app.task(name=__name__ + '.CGitListerTask')
def cgit_lister(**lister_args):
lister = new_lister(**lister_args)
lister.run()
CGitLister(**lister_args).run()
@app.task(name=__name__ + '.ping')
......
......@@ -11,7 +11,7 @@ def test_ping(swh_app, celery_session_worker):
@patch('swh.lister.cgit.tasks.CGitLister')
def test_lister_no_url_prefix(lister, swh_app, celery_session_worker):
def test_lister(lister, swh_app, celery_session_worker):
# setup the mocked CGitLister
lister.return_value = lister
lister.run.return_value = None
......@@ -25,29 +25,6 @@ def test_lister_no_url_prefix(lister, swh_app, celery_session_worker):
lister.assert_called_once_with(
url='https://git.kernel.org/',
url_prefix=None,
instance='kernel')
lister.db_last_index.assert_not_called()
lister.run.assert_called_once_with()
@patch('swh.lister.cgit.tasks.CGitLister')
def test_lister_with_url_prefix(lister, swh_app, celery_session_worker):
# setup the mocked CGitLister
lister.return_value = lister
lister.run.return_value = None
res = swh_app.send_task(
'swh.lister.cgit.tasks.CGitListerTask',
kwargs=dict(url='https://cgit.kde.org/',
url_prefix='https://anongit.kde.org/', instance='kde'))
assert res
res.wait()
assert res.successful()
lister.assert_called_once_with(
url='https://cgit.kde.org/',
url_prefix='https://anongit.kde.org/',
instance='kde')
lister.db_last_index.assert_not_called()
lister.run.assert_called_once_with()
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment