lint, etc

src/fides/api/alembic/migrations/versions/10c6b7709be3_adding_foreign_key_on_fides_key_for_.py

@@ -6,9 +6,8 @@
 
 """
 
-from alembic import op
 import sqlalchemy as sa
-
+from alembic import op
 
 # revision identifiers, used by Alembic.
 revision = "10c6b7709be3"

src/fides/api/alembic/migrations/versions/ae65da77c468_add_data_migration_to_reactivate_.py

@@ -8,11 +8,9 @@
 
 import sqlalchemy as sa
 from alembic import op
-
 from sqlalchemy import text
 from sqlalchemy.engine import Connection
 
-
 # revision identifiers, used by Alembic.
 revision = "ae65da77c468"
 down_revision = "10c6b7709be3"

src/fides/api/api/v1/endpoints/generic_overrides.py

@@ -9,7 +9,7 @@
 from sqlalchemy.orm import Session
 from sqlalchemy.sql.expression import select
 from starlette import status
-from starlette.status import HTTP_422_UNPROCESSABLE_ENTITY, HTTP_404_NOT_FOUND
+from starlette.status import HTTP_404_NOT_FOUND, HTTP_422_UNPROCESSABLE_ENTITY
 
 from fides.api.api.deps import get_db
 from fides.api.common_exceptions import KeyOrNameAlreadyExists
@@ -28,15 +28,15 @@
     DataUse,
     DataUseCreateOrUpdate,
 )
-from fides.api.util.errors import ForbiddenIsDefaultTaxonomyError
+from fides.api.util.errors import FidesError, ForbiddenIsDefaultTaxonomyError
 from fides.api.util.filter_utils import apply_filters_to_query
 from fides.common.api.scope_registry import (
     DATA_CATEGORY_CREATE,
+    DATA_CATEGORY_UPDATE,
     DATA_SUBJECT_CREATE,
     DATA_USE_CREATE,
-    DATASET_READ,
     DATA_USE_UPDATE,
-    DATA_CATEGORY_UPDATE,
+    DATASET_READ,
 )
 from fides.common.api.v1.urn_registry import V1_URL_PREFIX
 
@@ -162,6 +162,10 @@ def validate_and_create_taxonomy(
     """
     Validate and create a taxonomy element.
     """
+    if not data.fides_key:
+        raise FidesError(
+            f"Fides key is required to create a {model.__name__} resource"
+        )
     if isinstance(model, ModelWithDefaultField) and data.is_default:
         raise ForbiddenIsDefaultTaxonomyError(
             model.__name__, data.fides_key, action="create"
@@ -181,6 +185,10 @@ def validate_and_update_taxonomy(
     """
     Validate and update a taxonomy element.
     """
+    if not data.fides_key:
+        raise FidesError(
+            f"Fides key is required to update {resource.__name__}"
+        )
     if isinstance(resource, ModelWithDefaultField) and data.is_default:
         raise ForbiddenIsDefaultTaxonomyError(
             resource.__name__, data.fides_key, action="update"

src/fides/api/models/sql_models.py

@@ -36,7 +36,7 @@
 from sqlalchemy.exc import IntegrityError
 from sqlalchemy.ext.asyncio import AsyncSession, async_object_session
 from sqlalchemy.ext.hybrid import hybrid_property
-from sqlalchemy.orm import Session, relationship
+from sqlalchemy.orm import RelationshipProperty, Session, relationship
 from sqlalchemy.sql import Select
 from sqlalchemy.sql.elements import Case
 from sqlalchemy.sql.sqltypes import DateTime

tests/ctl/core/test_taxonomy_models.py

@@ -3,6 +3,7 @@
 import pytest
 from sqlalchemy.exc import IntegrityError
 from sqlalchemy.orm import Session
+
 from fides.api.models.sql_models import DataUse
 
 

tests/ops/api/v1/endpoints/test_taxonomy_overrides.py

@@ -9,8 +9,8 @@
 from fides.common.api.scope_registry import (
     DATA_USE,
     DATA_USE_CREATE,
-    STORAGE_READ,
     DATA_USE_UPDATE,
+    STORAGE_READ,
 )
 from fides.common.api.v1.urn_registry import V1_URL_PREFIX