Skip to content

Commit 555acfc

Browse files
committed
fix: Update tests to account for WebVTT regression test (id=3)
- test_add_test_empty_erc: Changed assertion to check id=4 (id=3 now used by WebVTT test) - test_update_expected_results_: Updated expected test count from 2 to 3 - sample1.webvtt: Fixed line endings to use CRLF as expected by code
1 parent f91b205 commit 555acfc

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
WEBVTT
2-
1+
WEBVTT
2+

tests/test_regression/test_controllers.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -243,7 +243,7 @@ def test_add_test_empty_erc(self):
243243
category_id=1,
244244
submit=True,
245245
))
246-
self.assertEqual(RegressionTest.query.filter(RegressionTest.id == 3).first(), None)
246+
self.assertEqual(RegressionTest.query.filter(RegressionTest.id == 4).first(), None)
247247

248248
def test_category_deletion_without_login(self):
249249
"""Check if it will move to the login page."""

tests/test_regression/test_update_regression.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ def test_update_expected_results_(self, mock_test, mock_os):
3939

4040
mock_os.path.isfile.return_value = True
4141
expected = True
42-
num_tests = 2 # store number of mock regression tests we have
42+
num_tests = 3 # store number of mock regression tests we have
4343

4444
response = update_expected_results('valid/path')
4545

0 commit comments

Comments
 (0)