Replace TODO with test.todo

Author: razor-x

src/lib/parse.test.ts

@@ -23,3 +23,13 @@ test('parseUrlSearchParams: with URLSearchParams input', (t) => {
     'with URLSearchParams input',
   )
 })
+
+test.todo('parse repeated array params like foo=bar&foo=baz')
+test.todo('parse bracket array params like foo[]=bar&foo[]=baz')
+test.todo('parse comma array params like foo=bar,baz')
+
+test.todo('cannot parse mixed array params like foo=bar,baz&foo=bar&foo[]=baz')
+
+// e.g., foo.bar= would conflict with foo.bar.a= or foo.bar.b=2
+// since this would be a null object containing values (null is still a value).
+test.todo('cannot parse conflicting object keys')

src/lib/parse.ts

@@ -33,19 +33,10 @@ const parseFromParamSchema = (
   path: string[],
 ): Record<string, unknown> | unknown => {
   if (typeof node === 'string') {
-    // TODO: For array parsing, try to lookup foo=, then foo[]= patterns,
-    // if only one match, try to detect commas, otherwise ignore commas.
-    // if both foo= and foo[]= this is a parse error
-    // more generally, try to find a matching key for this node in the searchParams
-    // and throw if conflicting keys are found, e.g, both foo= and foo[]=
     const key = path.join('.')
     return parse(key, searchParams.getAll(key), node)
   }
 
-  // TODO: Ensure that there are no conflicting object keys, e.g.,
-  // foo.bar= would conflict with foo.bar.a= or foo.bar.b=2
-  // since this would be a null object containing values (null is still a value).
-
   const entries = Object.entries(node).reduce<
     Array<[string, Record<string, unknown> | unknown]>
   >((acc, entry) => {