fix: Skip star imports when the --include-star-import flag is not used.

CHANGELOG.md

@@ -4,6 +4,8 @@ All notable changes to this project will be documented in this file.
 
 ## Unreleased
 
+- [🐞fix: Skip star imports when the `--include-star-import` flag is not used](https://github.com/hakancelik96/unimport/pull/60)
+
 - [🐞fix: finding functions during scanning](https://github.com/hakancelik96/unimport/pull/55)
 
 - [💪 `#noqa` comment support to skip import (#48) by @hakancelik96](https://github.com/hakancelik96/unimport/pull/54)

tests/test_refactor.py

@@ -193,6 +193,12 @@ def test_comment(self):
             expected, self.session.refactor(action),
         )
 
+    def test_star(self):
+        action = "from os import *\n" "walk\n"
+        self.assertEqual(
+            action, self.session.refactor(action),
+        )
+
 
 class TestDuplicateUnusedRefactor(RefactorTestCase):
     def test_full_unused(self):

unimport/scan.py

@@ -56,29 +56,31 @@ def visit_FunctionDef(self, node):
     def visit_Import(self, node, module_name=None):
         if self.skip_import(node):
             return
-        star = False
         module = None
         for alias in node.names:
-            name = alias.asname or alias.name
             package = module_name or alias.name
-            if (
-                package not in self.ignore_imports
-                and name not in self.ignore_import_names
+            alias_name = alias.asname or alias.name
+            star = True if alias_name == "*" else False
+            name = package if star else alias_name
+            is_package_or_name_ignore = (
+                package in self.ignore_imports
+                or name in self.ignore_import_names
+            )
+            if is_package_or_name_ignore or (
+                star and not self.include_star_import
             ):
-                if name == "*":
-                    star = True
-                    name = package
-                with contextlib.suppress(ImportError, ValueError):
-                    module = importlib.import_module(package)
-                self.imports.append(
-                    {
-                        "lineno": node.lineno,
-                        "name": name,
-                        "star": star,
-                        "module": module,
-                        "modules": [],
-                    }
-                )
+                return
+            with contextlib.suppress(ImportError, ValueError):
+                module = importlib.import_module(package)
+            self.imports.append(
+                {
+                    "lineno": node.lineno,
+                    "name": name,
+                    "star": star,
+                    "module": module,
+                    "modules": [],
+                }
+            )
 
     @recursive
     def visit_ImportFrom(self, node):