Merge pull request #226 from NetsBlox/matlab-logical-types

Convert to true/false not 1/0 when logical mwtype (matlab service)
NetsBlox · Mar 4, 2024 · 0fe9474 · 0fe9474
2 parents b12bb5d + ea6b980
commit 0fe9474
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/procedures/matlab/matlab.js b/src/procedures/matlab/matlab.js
@@ -65,7 +65,7 @@ MATLAB._parseArgument = function (arg) {
   const mwdata = flatValues
     .map((v) => {
       if (mwtype === "logical") {
-        return v ? 1 : 0;
+        return v ? true : false;
       } else if (mwtype === "string") {
         return v.toString();
       } else { // number

diff --git a/test/procedures/matlab/matlab.spec.js b/test/procedures/matlab/matlab.spec.js
@@ -126,7 +126,7 @@ describe(utils.suiteName(__filename), function () {
     it("should coerce booleans to 1/0", function () {
       const example = [true, false];
       const actual = MATLAB._parseArgument(example);
-      const expected = [1, 0];
+      const expected = [true, false];
       assert.deepEqual(actual.mwdata, expected);
     });