Xqt submitted this change.

View Change

[tests] Set expected_failure_if for ui_tests on Travis

This issue cannot be solved shortly.
The failures seems to be introduced by test method itself.

Bug: T282962
Change-Id: I0798bde22ea02a5c961cf2af1340cb148e0f9174
---
M tests/ui_tests.py
1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/tests/ui_tests.py b/tests/ui_tests.py
index a4d7f7f..f76c258 100644
--- a/tests/ui_tests.py
+++ b/tests/ui_tests.py
@@ -28,7 +28,10 @@
     terminal_interface_win32,
 )
 from tests.aspects import TestCase, TestCaseBase
-from tests.utils import FakeModule, unittest
+from tests.utils import expected_failure_if, FakeModule, unittest
+
+
+TRAVIS = os.getenv('TRAVIS', 'false') == 'true'
 
 
 class Stream:
@@ -143,6 +146,7 @@
         ('CRITICAL', CRITICAL, '', 'CRITICAL: CRITICAL\n'),
     ]
 
+    @expected_failure_if(TRAVIS)
     def test_outputlevels_logging(self):
         """Test logger with output levels."""
         for text, level, out, err in self.tests:
@@ -152,6 +156,7 @@
                 self.assertEqual(newstderr.getvalue(), err)
                 patch()  # reset terminal files
 
+    @expected_failure_if(TRAVIS)
     def test_output(self):
         pywikibot.output('output')
         self.assertEqual(newstdout.getvalue(), '')
@@ -162,11 +167,13 @@
         self.assertEqual(newstdout.getvalue(), 'output\n')
         self.assertEqual(newstderr.getvalue(), '')
 
+    @expected_failure_if(TRAVIS)
     def test_warning(self):
         pywikibot.warning('warning')
         self.assertEqual(newstdout.getvalue(), '')
         self.assertEqual(newstderr.getvalue(), 'WARNING: warning\n')
 
+    @expected_failure_if(TRAVIS)
     def test_error(self):
         pywikibot.error('error')
         self.assertEqual(newstdout.getvalue(), '')
@@ -177,6 +184,7 @@
         self.assertEqual(newstdout.getvalue(), '')
         self.assertEqual(newstderr.getvalue(), '')
 
+    @expected_failure_if(TRAVIS)
     def test_critical(self):
         pywikibot.critical('critical')
         self.assertEqual(newstdout.getvalue(), '')
@@ -187,6 +195,7 @@
         self.assertEqual(newstdout.getvalue(), '')
         self.assertEqual(newstderr.getvalue(), '')
 
+    @expected_failure_if(TRAVIS)
     def test_exception(self):
         class TestException(Exception):
 
@@ -200,6 +209,7 @@
         self.assertEqual(newstderr.getvalue(),
                          'ERROR: TestException: Testing Exception\n')
 
+    @expected_failure_if(TRAVIS)
     def test_exception_tb(self):
         class TestException(Exception):
 
@@ -227,6 +237,7 @@
 
     input_choice_output = 'question ([A]nswer 1, a[n]swer 2, an[s]wer 3): '
 
+    @expected_failure_if(TRAVIS)
     def testInput(self):
         newstdin.write('input to read\n')
         newstdin.seek(0)
@@ -237,6 +248,7 @@
         self.assertIsInstance(returned, str)
         self.assertEqual(returned, 'input to read')
 
+    @expected_failure_if(TRAVIS)
     def test_input_yn(self):
         newstdin.write('\n')
         newstdin.seek(0)
@@ -266,6 +278,7 @@
 
         self.assertEqual(returned, 'a')
 
+    @expected_failure_if(TRAVIS)
     def testInputChoiceCapital(self):
         newstdin.write('N\n')
         newstdin.seek(0)
@@ -274,6 +287,7 @@
         self.assertEqual(newstderr.getvalue(), self.input_choice_output)
         self.assertEqual(returned, 'n')
 
+    @expected_failure_if(TRAVIS)
     def testInputChoiceNonCapital(self):
         newstdin.write('n\n')
         newstdin.seek(0)
@@ -282,6 +296,7 @@
         self.assertEqual(newstderr.getvalue(), self.input_choice_output)
         self.assertEqual(returned, 'n')
 
+    @expected_failure_if(TRAVIS)
     def testInputChoiceIncorrectAnswer(self):
         newstdin.write('X\nN\n')
         newstdin.seek(0)
@@ -292,6 +307,7 @@
 
 
 @unittest.skipUnless(os.name == 'posix', 'requires Unix console')
+@expected_failure_if(TRAVIS)
 class TestTerminalOutputColorUnix(UITestCase):
 
     """Terminal output color tests."""
@@ -329,6 +345,7 @@
 
 
 @unittest.skipUnless(os.name == 'posix', 'requires Unix console')
+@expected_failure_if(TRAVIS)
 class TestTerminalUnicodeUnix(UITestCase):
 
     """Terminal output tests for unix."""
@@ -353,6 +370,7 @@
 
 
 @unittest.skipUnless(os.name == 'posix', 'requires Unix console')
+@expected_failure_if(TRAVIS)
 class TestTransliterationUnix(UITestCase):
 
     """Terminal output transliteration tests."""

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