Fix validation issue

eslint · Oct 16, 2024 · 76877f4 · 76877f4
1 parent a3cbf3c
commit 76877f4
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 7 deletions.
diff --git a/src/languages/json-language.js b/src/languages/json-language.js
@@ -93,15 +93,15 @@ export class JSONLanguage {
  "allowTrailingCommas must be a boolean if provided.",
  );
  }
- }
 
- // we know that allowTrailingCommas is a boolean here
+  // we know that allowTrailingCommas is a boolean here
 
- // only allowed in JSONC mode
- if (this.#mode !== "jsonc") {
- throw new Error(
- "allowTrailingCommas option is only available in JSONC.",
- );
+ // only allowed in JSONC mode
+ if (this.#mode !== "jsonc") {
+ throw new Error(
+ "allowTrailingCommas option is only available in JSONC.",
+ );
+ }
  }
  }
 

diff --git a/tests/languages/json-language.test.js b/tests/languages/json-language.test.js
@@ -63,6 +63,13 @@ describe("JSONLanguage", () => {
  language.validateLanguageOptions({});
  });
  });
+
+ it("should not throw an error when allowTrailingCommas is not provided and other keys are present", () => {
+ const language = new JSONLanguage({ mode: "jsonc" });
+ assert.doesNotThrow(() => {
+ language.validateLanguageOptions({ foo: "bar" });
+ });
+ });
  });
 
  describe("parse()", () => {