foster/denoland-deno
0
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2025-02-01 20:25:12 -05:00
Code Issues Packages 1 Wiki Activity

fix(std/tar): fix constant condition (#8010)

Browse source
This commit is contained in:
Marcos Casagrande 2020-10-19 13:33:51 +02:00 • committed by GitHub
parent f91c1155f0
commit d3dea24560
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
std/archive/tar.ts
View file

@ -485,7 +485,10 @@ class TarEntry implements Reader {
entryBytesLeft,
);
if (entryBytesLeft <= 0) return null;
if (entryBytesLeft <= 0) {
this.#consumed = true;
return null;
}
const block = new Uint8Array(bufSize);
const n = await readBlock(this.#reader, block);
@ -493,9 +496,7 @@ class TarEntry implements Reader {
this.#read += n || 0;
if (n === null || bytesLeft <= 0) {
// FIXME(bartlomieju): this condition makes no sense
// deno-lint-ignore no-constant-condition
if (null) this.#consumed = true;
if (n === null) this.#consumed = true;
return null;
}

10
std/archive/tar_test.ts
View file

@ -113,7 +113,9 @@ Deno.test("appendFileWithLongNameToTarArchive", async function (): Promise<
const untar = new Untar(tar.getReader());
const result = await untar.extract();
assert(result !== null);
assert(!result.consumed);
const untarText = new TextDecoder("utf-8").decode(await Deno.readAll(result));
assert(result.consumed);
// tests
assertEquals(result.fileName, fileName);
@ -137,6 +139,7 @@ Deno.test("untarAsyncIterator", async function (): Promise<void> {
// read data from a tar archive
const untar = new Untar(tar.getReader());
let lastEntry;
for await (const entry of untar) {
const expected = entries.shift();
assert(expected);
@ -145,11 +148,14 @@ Deno.test("untarAsyncIterator", async function (): Promise<void> {
if (expected.filePath) {
content = await Deno.readFile(expected.filePath);
}
assertEquals(content, await Deno.readAll(entry));
assertEquals(expected.name, entry.fileName);
}
if (lastEntry) assert(lastEntry.consumed);
lastEntry = entry;
}
assert(lastEntry);
assert(lastEntry.consumed);
assertEquals(entries.length, 0);
});