diff --git a/src/core/Dish.mjs b/src/core/Dish.mjs index 964380fb..be669cb7 100755 --- a/src/core/Dish.mjs +++ b/src/core/Dish.mjs @@ -37,17 +37,18 @@ class Dish { constructor(dishOrInput=null, type = null) { this.value = new ArrayBuffer(0); this.type = Dish.ARRAY_BUFFER; + const hasInput = dishOrInput !== undefined && dishOrInput !== null; // Case: dishOrInput is dish object - if (dishOrInput && + if (hasInput && Object.prototype.hasOwnProperty.call(dishOrInput, "value") && Object.prototype.hasOwnProperty.call(dishOrInput, "type")) { this.set(dishOrInput.value, dishOrInput.type); // input and type defined separately - } else if (dishOrInput && type !== null) { + } else if (hasInput && type !== null) { this.set(dishOrInput, type); // No type declared, so infer it. - } else if (dishOrInput) { + } else if (hasInput) { const inferredType = Dish.typeEnum(dishOrInput.constructor.name); this.set(dishOrInput, inferredType); } diff --git a/src/node/api.mjs b/src/node/api.mjs index a48ed3a9..1c92d7e5 100644 --- a/src/node/api.mjs +++ b/src/node/api.mjs @@ -102,7 +102,7 @@ function transformArgs(opArgsList, newArgs) { * @param input */ function ensureIsDish(input) { - if (!input) { + if (input === undefined || input === null) { return new NodeDish(); } diff --git a/tests/node/tests/NodeDish.mjs b/tests/node/tests/NodeDish.mjs index 958e1338..64a98265 100644 --- a/tests/node/tests/NodeDish.mjs +++ b/tests/node/tests/NodeDish.mjs @@ -38,6 +38,9 @@ TestRegister.addApiTests([ const numberDish = new Dish(333); assert.strictEqual(numberDish.type, 2); + const zeroDish = new Dish(0); + assert.strictEqual(zeroDish.type, 2); + const arrayBufferDish = new Dish(Buffer.from("some buffer input").buffer); assert.strictEqual(arrayBufferDish.type, 4); diff --git a/tests/node/tests/nodeApi.mjs b/tests/node/tests/nodeApi.mjs index b65b9abc..eec8ac2a 100644 --- a/tests/node/tests/nodeApi.mjs +++ b/tests/node/tests/nodeApi.mjs @@ -97,6 +97,11 @@ TestRegister.addApiTests([ assert(result instanceof NodeDish); }), + it("should process numeric zero input", () => { + const result = chef.toHex(0); + assert.strictEqual(result.toString(), "30"); + }), + it("should coerce to a string as you expect", () => { const result = chef.fromBase32(chef.toBase32("something")); assert.equal(String(result), "something"); @@ -212,6 +217,11 @@ TestRegister.addApiTests([ assert.strictEqual(result.toString(), "ONXW2ZJANFXHA5LU"); }), + it("chef.bake: should process numeric zero input", async () => { + const result = await chef.bake(0, "to hex"); + assert.strictEqual(result.toString(), "30"); + }), + it("chef.bake: should complain if recipe isnt a valid object", async () => { await assert.rejects(() => chef.bake("some input", 3264), { name: "TypeError", diff --git a/tests/node/tests/operations.mjs b/tests/node/tests/operations.mjs index c6f0e44f..4ce5c4ca 100644 --- a/tests/node/tests/operations.mjs +++ b/tests/node/tests/operations.mjs @@ -59,6 +59,13 @@ TestRegister.addApiTests([ assert.strictEqual(result.toString(), "7"); }), + it("ADD: processes numeric zero input", () => { + const result = chef.ADD(0, { + key: "4", + }); + assert.strictEqual(result.toString(), "4"); + }), + it("addLineNumbers: No arguments", () => { const result = addLineNumbers("sample input"); assert.equal(result.toString(), "1 sample input"); @@ -1203,4 +1210,3 @@ ExifImageHeight: 57`); ]); -