Fix 6206: Cloudflare function path resolving one step above (#6208)

* Fix 6206: Cloudflare function path resolving one step above
withastro · Feb 13, 2023 · 79f49ac · 79f49ac
1 parent 8b7cb64
commit 79f49ac
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/.changeset/lazy-coins-compete.md b/.changeset/lazy-coins-compete.md
@@ -0,0 +1,5 @@
+---
+'@astrojs/cloudflare': patch
+---
+
+Fix path file that was generated outside the functions folder
diff --git a/packages/integrations/cloudflare/src/index.ts b/packages/integrations/cloudflare/src/index.ts
@@ -203,8 +203,9 @@ export default function createIntegration(args?: Options): AstroIntegration {
 				}
 
 				if (isModeDirectory) {
-					const functionsUrl = new URL('functions', _config.root);
+					const functionsUrl = new URL('functions/', _config.root);
 					await fs.promises.mkdir(functionsUrl, { recursive: true });
+
 					const directoryUrl = new URL('[[path]].js', functionsUrl);
 					await fs.promises.rename(finalBuildUrl, directoryUrl);
 				}

diff --git a/packages/integrations/cloudflare/test/directory.test.js b/packages/integrations/cloudflare/test/directory.test.js
@@ -17,5 +17,6 @@ describe('mode: "directory"', () => {
 
 	it('generates functions folder inside the project root', async () => {
 		expect(await fixture.pathExists('../functions')).to.be.true;
+		expect(await fixture.pathExists('../functions/[[path]].js')).to.be.true;
 	});
 });