@jant/core 0.3.35 → 0.3.37

package/src/routes/api/__tests__/pages.test.ts

@@ -1,218 +0,0 @@
-import { describe, it, expect } from "vitest";
-import { createTestApp } from "../../../__tests__/helpers/app.js";
-import { pagesApiRoutes } from "../pages.js";
-
-describe("Pages API Routes", () => {
-  describe("GET /api/pages", () => {
-    it("returns empty list when no pages exist", async () => {
-      const { app } = createTestApp();
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages");
-      expect(res.status).toBe(200);
-
-      const body = await res.json();
-      expect(body.pages).toEqual([]);
-    });
-
-    it("returns pages list", async () => {
-      const { app, services } = createTestApp();
-      app.route("/api/pages", pagesApiRoutes);
-
-      await services.pages.create({ slug: "about", title: "About" });
-      await services.pages.create({ slug: "contact", title: "Contact" });
-
-      const res = await app.request("/api/pages");
-      const body = await res.json();
-
-      expect(body.pages).toHaveLength(2);
-    });
-  });
-
-  describe("GET /api/pages/:id", () => {
-    it("returns a page by id", async () => {
-      const { app, services } = createTestApp();
-      app.route("/api/pages", pagesApiRoutes);
-
-      const page = await services.pages.create({
-        slug: "about",
-        title: "About Us",
-      });
-
-      const res = await app.request(`/api/pages/${page.id}`);
-      expect(res.status).toBe(200);
-
-      const body = await res.json();
-      expect(body.title).toBe("About Us");
-      expect(body.slug).toBe("about");
-    });
-
-    it("returns 400 for invalid id", async () => {
-      const { app } = createTestApp();
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages/abc");
-      expect(res.status).toBe(400);
-    });
-
-    it("returns 404 for non-existent page", async () => {
-      const { app } = createTestApp();
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages/9999");
-      expect(res.status).toBe(404);
-    });
-  });
-
-  describe("POST /api/pages", () => {
-    it("returns 401 when not authenticated", async () => {
-      const { app } = createTestApp({ authenticated: false });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages", {
-        method: "POST",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ slug: "about", title: "About" }),
-      });
-
-      expect(res.status).toBe(401);
-    });
-
-    it("creates a page when authenticated", async () => {
-      const { app } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages", {
-        method: "POST",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({
-          slug: "about",
-          title: "About Us",
-          body: "We are Jant.",
-          status: "published",
-        }),
-      });
-
-      expect(res.status).toBe(201);
-      const body = await res.json();
-      expect(body.slug).toBe("about");
-      expect(body.title).toBe("About Us");
-    });
-
-    it("returns 400 for missing slug", async () => {
-      const { app } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages", {
-        method: "POST",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ title: "No Slug" }),
-      });
-
-      expect(res.status).toBe(400);
-    });
-  });
-
-  describe("PUT /api/pages/:id", () => {
-    it("returns 401 when not authenticated", async () => {
-      const { app, services } = createTestApp({ authenticated: false });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const page = await services.pages.create({
-        slug: "about",
-        title: "About",
-      });
-
-      const res = await app.request(`/api/pages/${page.id}`, {
-        method: "PUT",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ title: "Updated" }),
-      });
-
-      expect(res.status).toBe(401);
-    });
-
-    it("updates a page when authenticated", async () => {
-      const { app, services } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const page = await services.pages.create({
-        slug: "about",
-        title: "About",
-      });
-
-      const res = await app.request(`/api/pages/${page.id}`, {
-        method: "PUT",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ title: "Updated About" }),
-      });
-
-      expect(res.status).toBe(200);
-      const body = await res.json();
-      expect(body.title).toBe("Updated About");
-    });
-
-    it("returns 404 for non-existent page", async () => {
-      const { app } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages/9999", {
-        method: "PUT",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ title: "test" }),
-      });
-
-      expect(res.status).toBe(404);
-    });
-  });
-
-  describe("DELETE /api/pages/:id", () => {
-    it("returns 401 when not authenticated", async () => {
-      const { app, services } = createTestApp({ authenticated: false });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const page = await services.pages.create({
-        slug: "about",
-        title: "About",
-      });
-
-      const res = await app.request(`/api/pages/${page.id}`, {
-        method: "DELETE",
-      });
-
-      expect(res.status).toBe(401);
-    });
-
-    it("deletes a page when authenticated", async () => {
-      const { app, services } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const page = await services.pages.create({
-        slug: "about",
-        title: "About",
-      });
-
-      const res = await app.request(`/api/pages/${page.id}`, {
-        method: "DELETE",
-      });
-
-      expect(res.status).toBe(200);
-      const body = await res.json();
-      expect(body.success).toBe(true);
-
-      const found = await services.pages.getById(page.id);
-      expect(found).toBeNull();
-    });
-
-    it("returns 404 for non-existent page", async () => {
-      const { app } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages/9999", {
-        method: "DELETE",
-      });
-
-      expect(res.status).toBe(404);
-    });
-  });
-});

package/src/routes/api/pages.ts

@@ -1,73 +0,0 @@
-/**
- * Pages API Routes
- */
-
-import { Hono } from "hono";
-import { z } from "zod";
-import type { Bindings } from "../../types.js";
-import type { AppVariables } from "../../types/app-context.js";
-import { requireAuthApi } from "../../middleware/auth.js";
-import {
-  CreatePageSchema,
-  StatusSchema,
-  parseValidated,
-} from "../../lib/schemas.js";
-import { assertFound, parseIntParam, NotFoundError } from "../../lib/errors.js";
-
-type Env = { Bindings: Bindings; Variables: AppVariables };
-
-export const pagesApiRoutes = new Hono<Env>();
-
-// API update schema extends shared schema with nullable fields for explicit clearing
-const UpdatePageSchema = CreatePageSchema.partial().extend({
-  title: z.string().nullable().optional(),
-  body: z.string().nullable().optional(),
-  status: StatusSchema.optional(),
-});
-
-// List pages
-pagesApiRoutes.get("/", async (c) => {
-  const pages = await c.var.services.pages.list();
-  return c.json({ pages });
-});
-
-// Get single page
-pagesApiRoutes.get("/:id", async (c) => {
-  const id = parseIntParam(c.req.param("id"));
-  const page = assertFound(await c.var.services.pages.getById(id), "Page");
-  return c.json(page);
-});
-
-// Create page (requires auth)
-pagesApiRoutes.post("/", requireAuthApi(), async (c) => {
-  const body = parseValidated(CreatePageSchema, await c.req.json());
-
-  const page = await c.var.services.pages.create({
-    slug: body.slug,
-    title: body.title,
-    body: body.body,
-    status: body.status,
-  });
-
-  return c.json(page, 201);
-});
-
-// Update page (requires auth)
-pagesApiRoutes.put("/:id", requireAuthApi(), async (c) => {
-  const id = parseIntParam(c.req.param("id"));
-  const body = parseValidated(UpdatePageSchema, await c.req.json());
-
-  const page = assertFound(await c.var.services.pages.update(id, body), "Page");
-
-  return c.json(page);
-});
-
-// Delete page (requires auth)
-pagesApiRoutes.delete("/:id", requireAuthApi(), async (c) => {
-  const id = parseIntParam(c.req.param("id"));
-
-  const success = await c.var.services.pages.delete(id);
-  if (!success) throw new NotFoundError("Page");
-
-  return c.json({ success: true });
-});

package/src/routes/dash/__tests__/pages.test.ts

@@ -1,226 +0,0 @@
-/**
- * Tests for the page/nav management logic used by dashboard pages routes.
- *
- * Note: Route handler tests that import JSX components with @lingui/react/macro
- * cannot run in vitest (requires SWC plugin). These tests verify the service
- * layer operations that the routes orchestrate.
- */
-
-import { describe, it, expect, beforeEach } from "vitest";
-import { createTestDatabase } from "../../../__tests__/helpers/db.js";
-import { createPageService } from "../../../services/page.js";
-import { createNavItemService } from "../../../services/navigation.js";
-import { createPathRegistryService } from "../../../services/path-registry.js";
-import type { Database } from "../../../db/index.js";
-
-describe("Dashboard Pages - Nav Management Logic", () => {
-  let db: Database;
-  let pageService: ReturnType<typeof createPageService>;
-  let navItemService: ReturnType<typeof createNavItemService>;
-
-  beforeEach(() => {
-    const testDb = createTestDatabase();
-    db = testDb.db as unknown as Database;
-    pageService = createPageService(db, createPathRegistryService(db));
-    navItemService = createNavItemService(db);
-  });
-
-  describe("add page to nav", () => {
-    it("creates a page-type nav item for the page", async () => {
-      const page = await pageService.create({
-        slug: "about",
-        title: "About Us",
-      });
-
-      // Simulate what the route handler does
-      await navItemService.create({
-        type: "page",
-        label: page.title || page.slug,
-        url: `/${page.slug}`,
-        pageId: page.id,
-      });
-
-      const navItems = await navItemService.list();
-      expect(navItems).toHaveLength(1);
-      expect(navItems[0]?.type).toBe("page");
-      expect(navItems[0]?.label).toBe("About Us");
-      expect(navItems[0]?.url).toBe("/about");
-      expect(navItems[0]?.pageId).toBe(page.id);
-    });
-
-    it("uses slug as label when page has no title", async () => {
-      const page = await pageService.create({ slug: "about" });
-
-      await navItemService.create({
-        type: "page",
-        label: page.title || page.slug,
-        url: `/${page.slug}`,
-        pageId: page.id,
-      });
-
-      const navItems = await navItemService.list();
-      expect(navItems[0]?.label).toBe("about");
-    });
-
-    it("page appears in nav and not in listNotInNav after adding", async () => {
-      const page = await pageService.create({
-        slug: "about",
-        title: "About",
-      });
-
-      // Before adding to nav
-      let notInNav = await pageService.listNotInNav();
-      expect(notInNav).toHaveLength(1);
-
-      // Add to nav
-      await navItemService.create({
-        type: "page",
-        label: page.title || page.slug,
-        url: `/${page.slug}`,
-        pageId: page.id,
-      });
-
-      // After adding to nav
-      notInNav = await pageService.listNotInNav();
-      expect(notInNav).toHaveLength(0);
-
-      const navItems = await navItemService.list();
-      expect(navItems).toHaveLength(1);
-    });
-  });
-
-  describe("remove page from nav", () => {
-    it("removes the nav item but keeps the page", async () => {
-      const page = await pageService.create({
-        slug: "about",
-        title: "About",
-      });
-      await navItemService.create({
-        type: "page",
-        label: "About",
-        url: "/about",
-        pageId: page.id,
-      });
-
-      // Simulate what the route handler does: find and delete nav item by pageId
-      const allNavItems = await navItemService.list();
-      const found = allNavItems.find((item) => item.pageId === page.id);
-      expect(found).toBeDefined();
-      await navItemService.delete(found?.id as number);
-
-      // Nav item should be gone
-      const navItems = await navItemService.list();
-      expect(navItems).toHaveLength(0);
-
-      // Page should still exist
-      const foundPage = await pageService.getById(page.id);
-      expect(foundPage).not.toBeNull();
-
-      // Page should appear in "not in nav" list
-      const notInNav = await pageService.listNotInNav();
-      expect(notInNav).toHaveLength(1);
-      expect(notInNav[0]?.slug).toBe("about");
-    });
-  });
-
-  describe("reorder nav items", () => {
-    it("reorders nav items by position", async () => {
-      const a = await navItemService.create({
-        type: "link",
-        label: "A",
-        url: "/a",
-      });
-      const b = await navItemService.create({
-        type: "link",
-        label: "B",
-        url: "/b",
-      });
-
-      // Reverse order
-      await navItemService.reorder([b.id, a.id]);
-
-      const items = await navItemService.list();
-      expect(items[0]?.label).toBe("B");
-      expect(items[1]?.label).toBe("A");
-    });
-  });
-
-  describe("link CRUD", () => {
-    it("creates a link nav item", async () => {
-      await navItemService.create({
-        type: "link",
-        label: "Blog",
-        url: "/blog",
-      });
-
-      const navItems = await navItemService.list();
-      expect(navItems).toHaveLength(1);
-      expect(navItems[0]?.type).toBe("link");
-      expect(navItems[0]?.label).toBe("Blog");
-      expect(navItems[0]?.url).toBe("/blog");
-    });
-
-    it("updates a link nav item", async () => {
-      const item = await navItemService.create({
-        type: "link",
-        label: "Blog",
-        url: "/blog",
-      });
-
-      await navItemService.update(item.id, {
-        label: "Posts",
-        url: "/posts",
-      });
-
-      const updated = await navItemService.getById(item.id);
-      expect(updated?.label).toBe("Posts");
-      expect(updated?.url).toBe("/posts");
-    });
-
-    it("deletes a link nav item", async () => {
-      const item = await navItemService.create({
-        type: "link",
-        label: "Blog",
-        url: "/blog",
-      });
-
-      await navItemService.delete(item.id);
-
-      const found = await navItemService.getById(item.id);
-      expect(found).toBeNull();
-    });
-  });
-
-  describe("unified page listing", () => {
-    it("separates pages into nav and non-nav groups", async () => {
-      const aboutPage = await pageService.create({
-        slug: "about",
-        title: "About",
-      });
-      await pageService.create({ slug: "contact", title: "Contact" });
-
-      // Add about to nav
-      await navItemService.create({
-        type: "page",
-        label: "About",
-        url: "/about",
-        pageId: aboutPage.id,
-      });
-
-      // Also add a link nav item
-      await navItemService.create({
-        type: "link",
-        label: "External",
-        url: "https://example.com",
-      });
-
-      // Simulate the unified page view data fetch
-      const navItems = await navItemService.list();
-      const otherPages = await pageService.listNotInNav();
-
-      expect(navItems).toHaveLength(2); // page + link
-      expect(otherPages).toHaveLength(1); // only contact
-      expect(otherPages[0]?.slug).toBe("contact");
-    });
-  });
-});