bpo-25172: Reduce scope of crypt import tests (GH-17881)

(cherry picked from commit ed367815ee)

Co-authored-by: Steve Dower <steve.dower@python.org>
This commit is contained in:
Miss Islington (bot) 2020-01-09 09:19:59 -08:00 committed by GitHub
parent 20c990229e
commit 8c08518c25
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 4 deletions

View File

@ -6,20 +6,21 @@ try:
import crypt import crypt
IMPORT_ERROR = None IMPORT_ERROR = None
except ImportError as ex: except ImportError as ex:
if sys.platform != 'win32':
raise unittest.SkipTest(str(ex))
crypt = None crypt = None
IMPORT_ERROR = str(ex) IMPORT_ERROR = str(ex)
@unittest.skipIf(crypt, 'This should only run on windows') @unittest.skipUnless(sys.platform == 'win32', 'This should only run on windows')
@unittest.skipIf(crypt, 'import succeeded')
class TestWhyCryptDidNotImport(unittest.TestCase): class TestWhyCryptDidNotImport(unittest.TestCase):
def test_failure_only_for_windows(self):
self.assertEqual(sys.platform, 'win32')
def test_import_failure_message(self): def test_import_failure_message(self):
self.assertIn('not supported', IMPORT_ERROR) self.assertIn('not supported', IMPORT_ERROR)
@unittest.skipUnless(crypt, 'Not supported on Windows') @unittest.skipUnless(crypt, 'crypt module is required')
class CryptTestCase(unittest.TestCase): class CryptTestCase(unittest.TestCase):
def test_crypt(self): def test_crypt(self):