Use TypeError instead of AssertionError

Author: agriyakhetarpal

distutils/extension.py

@@ -107,19 +107,19 @@ def __init__(
         **kw,  # To catch unknown keywords
     ):
         if not isinstance(name, str):
-            raise AssertionError("'name' must be a string")  # noqa: TRY004
+            raise TypeError("'name' must be a string")  # noqa: TRY004
 
         # handle the string case first; since strings are iterable, disallow them
         if isinstance(sources, str):
-            raise AssertionError(  # noqa: TRY004
+            raise TypeError(
                 "'sources' must be an iterable of strings or PathLike objects, not a string"
             )
 
         # now we check if it's iterable and contains valid types
         try:
             self.sources = list(map(os.fspath, sources))
         except TypeError:
-            raise AssertionError(
+            raise TypeError(
                 "'sources' must be an iterable of strings or PathLike objects"
             )
 

distutils/tests/test_extension.py

@@ -63,16 +63,16 @@ def test_read_setup_file(self):
 
     def test_extension_init(self):
         # the first argument, which is the name, must be a string
-        with pytest.raises(AssertionError):
+        with pytest.raises(TypeError):
             Extension(1, [])
         ext = Extension('name', [])
         assert ext.name == 'name'
 
         # the second argument, which is the list of files, must
         # be an iterable of strings or PathLike objects, and not a string
-        with pytest.raises(AssertionError):
+        with pytest.raises(TypeError):
             Extension('name', 'file')
-        with pytest.raises(AssertionError):
+        with pytest.raises(TypeError):
             Extension('name', ['file', 1])
         ext = Extension('name', ['file1', 'file2'])
         assert ext.sources == ['file1', 'file2']