Change 'id' argument's type to ID in page query

torchbox · Aug 30, 2023 · 4a7fc4e · 4a7fc4e
1 parent 37df8d6
commit 4a7fc4e
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 13 deletions.
diff --git a/docs/general-usage/graphql-types.rst b/docs/general-usage/graphql-types.rst
@@ -80,7 +80,7 @@ The singular ``page`` field accepts the following arguments:
 
 .. code-block:: graphql
 
-    id: Int                       # Can be used on it's own
+    id: ID                        # Can be used on it's own
     slug: String                  # Can be used on it's own
     urlPath: String               # Can be used on it's own
     token: String                 # Must be used with one of the others. Usually contentType

diff --git a/grapple/types/pages.py b/grapple/types/pages.py
@@ -334,7 +334,7 @@ class Mixin:
         )
         page = graphene.Field(
             PageInterface,
-            id=graphene.Int(),
+            id=graphene.ID(),
             slug=graphene.String(),
             url_path=graphene.Argument(
                 graphene.String,

diff --git a/tests/test_blog.py b/tests/test_blog.py
@@ -136,7 +136,7 @@ def setUp(self):
 
     def test_blog_page(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     title
@@ -151,7 +151,7 @@ def test_blog_page(self):
 
     def test_related_author_page(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     author {
@@ -171,7 +171,7 @@ def test_related_author_page(self):
 
     def get_blocks_from_body(self, block_type, block_query="rawValue", page_id=None):
         query = f"""
-        query($id: Int) {{
+        query($id: ID) {{
             page(id: $id) {{
                 ... on BlogPage {{
                     body {{
@@ -248,7 +248,7 @@ def test_streamfield_richtextblock(self):
 
     def test_richtext(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     summary
@@ -471,7 +471,7 @@ def test_blog_body_objectives(self):
 
     def test_blog_embed(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     body {
@@ -656,7 +656,7 @@ def test_blog_body_snippetchooserblock_person(self):
     # Next 2 tests are used to test the Collection API, both ForeignKey and nested field extraction.
     def test_blog_page_related_links(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     relatedLinks {
@@ -676,7 +676,7 @@ def test_blog_page_related_links(self):
 
     def test_blog_page_related_urls(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     relatedUrls
@@ -696,7 +696,7 @@ def test_blog_page_paginated_authors(self):
         per_page = 5
 
         query = """
-        query ($id: Int, $page: PositiveInt, $perPage: PositiveInt) {
+        query ($id: ID, $page: PositiveInt, $perPage: PositiveInt) {
             page(id: $id) {
                 ... on BlogPage {
                     paginatedAuthors(page: $page, perPage: $perPage) {
@@ -905,7 +905,7 @@ def test_singular_blog_page_query(self):
 
     def test_blog_page_tags(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     tags {
@@ -1057,7 +1057,7 @@ def test_graphqlfield_method_with_extra_arg_in_structblock(self):
 
     def test_custom_property(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     customProperty

diff --git a/tests/test_grapple.py b/tests/test_grapple.py
@@ -352,7 +352,7 @@ def test_pages_content_type_filter(self):
 
     def test_page(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 contentType
                 parent {