"""n8n — INSTALL overlay (Phase 1d, DG4): override + extend-by-composition.

Reuses the generic "really serving" assertion, then ADDS the recipe-specific checks: /healthz answers
200, and a real browser loads the live n8n editor SPA over HTTPS (D2 install + D3 Playwright).
Assertion-only on the shared deployment."""

import os
import sys

sys.path.insert(0, os.path.join(os.path.dirname(__file__), "..", "..", "runner"))
from harness import browser as harness_browser  # noqa: E402
from harness import generic, lifecycle


def test_serving_and_editor(live_app, meta):
    # extend-by-composition: reuse the generic "really serving" assertion first ...
    generic.assert_serving(live_app, meta)

    # ... then the recipe-specific assertions.
    status = lifecycle.http_get(live_app, "/healthz")
    assert status == 200, f"expected 200 from {live_app}/healthz, got {status}"

    # A real browser loads the live n8n editor SPA over HTTPS.
    # n8n's boot is staged: /healthz returns 200 before / route is registered. So we may briefly
    # get 404 from /, then a 200 with the "starting up" placeholder, then the actual SPA. The
    # centralized harness.browser.goto_with_retry helper handles both the status-mismatch retry
    # AND transient PlaywrightError (net::ERR_*) — F2-3 hardening.
    from playwright.sync_api import sync_playwright

    url = f"https://{live_app}/"
    with sync_playwright() as p:
        browser = p.chromium.launch(args=["--no-sandbox"])
        try:
            ctx = browser.new_context(ignore_https_errors=True)
            page = ctx.new_page()
            resp = harness_browser.goto_with_retry(page, url, accept_statuses=(200, 304))
            assert resp is not None and resp.status in (
                200,
                304,
            ), f"page status {resp and resp.status}"
            body = page.content().lower()
            assert "n8n" in body or "<html" in body, "no n8n content served"
        finally:
            browser.close()
