From 673b5245c5b189ccdef1eae36b50acbdb5b438df Mon Sep 17 00:00:00 2001 From: Lavodan <21205085+Lavodan@users.noreply.github.com> Date: Thu, 4 Dec 2025 00:40:53 +0100 Subject: [PATCH] fix failing nationNameLength test (#2556) ## Description: Fixes the failing nationNameLength.test by usinbg the proper absolute path glob syntax Also switches to fast-glob because I don't see a reason to not use it. ## Please complete the following: - [x] I have added screenshots for all UI updates - [x] I process any text displayed to the user through translateText() and I've added it to the en.json file - [x] I have added relevant tests to the test directory - [x] I confirm I have thoroughly tested these changes and take full responsibility for any bugs introduced ## Please put your Discord username so you can be contacted if a bug or regression is found: Lavodan --- tests/NationNameLength.test.ts | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/tests/NationNameLength.test.ts b/tests/NationNameLength.test.ts index 5bfc4960b..9d7df858f 100644 --- a/tests/NationNameLength.test.ts +++ b/tests/NationNameLength.test.ts @@ -1,6 +1,5 @@ import fs from "fs"; import { globSync } from "glob"; -import path from "path"; type Nation = { name?: string; @@ -12,9 +11,7 @@ type Manifest = { describe("Map manifests: nation name length constraint", () => { test("All nations' names must be ≤ 27 characters", () => { - const manifestPaths = globSync( - path.resolve(process.cwd(), "resources/maps/**/manifest.json"), - ); + const manifestPaths = globSync("resources/maps/**/manifest.json"); expect(manifestPaths.length).toBeGreaterThan(0);