3 Commits

Author	SHA1	Message	Date
codex_a	155c8f7569	fix: nav/footer wireframe, honeypot CSP, explore topics, comment E2E coverage ... - Replace nav inline newsletter form with Subscribe CTA link per wireframe - Remove newsletter form from footer; add Connect section with social/RSS links - Fix honeypot inputs using hidden attribute (inline style blocked by CSP) - Add available_tags to HomePage.get_context for Explore Topics section - Add data-comment-form attribute to main comment form for reliable locating - Seed approved comment in E2E content for reply flow testing - Expand test_comments.py: moderation message, not-immediately-visible, missing fields, reply form visible, reply submission - Make COMMENT_RATE_LIMIT_PER_MINUTE configurable; set 100 in dev to prevent E2E test exhaustion; update rate limit unit test with override_settings - Update newsletter/home E2E tests to reflect nav form removal - Update unit test to assert no nav/footer newsletter forms Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>	2026-03-01 12:17:55 +00:00
codex_a	4992b0cb9d	fix: resolve 5 E2E test failures from first CI run ... - test_homepage_title_contains_brand: to_have_title() requires a string or regex, not a lambda; switch to re.compile('No Hype AI') - test_granular_preferences_save_dismisses_banner: wrong element clicked to open <details>; use 'details summary' locator directly - test_subscribe_invalid_email_shows_error: browser HTML5 email validation swallows the submit event before the JS handler fires; add 'novalidate' via evaluate() so the fetch still runs and the server returns 400 - test_copy_link_button_updates_text: clipboard API unavailable in headless Docker; add polyfill + pre-grant permissions in conftest page fixture so the JS success path runs and button text becomes 'Copied' - test_comments_section_absent_when_disabled: guard against Wagtail's add_child() resetting BooleanField defaults by calling an explicit .update(comments_enabled=False) + re-setting on the instance before save_revision().publish(); also tighten test to assert 200 + correct title Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>	2026-02-28 19:47:13 +00:00
codex_a	9d323d2040	feat: add comprehensive Playwright E2E test suite ... - Create e2e/ directory with 7 test modules covering: - Home page: title, nav links, theme toggle, newsletter form - Cookie consent: accept all, reject all, granular prefs, persistence - Article index: loads, tag filter, click-through navigation - Article detail: title/read-time, share section, comments, newsletter aside, related - Comments: valid submit → redirect, empty body → error display, disabled check - Newsletter: JS confirmation message, invalid email error, aside form, duplicate - Feeds: RSS/sitemap/robots.txt validity, tag feed, seeded content present - Extend seed_e2e_content management command with tagged article, about page, no-comments article, and legal pages for richer test coverage - Add seed command tests (create + idempotency) to keep coverage ≥ 90% - Add pr-e2e CI job (runs on pull_request): builds image, starts postgres + app, installs playwright, runs pytest e2e/ - Update nightly-e2e to run full e2e/ suite alongside legacy journey test - Add --ignore=e2e to unit-test pytest step (coverage must not include browser tests) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>	2026-02-28 19:30:43 +00:00