From 7d82619d7fe5c6dbb1c658bf8c615497e717827b Mon Sep 17 00:00:00 2001
From: Loknath Mishra <kloknath389@gmail.com>
Date: Wed, 13 May 2026 23:19:20 +0530
Subject: [PATCH] #2355 - update tests for ToBase32 to include BMP logic

---
 tests/node/tests/NodeDish.mjs     | 36 +++++++++++++++++++++++++++++++
 tests/node/tests/nodeApi.mjs      | 32 +++++++++++++++++++++++++++
 tests/operations/tests/Base32.mjs | 22 +++++++++++++++++++
 3 files changed, 90 insertions(+)

diff --git a/tests/node/tests/NodeDish.mjs b/tests/node/tests/NodeDish.mjs
index 3ec8b7e2..958e1338 100644
--- a/tests/node/tests/NodeDish.mjs
+++ b/tests/node/tests/NodeDish.mjs
@@ -65,6 +65,42 @@ TestRegister.addApiTests([
         assert.strictEqual(result.toString(), "493e8136b759370a415ef2cf2f7a69690441ff86592aba082bc2e2e0");
     }),
 
+    it("Composable Dish: toBase32 should support non-BMP Unicode alphabets", () => {
+        const alphabet = "🀇🀈🀉🀊🀋🀌🀍🀎🀏🀙🀚🀛🀜🀝🀞🀟🀠🀡🀐🀑🀒🀓🀔🀕🀖🀗🀘🀀🀁🀂🀃🀅";
+
+        const result = new Dish("hello")
+            .apply(toBase32, {alphabet})
+            .toString();
+
+        // Should not contain replacement characters
+        assert.equal(result.includes("�"), false);
+
+        // Should contain only symbols from the alphabet
+        for (const ch of Array.from(result)) {
+            assert.ok(Array.from(alphabet).includes(ch));
+        }
+
+        // "hello" => 8 Base32 symbols
+        assert.equal(Array.from(result).length, 8);
+    }),
+
+    it("Composable Dish: toBase32 should omit padding for 32-character Unicode alphabets", () => {
+        const alphabet = "🀇🀈🀉🀊🀋🀌🀍🀎🀏🀙🀚🀛🀜🀝🀞🀟🀠🀡🀐🀑🀒🀓🀔🀕🀖🀗🀘🀀🀁🀂🀃🀅";
+
+        const result = new Dish("hell")
+            .apply(toBase32, {alphabet})
+            .toString();
+
+        // Should not leak undefined from array indexing
+        assert.equal(result.includes("undefined"), false);
+
+        // Should not contain replacement characters
+        assert.equal(result.includes("�"), false);
+
+        // Unpadded Base32 output for 4-byte input should be 7 symbols
+        assert.equal(Array.from(result).length, 7);
+    }),
+
     it("Dish translation: ArrayBuffer to ArrayBuffer", () => {
         const dish = new Dish(new ArrayBuffer(10), 4);
         dish.get("array buffer");
diff --git a/tests/node/tests/nodeApi.mjs b/tests/node/tests/nodeApi.mjs
index 2510ef17..1d1146b1 100644
--- a/tests/node/tests/nodeApi.mjs
+++ b/tests/node/tests/nodeApi.mjs
@@ -109,6 +109,38 @@ TestRegister.addApiTests([
         assert.equal(3 + result, 35);
     }),
 
+    it("toBase32: should support non-BMP Unicode alphabets", () => {
+        const alphabet = "🀇🀈🀉🀊🀋🀌🀍🀎🀏🀙🀚🀛🀜🀝🀞🀟🀠🀡🀐🀑🀒🀓🀔🀕🀖🀗🀘🀀🀁🀂🀃🀅";
+
+        const result = chef.toBase32("hello", {alphabet}).toString();
+
+        // Should not contain replacement characters
+        assert.equal(result.includes("�"), false);
+
+        // Should contain only symbols from the alphabet
+        for (const ch of Array.from(result)) {
+            assert.ok(Array.from(alphabet).includes(ch));
+        }
+
+        // "hello" => 8 Base32 symbols
+        assert.equal(Array.from(result).length, 8);
+    }),
+
+    it("toBase32: should omit padding for 32-character Unicode alphabets", () => {
+        const alphabet = "🀇🀈🀉🀊🀋🀌🀍🀎🀏🀙🀚🀛🀜🀝🀞🀟🀠🀡🀐🀑🀒🀓🀔🀕🀖🀗🀘🀀🀁🀂🀃🀅";
+
+        const result = chef.toBase32("hell", {alphabet}).toString();
+
+        // Should not leak undefined from array indexing
+        assert.equal(result.includes("undefined"), false);
+
+        // Should not contain replacement characters
+        assert.equal(result.includes("�"), false);
+
+        // Unpadded Base32 output for 4-byte input should be 7 symbols
+        assert.equal(Array.from(result).length, 7);
+    }),
+
     it("chef.help: should exist", () => {
         assert(chef.help);
     }),
diff --git a/tests/operations/tests/Base32.mjs b/tests/operations/tests/Base32.mjs
index 760cdf14..558d7df6 100644
--- a/tests/operations/tests/Base32.mjs
+++ b/tests/operations/tests/Base32.mjs
@@ -172,5 +172,27 @@ TestRegister.addTests([
             },
         ],
     },
+    {
+        name: "To Base32: should support non-BMP Unicode alphabets",
+        input: "hello",
+        expectedOutput: "🀝🀈🀐🀔🀖🀀🀊🀟",
+        recipeConfig: [
+            {
+                op: "To Base32",
+                args: ["🀇🀈🀉🀊🀋🀌🀍🀎🀏🀙🀚🀛🀜🀝🀞🀟🀠🀡🀐🀑🀒🀓🀔🀕🀖🀗🀘🀀🀁🀂🀃🀅"],
+            },
+        ],
+    },
+    {
+        name: "To Base32: should omit padding for 32-character Unicode alphabets",
+        input: "hell",
+        expectedOutput: "🀝🀈🀐🀔🀖🀀🀇",
+        recipeConfig: [
+            {
+                op: "To Base32",
+                args: ["🀇🀈🀉🀊🀋🀌🀍🀎🀏🀙🀚🀛🀜🀝🀞🀟🀠🀡🀐🀑🀒🀓🀔🀕🀖🀗🀘🀀🀁🀂🀃🀅"],
+            },
+        ],
+    },
 ]);