Differential D1610 Diff 5555 swh/lister/cgit/tests/test_tasks.py

Changeset View

Standalone View

swh/lister/cgit/tests/test_tasks.py

This file was added.

				from unittest.mock import patch


				def test_ping(swh_app, celery_session_worker):
				res = swh_app.send_task(
				'swh.lister.cgit.tasks.ping')
				assert res
				res.wait()
				assert res.successful()
				assert res.result == 'OK'


				@patch('swh.lister.cgit.tasks.CGitLister')
				def test_lister_no_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://git.kernel.org/', instance='kernel'))
				assert res
				res.wait()
				assert res.successful()

				lister.assert_called_once_with(
				url='https://git.kernel.org/',
				ardumontUnsubmitted Done Inline Actions Same as before. If url_prefix is not really used (as it's the same as url), drop it here. It's best to test the other cases though. So add other tests (dubbed `test_lister_no_url_prefix`, `test_lister_url_and_url_prefix` or something...) with different instance initializations. Possible scenarios i see are: one with only url one with url and url_prefix (different) add other combinations i don't know about (if any ;) ardumont: Same as before. If url_prefix is not really used (as it's the same as url), drop it here. It's…
				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()