test: improve TypeScript test formatting and validation

Author: Torkan

test/ts/shouldFail/unionValidation.ts

@@ -5,10 +5,7 @@
 // Union Validation Tests - shouldFail
 // Tests for invalid union field syntax
 
-import {
-  getTodo,
-  createTodo,
-} from "../generated";
+import { getTodo, createTodo } from "../generated";
 
 // Test 1: Invalid union field syntax - using string instead of object notation
 export const invalidUnionString = await createTodo({
@@ -18,55 +15,49 @@ export const invalidUnionString = await createTodo({
     content: { note: "test" },
   },
   fields: [
-    "id", 
-    "title", 
+    "id",
+    "title",
     // @ts-expect-error - "content" should require object notation for union fields
-    "content"
-  ]
+    "content",
+  ],
 });
 
 // Test 2: Invalid union field in getTodo
 export const getWithInvalidUnionString = await getTodo({
   input: {},
   fields: [
-    "id", 
-    "title", 
+    "id",
+    "title",
     // @ts-expect-error - "content" should require object notation for union fields
-    "content"
-  ]
+    "content",
+  ],
 });
 
 // Test 3: Invalid array union field syntax
 export const invalidArrayUnionString = await createTodo({
   input: {
-    title: "Invalid Array Union Syntax", 
+    title: "Invalid Array Union Syntax",
     userId: "123e4567-e89b-12d3-a456-426614174000",
     attachments: [{ url: "https://example.com" }],
   },
   fields: [
-    "id", 
-    "title", 
+    "id",
+    "title",
     // @ts-expect-error - "attachments" should require object notation for union fields
-    "attachments"
-  ]
+    "attachments",
+  ],
 });
 
 // Test 4: Invalid multiple union fields as strings
 export const invalidBothUnionStrings = await createTodo({
   input: {
     title: "Invalid Both Union Syntax",
-    userId: "123e4567-e89b-12d3-a456-426614174000", 
+    userId: "123e4567-e89b-12d3-a456-426614174000",
     content: { note: "test" },
     attachments: [{ url: "https://example.com" }],
   },
-  fields: [
-    "id", 
-    "title", 
-    // @ts-expect-error - "content" should require object notation
-    "content", 
-    // @ts-expect-error - "attachments" should require object notation
-    "attachments"
-  ]
+  // @ts-expect-error - "content" & "attachments" should require object notation for union fields
+  fields: ["id", "title", "content", "attachments"],
 });
 
 // Test 5: Invalid union in calculation
@@ -81,11 +72,11 @@ export const invalidUnionInCalculation = await getTodo({
           "id",
           "title",
           // @ts-expect-error - "content" should require object notation even in calculations
-          "content"
-        ]
-      }
-    }
-  ]
+          "content",
+        ],
+      },
+    },
+  ],
 });
 
 // Test 6: Invalid union field with empty object
@@ -96,9 +87,9 @@ export const invalidUnionEmptyObject = await getTodo({
     "title",
     {
       // @ts-expect-error - content object cannot be empty, must specify fields
-      content: {}
-    }
-  ]
+      content: {},
+    },
+  ],
 });
 
 // Test 7: Invalid union field with wrong object structure
@@ -110,10 +101,10 @@ export const invalidUnionWrongStructure = await getTodo({
     {
       content: [
         // @ts-expect-error - "invalidMember" is not a valid union member
-        "invalidMember"
-      ]
-    }
-  ]
+        "invalidMember",
+      ],
+    },
+  ],
 });
 
 // Test 8: Invalid nested union field syntax in complex structure
@@ -129,12 +120,12 @@ export const invalidNestedUnionSyntax = await getTodo({
           // @ts-expect-error - "content" should require object notation
           "content",
           {
-            user: ["id", "name"]
-          }
-        ]
-      }
-    }
-  ]
+            user: ["id", "name"],
+          },
+        ],
+      },
+    },
+  ],
 });
 
 // Test 9: Invalid union member field selection
@@ -149,12 +140,12 @@ export const invalidUnionMemberFields = await getTodo({
           text: [
             "text",
             // @ts-expect-error - "nonExistentField" should not be valid for text member
-            "nonExistentField"
-          ]
-        }
-      ]
-    }
-  ]
+            "nonExistentField",
+          ],
+        },
+      ],
+    },
+  ],
 });
 
 // Test 10: Invalid array union with wrong member
@@ -166,10 +157,10 @@ export const invalidArrayUnionMember = await getTodo({
     {
       attachments: [
         // @ts-expect-error - "invalidAttachmentType" is not a valid union member
-        "invalidAttachmentType"
-      ]
-    }
-  ]
+        "invalidAttachmentType",
+      ],
+    },
+  ],
 });
 
 console.log("Union validation tests should FAIL compilation!");

test/ts/shouldPass/noFieldsTypeInference.ts

@@ -21,7 +21,8 @@ if (createUserNoFieldsInferred.success) {
   // This should compile - empty object type
   const isEmpty = Object.keys(data).length === 0;
 
-  // @ts-expect-error - Should NOT have id property when no fields requested
+  // Properties should NOT exist when no fields requested - this line should error
+  // @ts-expect-error
   const shouldNotExist = createUserNoFieldsInferred.data.id;
 }
 
@@ -35,7 +36,7 @@ export const createUserEmptyFieldsInferred = await createUser({
 });
 
 if (createUserEmptyFieldsInferred.success) {
-  const data: {} = createUserEmptyFieldsInferred.data;
+  const data: unknown = createUserEmptyFieldsInferred.data;
 
   // @ts-expect-error - Should NOT have name property when fields is empty
   const shouldNotExist = createUserEmptyFieldsInferred.data.name;