rename mixin classes to better match their responsibility

Author: picnixz

Lib/test/test_hmac.py

@@ -128,7 +128,7 @@ def hmac_digest(self, key, msg=None, digestmod=None):
         return _hashlib.hmac_digest(key, msg, digest=digestmod)
 
 
-class CheckerMixin:
+class ObjectCheckerMixin:
     """Mixin for checking HMAC objects (pure Python, OpenSSL or built-in)."""
 
     def check_object(self, h, hexdigest, hashname, digest_size, block_size):
@@ -149,7 +149,7 @@ def check_hexdigest(self, h, hexdigest, digest_size):
         self.assertEqual(h.hexdigest().upper(), hexdigest.upper())
 
 
-class TestVectorsMixin(CreatorMixin, DigestMixin, CheckerMixin):
+class AssertersMixin(CreatorMixin, DigestMixin, ObjectCheckerMixin):
     """Mixin class for common tests."""
 
     def hmac_new_by_name(self, key, msg=None, *, hashname):
@@ -308,7 +308,7 @@ def assert_hmac_extra_cases(
         self.check_object(h1, hexdigest, hashname, digest_size, block_size)
 
 
-class PyTestVectorsMixin(PyModuleMixin, TestVectorsMixin):
+class PyAssertersMixin(PyModuleMixin, AssertersMixin):
 
     def assert_hmac_extra_cases(
         self, key, msg, hexdigest, digestmod, hashname, digest_size, block_size
@@ -322,7 +322,7 @@ def assert_hmac_extra_cases(
         self.check_object(h, hexdigest, hashname, digest_size, block_size)
 
 
-class OpenSSLTestVectorsMixin(ThroughOpenSSLAPIMixin, TestVectorsMixin):
+class OpenSSLAssertersMixin(ThroughOpenSSLAPIMixin, AssertersMixin):
 
     def hmac_new_by_name(self, key, msg=None, *, hashname):
         self.assertIsInstance(hashname, str)
@@ -376,7 +376,7 @@ def setUpClass(cls):
             setattr(cls, name, name)
 
 
-class RFCTestCasesMixin(HashFunctionsTrait):
+class RFCTestCaseMixin(HashFunctionsTrait):
     """Test HMAC implementations against test vectors from the RFC."""
 
     def test_md5(self):
@@ -458,7 +458,6 @@ def test_sha2_512_rfc4231(self):
         self._test_sha2_rfc4231(self.sha512, 'sha512', 64, 128)
 
     def _test_sha2_rfc4231(self, hashfunc, hashname, digest_size, block_size):
-
         def hmactest(key, data, hexdigests):
             hexdigest = hexdigests[hashname]
 
@@ -578,34 +577,33 @@ def hmactest(key, data, hexdigests):
                  })
 
 
-class PyRFCTestCase(ThroughObjectMixin, PyTestVectorsMixin,
-                    WithOpenSSLHashFunctions, RFCTestCasesMixin,
+class PyRFCTestCase(ThroughObjectMixin, PyAssertersMixin,
+                    WithOpenSSLHashFunctions, RFCTestCaseMixin,
                     unittest.TestCase):
     """Python implementation of HMAC using hmac.HMAC().
 
     The underlying hash functions are OpenSSL-based.
     """
 
 
-class PyDotNewRFCTestCase(ThroughModuleAPIMixin, PyTestVectorsMixin,
-                          WithOpenSSLHashFunctions, RFCTestCasesMixin,
+class PyDotNewRFCTestCase(ThroughModuleAPIMixin, PyAssertersMixin,
+                          WithOpenSSLHashFunctions, RFCTestCaseMixin,
                           unittest.TestCase):
     """Python implementation of HMAC using hmac.new().
 
     The underlying hash functions are OpenSSL-based.
     """
 
 
-class OpenSSLRFCTestCase(OpenSSLTestVectorsMixin,
-                         WithOpenSSLHashFunctions, RFCTestCasesMixin,
+class OpenSSLRFCTestCase(OpenSSLAssertersMixin,
+                         WithOpenSSLHashFunctions, RFCTestCaseMixin,
                          unittest.TestCase):
     """OpenSSL implementation of HMAC.
 
     The underlying hash functions are also OpenSSL-based.
     """
 
 
-
 # TODO(picnixz): once we have a HACL* HMAC, we should also test the Python
 # implementation of HMAC with a HACL*-based hash function. For now, we only
 # test it partially via the '_sha2' module, but for completeness we could
@@ -668,7 +666,7 @@ def _invalid_digestmod_cases(self, func, key, msg, choices):
         return cases
 
 
-class ConstructorTestCaseMixin(CreatorMixin, DigestMixin, CheckerMixin):
+class ConstructorTestCaseMixin(CreatorMixin, DigestMixin, ObjectCheckerMixin):
     """HMAC constructor tests based on HMAC-SHA-2/256."""
 
     key = b"key"