foster/denoland-deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-11-21 15:04:11 -05:00
Code Issues Packages 1 Wiki Activity

Revert "feat(runtime): support urls for Deno.realPath and Deno.realPathSync (#10626)"

Browse source 
This reverts commit 910935c071.
This commit is contained in:
Luca Casonato 2021-05-17 15:26:07 +02:00 committed by GitHub
parent 94eabfeeba
commit 6d63167420
3 changed files with 4 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
cli/dts/lib.deno.ns.d.ts vendored
View file

@ -1467,7 +1467,7 @@ declare namespace Deno {
* Requires `allow-read` permission for the target path. * Requires `allow-read` permission for the target path.
* Also requires `allow-read` permission for the CWD if the target path is * Also requires `allow-read` permission for the CWD if the target path is
* relative.*/ * relative.*/
export function realPathSync(path: string | URL): string; export function realPathSync(path: string): string;
/** Resolves to the absolute normalized path, with symbolic links resolved. /** Resolves to the absolute normalized path, with symbolic links resolved.
* *
@ -1483,7 +1483,7 @@ declare namespace Deno {
* Requires `allow-read` permission for the target path. * Requires `allow-read` permission for the target path.
* Also requires `allow-read` permission for the CWD if the target path is * Also requires `allow-read` permission for the CWD if the target path is
* relative.*/ * relative.*/
export function realPath(path: string | URL): Promise<string>; export function realPath(path: string): Promise<string>;
export interface DirEntry { export interface DirEntry {
name: string; name: string;

17
cli/tests/unit/real_path_test.ts
View file

@ -1,11 +1,9 @@
// Copyright 2018-2021 the Deno authors. All rights reserved. MIT license. // Copyright 2018-2021 the Deno authors. All rights reserved. MIT license.
import { import {
assert, assert,
assertEquals,
assertMatch, assertMatch,
assertThrows, assertThrows,
assertThrowsAsync, assertThrowsAsync,
pathToAbsoluteFileUrl,
unitTest, unitTest,
} from "./test_util.ts"; } from "./test_util.ts";
@ -21,12 +19,6 @@ unitTest({ perms: { read: true } }, function realPathSyncSuccess(): void {
} }
}); });
unitTest({ perms: { read: true } }, function realPathSyncUrl(): void {
const relative = "cli/tests/fixture.json";
const url = pathToAbsoluteFileUrl(relative);
assertEquals(Deno.realPathSync(relative), Deno.realPathSync(url));
});
unitTest( unitTest(
{ {
perms: { read: true, write: true }, perms: { read: true, write: true },
@ -74,15 +66,6 @@ unitTest({ perms: { read: true } }, async function realPathSuccess(): Promise<
} }
}); });
unitTest(
{ perms: { read: true } },
async function realPathUrl(): Promise<void> {
const relative = "cli/tests/fixture.json";
const url = pathToAbsoluteFileUrl(relative);
assertEquals(await Deno.realPath(relative), await Deno.realPath(url));
},
);
unitTest( unitTest(
{ {
perms: { read: true, write: true }, perms: { read: true, write: true },

4
runtime/js/30_fs.js
View file

@ -128,11 +128,11 @@
} }
function realPathSync(path) { function realPathSync(path) {
return core.opSync("op_realpath_sync", pathFromURL(path)); return core.opSync("op_realpath_sync", path);
} }
function realPath(path) { function realPath(path) {
return core.opAsync("op_realpath_async", pathFromURL(path)); return core.opAsync("op_realpath_async", path);
} }
function removeSync( function removeSync(