jenkins-bot merged this change.

View Change

Skip TestIndexPageMappings and test_url_image

These tests occasionally fail causing multiple failures on Travis and
Appveyor jobs. We can also mark them as expectedFailure, but that
would make noise on unexpected success messages during the tests, making
detection of unexpected success messages that can actually be fixed harder.

Bug: T193637
Bug: T181913
Change-Id: If1765d94cc798f37b228730e5d5adec302f7824c
---
M tests/proofreadpage_tests.py
1 file changed, 2 insertions(+), 0 deletions(-)

diff --git a/tests/proofreadpage_tests.py b/tests/proofreadpage_tests.py
index 02fae23..826cdfc 100644
--- a/tests/proofreadpage_tests.py
+++ b/tests/proofreadpage_tests.py
@@ -284,6 +284,7 @@
         self.assertEqual(json.loads(page_text), json.loads(loaded_text))
 
     @require_modules('bs4')
+    @unittest.skip('T181913 and T114318')
     def test_url_image(self):
         """Test fetching of url image of the scan of ProofreadPage."""
         page = ProofreadPage(self.site, self.valid['title'])
@@ -479,6 +480,7 @@
         self._test_page_text()
 
 
+@unittest.skip('T193637 and T114318')
 class TestIndexPageMappings(IndexPageTestCase):
 
     """Test IndexPage class."""

To view, visit change 444424. To unsubscribe, or for help writing mail filters, visit settings.