pr fixes

demergent-labs · Oct 20, 2023 · 8795a5a · 8795a5a
1 parent f404192
commit 8795a5a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -131,6 +131,7 @@ jobs:
                     "examples/update",
                     "examples/vanilla_js",
                     "property_tests/tests/blob",
+                    "property_tests/tests/boolean",
                     "property_tests/tests/int",
                     "property_tests/tests/int8",
                     "property_tests/tests/int16",

diff --git a/property_tests/tests/bool/src/.gitkeep b/property_tests/tests/bool/src/.gitkeep
diff --git a/property_tests/tests/bool/test/test.ts b/property_tests/tests/bool/test/test.ts
@@ -11,18 +11,16 @@ const BoolTestArb = fc
         const returnCandidType = 'bool';
         const paramNames = bools.map((_, index) => `param${index}`);
 
-        const paramsAreNumbers = paramNames
+        const paramsAreBooleans = paramNames
             .map((paramName) => {
                 return `if (typeof ${paramName} !== 'boolean') throw new Error('${paramName} must be a boolean');`;
             })
             .join('\n');
 
-        const paramsAnd = paramNames.reduce((acc, paramName) => {
+        const returnStatement = paramNames.reduce((acc, paramName) => {
             return `${acc} && ${paramName}`;
         }, 'true');
 
-        const returnStatement = `${paramsAnd}`;
-
         const expectedResult = bools.reduce((acc, bool) => acc && bool, true);
 
         const paramSamples = bools;
@@ -43,7 +41,7 @@ const BoolTestArb = fc
             body: `
             ${paramsCorrectlyOrdered}
 
-            ${paramsAreNumbers}
+            ${paramsAreBooleans}
 
             return ${returnStatement};
         `,