fix: add lookup_value_regex for taxonomy rest api (#101)

openedx · Oct 23, 2023 · be5d527 · be5d527
1 parent e3be15c
commit be5d527
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/openedx_tagging/core/tagging/rest_api/v1/views.py b/openedx_tagging/core/tagging/rest_api/v1/views.py
@@ -173,6 +173,7 @@ class TaxonomyView(ModelViewSet):
 
     """
 
+    lookup_value_regex = r"\d+"
     serializer_class = TaxonomySerializer
     permission_classes = [TaxonomyObjectPermissions]
 

diff --git a/tests/openedx_tagging/core/tagging/test_views.py b/tests/openedx_tagging/core/tagging/test_views.py
@@ -195,6 +195,13 @@ def test_detail_taxonomy_404(self) -> None:
         response = self.client.get(url)
         assert response.status_code == status.HTTP_404_NOT_FOUND
 
+    def test_detail_taxonomy_invalud_pk(self) -> None:
+        url = TAXONOMY_DETAIL_URL.format(pk="invalid")
+
+        self.client.force_authenticate(user=self.staff)
+        response = self.client.get(url)
+        assert response.status_code == status.HTTP_404_NOT_FOUND
+
     @ddt.data(
         (None, status.HTTP_401_UNAUTHORIZED),
         ("user", status.HTTP_403_FORBIDDEN),
-Original file line number
+Diff line change
@@ Expand Up / @@ -173,6 +173,7 @@ class TaxonomyView(ModelViewSet): @@
         """
+        lookup_value_regex = r"\d+"
         serializer_class = TaxonomySerializer
         permission_classes = [TaxonomyObjectPermissions]
@@ Expand Down @@