fix(test): cI and show warning when pathname duplicated (#137)

denoland · Aug 22, 2023 · 6402993 · 6402993
1 parent 9115589
commit 6402993
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 7 deletions.
diff --git a/blog.tsx b/blog.tsx
@@ -284,6 +284,10 @@ async function loadPost(postsDirectory: string, path: string) {
     readTime: readingTime(content),
     renderMath: data.get("render_math"),
   };
+
+  if (POSTS.get(pathname)) {
+    console.warn(`Duplicate blog post path: ${pathname}`);
+  }
   POSTS.set(pathname, post);
 }
 

diff --git a/blog_test.ts b/blog_test.ts
@@ -152,18 +152,20 @@ Deno.test("posts/ fourth", async () => {
   );
 });
 
-Deno.test("posts/ fifth", async () => {
-  const resp = await testHandler(new Request("https://blog.deno.dev/fifth"));
+Deno.test("posts/ seventh", async () => {
+  const resp = await testHandler(
+    new Request("https://blog.deno.dev/uses-pathname"),
+  );
   assert(resp);
   assertEquals(resp.status, 200);
   assertEquals(resp.headers.get("content-type"), "text/html; charset=utf-8");
   const body = await resp.text();
   assertStringIncludes(body, `<html lang="en-GB">`);
   assertStringIncludes(
     body,
-    `<link rel="canonical" href="https://blog.deno.dev/fifth" />`,
+    `<link rel="canonical" href="https://blog.deno.dev/uses-pathname" />`,
   );
-  assertStringIncludes(body, `fifth post`);
+  assertStringIncludes(body, `seventh post`);
   assertStringIncludes(body, `<time dateTime="2022-05-02T00:00:00.000Z">`);
   assertStringIncludes(body, `<p>Lorem Ipsum is simply dummy text`);
 });

diff --git a/testdata/posts/fifth-uses-pathname.md → testdata/posts/seventh.md b/testdata/posts/fifth-uses-pathname.md → testdata/posts/seventh.md
@@ -1,8 +1,8 @@
 ---
-title: Fifth post
-pathname: "/fifth"
+title: Seventh post
+pathname: "/uses-pathname"
 publish_date: 2022-05-02
-abstract: This is the fifth post.
+abstract: This is the seventh post.
 ---
 
 Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem