From 408ad0d2b043f966beef3b7ed7f317e8228b8ddc Mon Sep 17 00:00:00 2001
From: Jeppe Reinhold <jeppe@reinhold.is>
Date: Wed, 7 Feb 2024 22:29:46 +0100
Subject: [PATCH] expect boolean args to be booleans instead of strings

---
 code/lib/preview-api/src/modules/store/args.test.ts | 7 ++++---
 code/lib/preview-api/src/modules/store/args.ts      | 2 +-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/code/lib/preview-api/src/modules/store/args.test.ts b/code/lib/preview-api/src/modules/store/args.test.ts
index 0bfc4ab41a20..0d39874c766f 100644
--- a/code/lib/preview-api/src/modules/store/args.test.ts
+++ b/code/lib/preview-api/src/modules/store/args.test.ts
@@ -67,8 +67,9 @@ describe('mapArgsToTypes', () => {
   });
 
   it('maps booleans', () => {
+    expect(mapArgsToTypes({ a: true }, { a: { type: booleanType } })).toStrictEqual({ a: true });
     expect(mapArgsToTypes({ a: 'true' }, { a: { type: booleanType } })).toStrictEqual({ a: true });
-    expect(mapArgsToTypes({ a: 'false' }, { a: { type: booleanType } })).toStrictEqual({
+    expect(mapArgsToTypes({ a: false }, { a: { type: booleanType } })).toStrictEqual({
       a: false,
     });
     expect(mapArgsToTypes({ a: 'yes' }, { a: { type: booleanType } })).toStrictEqual({ a: false });
@@ -127,7 +128,7 @@ describe('mapArgsToTypes', () => {
         {
           key: {
             arr: ['1', '2'],
-            obj: { bool: 'true' },
+            obj: { bool: true },
           },
         },
         {
@@ -157,7 +158,7 @@ describe('mapArgsToTypes', () => {
           key: [
             {
               arr: ['1', '2'],
-              obj: { bool: 'true' },
+              obj: { bool: true },
             },
           ],
         },
diff --git a/code/lib/preview-api/src/modules/store/args.ts b/code/lib/preview-api/src/modules/store/args.ts
index 2634015bce76..7ce46f94a512 100644
--- a/code/lib/preview-api/src/modules/store/args.ts
+++ b/code/lib/preview-api/src/modules/store/args.ts
@@ -19,7 +19,7 @@ const map = (arg: unknown, argType: InputType): any => {
     case 'number':
       return Number(arg);
     case 'boolean':
-      return arg === 'true';
+      return String(arg) === 'true';
     case 'array':
       if (!type.value || !Array.isArray(arg)) return INCOMPATIBLE;
       return arg.reduce((acc, item, index) => {