add test for shuffle transform

gka · gka · commit b238a19b84af · 2025-06-01T01:39:23.000+02:00
diff --git a/src/lib/transforms/sort.test.ts b/src/lib/transforms/sort.test.ts
@@ -1,5 +1,5 @@
 import { describe, it, expect } from 'vitest';
-import { sort } from './sort.js';
+import { sort, shuffle } from './sort.js';
 import type { DataRecord } from '$lib/types.js';
 
 const data: DataRecord[] = [
@@ -43,3 +43,13 @@ describe('sort transform', () => {
         );
     });
 });
+
+describe('shuffle transform', () => {
+    it('shuffles the data', () => {
+        const shuffled = shuffle({ data }, { seed: 1 });
+        expect(shuffled.sort).toBe(null);
+        expect(shuffled.data).toHaveLength(data.length);
+        expect(shuffled.data).not.toStrictEqual(data);
+        expect(shuffled.data.sort((a, b) => a.A - b.A)).toStrictEqual(sortedByA);
+    });
+});
diff --git a/src/lib/transforms/sort.ts b/src/lib/transforms/sort.ts
@@ -63,7 +63,7 @@ export function sort(
 }
 
 /**
- * reverses the data row order
+ * shuffles the data row order
  */
 export function shuffle(
     { data, ...channels }: TransformArg<DataRow[]>,
@@ -76,7 +76,8 @@ export function shuffle(
         ...channels,
         // set the sort channel to null to disable the implicit
         // alphabetical ordering of ordinal domains
-        sort: null
+        sort: null,
+        [IS_SORTED]: true
     };
 }
 
@@ -89,6 +90,7 @@ export function reverse({ data, ...channels }: TransformArg<DataRow[]>) {
         ...channels,
         // set the sort channel to null to disable the implicit
         // alphabetical ordering of ordinal domains
-        sort: null
+        sort: null,
+        [IS_SORTED]: true
     };
 }
