allow platform specific diffs in error message

supabase · Feb 3, 2025 · 26a6bf6 · 26a6bf6
1 parent 062a52a
commit 26a6bf6
Show file tree

Hide file tree

Showing 4 changed files with 32 additions and 24 deletions.
diff --git a/test/expected/resolve_error_from_parser.out b/test/expected/resolve_error_from_parser.out
@@ -1,11 +1,20 @@
-select graphql.resolve($$
-{ { {
-    shouldFail
+-- Platform specific diffs so we have to test the properties here rather than exact response
+with d(val) as (
+  select graphql.resolve($$
+  { { {
+      shouldFail
+    }
   }
-}
-$$);
-                                                      resolve                                                      
--------------------------------------------------------------------------------------------------------------------
- {"errors": [{"message": "query parse error: Parse error at 2:3\nUnexpected `{[Punctuator]`\nExpected `Name`\n"}]}
+  $$)::json
+)
+select
+  (
+    json_typeof(val -> 'errors') = 'array'
+    and json_array_length(val -> 'errors') = 1
+  ) as is_valid
+from d;
+ is_valid 
+----------
+ t
 (1 row)
 
diff --git a/test/expected/test_error_parsing.out b/test/expected/test_error_parsing.out
diff --git a/test/sql/resolve_error_from_parser.sql b/test/sql/resolve_error_from_parser.sql
@@ -1,6 +1,16 @@
-select graphql.resolve($$
-{ { {
-    shouldFail
+-- Platform specific diffs so we have to test the properties here rather than exact response
+with d(val) as (
+  select graphql.resolve($$
+  { { {
+      shouldFail
+    }
   }
-}
-$$);
+  $$)::json
+)
+
+select
+  (
+    json_typeof(val -> 'errors') = 'array'
+    and json_array_length(val -> 'errors') = 1
+  ) as is_valid
+from d;
diff --git a/test/sql/test_error_parsing.sql b/test/sql/test_error_parsing.sql