From a173270ef1c98072eeafe6cfcc4699c1dfcab5bf Mon Sep 17 00:00:00 2001 From: Calen Pennington Date: Thu, 9 Jun 2016 16:25:55 -0400 Subject: [PATCH] Fix pylint-identified issues in i18n_tests --- common/lib/i18n_tests/test_extract_and_generate.py | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/common/lib/i18n_tests/test_extract_and_generate.py b/common/lib/i18n_tests/test_extract_and_generate.py index 96a73bf736..6766b9ef55 100644 --- a/common/lib/i18n_tests/test_extract_and_generate.py +++ b/common/lib/i18n_tests/test_extract_and_generate.py @@ -28,6 +28,8 @@ class TestGenerate(TestCase): @classmethod def setUpClass(cls): + super(TestGenerate, cls).setUpClass() + sys.stderr.write( "\nThis test tests that i18n extraction (`paver i18n_extract`) works properly. " "If you experience failures, please check that all instances of `gettext` and " @@ -53,6 +55,8 @@ class TestGenerate(TestCase): super(TestGenerate, cls).tearDownClass() def setUp(self): + super(TestGenerate, self).setUp() + # Subtract 1 second to help comparisons with file-modify time succeed, # since os.path.getmtime() is not millisecond-accurate self.start_time = datetime.now(UTC) - timedelta(seconds=1) @@ -83,8 +87,9 @@ class TestGenerate(TestCase): path = os.path.join(CONFIGURATION.get_messages_dir(locale), mofile) exists = os.path.exists(path) self.assertTrue(exists, msg='Missing file in locale %s: %s' % (locale, mofile)) - self.assertTrue( - datetime.fromtimestamp(os.path.getmtime(path), UTC) >= self.start_time, + self.assertGreaterEqual( + datetime.fromtimestamp(os.path.getmtime(path), UTC), + self.start_time, msg='File not recently modified: %s' % path ) # Segmenting means that the merge headers don't work they way they