Raise ModuleNotFoundError instead of returning None

Author: dr-carlos

Lib/test/test_import/__init__.py

@@ -1254,17 +1254,19 @@ class Spec2:
             _imp.create_dynamic(Spec2())
 
     def test_create_builtin(self):
-        for internal_mod in (sys, builtins):
-            class Spec:
-                name = internal_mod.__name__
+        class Spec:
+            name = "sys"
 
-            self.assertIs(_imp.create_builtin(Spec()), internal_mod)
+        spec = Spec()
+        self.assertIs(_imp.create_builtin(spec), sys)
 
-        class Spec:
-            name = "nonexistent_lib"
+        spec.name = "builtins"
+        self.assertIs(_imp.create_builtin(spec), builtins)
 
         # gh-142029
-        self.assertIs(_imp.create_builtin(Spec()), None)
+        spec.name = "nonexistent_lib"
+        with self.assertRaises(ModuleNotFoundError):
+            _imp.create_builtin(spec)
 
     def test_filter_syntax_warnings_by_module(self):
         module_re = r'test\.test_import\.data\.syntax_warnings\z'

Python/import.c

@@ -2434,7 +2434,8 @@ create_builtin(
     if (initfunc == NULL) {
         struct _inittab *entry = lookup_inittab_entry(&info);
         if (entry == NULL) {
-            mod = Py_NewRef(Py_None);
+            mod = NULL;
+            _PyErr_SetModuleNotFoundError(name);
             goto finally;
         }