hendrik/lens
1
0
Fork 0
mirror of https://github.com/lensapp/lens.git synced 2025-05-20 05:10:56 +00:00
Code Issues Projects Releases Wiki Activity

feat: Handle non existing files or directories in linking

Browse Source 
Signed-off-by: Janne Savolainen <janne.savolainen@live.fi>
This commit is contained in:
Janne Savolainen 2023-04-24 08:51:56 +03:00
parent 603afc19a7
commit fc683158ca
No known key found for this signature in database
GPG Key ID: 8C6CFB2FFFE8F68A
2 changed files with 105 additions and 67 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
packages/infrastructure/lens-link/src/get-symlink-paths.injectable.ts
View File

@ -1,4 +1,4 @@
import { flatten, map, partition, uniq, uniqBy } from "lodash/fp";
import { flatten, map, partition, uniq, uniqBy, filter } from "lodash/fp";
import type { PackageJsonAndPath } from "./package-json-and-path";
import { globInjectable } from "./fs/glob.injectable";
import { resolvePathInjectable } from "./path/resolve-path.injectable";
@ -8,6 +8,7 @@ import { pipeline } from "@ogre-tools/fp";
import { getLensLinkDirectoryInjectable } from "./get-lens-link-directory.injectable";
import path from "path";
import { isFileOrDirectoryInjectable } from "./fs/is-file-or-directory.injectable";
import { existsInjectable } from "./fs/exists.injectable";
const shouldBeGlobbed = (possibleGlobString: string) => possibleGlobString.includes("*");
const simplifyGlobbing = new RegExp("(\\/\\*\\/\\*\\*|\\/\\*\\*|\\/\\*\\*\\/\\*|\\/\\*)$");
@ -21,6 +22,7 @@ export const getSymlinkPathsInjectable = getInjectable({
const resolvePath = di.inject(resolvePathInjectable);
const getLensLinkDirectory = di.inject(getLensLinkDirectoryInjectable);
const isFileOrDirectory = di.inject(isFileOrDirectoryInjectable);
const exists = di.inject(existsInjectable);
return async (packageJsons: PackageJsonAndPath[]) => {
return pipeline(
@ -46,7 +48,13 @@ export const getSymlinkPathsInjectable = getInjectable({
uniq,
map(async (fileOrDirectory) => {
map(async (fileOrDirectory) => ({ fileOrDirectory, exists: await exists(fileOrDirectory) })),
awaitAll,
filter(({ exists }) => exists),
map(async ({ fileOrDirectory }) => {
const target = resolvePath(moduleDirectory, fileOrDirectory);
return {

32
packages/infrastructure/lens-link/src/lens-link.test.ts
View File

@ -198,6 +198,9 @@ describe("lens-link", () => {
await ensureEmptyDirectoryMock.resolve();
await ensureEmptyDirectoryMock.resolve();
await existsMock.resolve(true);
await existsMock.resolve(true);
await isFileOrDirectoryMock.resolve("dir");
await isFileOrDirectoryMock.resolve("dir");
});
@ -290,6 +293,11 @@ describe("lens-link", () => {
beforeEach(async () => {
await globMock.resolve(["some-directory-from-glob/some-file-from-glob.txt", "some-duplicate-file"]);
await existsMock.resolve(true);
await existsMock.resolve(true);
await existsMock.resolve(true);
await existsMock.resolve(true);
await isFileOrDirectoryMock.resolve("file");
await isFileOrDirectoryMock.resolve("dir");
await isFileOrDirectoryMock.resolve("dir");
@ -351,7 +359,7 @@ describe("lens-link", () => {
await readJsonFileMock.resolveSpecific(([path]) => path === "/some-directory/some-module/package.json", {
name: "some-module",
files: ["some-build-directory", "some-file"],
files: ["some-build-directory", "some-file", "some-non-existing-file.txt"],
main: "some-build-directory/index.js",
});
@ -386,6 +394,27 @@ describe("lens-link", () => {
expect(createSymlinkMock).not.toHaveBeenCalled();
});
it("calls to detect if file or directory exists", () => {
expect(existsMock.mock.calls).toEqual([
["some-build-directory"],
["some-file"],
["some-non-existing-file.txt"],
["some-other-build-directory"],
]);
});
it("does not check for files and directories yet", () => {
expect(isFileOrDirectoryMock).not.toHaveBeenCalled();
});
describe("when existence of files and directories resolve with some non existing", () => {
beforeEach(async () => {
await existsMock.resolve(true);
await existsMock.resolve(true);
await existsMock.resolve(false);
await existsMock.resolve(true);
});
it("calls for if symlinks to be created are for files or directories", () => {
expect(isFileOrDirectoryMock.mock.calls).toEqual([
["/some-directory/some-module/some-build-directory"],
@ -473,3 +502,4 @@ describe("lens-link", () => {
});
});
});
});