linted

src/registrar/models/user_portfolio_permission.py

@@ -2,7 +2,12 @@ from django.db import models
 from django.forms import ValidationError
 from registrar.models.user_domain_role import UserDomainRole
 from registrar.utility.waffle import flag_is_active_for_user
-from registrar.models.utility.portfolio_helper import UserPortfolioPermissionChoices, UserPortfolioRoleChoices, DomainRequestPermissionDisplay, MemberPermissionDisplay
+from registrar.models.utility.portfolio_helper import (
+    UserPortfolioPermissionChoices,
+    UserPortfolioRoleChoices,
+    DomainRequestPermissionDisplay,
+    MemberPermissionDisplay,
+)
 from .utility.time_stamped_model import TimeStampedModel
 from django.contrib.postgres.fields import ArrayField
 
@@ -115,7 +120,7 @@ class UserPortfolioPermission(TimeStampedModel):
             UserPortfolioPermissionChoices.VIEW_ALL_REQUESTS,
             UserPortfolioPermissionChoices.EDIT_REQUESTS,
         ]
-        
+
         if all(perm in all_permissions for perm in all_domain_perms):
             return DomainRequestPermissionDisplay.VIEWER_REQUESTER
         elif UserPortfolioPermissionChoices.VIEW_ALL_REQUESTS in all_permissions:

src/registrar/models/utility/portfolio_helper.py

@@ -51,6 +51,7 @@ class DomainRequestPermissionDisplay(StrEnum):
     - VIEWER: "Viewer"
     - NONE: "None"
     """
+
     VIEWER_REQUESTER = "Viewer Requester"
     VIEWER = "Viewer"
     NONE = "None"
@@ -64,6 +65,7 @@ class MemberPermissionDisplay(StrEnum):
     - VIEWER: "Viewer"
     - NONE: "None"
     """
+
     MANAGER = "Manager"
     VIEWER = "Viewer"
     NONE = "None"

src/registrar/tests/test_admin_domain.py

@@ -30,7 +30,7 @@ from .common import (
 from unittest.mock import ANY, call, patch
 
 import boto3_mocking  # type: ignore
-import logging
+
 
 class TestDomainAdminAsStaff(MockEppLib):
     """Test DomainAdmin class as staff user.

src/registrar/utility/csv_export.py

@@ -198,8 +198,8 @@ class BaseExport(ABC):
 
         # We can infer that if we're passing in annotations,
         # we want to grab the result of said annotation.
-        if computed_fields :
+        if computed_fields:
-            related_table_fields.extend(computed_fields .keys())
+            related_table_fields.extend(computed_fields.keys())
 
         # Get prexisting fields on the model
         model_fields = set()
@@ -213,26 +213,6 @@ class BaseExport(ABC):
 
         return cls.update_queryset(queryset, **kwargs)
 
-    @classmethod
-    def export_data_to_csv(cls, csv_file, **kwargs):
-        """
-        All domain metadata:
-        Exports domains of all statuses plus domain managers.
-        """
-
-        writer = csv.writer(csv_file)
-        columns = cls.get_columns()
-        models_dict = cls.get_model_annotation_dict(**kwargs)
-
-        # Write to csv file before the write_csv
-        cls.write_csv_before(writer, **kwargs)
-
-        # Write the csv file
-        rows = cls.write_csv(writer, columns, models_dict)
-
-        # Return rows that for easier parsing and testing
-        return rows
-
     @classmethod
     def get_annotated_queryset(cls, **kwargs):
         """Returns an annotated queryset based off of all query conditions."""

src/registrar/utility/enums.py

@@ -49,6 +49,7 @@ class DefaultUserValues(StrEnum):
     - SYSTEM: "System" <= Default username
     - UNRETRIEVED: "Unretrieved" <= Default email state
     """
+
     HELP_EMAIL = "help@get.gov"
     SYSTEM = "System"
     UNRETRIEVED = "Unretrieved"

src/registrar/views/portfolio_members_json.py

@@ -1,7 +1,6 @@
 from django.http import JsonResponse
 from django.core.paginator import Paginator
 from django.db.models import Value, F, CharField, TextField, Q, Case, When, OuterRef, Subquery
-from django.db.models.expressions import Func
 from django.db.models.functions import Cast, Coalesce, Concat
 from django.contrib.postgres.aggregates import ArrayAgg
 from django.urls import reverse
@@ -214,4 +213,3 @@ class PortfolioMembersJson(PortfolioMembersPermission, View):
             "svg_icon": ("visibility" if view_only else "settings"),
         }
         return member_json
-