Skip to content

test: simplify the suite to the behavior it protects - #145

Merged
datlechin merged 4 commits into
mainfrom
test/simplify
Nov 5, 2026
Merged

datlechin merged 4 commits into
mainfrom
test/simplify

Conversation

@datlechin

@datlechin datlechin commented Nov 5, 2026 •

Copy link
Copy Markdown
Member
  • Pest: 102 files, 2377 cases (2106 + 271 ssr) to 48 files, 1034 cases (994 + 40 ssr). Local run: non-ssr 37s to 27s, ssr 18s to 10s.
  • JS: 32 files, 309 tests to 27 files, 223 tests.
  • Test PHP: 24.3k to 9.1k lines.

Removed:

  • Copy and wording pins: Content/* guards (banned claims, positioning, typed counts, copy style, reviewed facts), forbidden variants, app label glosses, legal text line by line, banner and language bar lengths. They failed on prose edits, not on bugs.
  • Duplicates: per-page "renders in every locale" and "is indexed" (Pages/PagesTest and the sitemap own them), NFC three times, catalog key parity (typecheck owns it), cookies and credentials twice.
  • SSR markup checks beyond render, filled tokens, own catalog and internal links.
  • CSS, source and design greps: Chrome/*, Theme/*, typography, contrast, attribution source reads, event names.
  • Pinned data values: prices, engine and feature counts, versions, dates, names. This includes the "every price unchanged" pin; prices still follow checkout by hand (CLAUDE.md, Content).
  • SuiteHygieneTest.

Kept and moved:

# Conflicts:
#	tests/Feature/Content/ReviewedFactsTest.php
#	tests/Feature/Data/FactsDataTest.php
#	tests/Feature/Landing/LegalPagesTest.php
#	tests/Feature/Localization/AppLabelsTest.php
#	tests/js/prose-html.test.ts
#	tests/js/public-request-credentials.test.ts
@datlechin
datlechin merged commit 2909c1e into main Nov 5, 2026
4 checks passed
@datlechin
datlechin deleted the test/simplify branch November 5, 2026 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant