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()", () => {