diff options
Diffstat (limited to 'test')
-rw-r--r-- | test/cli/install/bun-add.test.ts | 22 | ||||
-rw-r--r-- | test/cli/install/bun-install.test.ts | 21 |
2 files changed, 23 insertions, 20 deletions
diff --git a/test/cli/install/bun-add.test.ts b/test/cli/install/bun-add.test.ts index c00c0ba7b..14ae44f73 100644 --- a/test/cli/install/bun-add.test.ts +++ b/test/cli/install/bun-add.test.ts @@ -20,7 +20,11 @@ import { beforeAll(dummyBeforeAll); afterAll(dummyAfterAll); +let port: string; let add_dir: string; +beforeAll(() => { + port = new URL(root_url).port; +}); beforeEach(async () => { add_dir = await mkdtemp(join(await realpath(tmpdir()), "bun-add.test")); @@ -102,9 +106,9 @@ it("should reject missing package", async () => { }); expect(stderr).toBeDefined(); const err = await new Response(stderr).text(); - expect(err.includes("bun add")).toBeTrue(); - expect(err.includes("error: MissingPackageJSON")).toBeTrue(); - expect(err.includes(`note: error occured while resolving file:${add_path}`)).toBeTrue(); + expect(err).toContain("bun add"); + expect(err).toContain("error: MissingPackageJSON"); + expect(err).toContain(`note: error occured while resolving file:${add_path}`); expect(stdout).toBeDefined(); const out = await new Response(stdout).text(); @@ -144,9 +148,9 @@ it("should reject invalid path without segfault", async () => { }); expect(stderr).toBeDefined(); const err = await new Response(stderr).text(); - expect(err.includes("bun add")).toBeTrue(); - expect(err.includes("error: MissingPackageJSON")).toBeTrue(); - expect(err.includes(`note: error occured while resolving file://${add_path}`)).toBeTrue(); + expect(err).toContain("bun add"); + expect(err).toContain("error: MissingPackageJSON"); + expect(err).toContain(`note: error occured while resolving file://${add_path}`); expect(stdout).toBeDefined(); const out = await new Response(stdout).text(); @@ -189,7 +193,7 @@ it("should handle semver-like names", async () => { }); expect(stderr).toBeDefined(); const err = await new Response(stderr).text(); - expect(err.split(/\r?\n/)).toContain('error: package "1.2.3" not found localhost/1.2.3 404'); + expect(err.split(/\r?\n/)).toContain(`error: package "1.2.3" not found localhost:${port}/1.2.3 404`); expect(stdout).toBeDefined(); expect(await new Response(stdout).text()).toBe(""); expect(await exited).toBe(1); @@ -232,7 +236,7 @@ it("should handle @scoped names", async () => { }); expect(stderr).toBeDefined(); const err = await new Response(stderr).text(); - expect(err.split(/\r?\n/)).toContain('error: package "@bar/baz" not found localhost/@bar%2fbaz 404'); + expect(err.split(/\r?\n/)).toContain(`error: package "@bar/baz" not found localhost:${port}/@bar%2fbaz 404`); expect(stdout).toBeDefined(); expect(await new Response(stdout).text()).toBe(""); expect(await exited).toBe(1); @@ -1512,7 +1516,7 @@ async function installRedirectsToAdd(saveFlagFirst: boolean) { " 1 packages installed", ]); expect(await exited).toBe(0); - expect((await file(join(package_dir, "package.json")).text()).includes("bun-add.test")); + expect(await file(join(package_dir, "package.json")).text()).toInclude("bun-add.test"); } it("should add dependency alongside peerDependencies", async () => { diff --git a/test/cli/install/bun-install.test.ts b/test/cli/install/bun-install.test.ts index 25cd0aca7..354909745 100644 --- a/test/cli/install/bun-install.test.ts +++ b/test/cli/install/bun-install.test.ts @@ -96,7 +96,7 @@ it("should handle missing package", async () => { }); expect(stderr).toBeDefined(); const err = await new Response(stderr).text(); - expect(err.split(/\r?\n/)).toContain('error: package "foo" not found localhost/foo 404'); + expect(err.split(/\r?\n/)).toContain(`error: package "foo" not found localhost:${new URL(root_url).port}/foo 404`); expect(stdout).toBeDefined(); expect(await new Response(stdout).text()).toBeEmpty(); expect(await exited).toBe(1); @@ -6022,13 +6022,13 @@ describe("Registry URLs", () => { ["https://registry.npmjs.org/", false], ["https://artifactory.xxx.yyy/artifactory/api/npm/my-npm/", false], // https://github.com/oven-sh/bun/issues/3899 ["https://artifactory.xxx.yyy/artifactory/api/npm/my-npm", false], // https://github.com/oven-sh/bun/issues/5368 - ["", true], + // ["", true], ["https:example.org", false], ["https://////example.com///", false], ["https://example.com/https:example.org", false], ["https://example.com/[]?[]#[]", false], ["https://example/%?%#%", false], - ["c:", false], + ["c:", true], ["c:/", false], ["https://點看", false], // gets converted to punycode ["https://xn--c1yn36f/", false], @@ -6067,11 +6067,10 @@ describe("Registry URLs", () => { const err = await new Response(stderr).text(); if (fails) { - const url = regURL.at(-1) === "/" ? regURL : regURL + "/"; - expect(err.includes(`Failed to join registry \"${url}\" and package \"notapackage\" URLs`)).toBeTrue(); - expect(err.includes("error: InvalidURL")).toBeTrue(); + expect(err).toContain(`Failed to join registry "${regURL}" and package "notapackage" URLs`); + expect(err).toContain("error: InvalidURL"); } else { - expect(err.includes("error: notapackage@0.0.2 failed to resolve")).toBeTrue(); + expect(err).toContain("error: notapackage@0.0.2 failed to resolve"); } // fails either way, since notapackage is, well, not a real package. expect(await exited).not.toBe(0); @@ -6108,8 +6107,8 @@ describe("Registry URLs", () => { expect(stderr).toBeDefined(); const err = await new Response(stderr).text(); - expect(err.includes(`Failed to join registry \"${regURL}/\" and package \"notapackage\" URLs`)).toBeTrue(); - expect(err.includes("warn: InvalidURL")).toBeTrue(); + expect(err).toContain(`Failed to join registry "${regURL}" and package "notapackage" URLs`); + expect(err).toContain("warn: InvalidURL"); expect(await exited).toBe(0); }); @@ -6149,8 +6148,8 @@ describe("Registry URLs", () => { expect(stderr).toBeDefined(); const err = await new Response(stderr).text(); - expect(err.includes(`Failed to join registry \"${regURL}\" and package \"notapackage\" URLs`)).toBeTrue(); - expect(err.includes("warn: InvalidURL")).toBeTrue(); + expect(err).toContain(`Failed to join registry "${regURL}" and package "notapackage" URLs`); + expect(err).toContain("warn: InvalidURL"); expect(await exited).toBe(0); }); |