Merge pull request jspsych#3304 from jspsych/v8-null-default-test

Add a test case for trial parameters with `null` values
vzhang03 · Jun 6, 2024 · 505be4b · 505be4b
2 parents 233deb6 + 5837568
commit 505be4b
Showing 1 changed file with 15 additions and 0 deletions.
diff --git a/packages/jspsych/src/timeline/Trial.spec.ts b/packages/jspsych/src/timeline/Trial.spec.ts
@@ -462,6 +462,21 @@ describe("Trial", () => {
         );
       });
 
+      it("allows null values for parameters with a non-null default value", async () => {
+        TestPlugin.setParameterInfos({
+          allowedNullString: { type: ParameterType.STRING, default: "foo" },
+        });
+
+        const trial = createTrial({ type: TestPlugin, allowedNullString: null });
+        await trial.run();
+
+        expect(trial.pluginInstance.trial).toHaveBeenCalledWith(
+          expect.anything(),
+          expect.objectContaining({ allowedNullString: null }),
+          expect.anything()
+        );
+      });
+
       describe("with missing required parameters", () => {
         it("errors on missing simple parameters", async () => {
           TestPlugin.setParameterInfos({ requiredString: { type: ParameterType.STRING } });