Merge pull request #3 from david50407/fix/untar-last-part

Correct untar padding removal when last part has size = 512

deno.json

@@ -1,6 +1,6 @@
 {
   "name": "@doctor/tar-stream",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "exports": {
     ".": "./mod.ts",
     "./tar": "./tar.ts",

tests.ts

@@ -277,3 +277,29 @@ Deno.test('expandTarArchiveCheckingBodiesByteStream', async function () {
     }
   }
 })
+
+Deno.test('UnTarStream() with size equals to multiple of 512', async () => {
+  const size = 512 * 3
+  const data = Uint8Array.from({ length: size }, () => Math.floor(Math.random() * 256))
+
+  const readable = ReadableStream.from<TarInput>([
+    {
+      pathname: 'name',
+      size,
+      iterable: [data.slice()],
+    },
+  ])
+    .pipeThrough(new TarStream())
+    .pipeThrough(new UnTarStream())
+
+  for await (const item of readable) {
+    if (item.readable) {
+      assertEquals(
+        Uint8Array.from(
+          (await Array.fromAsync(item.readable)).map((x) => [...x]).flat()
+        ),
+        data
+      )
+    }
+  }
+})

untar.ts

@@ -192,7 +192,7 @@ export class UnTarStream {
                   lock = true
                   const { done, value } = await async function () {
                     const x = await reader.read()
-                    if (!x.done && i-- === 1) {
+                    if (!x.done && i-- === 1 && size % 512) {
                       x.value = x.value.slice(0, size % 512) // Slice off suffix padding.
                     }
                     return x