Fix issues with delete item PR

ProdPerfect · Jun 27, 2021 · d265f00 · d265f00
1 parent 0ee5c63
commit d265f00
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 13 deletions.
diff --git a/monday/query_joins.py b/monday/query_joins.py
@@ -6,18 +6,6 @@
 # Eventually I will organize this file better but you know what today is not that day.
 
 # ITEM RESOURCE QUERIES
-def delete_item_query(item_id):
-    query = '''
-    mutation
-    {
-        delete_item (item_id: %s)
-        {
-            id
-        }
-    }''' % (item_id)
-    return query
-
-
 def mutate_item_query(board_id, group_id, item_name, column_values,
                       create_labels_if_missing):
     # Monday does not allow passing through non-JSON null values here,
@@ -139,6 +127,18 @@ def update_item_query(board_id, item_id, column_id, value):
     return query
 
 
+def delete_item_query(item_id):
+    query = '''
+    mutation
+    {
+        delete_item (item_id: %s)
+        {
+            id
+        }
+    }''' % (item_id)
+    return query
+
+
 def update_multiple_column_values_query(board_id, item_id, column_values):
     query = '''mutation
         {

diff --git a/monday/tests/test_item_resource.py b/monday/tests/test_item_resource.py
@@ -55,4 +55,12 @@ def test_add_file_to_column_query(self):
 
     def test_delete_item_by_id(self):
         query = delete_item_query(item_id=self.item_id)
-        self.assertIn(str(self.item_id))
+        self.assertIn(str(self.item_id), query)
+        self.assertEqual('''
+        mutation
+        {
+            delete_item (item_id: 24)
+            {
+                id
+            }
+        }'''.replace(" ", ""), query.replace(" ", ""))