Refactor: Resolve Flake8 F401 and F403 and F405

It's bad practice to use * imports (F405) since the methods or
variables being imported could be undefined. Explicitly importing
makes this more clear. Also F403 violation makes it difficult to
detect undefined names.

Additionally F401 violation to allow an indirect module import makes
things harder to debug. It's better to be explicit.

https://www.flake8rules.com/rules/F401.html
https://www.flake8rules.com/rules/F403.html
https://www.flake8rules.com/rules/F405.html

Change-Id: Ib938f865d3ac6456216a119c830d3b013932bf88
Signed-off-by: Thanh Ha <[email protected]>

lftools/cli/nexus2/__init__.py

@@ -13,12 +13,14 @@
 __author__ = "DW Talton"
 
 
+import click
+
 from lftools.api.endpoints import nexus2
 
-from .privilege import *
-from .repository import *
-from .role import *
-from .user import *
+from .privilege import privilege
+from .repository import repo
+from .role import role
+from .user import user
 
 
 @click.group(name="nexus2")

lftools/cli/nexus3/__init__.py

@@ -12,14 +12,18 @@
 
 __author__ = "DW Talton"
 
-from .asset import *
-from .privilege import *
-from .repository import *
-from .role import *
-from .script import *
-from .tag import *
-from .task import *
-from .user import *
+import click
+
+from lftools.api.endpoints import nexus3
+
+from .asset import asset
+from .privilege import privilege
+from .repository import repository
+from .role import role
+from .script import script
+from .tag import tag
+from .task import task
+from .user import user
 
 
 @click.group(name="nexus3")

lftools/cli/nexus3/repository.py

@@ -17,8 +17,6 @@
 
 import click
 
-from lftools.api.endpoints import nexus3  # noqa: F401
-
 log = logging.getLogger(__name__)
 
 

lftools/cli/nexus3/role.py

@@ -18,8 +18,6 @@
 import click
 from tabulate import tabulate
 
-from lftools.api.endpoints import nexus3  # noqa: F401
-
 log = logging.getLogger(__name__)
 
 

lftools/cli/nexus3/script.py

@@ -16,8 +16,6 @@
 
 import click
 
-from lftools.api.endpoints import nexus3  # noqa: F401
-
 log = logging.getLogger(__name__)
 
 

lftools/cli/nexus3/task.py

@@ -17,8 +17,6 @@
 import click
 from tabulate import tabulate
 
-from lftools.api.endpoints import nexus3  # noqa: F401
-
 log = logging.getLogger(__name__)
 
 

lftools/cli/nexus3/user.py

@@ -17,8 +17,6 @@
 import click
 from tabulate import tabulate
 
-from lftools.api.endpoints import nexus3  # noqa: F401
-
 log = logging.getLogger(__name__)