mirror of
https://github.com/denoland/deno.git
synced 2025-02-08 07:16:56 -05:00
![Marvin Hagemeister](/assets/img/avatar_default.png)
This commit changes "Deno.cwd()" (as well as "process.cwd()") to no longer require full "--allow-read" permission. This change was meant to be done in Deno 2.0.0, but somehow it slipped. Requiring full read permission just to read the CWD is a mistake, because CWD can already be obtained with no permission by throwing an error in JS and inspecting its stack. Fixes https://github.com/denoland/deno/issues/27110 --------- Co-authored-by: Bartek Iwańczuk <biwanczuk@gmail.com>
53 lines
1.3 KiB
TypeScript
53 lines
1.3 KiB
TypeScript
// Copyright 2018-2025 the Deno authors. MIT license.
|
|
import { assert, assertEquals, assertThrows } from "./test_util.ts";
|
|
|
|
Deno.test({ permissions: { read: true } }, function dirCwdNotNull() {
|
|
assert(Deno.cwd() != null);
|
|
});
|
|
|
|
Deno.test(
|
|
{ permissions: { read: true, write: true } },
|
|
function dirCwdChdirSuccess() {
|
|
const initialdir = Deno.cwd();
|
|
const path = Deno.makeTempDirSync();
|
|
Deno.chdir(path);
|
|
const current = Deno.cwd();
|
|
if (Deno.build.os === "darwin") {
|
|
assertEquals(current, "/private" + path);
|
|
} else {
|
|
assertEquals(current, path);
|
|
}
|
|
Deno.chdir(initialdir);
|
|
},
|
|
);
|
|
|
|
Deno.test({ permissions: { read: true, write: true } }, function dirCwdError() {
|
|
// excluding windows since it throws resource busy, while removeSync
|
|
if (["linux", "darwin"].includes(Deno.build.os)) {
|
|
const initialdir = Deno.cwd();
|
|
const path = Deno.makeTempDirSync();
|
|
Deno.chdir(path);
|
|
Deno.removeSync(path);
|
|
try {
|
|
assertThrows(() => {
|
|
Deno.cwd();
|
|
}, Deno.errors.NotFound);
|
|
} finally {
|
|
Deno.chdir(initialdir);
|
|
}
|
|
}
|
|
});
|
|
|
|
Deno.test(
|
|
{ permissions: { read: true, write: true } },
|
|
function dirChdirError() {
|
|
const path = Deno.makeTempDirSync() + "test";
|
|
assertThrows(
|
|
() => {
|
|
Deno.chdir(path);
|
|
},
|
|
Deno.errors.NotFound,
|
|
`chdir '${path}'`,
|
|
);
|
|
},
|
|
);
|