Skip to content

Commit c0fc947

Browse files
committed
Mask file-type bits in permission assertions
Deno.statSync().mode includes file-type bits (e.g. 0o100644 for regular files), not just permission bits. Use & 0o777 to compare only permission bits in assertEquals calls.
1 parent 393bce9 commit c0fc947

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

tests/unit/file-permissions.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ unitTest(
3232

3333
ensureUserWritable(file);
3434

35-
assertEquals(safeModeFromFile(file), 0o644,
36-
"Mode should be exactly 0o644 (0o444 | 0o200) — only user write bit added");
35+
assertEquals(safeModeFromFile(file)! & 0o777, 0o644,
36+
"Permission bits should be 0o644 (0o444 | 0o200) — only user write bit added");
3737
}),
3838
{ ignore: isWindows },
3939
);
@@ -74,7 +74,7 @@ unitTest(
7474

7575
ensureUserWritable(dest);
7676

77-
assertEquals(safeModeFromFile(dest), 0o644,
77+
assertEquals(safeModeFromFile(dest)! & 0o777, 0o644,
7878
"Copied file should be user-writable after ensureUserWritable");
7979
}),
8080
{ ignore: isWindows },

0 commit comments

Comments
 (0)