Skip to content

gh-93334: Fix homonym edge case in PathFinder.find_spec() #98100

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
brettcannon merged 14 commits into from
Sep 5, 2025
Merged

gh-93334: Fix homonym edge case in PathFinder.find_spec() #98100

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
17ddcd0
gh-93334: Fix homonym edge case in PathFinder.find_spec
jacobtylerwalls Oct 8, 2022
7ee7ebe
fixup! Fix homonym edge case
jacobtylerwalls Oct 8, 2022
ef08ec0
Merge branch 'main' into homonym-find-spec
jacobtylerwalls Apr 8, 2023
c525fb0
Merge branch 'main' into homonym-find-spec
jacobtylerwalls Feb 24, 2024
8092f01
Update expected result
jacobtylerwalls Feb 24, 2024
d0f5a82
Fix expected result
jacobtylerwalls Feb 24, 2024
78a1232
fix failing test
jacobtylerwalls Feb 24, 2024
efb61c1
reuse string
jacobtylerwalls Feb 24, 2024
1369c60
Merge branch 'main' into homonym-find-spec
jacobtylerwalls Jul 13, 2024
7b77fac
Raise ModuleNotFoundError, update news and tests
jacobtylerwalls Jul 13, 2024
1d59e5c
Let build know about new test package
jacobtylerwalls Jul 13, 2024
3910d92
Merge branch 'main' into homonym-find-spec
brettcannon Sep 13, 2024
fd24f87
Address review
jacobtylerwalls Sep 15, 2024
5a8fafb
Merge branch 'main' into homonym-find-spec
brettcannon Sep 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Address review
  • Loading branch information
@jacobtylerwalls
jacobtylerwalls committed Sep 15, 2024
commit fd24f8794c3fdd5b38f3e8934e1dec9c9889b52f
5 changes: 3 additions & 2 deletions Lib/importlib/_bootstrap_external.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1109,10 +1109,11 @@ def _get_parent_path(self):
module = sys.modules[parent_module_name]
except KeyError as e:
raise ModuleNotFoundError(
f"{parent_module_name} must be imported before finding {self._name}.",
f"{parent_module_name!r} must be imported before finding {self._name!r}.",
name=parent_module_name,
) from e
return getattr(module, path_attr_name)
else:
return getattr(module, path_attr_name)

def _recalculate(self):
# If the parent's path has changed, recalculate _path
Expand Down
5 changes: 3 additions & 2 deletions Lib/test/test_importlib/test_namespace_pkgs.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -323,10 +323,11 @@ class NamespaceSubpackageSameName(NamespacePackageTest):

def test_namespace_subpackage_shares_name_with_directory(self):
submodule_path = 'project4.foo'
msg = 'project4 must be imported before finding project4.foo.'
with self.assertRaisesRegex(ModuleNotFoundError, msg):
with self.assertRaises(ModuleNotFoundError) as cm:
importlib.machinery.PathFinder.find_spec(submodule_path)

self.assertEqual(cm.exception.name, 'project4')


class ReloadTests(NamespacePackageTest):
paths = ['portion1']
Expand Down