Add basic tests (#39)

- end-to-end build & dev in CI - better type-checking Related to #35
mxdvl · May 2, 2023 · e120fb9 · e120fb9
2 parents 2bdf667 + 52cf599
commit e120fb9
Show file tree

Hide file tree

Showing 6 changed files with 87 additions and 191 deletions.
diff --git a/.github/workflows/integrate.yml b/.github/workflows/integrate.yml
@@ -30,3 +30,9 @@ jobs:
 
       - name: Lint
         run: deno lint
+
+      - name: Test
+        run: deno test -A --no-check
+
+      - name: Check
+        run: deno check src/**.ts
diff --git a/deno.lock b/deno.lock
diff --git a/src/build.test.ts b/src/build.test.ts
@@ -0,0 +1,60 @@
+import { assert } from "https://deno.land/[email protected]/testing/asserts.ts";
+import { build } from "./build.ts";
+
+const base = "mononykus/";
+const site_dir = "src/_site";
+
+Deno.test({
+	name: "Able to build the current project",
+	fn: async () => {
+		await build({ base, site_dir });
+	},
+});
+
+Deno.test({
+	name: "Able to develop the current project",
+	fn: async () => {
+		const command = new Deno.Command(
+			Deno.execPath(),
+			{
+				args: [
+					"task",
+					"dev",
+				],
+				stdout: "null",
+				stderr: "null",
+			},
+		);
+
+		const process = command.spawn();
+
+		await new Promise<void>((resolve) => {
+			const check_if_port_is_open = async () => {
+				try {
+					const { status } = await fetch("http://localhost:4507/mononykus/", {
+						method: "HEAD",
+					});
+					if (status === 200) resolve();
+				} catch (_) {
+					setTimeout(check_if_port_is_open, 60);
+				}
+			};
+			return check_if_port_is_open();
+		});
+
+		const response = await fetch("http://localhost:4507/mononykus/");
+
+		const html = await response.text();
+
+		process.kill();
+		await process.output();
+
+		assert(html.startsWith(
+			"<!DOCTYPE html>",
+		));
+		assert(html.includes(
+			"<title>Mononykus – Deno + Svelte</title>",
+		));
+	},
+	sanitizeResources: true,
+});
diff --git a/src/build.ts b/src/build.ts
@@ -133,7 +133,7 @@ export const build = async (
 	const out_dir = slashify(_out_dir);
 	const site_dir = slashify(_site_dir);
 
-	clean(out_dir);
+	await clean(out_dir);
 
 	await rebuild({ base, out_dir, site_dir, minify });
 
@@ -152,7 +152,7 @@ export const watch = async (
 	const out_dir = slashify(_out_dir);
 	const site_dir = slashify(_site_dir);
 
-	clean(out_dir);
+	await clean(out_dir);
 
 	const _rebuild = () => rebuild({ base, out_dir, site_dir, minify });
 

diff --git a/src/esbuild_plugins/get_route_html.ts b/src/esbuild_plugins/get_route_html.ts
@@ -1,3 +1,4 @@
+// @ts-expect-error -- the package is untyped
 import { format } from "npm:prettier";
 
 interface TemplateOptions {

diff --git a/src/server.ts b/src/server.ts
@@ -1,19 +1,22 @@
 import { Handler } from "https://deno.land/[email protected]/http/server.ts";
 import { serveDir } from "https://deno.land/[email protected]/http/file_server.ts";
+import { normalize as normalise } from "https://deno.land/[email protected]/path/posix.ts";
 
 interface ServerOptions {
-	base?: string;
+	base: string;
 	out_dir: string;
 }
 
 export const create_handler = (
-	{ base = "", out_dir }: ServerOptions,
+	{ base, out_dir }: ServerOptions,
 ): Handler => ((req) => {
 	const url = new URL(req.url);
 
-	if (url.pathname.startsWith("/" + base)) {
+	if (url.pathname.startsWith(normalise("/" + base))) {
 		return serveDir(req, { fsRoot: out_dir, urlRoot: base });
 	} else {
-		return Response.redirect(new URL(base + url.pathname, url.origin));
+		return Response.redirect(
+			new URL(normalise(base + url.pathname), url.origin),
+		);
 	}
 });