Fix false negative for deprecated-module when a __import__ method is used instead of import sentence. (#10454)

Using the following python code style

```python
__import__("deprecated_module")
```

The deprecated-module is ignored since that it expects

```python
import deprecated_module
```

This change adds the posibility to process these kind of style too

This style is used from popular auto-code snippets:
 - https://github.com/honza/vim-snippets/blob/f0a3184/snippets/python.snippets#L209

Fix #10453

Author: moylop260

CONTRIBUTORS.txt

@@ -91,6 +91,7 @@ contributors:
   * Fix consider-using-ternary for 'True and True and True or True' case
   * Add bad-docstring-quotes and docstring-first-line-empty
   * Add missing-timeout
+  * Fix false negative for `deprecated-module` when a `__import__` method is used instead of `import` sentence
 - Frank Harrison <[email protected]> (doublethefish)
 - Zen Lee <[email protected]>
 - Pierre-Yves David <[email protected]>

doc/whatsnew/fragments/10453.false_negative

@@ -0,0 +1,3 @@
+Fix false negative for `deprecated-module` when a `__import__` method is used instead of `import` sentence.
+
+Refs #10453

pylint/checkers/deprecated.py

@@ -97,6 +97,7 @@ def visit_attribute(self, node: astroid.Attribute) -> None:
         "deprecated-method",
         "deprecated-argument",
         "deprecated-class",
+        "deprecated-module",
     )
     def visit_call(self, node: nodes.Call) -> None:
         """Called when a :class:`nodes.Call` node is visited."""
@@ -105,6 +106,15 @@ def visit_call(self, node: nodes.Call) -> None:
             # Calling entry point for deprecation check logic.
             self.check_deprecated_method(node, inferred)
 
+            if (
+                isinstance(inferred, nodes.FunctionDef)
+                and inferred.qname() == "builtins.__import__"
+                and len(node.args) == 1
+                and (mod_path_node := utils.safe_infer(node.args[0]))
+                and isinstance(mod_path_node, astroid.Const)
+            ):
+                self.check_deprecated_module(node, mod_path_node.value)
+
     @utils.only_required_for_messages(
         "deprecated-module",
         "deprecated-class",

script/.contributors_aliases.json

@@ -109,6 +109,10 @@
     "mails": ["[email protected]"],
     "name": "Hayden Richards"
   },
+  "[email protected]": {
+    "mails": ["[email protected]", "[email protected]"],
+    "name": "Moisés Lopez"
+  },
   "[email protected]": {
     "comment": ": Fix for invalid toml config",
     "mails": ["[email protected]"],

tests/functional/d/deprecated/deprecated_module__import__.py

@@ -0,0 +1,9 @@
+"""Test deprecated modules using '__import__' builtin method."""
+# pylint: disable=invalid-name
+
+imp = __import__("mypackage")  # [deprecated-module]
+
+imp_meth = __import__("mypackage").meth()  # [deprecated-module]
+
+lib = "mypackage"
+infer_imp = __import__(lib)  # [deprecated-module]

tests/functional/d/deprecated/deprecated_module__import__.rc

@@ -0,0 +1,2 @@
+[Imports]
+deprecated-modules=mypackage

tests/functional/d/deprecated/deprecated_module__import__.txt

@@ -0,0 +1,3 @@
+deprecated-module:4:6:4:29::Deprecated module 'mypackage':UNDEFINED
+deprecated-module:6:11:6:34::Deprecated module 'mypackage':UNDEFINED
+deprecated-module:9:12:9:27::Deprecated module 'mypackage':UNDEFINED