BREAKING: change format for permutateThemes output for single-dimensi…

…onal themes (#167)
tokens-studio · Jun 30, 2023 · 3425572 · 3425572
1 parent 260d152
commit 3425572
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 5 deletions.
diff --git a/.changeset/quick-onions-lie.md b/.changeset/quick-onions-lie.md
@@ -0,0 +1,5 @@
+---
+'@tokens-studio/sd-transforms': minor
+---
+
+BREAKING: permutateThemes to return the same format regardless of whether multi-dimensional themes are used or not. Format: `{ themeName: [sets] }`.
diff --git a/src/permutateThemes.ts b/src/permutateThemes.ts
@@ -4,6 +4,12 @@ declare interface Options {
   separator?: string;
 }
 
+function mapThemesToSetsObject(themes: ThemeObject[]) {
+  return Object.fromEntries(
+    themes.map(theme => [theme.name, filterTokenSets(theme.selectedTokenSets)]),
+  );
+}
+
 export function permutateThemes(themes: ThemeObject[], { separator = '-' } = {} as Options) {
   if (themes.some(theme => theme.group)) {
     // Sort themes by groups
@@ -19,9 +25,7 @@ export function permutateThemes(themes: ThemeObject[], { separator = '-' } = {}
     });
 
     if (Object.keys(groups).length <= 1) {
-      return Object.fromEntries(
-        themes.map(theme => [theme.name, filterTokenSets(theme.selectedTokenSets)]),
-      );
+      return mapThemesToSetsObject(themes);
     }
 
     // Create theme permutations
@@ -44,7 +48,7 @@ export function permutateThemes(themes: ThemeObject[], { separator = '-' } = {}
       }),
     );
   } else {
-    return themes;
+    return mapThemesToSetsObject(themes);
   }
 }
 

diff --git a/test/spec/permutateThemes.spec.ts b/test/spec/permutateThemes.spec.ts
@@ -196,6 +196,9 @@ describe('transform dimension', () => {
       },
     ];
 
-    expect(permutateThemes(singleDimensionalThemes)).to.eql(singleDimensionalThemes);
+    expect(permutateThemes(singleDimensionalThemes)).to.eql({
+      dark: ['core', 'dark', 'theme'],
+      light: ['core', 'light', 'theme'],
+    });
   });
 });