print in red if user makes duplicated name

Author: vitalik

ninja/openapi/schema.py

@@ -1,9 +1,10 @@
 import itertools
 import re
-import warnings
 from http.client import responses
 from typing import TYPE_CHECKING, Any, Dict, Generator, List, Optional, Set, Tuple
 
+from django.utils.termcolors import make_style
+
 from ninja.constants import NOT_SET
 from ninja.operation import Operation
 from ninja.params.models import TModel, TModels
@@ -28,6 +29,9 @@ def get_schema(api: "NinjaAPI", path_prefix: str = "") -> "OpenAPISchema":
     return openapi
 
 
+bold_red_style = make_style(opts=("bold",), fg="red")
+
+
 class OpenAPISchema(dict):
     def __init__(self, api: "NinjaAPI", path_prefix: str) -> None:
         self.api = api
@@ -103,9 +107,10 @@ def deep_dict_update(
     def operation_details(self, operation: Operation) -> DictStrAny:
         op_id = operation.operation_id or self.api.get_openapi_operation_id(operation)
         if op_id in self.all_operation_ids:
-            warnings.warn(
-                f'operation_id "{op_id}" is already used (func: {operation.view_func})',
-                stacklevel=2,
+            print(
+                bold_red_style(
+                    f'Warning: operation_id "{op_id}" is already used (Try giving a different name to: {operation.view_func.__module__}.{operation.view_func.__name__})'
+                )
             )
         self.all_operation_ids.add(op_id)
         result = {

tests/test_openapi_schema.py

@@ -792,7 +792,7 @@ def test_get_openapi_urls():
         get_openapi_urls(api)
 
 
-def test_unique_operation_ids():
+def test_unique_operation_ids(capsys):
     api = NinjaAPI()
 
     @api.get("/1")
@@ -803,9 +803,9 @@ def same_name(request):
     def same_name(request):  # noqa: F811
         pass
 
-    match = 'operation_id "test_openapi_schema_same_name" is already used'
-    with pytest.warns(UserWarning, match=match):
-        api.get_openapi_schema()
+    api.get_openapi_schema()
+    captured = capsys.readouterr()
+    assert '"test_openapi_schema_same_name" is already used ' in captured.out
 
 
 def test_docs_decorator():