ensure consistent sort order when pattern matches >1 files

chris48s · Aug 19, 2024 · 195bc17 · 195bc17
1 parent d2aa505
commit 195bc17
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/src/cli.spec.js b/src/cli.spec.js
@@ -1006,13 +1006,6 @@ describe("CLI", function () {
       }).then(() => {
         const expected = {
           results: [
-            {
-              code: 0,
-              errors: [],
-              fileLocation: "./testfiles/files/valid.json",
-              schemaLocation: "./testfiles/schemas/schema.json",
-              valid: true,
-            },
             {
               code: 99,
               errors: [
@@ -1037,6 +1030,13 @@ describe("CLI", function () {
               schemaLocation: "./testfiles/schemas/schema.json",
               valid: null,
             },
+            {
+              code: 0,
+              errors: [],
+              fileLocation: "./testfiles/files/valid.json",
+              schemaLocation: "./testfiles/schemas/schema.json",
+              valid: true,
+            },
           ],
         };
         assert.deepStrictEqual(JSON.parse(logger.stdout[0]), expected);

diff --git a/src/glob.js b/src/glob.js
@@ -3,7 +3,9 @@ import logger from "./logger.js";
 
 async function getFiles(pattern) {
   try {
-    return await glob(pattern, { dot: true, dotRelative: true });
+    let matches = await glob(pattern, { dot: true, dotRelative: true });
+    matches.sort((a, b) => a.localeCompare(b));
+    return matches;
   } catch (e) {
     logger.error(e.message);
     return [];