chore: flake8 pre commit (#345)

Add flake8 to pre-commit checks

Author: sgoral-splunk

.pre-commit-config.yaml

@@ -13,3 +13,7 @@ repos:
     hooks:
       - id: docformatter
         args: [--in-place]
+-   repo: https://github.com/PyCQA/flake8
+    rev: 5.0.4
+    hooks:
+    -   id: flake8

tests/integration/_search.py

@@ -13,15 +13,16 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
+import context
 import os.path as op
 import sys
 import time
 
-sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
-import context
 from splunklib import client
 from splunklib import results as splunklib_results
 
+sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
+
 
 def search(session_key, query):
     service = client.connect(host=context.host, token=session_key)

tests/integration/data/solnlib_demo/bin/solnlib_demo_collector.py

@@ -2,7 +2,7 @@
 import uuid
 
 from solnlib import log
-from solnlib.modular_input import *
+from solnlib.modular_input import ModularInput, Argument
 
 # Set log context
 log.Logs.set_context(namespace="solnlib_demo", root_logger_log_file="collector")

tests/integration/test__kvstore.py

@@ -14,6 +14,7 @@
 # limitations under the License.
 #
 
+import context
 import json
 import os.path as op
 import sys
@@ -22,11 +23,11 @@
 
 import pytest
 
-sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
-import context
 from splunklib import binding, client
 from splunklib.binding import HTTPError
 
+sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
+
 
 def test_kvstore():
     session_key = context.get_session_key()

tests/integration/test_acl.py

@@ -14,13 +14,12 @@
 # limitations under the License.
 #
 
+import context
 import os.path as op
 import sys
+from solnlib import acl
 
 sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
-import context
-
-from solnlib import acl
 
 
 def test_acl_manager():

tests/integration/test_conf_manager.py

@@ -14,16 +14,15 @@
 # limitations under the License.
 #
 
+import context
 import os.path as op
 import sys
+import pytest
+from solnlib import conf_manager
 from unittest import mock
 
-import pytest
 
 sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
-import context
-
-from solnlib import conf_manager
 
 
 def _build_conf_manager(session_key: str) -> conf_manager.ConfManager:

tests/integration/test_credentials.py

@@ -14,16 +14,14 @@
 # limitations under the License.
 #
 
+import context
 import os.path as op
 import sys
 from typing import Optional
-
 import pytest
+from solnlib import credentials
 
 sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
-import context
-
-from solnlib import credentials
 
 
 def _build_credential_manager(

tests/integration/test_hec_config.py

@@ -14,14 +14,14 @@
 # limitations under the License.
 #
 
+import context
 import os.path as op
 import sys
 
-sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
-import context
-
 from solnlib import hec_config
 
+sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
+
 
 def test_hec_config():
     session_key = context.get_session_key()

tests/integration/test_hec_event_writer.py

@@ -13,16 +13,17 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
+import context
 import os.path as op
 import sys
 import time
 
-sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
-import context
 from _search import search
 
 from solnlib.modular_input import event_writer as hew
 
+sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
+
 
 def test_hec_event_writer():
     session_key = context.get_session_key()

tests/integration/test_logger.py

@@ -13,13 +13,13 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
+import context
 import os.path as op
 import sys
 import time
+from _search import search
 
 sys.path.insert(0, op.dirname(op.dirname(op.abspath(__file__))))
-import context
-from _search import search
 
 
 def test_CVE_2023_32712():