switch from deny listing to allow listing

This ensures the linter will catch violations on newly added models.

Author: MatMoore

scripts/lint_model_usage_in_views.py

@@ -1,5 +1,5 @@
 #!/usr/bin/env python3
-"""Simple linter to spot denylisted model usage within Django view modules."""
+"""Check for model usage without provider scoping within Django view modules."""
 
 import argparse
 import re
@@ -9,13 +9,16 @@
 
 REPO_ROOT = Path(__file__).parent.parent
 
-DENYLISTED_MODELS = (
-    "Appointment",
-    "Clinic",
-    "Participant",
-    "Symptom",
-    "BreastCancerHistoryItem",
+ALLOWLISTED_MODELS = (
+    "User",
+    "Provider",
+    "BreastAugmentationHistoryItem",  # Temporarily allowed
+    "CystHistoryItem",  # Temporarily allowed
+    "ImplantedMedicalDeviceHistoryItem",  # Temporarily allowed
+    "MastectomyOrLumpectomyHistoryItem",  # Temporarily allowed
+    "ParticipantReportedMammogram",  # Temporarily allowed
 )
+
 TARGETS = {
     "model.objects": re.compile(r"(?P<model_name>\w+)\.objects"),
     "model.get_object_or_404": re.compile(r"(?P<model_name>\w+)\.get_object_or_404"),
@@ -75,7 +78,7 @@ def find_matches(paths):
             for label, regex in TARGETS.items():
                 if match := regex.search(line):
                     model_name = match.group("model_name")
-                    if model_name in DENYLISTED_MODELS:
+                    if model_name not in ALLOWLISTED_MODELS:
                         yield Match(
                             path=path,
                             line_number=line_number,