fix(parse): accept list what includes empty element

Author: TomokiMiyauci

_tools/meta.ts

@@ -37,10 +37,15 @@ export const makeOptions = (version: string): BuildOptions => ({
   },
   packageManager: "pnpm",
   mappings: {
-    "https://deno.land/x/isx@1.1.0/is_string.ts": {
+    "https://deno.land/x/isx@1.3.1/is_string.ts": {
       name: "@miyauci/isx",
-      version: "1.1.0",
-      subPath: "is_string",
+      version: "1.3.1",
+      subPath: "is_string.js",
+    },
+    "https://deno.land/x/[email protected]/iterable/is_not_empty.ts": {
+      name: "@miyauci/isx",
+      version: "1.3.1",
+      subPath: "iterable/is_not_empty.js",
     },
     "https://deno.land/x/[email protected]/token.ts": {
       name: "@httpland/http-utils",

deno.lock

@@ -1,7 +1,8 @@
 // Copyright 2023-latest the httpland authors. All rights reserved. MIT license.
 // This module is browser compatible.
 
-export { isString } from "https://deno.land/x/[email protected]/is_string.ts";
+export { isString } from "https://deno.land/x/[email protected]/is_string.ts";
+export { isNotEmpty } from "https://deno.land/x/[email protected]/iterable/is_not_empty.ts";
 export {
   isToken,
   type Token,

deps.ts

@@ -1,7 +1,7 @@
 // Copyright 2023-latest the httpland authors. All rights reserved. MIT license.
 // This module is browser compatible.
 
-import { isToken, parseListFields } from "./deps.ts";
+import { isNotEmpty, isToken, parseListFields } from "./deps.ts";
 import { Msg } from "./constants.ts";
 import type { AcceptRanges } from "./types.ts";
 
@@ -19,11 +19,12 @@ import type { AcceptRanges } from "./types.ts";
  */
 export function parseAcceptRanges(input: string): AcceptRanges {
   const acceptableRanges = parseListFields(input);
+  const msg = `${Msg.InvalidSyntax} "${input}"`;
+
+  if (!isNotEmpty(acceptableRanges)) throw new SyntaxError(msg);
 
   acceptableRanges.forEach((token) => {
-    if (!isToken(token)) {
-      throw SyntaxError(`${Msg.InvalidSyntax} "${input}"`);
-    }
+    if (!isToken(token)) throw new SyntaxError(msg);
   });
 
   return acceptableRanges as AcceptRanges;

parse.ts

@@ -18,8 +18,6 @@ describe("parseAcceptRanges", () => {
       ",",
       ",,,",
       ", , ,",
-      "a,",
-      "a, , a",
       "あ",
       "a, あ",
       `""`,
@@ -38,6 +36,7 @@ describe("parseAcceptRanges", () => {
       [" abc ", ["abc"]],
       ["abc, def", ["abc", "def"]],
       ["bytes,none", ["bytes", "none"]],
+      ["bytes, ,,none", ["bytes", "none"]],
     ];
 
     table.forEach(([input, expected]) => {