Skip to content

Commit a2245c5

Browse files
add tests for keep_names config option
1 parent 80835ca commit a2245c5

File tree

1 file changed

+78
-0
lines changed

1 file changed

+78
-0
lines changed

packages/wrangler/src/__tests__/deploy.test.ts

Lines changed: 78 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6571,6 +6571,84 @@ addEventListener('fetch', event => {});`
65716571
`);
65726572
expect(std.err).toMatchInlineSnapshot(`""`);
65736573
});
6574+
6575+
it("should apply esbuild's keep-names functionality by default", async () => {
6576+
writeWranglerConfig({
6577+
main: "./index.js",
6578+
legacy_env: false,
6579+
env: {
6580+
testEnv: {},
6581+
},
6582+
});
6583+
fs.writeFileSync(
6584+
"./index.js",
6585+
`
6586+
export
6587+
default {
6588+
fetch() {
6589+
function sayHello() {
6590+
return "Hello World with keep_names";
6591+
}
6592+
return new Response(sayHello());
6593+
}
6594+
}
6595+
`
6596+
);
6597+
6598+
const underscoreUnderscoreNameRegex = /__name\(.*?\)/;
6599+
6600+
mockUploadWorkerRequest({
6601+
env: "testEnv",
6602+
expectedType: "esm",
6603+
legacyEnv: false,
6604+
expectedEntry: (str) => {
6605+
expect(str).toMatch(underscoreUnderscoreNameRegex);
6606+
},
6607+
});
6608+
6609+
mockSubDomainRequest();
6610+
await runWrangler("deploy -e testEnv index.js");
6611+
});
6612+
6613+
it("should apply esbuild's keep-names functionality unless keep_names is set to false", async () => {
6614+
writeWranglerConfig({
6615+
main: "./index.js",
6616+
legacy_env: false,
6617+
env: {
6618+
testEnv: {
6619+
keep_names: false,
6620+
},
6621+
},
6622+
});
6623+
fs.writeFileSync(
6624+
"./index.js",
6625+
`
6626+
export
6627+
default {
6628+
fetch() {
6629+
function sayHello() {
6630+
return "Hello World without keep_names";
6631+
}
6632+
return new Response(sayHello());
6633+
}
6634+
}
6635+
`
6636+
);
6637+
6638+
const underscoreUnderscoreNameRegex = /__name\(.*?\)/;
6639+
6640+
mockUploadWorkerRequest({
6641+
env: "testEnv",
6642+
expectedType: "esm",
6643+
legacyEnv: false,
6644+
expectedEntry: (str) => {
6645+
expect(str).not.toMatch(underscoreUnderscoreNameRegex);
6646+
},
6647+
});
6648+
6649+
mockSubDomainRequest();
6650+
await runWrangler("deploy -e testEnv index.js");
6651+
});
65746652
});
65756653

65766654
describe("durable object migrations", () => {

0 commit comments

Comments
 (0)