From 7c2ca748b73d67ece7330bc9210bfac4e907bd7e Mon Sep 17 00:00:00 2001 From: Jon <66625200+Kyro3400@users.noreply.github.com> Date: Wed, 13 Nov 2024 09:42:03 -0500 Subject: [PATCH] Update json.test.js --- tests/json.test.js | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/tests/json.test.js b/tests/json.test.js index 02092b42..d9af3742 100644 --- a/tests/json.test.js +++ b/tests/json.test.js @@ -19,7 +19,8 @@ const reservedDomainsPath = path.resolve("reserved"); const rootDomainFiles = ["is-a-fullstack.dev.json", "is-cool.dev.json", "is-local.org.json", "is-not-a.dev.json", "localplayer.dev.json"]; const files = fs.readdirSync(domainsPath); const reservedFiles = fs.existsSync(reservedDomainsPath) ? fs.readdirSync(reservedDomainsPath) : []; - + +const Domains = [".is-a-fullstack.dev", ".is-cool.dev", ".is-local.org", ".is-not-a.dev", ".localplayer.dev"] function validateRequiredFields(t, obj, requiredFields, file) { Object.keys(requiredFields).forEach((key) => { @@ -46,16 +47,13 @@ t("All files should be valid JSON", (t) => { }); }); -const Domains = [".is-a-fullstack.dev", ".is-cool.dev", ".is-local.org", ".is-not-a.dev", ".localplayer.dev"] - t("All files should have valid file names", (t) => { files.forEach((file) => { t.true(file.endsWith(".json"), `${file}: File does not have .json extension`); - t.false(file.includes(`${Domains}`), `${file}: File name should not contain .is-a-fullstack.dev`); - t.false(file.includes(".is-cool.dev"), `${file}: File name should not contain .is-cool.dev`); - t.false(file.includes(".is-local.org"), `${file}: File name should not contain .is-local.org`); - t.false(file.includes(".is-not-a.dev"), `${file}: File name should not contain .is-not-a.dev`); - t.false(file.includes(".localplayer.dev"), `${file}: File name should not contain .localplayer.dev`); + + // Check for any unwanted domain in file names + t.false(Domains.some(domain => file.includes(domain)), `${file}: File name should not contain any restricted domain extensions`); + t.true(file === file.toLowerCase(), `${file}: File name should be lowercase`); // Ignore root domain