chore(py): Consistently name urls using organization- prefix (#45180)

Author: evanpurkhiser

src/sentry/api/urls.py

@@ -652,7 +652,7 @@
     url(
         r"^accept-invite/(?P<organization_slug>[^\/]+)/(?P<member_id>[^\/]+)/(?P<token>[^\/]+)/$",
         AcceptOrganizationInvite.as_view(),
-        name="sentry-api-0-accept-organization-invite-with-org",
+        name="sentry-api-0-organization-accept-organization-invite",
     ),
     url(
         r"^accept-invite/(?P<member_id>[^\/]+)/(?P<token>[^\/]+)/$",
@@ -695,27 +695,27 @@
                 url(
                     r"^(?P<monitor_id>[^\/]+)/$",
                     MonitorDetailsEndpoint.as_view(),
-                    name="sentry-api-0-monitor-details-with-org",
+                    name="sentry-api-0-organization-monitor-details",
                 ),
                 url(
                     r"^(?P<monitor_id>[^\/]+)/checkins/$",
                     MonitorCheckInsEndpoint.as_view(),
-                    name="sentry-api-0-monitor-check-in-index-with-org",
+                    name="sentry-api-0-organization-monitor-check-in-index",
                 ),
                 url(
                     r"^(?P<monitor_id>[^\/]+)/checkins/(?P<checkin_id>[^\/]+)/$",
                     MonitorCheckInDetailsEndpoint.as_view(),
-                    name="sentry-api-0-monitor-check-in-details-with-org",
+                    name="sentry-api-0-organization-monitor-check-in-details",
                 ),
                 url(
                     r"^(?P<monitor_id>[^\/]+)/checkins/(?P<checkin_id>[^\/]+)/attachment/$",
                     MonitorCheckInAttachmentEndpoint.as_view(),
-                    name="sentry-api-0-monitor-check-in-attachment-with-org",
+                    name="sentry-api-0-organization-monitor-check-in-attachment",
                 ),
                 url(
                     r"^(?P<monitor_id>[^\/]+)/stats/$",
                     MonitorStatsEndpoint.as_view(),
-                    name="sentry-api-0-monitor-stats-with-org",
+                    name="sentry-api-0-organization-monitor-stats",
                 ),
             ]
         ),
@@ -2335,7 +2335,7 @@
     url(
         r"^organizations/(?P<organization_slug>[^\/]+)/issues/(?P<issue_id>[^\/]+)/participants/$",
         GroupParticipantsEndpoint.as_view(),
-        name="sentry-api-0-group-stats-with-org",
+        name="sentry-api-0-organization-group-stats",
     ),
     url(
         r"^issues/(?P<issue_id>[^\/]+)/participants/$",
@@ -2346,7 +2346,7 @@
     url(
         r"^organizations/(?P<organization_slug>[^\/]+)/shared/(?:issues|groups)/(?P<share_id>[^\/]+)/$",
         SharedGroupDetailsEndpoint.as_view(),
-        name="sentry-api-0-shared-group-details-with-org",
+        name="sentry-api-0-organization-shared-group-details",
     ),
     url(
         r"^shared/(?:issues|groups)/(?P<share_id>[^\/]+)/$",

src/sentry/web/urls.py

@@ -366,7 +366,7 @@
     url(
         r"^accept/(?P<organization_slug>[^/]+)/(?P<member_id>\d+)/(?P<token>\w+)/$",
         GenericReactPageView.as_view(auth_required=False),
-        name="sentry-accept-invite-with-org",
+        name="sentry-organization-accept-invite",
     ),
     # User settings use generic_react_page_view, while any view acting on
     # behalf of an organization should use react_page_view

tests/sentry/api/endpoints/test_accept_organization_invite.py

@@ -28,15 +28,15 @@ def _get_paths(self, args):
         return (
             reverse("sentry-api-0-accept-organization-invite", args=args),
             reverse(
-                "sentry-api-0-accept-organization-invite-with-org",
+                "sentry-api-0-organization-accept-organization-invite",
                 args=[self.organization.slug] + args,
             ),
         )
 
     def _get_urls(self):
         return (
             "sentry-api-0-accept-organization-invite",
-            "sentry-api-0-accept-organization-invite-with-org",
+            "sentry-api-0-organization-accept-organization-invite",
         )
 
     def _get_path(self, url, args):
@@ -350,7 +350,7 @@ def test_mismatched_org_slug(self):
         )
 
         path = reverse(
-            "sentry-api-0-accept-organization-invite-with-org", args=["asdf", om.id, om.token]
+            "sentry-api-0-organization-accept-organization-invite", args=["asdf", om.id, om.token]
         )
 
         resp = self.client.get(path)

tests/sentry/api/endpoints/test_group_participants.py

@@ -19,7 +19,7 @@ def _get_path_functions(self):
         return (
             lambda group: reverse("sentry-api-0-group-stats", args=[group.id]),
             lambda group: reverse(
-                "sentry-api-0-group-stats-with-org", args=[self.organization.slug, group.id]
+                "sentry-api-0-organization-group-stats", args=[self.organization.slug, group.id]
             ),
         )
 

tests/sentry/api/endpoints/test_monitor_checkin_attachment.py

@@ -12,7 +12,7 @@
 
 @region_silo_test(stable=True)
 class UploadMonitorCheckInAttachmentTest(APITestCase):
-    endpoint = "sentry-api-0-monitor-check-in-attachment-with-org"
+    endpoint = "sentry-api-0-organization-monitor-check-in-attachment"
 
     def setUp(self):
         super().setUp()

tests/sentry/api/endpoints/test_monitor_checkin_details.py

@@ -11,7 +11,7 @@
 @region_silo_test(stable=True)
 class UpdateMonitorCheckInTest(APITestCase):
     endpoint = "sentry-api-0-monitor-check-in-details"
-    endpoint_with_org = "sentry-api-0-monitor-check-in-details-with-org"
+    endpoint_with_org = "sentry-api-0-organization-monitor-check-in-details"
 
     def setUp(self):
         super().setUp()

tests/sentry/api/endpoints/test_monitor_checkins.py

@@ -16,7 +16,7 @@
 @freeze_time()
 class CreateMonitorCheckInTest(MonitorTestCase):
     endpoint = "sentry-api-0-monitor-check-in-index"
-    endpoint_with_org = "sentry-api-0-monitor-check-in-index-with-org"
+    endpoint_with_org = "sentry-api-0-organization-monitor-check-in-index"
 
     def setUp(self):
         super().setUp()

tests/sentry/api/endpoints/test_monitor_details.py

@@ -6,7 +6,7 @@
 @region_silo_test(stable=True)
 class MonitorDetailsTest(MonitorTestCase):
     endpoint = "sentry-api-0-monitor-details"
-    endpoint_with_org = "sentry-api-0-monitor-details-with-org"
+    endpoint_with_org = "sentry-api-0-organization-monitor-details"
 
     def setUp(self):
         super().setUp()
@@ -44,7 +44,7 @@ def test_invalid_monitor_id(self):
 @region_silo_test(stable=True)
 class UpdateMonitorTest(MonitorTestCase):
     endpoint = "sentry-api-0-monitor-details"
-    endpoint_with_org = "sentry-api-0-monitor-details-with-org"
+    endpoint_with_org = "sentry-api-0-organization-monitor-details"
 
     def setUp(self):
         super().setUp()
@@ -281,7 +281,7 @@ def test_cannot_change_project(self):
 @region_silo_test()
 class DeleteMonitorTest(MonitorTestCase):
     endpoint = "sentry-api-0-monitor-details"
-    endpoint_with_org = "sentry-api-0-monitor-details-with-org"
+    endpoint_with_org = "sentry-api-0-organization-monitor-details"
 
     def setUp(self):
         super().setUp()