Merge pull request #10430 from AlanCoding/awxkit_tox

Run flake8 on awxkit

Reviewed-by: Jake McDermott <yo@jakemcdermott.me>

awxkit/awxkit/api/pages/inventory.py

@@ -1,10 +1,9 @@
 from contextlib import suppress
 import logging
 import json
-import re
 
 from awxkit.api.pages import Credential, Organization, Project, UnifiedJob, UnifiedJobTemplate
-from awxkit.utils import filter_by_class, random_title, update_payload, not_provided, PseudoNamespace, poll_until, random_utf8
+from awxkit.utils import filter_by_class, random_title, update_payload, not_provided, PseudoNamespace, poll_until
 from awxkit.api.mixins import DSAdapter, HasCreate, HasInstanceGroups, HasNotifications, HasVariables, HasCopy
 from awxkit.api.resources import resources
 import awxkit.exceptions as exc

awxkit/awxkit/awx/utils.py

@@ -1,6 +1,6 @@
 from contextlib import contextmanager, suppress
 
-from awxkit import api, utils, exceptions
+from awxkit import api, exceptions
 from awxkit.config import config
 
 

awxkit/awxkit/utils/__init__.py

@@ -1,4 +1,3 @@
-from contextlib import contextmanager
 from datetime import datetime, timedelta, tzinfo
 import inspect
 import logging

awxkit/test/cli/test_client.py

@@ -1,5 +1,3 @@
-from io import StringIO
-
 import pytest
 from requests.exceptions import ConnectionError
 

awxkit/test/cli/test_format.py

@@ -4,7 +4,7 @@ import json
 import yaml
 
 from awxkit.api.pages import Page
-from awxkit.api.pages.users import Users, User
+from awxkit.api.pages.users import Users
 from awxkit.cli import CLI
 from awxkit.cli.format import format_response
 from awxkit.cli.resource import Import

awxkit/test/cli/test_options.py

@@ -1,11 +1,7 @@
 import argparse
-import json
 import unittest
 from io import StringIO
 
-import pytest
-from requests import Response
-
 from awxkit.api.pages import Page
 from awxkit.cli.options import ResourceOptionsParser
 

awxkit/test/test_credentials.py

@@ -1,4 +1,3 @@
-from unittest.mock import patch
 import pytest
 
 

awxkit/test/test_dependency_resolver.py

@@ -403,10 +403,7 @@ def test_teardown_dependency_store_cleared():
     g = G().create(f=True, e=True)
     f = g._dependency_store[F]
     e = g._dependency_store[E]
-    b = f._dependency_store[B]
-    d = e._dependency_store[D]
     c = e._dependency_store[C]
-    a = c._dependency_store[A]
 
     g.teardown()
 
@@ -443,10 +440,7 @@ def test_teardown_ds_cleared():
     g = G().create(f=True, e=True)
     f = g._dependency_store[F]
     e = g._dependency_store[E]
-    b = f._dependency_store[B]
-    d = e._dependency_store[D]
     c = e._dependency_store[C]
-    a = c._dependency_store[A]
 
     g.teardown()
 

tox.ini

@@ -9,7 +9,7 @@ setenv =
   BLACK_ARGS = --check
 commands =
   make black
-  flake8 awx
+  flake8 awx awxkit
   yamllint -s .
 
 [flake8]