fix: return Uint8ClampedArray instead of Buffer in generateImage func…

…tion
Short-io · Jun 5, 2024 · 46d32ac · 46d32ac
1 parent a318ec4
commit 46d32ac
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/src/png.ts b/src/png.ts
@@ -38,5 +38,6 @@ export async function generateImage({
         })
         qrImage.composite(layers);
     }
-    return await qrImage.png().toBuffer();
+    const { data } = await qrImage.png().toBuffer({ resolveWithObject: true});
+    return new Uint8ClampedArray(data.buffer);
 }
diff --git a/src/tests/test.ts b/src/tests/test.ts
@@ -238,6 +238,12 @@ interface TestParams {
         filename: "qr_with_logo.pdf",
         params: { logo: await readFile(`${goldenDir}/logo.png`) },
     },
+    {
+        name: "PNG memory usage",
+        fn: getPNG,
+        filename: "qr_large.png",
+        params: { size: 300, margin: 10 },
+    }
 ] as TestParams[]).forEach((testData) => {
     test.serial(testData.name, async (t) => {
         const image = await testData.fn(text, {

diff --git a/test_data/golden/qr_large.png b/test_data/golden/qr_large.png