deno/tests/unit_node/_fs/_fs_fstat_test.ts
Daniel Osvaldo Rahmanto 432761aac5
Some checks are pending
ci / build libs (push) Blocked by required conditions
ci / test debug windows-x86_64 (push) Blocked by required conditions
ci / test release windows-x86_64 (push) Blocked by required conditions
ci / pre-build (push) Waiting to run
ci / test debug linux-aarch64 (push) Blocked by required conditions
ci / test release linux-aarch64 (push) Blocked by required conditions
ci / test debug macos-aarch64 (push) Blocked by required conditions
ci / test release macos-aarch64 (push) Blocked by required conditions
ci / bench release linux-x86_64 (push) Blocked by required conditions
ci / lint debug linux-x86_64 (push) Blocked by required conditions
ci / lint debug macos-x86_64 (push) Blocked by required conditions
ci / lint debug windows-x86_64 (push) Blocked by required conditions
ci / test debug linux-x86_64 (push) Blocked by required conditions
ci / test release linux-x86_64 (push) Blocked by required conditions
ci / test debug macos-x86_64 (push) Blocked by required conditions
ci / test release macos-x86_64 (push) Blocked by required conditions
ci / publish canary (push) Blocked by required conditions
fix(ext/node): fs.stat and fs.statSync compatibility (#30637)
Closes #30570

Changes in this PR:
- Implement `ino`, `nlink`, and `blocks` properties of `Deno.FileInfo`
on Windows. These changes are automatically reflected to the
corresponding node stat function. In order to do so, I had to tinker
with the
[createByteStruct](a3a904da14/ext/fs/30_fs.js (L297))
function to create another optional int type, apart from `?u64`. It's
common for small sized files on Windows (particularly NTFS file system)
to have a `Stats.blocks` property of 0, and currently all 0 values with
type `?u64` will be coerced into `null` by `createByteStruct`.
- Refactor the `BigIntStats` and `Stats` class, to use the same class
with Node.js that are provided from
[utils.mjs](7f8e488c36/ext/node/polyfills/internal/fs/utils.mjs (L577)).
Also ensures that all properties are not `null` or `undefined`.
- Addresses the `prefer-primordials` lint rule.
2025-09-15 15:29:30 +02:00

108 lines
3.1 KiB
TypeScript

// Copyright 2018-2025 the Deno authors. MIT license.
import { fstat, fstatSync } from "node:fs";
import { fail } from "@std/assert";
import type { BigIntStats, Stats } from "node:fs";
import { assertStats, assertStatsBigInt } from "../_test_utils.ts";
Deno.test({
name: "ASYNC: get a file Stats",
// TODO(bartlomieju): this test is broken in Deno 2, because `file.rid` is undefined.
// The fs APIs should be rewritten to use actual FDs, not RIDs
ignore: true,
async fn() {
const filePath = await Deno.makeTempFile();
using file = await Deno.open(filePath);
await new Promise<Stats>((resolve, reject) => {
// @ts-ignore (iuioiua) `file.rid` should no longer be needed once FDs are used
fstat(file.rid, (err: Error | null, stat: Stats) => {
if (err) reject(err);
resolve(stat);
});
})
.then(
(stat) => {
assertStats(stat, file.statSync());
},
() => fail(),
)
.finally(() => {
Deno.removeSync(filePath);
});
},
});
Deno.test({
name: "ASYNC: get a file BigInt Stats",
// TODO(bartlomieju): this test is broken in Deno 2, because `file.rid` is undefined.
// The fs APIs should be rewritten to use actual FDs, not RIDs
ignore: true,
async fn() {
const filePath = await Deno.makeTempFile();
using file = await Deno.open(filePath);
await new Promise<BigIntStats>((resolve, reject) => {
fstat(
// @ts-ignore (iuioiua) `file.rid` should no longer be needed once FDs are used
file.rid,
{ bigint: true },
(err: Error | null, stat: BigIntStats) => {
if (err) reject(err);
resolve(stat);
},
);
})
.then(
(stat) => assertStatsBigInt(stat, file.statSync()),
() => fail(),
)
.finally(() => {
Deno.removeSync(filePath);
});
},
});
Deno.test({
name: "SYNC: get a file Stats",
// TODO(bartlomieju): this test is broken in Deno 2, because `file.rid` is undefined.
// The fs APIs should be rewritten to use actual FDs, not RIDs
ignore: true,
fn() {
const filePath = Deno.makeTempFileSync();
using file = Deno.openSync(filePath);
try {
// @ts-ignore (iuioiua) `file.rid` should no longer be needed once FDs are used
assertStats(fstatSync(file.rid), file.statSync());
} finally {
Deno.removeSync(filePath);
}
},
});
Deno.test({
name: "SYNC: get a file BigInt Stats",
// TODO(bartlomieju): this test is broken in Deno 2, because `file.rid` is undefined.
// The fs APIs should be rewritten to use actual FDs, not RIDs
ignore: true,
fn() {
const filePath = Deno.makeTempFileSync();
using file = Deno.openSync(filePath);
try {
// HEAD
// @ts-ignore (iuioiua) `file.rid` should no longer be needed once FDs are used
assertStatsBigInt(fstatSync(file.rid, { bigint: true }), file.statSync());
//
assertStatsBigInt(
// @ts-ignore (iuioiua) `file.rid` should no longer be needed once FDs are used
fstatSync(file.rid, { bigint: true }),
file.statSync(),
);
//main
} finally {
Deno.removeSync(filePath);
}
},
});