c8340f8
@@ -68,7 +68,7 @@
export const limbHref = (
  action: LimbAction | "replan",
  under?: string,
): string =>
  `${milestoneHref(base, number)}/limb?action=${action}${under === undefined ? "" : `&under=${under}`}`;
  `${milestoneHref(base, number)}/limb?action=${action}${under === undefined ? "" : `&under=${encodeURIComponent(under)}`}`;
/**
 * Raise and Lower for a limb among its siblings in one group (`top`, or a
@@ -0,0 +1,21 @@
import { expect, test } from "bun:test";
import { limbHref, milestoneHref } from "../src/web/pages/milestones.ts";
test("milestoneHref is the base, /milestones/, and the number", () => {
  expect(milestoneHref("/a/b", 7)).toBe("/a/b/milestones/7");
});
test("limbHref names the action, and `under` only when given", () => {
  expect(limbHref("/a/b", 7, "defer")).toBe("/a/b/milestones/7/limb?action=defer");
  expect(limbHref("/a/b", 7, "raise", "top")).toBe(
    "/a/b/milestones/7/limb?action=raise&under=top",
  );
});
test("limbHref encodes `under` so & and # cannot cut the link", () => {
  const href = limbHref("/a/b", 7, "lower", "x&y#z");
  expect(href).toBe("/a/b/milestones/7/limb?action=lower&under=x%26y%23z");
  const q = new URL(href, "http://x").searchParams;
  expect(q.get("under")).toBe("x&y#z");
  expect(q.get("action")).toBe("lower");
});
		@@ -68,7 +68,7 @@ export const limbHref = (
68	68	action: LimbAction \| "replan",
69	69	under?: string,
70	70	): string =>
71		`${milestoneHref(base, number)}/limb?action=${action}${under === undefined ? "" : `&under=${under}`}`;
	71	`${milestoneHref(base, number)}/limb?action=${action}${under === undefined ? "" : `&under=${encodeURIComponent(under)}`}`;
72	72
73	73	/**
74	74	* Raise and Lower for a limb among its siblings in one group (`top`, or a
		@@ -0,0 +1,21 @@
	1	import { expect, test } from "bun:test";
	2	import { limbHref, milestoneHref } from "../src/web/pages/milestones.ts";
	3
	4	test("milestoneHref is the base, /milestones/, and the number", () => {
	5	expect(milestoneHref("/a/b", 7)).toBe("/a/b/milestones/7");
	6	});
	7
	8	test("limbHref names the action, and `under` only when given", () => {
	9	expect(limbHref("/a/b", 7, "defer")).toBe("/a/b/milestones/7/limb?action=defer");
	10	expect(limbHref("/a/b", 7, "raise", "top")).toBe(
	11	"/a/b/milestones/7/limb?action=raise&under=top",
	12	);
	13	});
	14
	15	test("limbHref encodes `under` so & and # cannot cut the link", () => {
	16	const href = limbHref("/a/b", 7, "lower", "x&y#z");
	17	expect(href).toBe("/a/b/milestones/7/limb?action=lower&under=x%26y%23z");
	18	const q = new URL(href, "http://x").searchParams;
	19	expect(q.get("under")).toBe("x&y#z");
	20	expect(q.get("action")).toBe("lower");
	21	});