9e1d057878
Batch of fixes from the automated QA pass on develop. Each was reproduced and then verified fixed live (browser/curl); logic-bearing fixes have unit tests. Functional bugs: - #122 collab-token was capped by the anonymous public-share-AI throttler (5/min); skip all non-AUTH named throttlers on this auth-guarded, client-cached route. - #123 editor onAuthenticationFailed threw `jwtDecode(undefined)` and never reconnected; read the token via a ref, guard the decode (incl. missing exp), and refetch+reconnect on any auth failure. - #124 a slash command containing a space ("/Heading 1") inserted literal text; enable allowSpaces and close the menu when the query matches no items. - #125 space slug auto-gen produced uppercase initials for multi-word names; computeSpaceSlug now yields a lowercase alphanumeric slug. - #126 AI chat window position/size now persisted (atomWithStorage) across reload; also fixes a latent ResizeObserver-attach bug on first open. - #127 workspace name update accepted URLs; add @NoUrls (parity with setup). - #132 icon-columns 4/5 passed calc() into SVG width/height attrs (console spam); size via style. share-for-page query returns null instead of undefined. - #134 "Reindex now" counter looked stuck: reindex runs async; the client now polls coverage (bounded) so the counter climbs live; misleading server comment reworded. UX / consistency: - #128 add success toasts to favorite/label/avatar/member-(de)activate. - #129 "1 result found" pluralization; hide the single-option Type filter. - #130 replace raw Zod strings with friendly messages (name/password/group). - #131 unify "Untitled" casing in tree/breadcrumb/tab; stop force-uppercasing space-name chips; fix confirm-dialog labels (Cancel / Remove), invite placeholder typo, Export/Move-to-space labels. - #133 disable profile Save when clean; toast on unsupported avatar image; style the invalid-invitation page with a CTA; hide Share for read-only users; align the dictation "not configured" message; "Go to login page" typo. Tests: computeSpaceSlug, workspace-name NoUrls DTO, share-query null normalization, slash getSuggestionItems empty-close. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
50 lines
1.6 KiB
TypeScript
50 lines
1.6 KiB
TypeScript
import { describe, it, expect } from "vitest";
|
|
import { computeSpaceSlug } from "@/lib/utils.tsx";
|
|
|
|
// `computeSpaceSlug` derives a space slug that must satisfy the server-side
|
|
// @IsAlphanumeric / ^[a-zA-Z0-9]+$ constraint: lowercase the name and strip
|
|
// every non-[a-z0-9] character (spaces, punctuation, and non-ascii letters).
|
|
// No hyphens, no uppercase, no separators survive.
|
|
describe("computeSpaceSlug", () => {
|
|
it("strips the space between two words", () => {
|
|
expect(computeSpaceSlug("Product Team")).toBe("productteam");
|
|
});
|
|
|
|
it("lowercases and joins a two-word name", () => {
|
|
expect(computeSpaceSlug("Hello World")).toBe("helloworld");
|
|
});
|
|
|
|
it("lowercases a single word with no separators", () => {
|
|
expect(computeSpaceSlug("SingleWord")).toBe("singleword");
|
|
});
|
|
|
|
it("lowercases an all-caps word and removes the inner space", () => {
|
|
expect(computeSpaceSlug("UPPER case")).toBe("uppercase");
|
|
});
|
|
|
|
it("drops non-ascii characters, keeping ascii letters and digits", () => {
|
|
// "Привет" (Cyrillic) is stripped entirely; only "a", "b" and "1" remain.
|
|
expect(computeSpaceSlug("a b Привет 1")).toBe("ab1");
|
|
});
|
|
|
|
it("returns an empty string for whitespace-only input", () => {
|
|
expect(computeSpaceSlug(" ")).toBe("");
|
|
});
|
|
|
|
it("always produces output matching /^[a-z0-9]*$/", () => {
|
|
const samples = [
|
|
"Product Team",
|
|
"Hello World",
|
|
"SingleWord",
|
|
"UPPER case",
|
|
"a b Привет 1",
|
|
" ",
|
|
"Mixed-123 !@#",
|
|
"Café Münster",
|
|
];
|
|
for (const sample of samples) {
|
|
expect(computeSpaceSlug(sample)).toMatch(/^[a-z0-9]*$/);
|
|
}
|
|
});
|
|
});
|