🚨 Add tests for workbook (#2020)

Author: KATO-Hiro

src/test/lib/utils/url.test.ts

@@ -1,6 +1,6 @@
 import { expect, test } from 'vitest';
 
-import { isValidUrl, sanitizeUrl } from '$lib/utils/url';
+import { isValidUrl, isValidUrlSlug, sanitizeUrl } from '$lib/utils/url';
 
 type TestCaseForUrlValidation = {
   rawUrl: string;
@@ -61,6 +61,67 @@ describe('URL', () => {
     }
   });
 
+  describe('is valid URL slug', () => {
+    describe('when valid URL slugs are given', () => {
+      const testCases = [
+        { rawUrl: 'a' },
+        { rawUrl: 'a'.repeat(2) },
+        { rawUrl: 'a'.repeat(30) },
+        { rawUrl: 'bfs' },
+        { rawUrl: 'dfs' },
+        { rawUrl: 'dp' },
+        { rawUrl: 'union-find' },
+        { rawUrl: 'warshall-floyd' },
+        { rawUrl: 'digit-dp' },
+        { rawUrl: '2-sat' },
+        { rawUrl: 'directed-acyclic-graph' },
+      ];
+
+      runTests('isValidUrlSlug', testCases, ({ rawUrl }: TestCaseForUrlValidation) => {
+        expect(isValidUrlSlug(rawUrl)).toBeTruthy();
+      });
+    });
+
+    describe('when invalid URL slugs are given', () => {
+      const testCases = [
+        { rawUrl: '' },
+        { rawUrl: '-bfs' },
+        { rawUrl: 'bfs-' },
+        { rawUrl: 'bfs--dfs' },
+        { rawUrl: 'Bfs' },
+        { rawUrl: 'A' },
+        { rawUrl: 'BFS' },
+        { rawUrl: 'Union_Find' },
+        { rawUrl: 'union_find' },
+        { rawUrl: 'union.find' },
+        { rawUrl: 'union/find' },
+        { rawUrl: 'union@find' },
+        { rawUrl: '@union-find' },
+        { rawUrl: 'union-find@' },
+        { rawUrl: 'directed acyclic graph' },
+        { rawUrl: '-' },
+        { rawUrl: '--' },
+        { rawUrl: 'ー' },
+        { rawUrl: '１' },
+        { rawUrl: '２' },
+        { rawUrl: '９' },
+        { rawUrl: '１０' },
+      ];
+
+      runTests('isValidUrlSlug', testCases, ({ rawUrl }: TestCaseForUrlValidation) => {
+        expect(isValidUrlSlug(rawUrl)).toBeFalsy();
+      });
+    });
+
+    function runTests(
+      testName: string,
+      testCases: TestCasesForUrlValidation,
+      testFunction: (testCase: TestCaseForUrlValidation) => void,
+    ) {
+      test.each(testCases)(`${testName}(rawUrl: $rawUrl)`, testFunction);
+    }
+  });
+
   describe('sanitize URL', () => {
     describe('when sanitization is not required URLs are given', () => {
       const testCases = [

src/test/lib/utils/workbook.test.ts

@@ -1,9 +1,9 @@
 import { expect, test } from 'vitest';
-import { parseWorkBookId } from '$lib/utils/workbook';
+import { parseWorkBookId, parseWorkBookUrlSlug } from '$lib/utils/workbook';
 
 type TestCase = {
   slug: string;
-  expected: number | null;
+  expected: number | string | null;
 };
 
 type TestCases = TestCase[];
@@ -90,3 +90,47 @@ describe('Workbook', () => {
     }
   });
 });
+
+describe('parse workbook URL slug', () => {
+  describe('when valid URL slugs are given', () => {
+    const testCases = [
+      { slug: 'a', expected: 'a' },
+      { slug: 'a'.repeat(30), expected: 'a'.repeat(30) },
+      { slug: 'bfs', expected: 'bfs' },
+      { slug: 'union-find', expected: 'union-find' },
+      { slug: '2-sat', expected: '2-sat' },
+    ];
+
+    runTests('parseWorkBookUrlSlug', testCases, ({ slug, expected }: TestCase) => {
+      expect(parseWorkBookUrlSlug(slug)).toBe(expected);
+    });
+  });
+
+  describe('when invalid URL slugs are given', () => {
+    const testCases = [
+      { slug: '', expected: null },
+      { slug: ' ', expected: null },
+      { slug: 'bfs dfs', expected: null },
+      { slug: 'invalid@slug', expected: null },
+      { slug: 'invalid#slug', expected: null },
+      { slug: 'invalid/slug', expected: null },
+      { slug: 'UPPERCASE', expected: null },
+      { slug: 'slug_with_underscore', expected: null },
+      { slug: '-invalid-start', expected: null },
+      { slug: 'invalid-end-', expected: null },
+      { slug: 'double--dash', expected: null },
+    ];
+
+    runTests('parseWorkBookUrlSlug', testCases, ({ slug, expected }: TestCase) => {
+      expect(parseWorkBookUrlSlug(slug)).toBe(expected);
+    });
+  });
+
+  function runTests(
+    testName: string,
+    testCases: TestCases,
+    testFunction: (testCase: TestCase) => void,
+  ) {
+    test.each(testCases)(`${testName}(slug: $slug)`, testFunction);
+  }
+});