Skip to content

Commit 861a33a

Browse files
committed
Fix tests
Now the build is recorded as success
1 parent 13b0086 commit 861a33a

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

readthedocs/rtd_tests/tests/test_doc_building.py

+5-7
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,6 @@ def test_command_not_recorded(self):
138138
'output': '',
139139
'state': 'finished',
140140
'builder': mock.ANY,
141-
'exit_code': 0,
142141
})
143142

144143
def test_record_command_as_success(self):
@@ -177,7 +176,7 @@ def test_record_command_as_success(self):
177176
self.mocks.mocks['api_v2.build']().put.assert_called_with({
178177
'id': DUMMY_BUILD_ID,
179178
'version': self.version.pk,
180-
'success': False,
179+
'success': True,
181180
'project': self.project.pk,
182181
'setup_error': u'',
183182
'length': mock.ANY,
@@ -186,7 +185,7 @@ def test_record_command_as_success(self):
186185
'output': '',
187186
'state': 'finished',
188187
'builder': mock.ANY,
189-
'exit_code': 1,
188+
'exit_code': 0,
190189
})
191190

192191
def test_incremental_state_update_with_no_update(self):
@@ -744,10 +743,9 @@ def test_command_not_recorded(self):
744743
self.mocks.mocks['api_v2.build']().put.assert_called_with({
745744
'id': DUMMY_BUILD_ID,
746745
'version': self.version.pk,
747-
'success': False,
746+
'success': True,
748747
'project': self.project.pk,
749748
'setup_error': '',
750-
'exit_code': 1,
751749
'length': 0,
752750
'error': '',
753751
'setup': '',
@@ -797,10 +795,10 @@ def test_record_command_as_success(self):
797795
self.mocks.mocks['api_v2.build']().put.assert_called_with({
798796
'id': DUMMY_BUILD_ID,
799797
'version': self.version.pk,
800-
'success': False,
798+
'success': True,
801799
'project': self.project.pk,
802800
'setup_error': '',
803-
'exit_code': 1,
801+
'exit_code': 0,
804802
'length': 0,
805803
'error': '',
806804
'setup': '',

0 commit comments

Comments
 (0)