Skip to content

Commit 41c5d30

Browse files
committed
improve test
1 parent b1676c7 commit 41c5d30

File tree

1 file changed

+34
-1
lines changed

1 file changed

+34
-1
lines changed

readthedocs/rtd_tests/tests/test_core_utils.py

+34-1
Original file line numberDiff line numberDiff line change
@@ -34,13 +34,46 @@ def test_trigger_skipped_project(self, update_docs_task):
3434
self.assertEqual(result, (None, None))
3535
self.assertFalse(update_docs_task.signature.called)
3636
self.assertFalse(update_docs_task.signature().apply_async.called)
37+
38+
@mock.patch('readthedocs.projects.tasks.update_docs_task')
39+
def test_trigger_build_when_version_not_provided_default_version_exist(self, update_docs_task):
40+
self.assertFalse(Version.objects.filter(slug='test-default-version').exists())
41+
42+
project_1 = get(Project)
43+
version_1 = get(Version, project=project_1, slug='test-default-version', active=True)
44+
45+
project_1.default_version = 'test-default-version'
46+
project_1.save()
47+
48+
default_version = project_1.get_default_version()
49+
self.assertEqual(default_version, 'test-default-version')
50+
51+
trigger_build(project=project_1)
52+
kwargs = {
53+
'version_pk': version_1.pk,
54+
'record': True,
55+
'force': False,
56+
'build_pk': mock.ANY,
57+
}
58+
59+
update_docs_task.signature.assert_has_calls([
60+
mock.call(
61+
args=(project_1.pk,),
62+
kwargs=kwargs,
63+
options=mock.ANY,
64+
immutable=True,
65+
),
66+
])
3767

3868
@mock.patch('readthedocs.projects.tasks.update_docs_task')
39-
def test_trigger_build_when_version_not_provided(self, update_docs_task):
69+
def test_trigger_build_when_version_not_provided_default_version_doesnt_exist(self, update_docs_task):
4070

4171
trigger_build(project=self.project)
4272
default_version = self.project.get_default_version()
4373
version_ = self.project.versions.get(slug=default_version)
74+
75+
self.assertEqual(version_.slug, LATEST)
76+
4477
kwargs = {
4578
'version_pk': version_.pk,
4679
'record': True,

0 commit comments

Comments
 (0)