[issue30202] Update test.test_importlib.test_abc to test find_spec()

Change by Brett Cannon : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue30202] Update test.test_importlib.test_abc to test find_spec()

Brett Cannon added the comment: New changeset a0d73a143af404deecb9c4fcdbd3ddbafd96b41b by Brett Cannon (Joannah Nanjekye) in branch 'master': bpo-30202 : Update test.test_importlib.test_abc to test find_spec() (GH-12847) https://github.com/python/cpython/commit/a0d73a143af404deecb9c4fcdbd3ddb

[issue30202] Update test.test_importlib.test_abc to test find_spec()

Change by Joannah Nanjekye : -- stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscr

[issue30202] Update test.test_importlib.test_abc to test find_spec()

Joannah Nanjekye added the comment: Since it was almost 2 years since the last conversation, i opened a PR for this. @brett.cannon I may have not looked at all possible areas to use find_spec(), so please let me know in the reviews for this PR. -- _

[issue30202] Update test.test_importlib.test_abc to test find_spec()

Change by Joannah Nanjekye : -- nosy: +nanjekyejoannah stage: patch review -> needs patch ___ Python tracker ___ ___ Python-bugs-lis

[issue30202] Update test.test_importlib.test_abc to test find_spec()

Change by Joannah Nanjekye : -- keywords: +patch pull_requests: +12772 stage: needs patch -> patch review ___ Python tracker ___ ___

[issue30202] Update test.test_importlib.test_abc to test find_spec()

Sayan Chowdhury added the comment: I am picking up this issue. -- nosy: +sayanchowdhury ___ Python tracker ___ ___ Python-bugs-list ma

[issue30202] Update test.test_importlib.test_abc to test find_spec()

New submission from Brett Cannon: It looks like test_abc isn't really testing find_spec() very much compared to find_module(). There might also be some tests still using find_module() that should be updated to use find_spec() instead. -- components: Tests messages: 292552 nosy: brett.c