Fix F811 rule, move fixtures to conftest.py

Moved fixtures from test_utils module to a special conftest.py file that
is grabbed automatically by pytest. This fixes the F811 Flake8 rule
complaining about test function argument names clashing with the pytest
fixture function name.

conftest.py

@@ -0,0 +1,22 @@
+from pytest import fixture
+
+from app import create_app, db
+from test_utils import rename_host_table_and_indexes
+
+
+@fixture
+def flask_app_fixture():
+    rename_host_table_and_indexes()
+
+    app = create_app(config_name="testing")
+
+    # binds the app to the current context
+    with app.app_context() as ctx:
+        # create all tables
+        db.create_all()
+        ctx.push()
+        yield app
+        ctx.pop
+
+        db.session.remove()
+        db.drop_all()

test_db_model.py

@@ -2,7 +2,6 @@
 
 from app import db
 from app.models import Host
-from test_utils import flask_app_fixture
 
 """
 These tests are for testing the db model classes outside of the api.

test_host_dedup_logic.py

@@ -4,7 +4,7 @@
 from app import db
 from app.models import Host
 from pytest import mark
-from test_utils import flask_app_fixture
+
 
 ACCOUNT_NUMBER = "000102"
 

test_utils.py

@@ -1,9 +1,7 @@
 import contextlib
 import os
-import pytest
 import unittest.mock
 
-from app import create_app, db
 from app.models import Host
 
 
@@ -33,21 +31,3 @@ def rename_host_table_and_indexes():
     for index in Host.__table_args__:
         if temp_table_name_suffix not in index.name:
             index.name = index.name + temp_table_name_suffix
-
-
-@pytest.fixture
-def flask_app_fixture():
-    rename_host_table_and_indexes()
-
-    app = create_app(config_name="testing")
-
-    # binds the app to the current context
-    with app.app_context() as ctx:
-        # create all tables
-        db.create_all()
-        ctx.push()
-        yield app
-        ctx.pop
-
-        db.session.remove()
-        db.drop_all()