From 04f3e1ba03b619dcd13d63d0bd0eab192191d6e1 Mon Sep 17 00:00:00 2001
From: peterfarrell <peter_farrell@saa.senate.gov>
Date: Mon, 19 Aug 2019 15:05:15 -0500
Subject: [PATCH] Fixed #6875 -- Made OpenAPI Schema operationId casing
 consistent.

---
 rest_framework/schemas/openapi.py | 9 ++++++---
 tests/schemas/test_openapi.py     | 4 ++--
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/rest_framework/schemas/openapi.py b/rest_framework/schemas/openapi.py
index 343d0b6247..dbd99587f8 100644
--- a/rest_framework/schemas/openapi.py
+++ b/rest_framework/schemas/openapi.py
@@ -111,7 +111,7 @@ def _get_operation_id(self, path, method):
         """
         method_name = getattr(self.view, 'action', method.lower())
         if is_list_view(path, method, self.view):
-            action = 'List'
+            action = 'list'
         elif method_name not in self.method_mapping:
             action = method_name
         else:
@@ -135,10 +135,13 @@ def _get_operation_id(self, path, method):
                 name = name[:-7]
             elif name.endswith('View'):
                 name = name[:-4]
-            if name.endswith(action):  # ListView, UpdateAPIView, ThingDelete ...
+
+            # Due to camel-casing of classes and `action` being lowercase, apply title in order to find if action truly
+            # comes at the end of the name
+            if name.endswith(action.title()):  # ListView, UpdateAPIView, ThingDelete ...
                 name = name[:-len(action)]
 
-        if action == 'List' and not name.endswith('s'):  # ListThings instead of ListThing
+        if action == 'list' and not name.endswith('s'):  # listThings instead of listThing
             name += 's'
 
         return action + name
diff --git a/tests/schemas/test_openapi.py b/tests/schemas/test_openapi.py
index e0fe3c9abe..9228dfb27e 100644
--- a/tests/schemas/test_openapi.py
+++ b/tests/schemas/test_openapi.py
@@ -80,7 +80,7 @@ def test_path_without_parameters(self):
 
         operation = inspector.get_operation(path, method)
         assert operation == {
-            'operationId': 'ListExamples',
+            'operationId': 'listExamples',
             'parameters': [],
             'responses': {
                 '200': {
@@ -402,7 +402,7 @@ def test_operation_id_generation(self):
         inspector.view = view
 
         operationId = inspector._get_operation_id(path, method)
-        assert operationId == 'ListExamples'
+        assert operationId == 'listExamples'
 
     def test_repeat_operation_ids(self):
         router = routers.SimpleRouter()