Compare commits
4 Commits
7ecb26cc92
...
latest
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
cc9b706b70 | ||
|
|
b0593ab431 | ||
|
|
3d5bfd5401 | ||
|
|
fa6adea3c1 |
10
CHANGELOG.md
10
CHANGELOG.md
@@ -1,3 +1,13 @@
|
||||
## [1.1.13] - 2026-02-15
|
||||
|
||||
* This release fixes the intermittent setupSuperUser bootstrap timeout by making superuser-form detection and submission more robust across timing and DOM variations, with added integration coverage and full passing E2E/integration tests.
|
||||
|
||||
|
||||
## [1.1.12] - 2026-02-14
|
||||
|
||||
* This release fixes the intermittent Matomo installer failure in the setupSuperUser step by adding more robust waiting logic and introduces E2E tests for deployments under very tight resource constraints.
|
||||
|
||||
|
||||
## [1.1.11] - 2026-02-14
|
||||
|
||||
* This release improves matomo-bootstrap installer resilience by adding robust setupSuperUser field and button detection to prevent intermittent bootstrap failures.
|
||||
|
||||
@@ -20,7 +20,7 @@
|
||||
rec {
|
||||
matomo-bootstrap = python.pkgs.buildPythonApplication {
|
||||
pname = "matomo-bootstrap";
|
||||
version = "1.1.11"; # keep in sync with pyproject.toml
|
||||
version = "1.1.13"; # keep in sync with pyproject.toml
|
||||
pyproject = true;
|
||||
src = self;
|
||||
|
||||
|
||||
@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
|
||||
|
||||
[project]
|
||||
name = "matomo-bootstrap"
|
||||
version = "1.1.11"
|
||||
version = "1.1.13"
|
||||
description = "Headless bootstrap tooling for Matomo (installation + API token provisioning)"
|
||||
readme = "README.md"
|
||||
requires-python = ">=3.10"
|
||||
|
||||
@@ -34,6 +34,9 @@ INSTALLER_TABLES_CREATION_TIMEOUT_S = int(
|
||||
INSTALLER_TABLES_ERASE_TIMEOUT_S = int(
|
||||
os.environ.get("MATOMO_INSTALLER_TABLES_ERASE_TIMEOUT_S", "120")
|
||||
)
|
||||
INSTALLER_SUPERUSER_RELOAD_INTERVAL_S = int(
|
||||
os.environ.get("MATOMO_INSTALLER_SUPERUSER_RELOAD_INTERVAL_S", "30")
|
||||
)
|
||||
INSTALLER_DEBUG_DIR = os.environ.get(
|
||||
"MATOMO_INSTALLER_DEBUG_DIR", "/tmp/matomo-bootstrap"
|
||||
).rstrip("/")
|
||||
@@ -74,6 +77,11 @@ SUPERUSER_LOGIN_SELECTORS = (
|
||||
"input[name='login']",
|
||||
"form#generalsetupform input[name='login']",
|
||||
)
|
||||
SUPERUSER_FORM_SELECTORS = (
|
||||
"form#generalsetupform",
|
||||
"form[action*='setupSuperUser']",
|
||||
"form[action*='action=setupSuperUser']",
|
||||
)
|
||||
SUPERUSER_PASSWORD_SELECTORS = (
|
||||
"#password-0",
|
||||
"#password",
|
||||
@@ -376,6 +384,19 @@ def _has_superuser_login_field(page, *, timeout_s: float = 0.2) -> bool:
|
||||
return loc is not None
|
||||
|
||||
|
||||
def _has_superuser_form_container(page, *, timeout_s: float = 0.2) -> bool:
|
||||
loc, _ = _first_present_css_locator(
|
||||
page, SUPERUSER_FORM_SELECTORS, timeout_s=timeout_s
|
||||
)
|
||||
return loc is not None
|
||||
|
||||
|
||||
def _superuser_form_ready(page, *, timeout_s: float = 0.2) -> bool:
|
||||
return _has_superuser_login_field(
|
||||
page, timeout_s=timeout_s
|
||||
) or _has_superuser_form_container(page, timeout_s=timeout_s)
|
||||
|
||||
|
||||
def _has_first_website_name_field(page, *, timeout_s: float = 0.2) -> bool:
|
||||
loc, _ = _first_present_css_locator(
|
||||
page, FIRST_WEBSITE_NAME_SELECTORS, timeout_s=timeout_s
|
||||
@@ -388,6 +409,52 @@ def _has_continue_to_matomo_action(page, *, timeout_s: float = 0.2) -> bool:
|
||||
return loc is not None
|
||||
|
||||
|
||||
def _wait_for_superuser_login_field(
|
||||
page, *, timeout_s: float, poll_interval_ms: int = 300
|
||||
) -> bool:
|
||||
if timeout_s <= 0:
|
||||
return _superuser_form_ready(page, timeout_s=0.2)
|
||||
|
||||
deadline = time.time() + timeout_s
|
||||
last_wait_log_at = 0.0
|
||||
last_reload_at = time.time()
|
||||
|
||||
while time.time() < deadline:
|
||||
_wait_dom_settled(page)
|
||||
if _superuser_form_ready(page, timeout_s=0.2):
|
||||
return True
|
||||
|
||||
now = time.time()
|
||||
if (
|
||||
INSTALLER_SUPERUSER_RELOAD_INTERVAL_S > 0
|
||||
and now - last_reload_at >= INSTALLER_SUPERUSER_RELOAD_INTERVAL_S
|
||||
):
|
||||
try:
|
||||
page.reload(wait_until="domcontentloaded")
|
||||
_wait_dom_settled(page)
|
||||
_log(
|
||||
"[install] Reloaded setupSuperUser page while waiting "
|
||||
"for superuser form."
|
||||
)
|
||||
except Exception as exc:
|
||||
_log(f"[install] setupSuperUser reload attempt failed: {exc}")
|
||||
last_reload_at = now
|
||||
if _superuser_form_ready(page, timeout_s=0.2):
|
||||
return True
|
||||
|
||||
if now - last_wait_log_at >= 5:
|
||||
_log(
|
||||
"[install] setupSuperUser reached but superuser form is not visible yet; "
|
||||
f"waiting (url={page.url}, step={_get_step_hint(page.url)})"
|
||||
)
|
||||
_page_warnings(page)
|
||||
last_wait_log_at = now
|
||||
|
||||
page.wait_for_timeout(poll_interval_ms)
|
||||
|
||||
return _superuser_form_ready(page, timeout_s=0.2)
|
||||
|
||||
|
||||
def _fill_required_input(page, selectors, value: str, *, label: str) -> None:
|
||||
loc, _ = _first_present_css_locator(page, selectors, timeout_s=1.0)
|
||||
if loc is None:
|
||||
@@ -417,6 +484,7 @@ def _fill_optional_input(page, selectors, value: str) -> bool:
|
||||
def _installer_interactive(page) -> bool:
|
||||
checks = [
|
||||
_has_superuser_login_field(page),
|
||||
_has_superuser_form_container(page),
|
||||
_has_first_website_name_field(page),
|
||||
_has_continue_to_matomo_action(page),
|
||||
]
|
||||
@@ -424,6 +492,95 @@ def _installer_interactive(page) -> bool:
|
||||
return any(checks) or loc is not None
|
||||
|
||||
|
||||
def _submit_superuser_form_via_dom(
|
||||
page, *, user: str, password: str, email: str
|
||||
) -> bool:
|
||||
try:
|
||||
return bool(
|
||||
page.evaluate(
|
||||
"""
|
||||
([user, password, email]) => {
|
||||
const form =
|
||||
document.querySelector("form#generalsetupform")
|
||||
|| document.querySelector("form[action*='setupSuperUser']")
|
||||
|| document.querySelector("form[action*='action=setupSuperUser']");
|
||||
if (!form) return false;
|
||||
|
||||
const pick = (selectors) => {
|
||||
for (const selector of selectors) {
|
||||
const candidate = form.querySelector(selector);
|
||||
if (candidate) return candidate;
|
||||
}
|
||||
return null;
|
||||
};
|
||||
|
||||
const loginInput = pick([
|
||||
"input[name='login']",
|
||||
"input#login",
|
||||
"input[id^='login-']",
|
||||
"input[name*='login']",
|
||||
"input[name*='user']",
|
||||
"input[type='text']",
|
||||
]);
|
||||
const passwordInput = pick([
|
||||
"input[name='password']",
|
||||
"input#password",
|
||||
"input[id^='password-']",
|
||||
"input[type='password']:not([name='password_bis'])",
|
||||
"input[type='password']",
|
||||
]);
|
||||
const repeatPasswordInput = pick([
|
||||
"input[name='password_bis']",
|
||||
"input#password_bis",
|
||||
"input[id^='password_bis-']",
|
||||
"input[name*='repeat']",
|
||||
]);
|
||||
const emailInput = pick([
|
||||
"input[name='email']",
|
||||
"input#email",
|
||||
"input[id^='email-']",
|
||||
"input[type='email']",
|
||||
"input[name*='mail']",
|
||||
]);
|
||||
|
||||
if (!loginInput || !passwordInput || !emailInput) return false;
|
||||
|
||||
const setValue = (element, value) => {
|
||||
element.value = value;
|
||||
element.dispatchEvent(new Event("input", { bubbles: true }));
|
||||
element.dispatchEvent(new Event("change", { bubbles: true }));
|
||||
};
|
||||
|
||||
setValue(loginInput, user);
|
||||
setValue(passwordInput, password);
|
||||
if (repeatPasswordInput) {
|
||||
setValue(repeatPasswordInput, password);
|
||||
}
|
||||
setValue(emailInput, email);
|
||||
|
||||
const submit = form.querySelector(
|
||||
"button[type='submit'],input[type='submit']"
|
||||
);
|
||||
if (submit) {
|
||||
submit.click();
|
||||
return true;
|
||||
}
|
||||
|
||||
if (typeof form.requestSubmit === "function") {
|
||||
form.requestSubmit();
|
||||
} else {
|
||||
form.submit();
|
||||
}
|
||||
return true;
|
||||
}
|
||||
""",
|
||||
[user, password, email],
|
||||
)
|
||||
)
|
||||
except Exception:
|
||||
return False
|
||||
|
||||
|
||||
def _wait_for_installer_interactive(page, *, timeout_s: int) -> None:
|
||||
_log(f"[install] Waiting for interactive installer UI (timeout={timeout_s}s)...")
|
||||
deadline = time.time() + timeout_s
|
||||
@@ -481,6 +638,12 @@ def _click_next_with_wait(page, *, timeout_s: int) -> str:
|
||||
f"staying on step {current_step} (url {current_url})"
|
||||
)
|
||||
return current_step
|
||||
if _has_superuser_form_container(page, timeout_s=0.2):
|
||||
_log(
|
||||
"[install] Superuser form container became available without explicit click; "
|
||||
f"staying on step {current_step} (url {current_url})"
|
||||
)
|
||||
return current_step
|
||||
if _has_first_website_name_field(page, timeout_s=0.2):
|
||||
_log(
|
||||
"[install] First website form became available without explicit click; "
|
||||
@@ -735,13 +898,22 @@ class WebInstaller(Installer):
|
||||
|
||||
progress_deadline = time.time() + INSTALLER_STEP_DEADLINE_S
|
||||
|
||||
while not _has_superuser_login_field(page):
|
||||
if time.time() >= progress_deadline:
|
||||
while not _superuser_form_ready(page):
|
||||
now = time.time()
|
||||
if now >= progress_deadline:
|
||||
raise RuntimeError(
|
||||
"Installer did not reach superuser step "
|
||||
"Installer did not reach superuser form "
|
||||
f"within {INSTALLER_STEP_DEADLINE_S}s "
|
||||
f"(url={page.url}, step={_get_step_hint(page.url)})."
|
||||
)
|
||||
|
||||
current_step = _get_step_hint(page.url)
|
||||
if "setupSuperUser" in current_step:
|
||||
remaining_s = max(0.0, progress_deadline - now)
|
||||
if _wait_for_superuser_login_field(page, timeout_s=remaining_s):
|
||||
break
|
||||
continue
|
||||
|
||||
if _resolve_tables_creation_conflict(
|
||||
page, timeout_s=INSTALLER_TABLES_ERASE_TIMEOUT_S
|
||||
):
|
||||
@@ -755,73 +927,40 @@ class WebInstaller(Installer):
|
||||
_click_next_with_wait(page, timeout_s=step_timeout)
|
||||
_page_warnings(page)
|
||||
|
||||
_fill_required_input(
|
||||
submitted_superuser = _submit_superuser_form_via_dom(
|
||||
page,
|
||||
SUPERUSER_LOGIN_SELECTORS,
|
||||
config.admin_user,
|
||||
label="superuser login",
|
||||
user=config.admin_user,
|
||||
password=config.admin_password,
|
||||
email=config.admin_email,
|
||||
)
|
||||
_fill_required_input(
|
||||
page,
|
||||
SUPERUSER_PASSWORD_SELECTORS,
|
||||
config.admin_password,
|
||||
label="superuser password",
|
||||
)
|
||||
_fill_optional_input(
|
||||
page, SUPERUSER_PASSWORD_REPEAT_SELECTORS, config.admin_password
|
||||
)
|
||||
_fill_required_input(
|
||||
page,
|
||||
SUPERUSER_EMAIL_SELECTORS,
|
||||
config.admin_email,
|
||||
label="superuser email",
|
||||
)
|
||||
_page_warnings(page)
|
||||
|
||||
submitted_superuser = False
|
||||
try:
|
||||
submitted_superuser = bool(
|
||||
page.evaluate(
|
||||
"""
|
||||
([user, password, email]) => {
|
||||
const form = document.querySelector("form#generalsetupform");
|
||||
if (!form) return false;
|
||||
|
||||
const loginInput = form.querySelector("input[name='login']");
|
||||
const passwordInput = form.querySelector("input[name='password']");
|
||||
const repeatPasswordInput = form.querySelector("input[name='password_bis']");
|
||||
const emailInput = form.querySelector("input[name='email']");
|
||||
if (!loginInput || !passwordInput || !emailInput) return false;
|
||||
|
||||
loginInput.value = user;
|
||||
passwordInput.value = password;
|
||||
if (repeatPasswordInput) {
|
||||
repeatPasswordInput.value = password;
|
||||
}
|
||||
emailInput.value = email;
|
||||
|
||||
if (typeof form.requestSubmit === "function") {
|
||||
form.requestSubmit();
|
||||
} else {
|
||||
form.submit();
|
||||
}
|
||||
return true;
|
||||
}
|
||||
""",
|
||||
[
|
||||
config.admin_user,
|
||||
config.admin_password,
|
||||
config.admin_email,
|
||||
],
|
||||
)
|
||||
)
|
||||
except Exception:
|
||||
submitted_superuser = False
|
||||
|
||||
if submitted_superuser:
|
||||
_wait_dom_settled(page)
|
||||
_log("[install] Submitted superuser form via form.requestSubmit().")
|
||||
else:
|
||||
_fill_required_input(
|
||||
page,
|
||||
SUPERUSER_LOGIN_SELECTORS,
|
||||
config.admin_user,
|
||||
label="superuser login",
|
||||
)
|
||||
_fill_required_input(
|
||||
page,
|
||||
SUPERUSER_PASSWORD_SELECTORS,
|
||||
config.admin_password,
|
||||
label="superuser password",
|
||||
)
|
||||
_fill_optional_input(
|
||||
page, SUPERUSER_PASSWORD_REPEAT_SELECTORS, config.admin_password
|
||||
)
|
||||
_fill_required_input(
|
||||
page,
|
||||
SUPERUSER_EMAIL_SELECTORS,
|
||||
config.admin_email,
|
||||
label="superuser email",
|
||||
)
|
||||
_page_warnings(page)
|
||||
|
||||
submit_loc, submit_label = _first_present_css_locator(
|
||||
page, SUPERUSER_SUBMIT_SELECTORS, timeout_s=0.5
|
||||
)
|
||||
@@ -838,10 +977,10 @@ class WebInstaller(Installer):
|
||||
superuser_progress_deadline = time.time() + INSTALLER_STEP_TIMEOUT_S
|
||||
while time.time() < superuser_progress_deadline:
|
||||
_wait_dom_settled(page)
|
||||
if not _has_superuser_login_field(page):
|
||||
if not _superuser_form_ready(page):
|
||||
break
|
||||
page.wait_for_timeout(300)
|
||||
if _has_superuser_login_field(page):
|
||||
if _superuser_form_ready(page):
|
||||
_page_warnings(page)
|
||||
raise RuntimeError(
|
||||
"Superuser form submit did not progress to first website setup "
|
||||
|
||||
32
tests/e2e/docker-compose.slow.yml
Normal file
32
tests/e2e/docker-compose.slow.yml
Normal file
@@ -0,0 +1,32 @@
|
||||
services:
|
||||
db:
|
||||
cpus: 0.35
|
||||
mem_reservation: 192m
|
||||
mem_limit: 320m
|
||||
healthcheck:
|
||||
interval: 10s
|
||||
timeout: 5s
|
||||
retries: 90
|
||||
|
||||
matomo:
|
||||
cpus: 0.35
|
||||
mem_reservation: 192m
|
||||
mem_limit: 384m
|
||||
healthcheck:
|
||||
interval: 15s
|
||||
timeout: 8s
|
||||
retries: 120
|
||||
start_period: 120s
|
||||
|
||||
bootstrap:
|
||||
cpus: 0.75
|
||||
mem_reservation: 512m
|
||||
mem_limit: 1g
|
||||
environment:
|
||||
MATOMO_TIMEOUT: "120"
|
||||
MATOMO_PLAYWRIGHT_NAV_TIMEOUT_MS: "120000"
|
||||
MATOMO_INSTALLER_READY_TIMEOUT_S: "420"
|
||||
MATOMO_INSTALLER_STEP_TIMEOUT_S: "120"
|
||||
MATOMO_INSTALLER_STEP_DEADLINE_S: "420"
|
||||
MATOMO_INSTALLER_TABLES_CREATION_TIMEOUT_S: "360"
|
||||
MATOMO_INSTALLER_TABLES_ERASE_TIMEOUT_S: "240"
|
||||
@@ -7,13 +7,21 @@ import urllib.request
|
||||
|
||||
|
||||
COMPOSE_FILE = os.environ.get("MATOMO_STACK_COMPOSE_FILE", "docker-compose.yml")
|
||||
SLOW_COMPOSE_FILE = os.environ.get(
|
||||
"MATOMO_STACK_SLOW_COMPOSE_FILE", "tests/e2e/docker-compose.slow.yml"
|
||||
)
|
||||
|
||||
# Pick a non-default port to avoid collisions with other CI stacks that use 8080
|
||||
MATOMO_PORT = os.environ.get("MATOMO_PORT", "18080")
|
||||
MATOMO_HOST_URL = os.environ.get("MATOMO_STACK_URL", f"http://127.0.0.1:{MATOMO_PORT}")
|
||||
MATOMO_SLOW_PORT = os.environ.get("MATOMO_SLOW_PORT", "18081")
|
||||
MATOMO_SLOW_HOST_URL = os.environ.get(
|
||||
"MATOMO_SLOW_STACK_URL", f"http://127.0.0.1:{MATOMO_SLOW_PORT}"
|
||||
)
|
||||
|
||||
# How long we wait for Matomo HTTP to respond at all (seconds)
|
||||
WAIT_TIMEOUT_SECONDS = int(os.environ.get("MATOMO_STACK_WAIT_TIMEOUT", "180"))
|
||||
SLOW_WAIT_TIMEOUT_SECONDS = int(os.environ.get("MATOMO_SLOW_STACK_WAIT_TIMEOUT", "420"))
|
||||
|
||||
|
||||
def _run(
|
||||
@@ -32,8 +40,13 @@ def _run(
|
||||
)
|
||||
|
||||
|
||||
def _compose_cmd(*args: str) -> list[str]:
|
||||
return ["docker", "compose", "-f", COMPOSE_FILE, *args]
|
||||
def _compose_cmd(*args: str, compose_files: list[str] | None = None) -> list[str]:
|
||||
files = compose_files or [COMPOSE_FILE]
|
||||
cmd = ["docker", "compose"]
|
||||
for compose_file in files:
|
||||
cmd.extend(["-f", compose_file])
|
||||
cmd.extend(args)
|
||||
return cmd
|
||||
|
||||
|
||||
def _wait_for_http_any_status(url: str, timeout_s: int) -> None:
|
||||
@@ -108,12 +121,19 @@ class TestRootDockerComposeStack(unittest.TestCase):
|
||||
extra_env={"MATOMO_PORT": MATOMO_PORT},
|
||||
)
|
||||
|
||||
def test_root_docker_compose_yml_stack_bootstraps_and_token_works(self) -> None:
|
||||
# Build bootstrap image from Dockerfile (as defined in docker-compose.yml)
|
||||
def _assert_stack_bootstraps_and_token_works(
|
||||
self,
|
||||
*,
|
||||
compose_files: list[str],
|
||||
matomo_port: str,
|
||||
matomo_host_url: str,
|
||||
wait_timeout_seconds: int,
|
||||
bootstrap_retries: int = 2,
|
||||
) -> None:
|
||||
build = _run(
|
||||
_compose_cmd("build", "bootstrap"),
|
||||
_compose_cmd("build", "bootstrap", compose_files=compose_files),
|
||||
check=False,
|
||||
extra_env={"MATOMO_PORT": MATOMO_PORT},
|
||||
extra_env={"MATOMO_PORT": matomo_port},
|
||||
)
|
||||
self.assertEqual(
|
||||
build.returncode,
|
||||
@@ -121,11 +141,10 @@ class TestRootDockerComposeStack(unittest.TestCase):
|
||||
f"compose build failed\nstdout:\n{build.stdout}\nstderr:\n{build.stderr}",
|
||||
)
|
||||
|
||||
# Start db + matomo (bootstrap is one-shot and started via "run")
|
||||
up = _run(
|
||||
_compose_cmd("up", "-d", "db", "matomo"),
|
||||
_compose_cmd("up", "-d", "db", "matomo", compose_files=compose_files),
|
||||
check=False,
|
||||
extra_env={"MATOMO_PORT": MATOMO_PORT},
|
||||
extra_env={"MATOMO_PORT": matomo_port},
|
||||
)
|
||||
self.assertEqual(
|
||||
up.returncode,
|
||||
@@ -133,17 +152,14 @@ class TestRootDockerComposeStack(unittest.TestCase):
|
||||
f"compose up failed\nstdout:\n{up.stdout}\nstderr:\n{up.stderr}",
|
||||
)
|
||||
|
||||
# Wait until Matomo answers on the published port
|
||||
_wait_for_http_any_status(MATOMO_HOST_URL + "/", WAIT_TIMEOUT_SECONDS)
|
||||
_wait_for_http_any_status(matomo_host_url + "/", wait_timeout_seconds)
|
||||
|
||||
# Run bootstrap: it should print ONLY the token to stdout.
|
||||
# Retry once because first-run installer startup can be flaky on slow CI.
|
||||
boot_attempts: list[subprocess.CompletedProcess] = []
|
||||
for _ in range(2):
|
||||
for _ in range(bootstrap_retries):
|
||||
boot = _run(
|
||||
_compose_cmd("run", "--rm", "bootstrap"),
|
||||
_compose_cmd("run", "--rm", "bootstrap", compose_files=compose_files),
|
||||
check=False,
|
||||
extra_env={"MATOMO_PORT": MATOMO_PORT},
|
||||
extra_env={"MATOMO_PORT": matomo_port},
|
||||
)
|
||||
boot_attempts.append(boot)
|
||||
if boot.returncode == 0:
|
||||
@@ -152,9 +168,26 @@ class TestRootDockerComposeStack(unittest.TestCase):
|
||||
|
||||
if boot.returncode != 0:
|
||||
matomo_logs = _run(
|
||||
_compose_cmd("logs", "--no-color", "--tail=200", "matomo"),
|
||||
_compose_cmd(
|
||||
"logs",
|
||||
"--no-color",
|
||||
"--tail=250",
|
||||
"matomo",
|
||||
compose_files=compose_files,
|
||||
),
|
||||
check=False,
|
||||
extra_env={"MATOMO_PORT": MATOMO_PORT},
|
||||
extra_env={"MATOMO_PORT": matomo_port},
|
||||
)
|
||||
db_logs = _run(
|
||||
_compose_cmd(
|
||||
"logs",
|
||||
"--no-color",
|
||||
"--tail=200",
|
||||
"db",
|
||||
compose_files=compose_files,
|
||||
),
|
||||
check=False,
|
||||
extra_env={"MATOMO_PORT": matomo_port},
|
||||
)
|
||||
attempts_dump = "\n\n".join(
|
||||
[
|
||||
@@ -169,7 +202,8 @@ class TestRootDockerComposeStack(unittest.TestCase):
|
||||
self.fail(
|
||||
"bootstrap container failed after retry.\n"
|
||||
f"{attempts_dump}\n\n"
|
||||
f"[matomo logs]\n{matomo_logs.stdout}\n{matomo_logs.stderr}"
|
||||
f"[matomo logs]\n{matomo_logs.stdout}\n{matomo_logs.stderr}\n\n"
|
||||
f"[db logs]\n{db_logs.stdout}\n{db_logs.stderr}"
|
||||
)
|
||||
|
||||
token = (boot.stdout or "").strip()
|
||||
@@ -179,9 +213,8 @@ class TestRootDockerComposeStack(unittest.TestCase):
|
||||
f"Expected token_auth on stdout, got stdout={boot.stdout!r} stderr={boot.stderr!r}",
|
||||
)
|
||||
|
||||
# Verify token works against Matomo API
|
||||
api_url = (
|
||||
f"{MATOMO_HOST_URL}/index.php"
|
||||
f"{matomo_host_url}/index.php"
|
||||
f"?module=API&method=SitesManager.getSitesWithAtLeastViewAccess"
|
||||
f"&format=json&token_auth={token}"
|
||||
)
|
||||
@@ -190,6 +223,26 @@ class TestRootDockerComposeStack(unittest.TestCase):
|
||||
|
||||
self.assertIsInstance(data, list)
|
||||
|
||||
def test_root_docker_compose_yml_stack_bootstraps_and_token_works(self) -> None:
|
||||
self._assert_stack_bootstraps_and_token_works(
|
||||
compose_files=[COMPOSE_FILE],
|
||||
matomo_port=MATOMO_PORT,
|
||||
matomo_host_url=MATOMO_HOST_URL,
|
||||
wait_timeout_seconds=WAIT_TIMEOUT_SECONDS,
|
||||
bootstrap_retries=2,
|
||||
)
|
||||
|
||||
def test_root_docker_compose_yml_stack_bootstraps_under_resource_pressure(
|
||||
self,
|
||||
) -> None:
|
||||
self._assert_stack_bootstraps_and_token_works(
|
||||
compose_files=[COMPOSE_FILE, SLOW_COMPOSE_FILE],
|
||||
matomo_port=MATOMO_SLOW_PORT,
|
||||
matomo_host_url=MATOMO_SLOW_HOST_URL,
|
||||
wait_timeout_seconds=SLOW_WAIT_TIMEOUT_SECONDS,
|
||||
bootstrap_retries=3,
|
||||
)
|
||||
|
||||
|
||||
class TestRootDockerComposeDefinition(unittest.TestCase):
|
||||
def test_bootstrap_service_waits_for_healthy_matomo_and_has_readiness_knobs(
|
||||
@@ -217,6 +270,28 @@ class TestRootDockerComposeDefinition(unittest.TestCase):
|
||||
self.assertIn("healthcheck:", matomo_block)
|
||||
self.assertIn("curl -fsS http://127.0.0.1/ >/dev/null || exit 1", matomo_block)
|
||||
|
||||
def test_slow_override_sets_tight_resources_and_longer_timeouts(self) -> None:
|
||||
cfg = _run(
|
||||
_compose_cmd("config", compose_files=[COMPOSE_FILE, SLOW_COMPOSE_FILE]),
|
||||
check=True,
|
||||
extra_env={"MATOMO_PORT": MATOMO_SLOW_PORT},
|
||||
)
|
||||
self.assertEqual(cfg.returncode, 0, cfg.stderr)
|
||||
|
||||
matomo_block = _extract_service_block(cfg.stdout, "matomo")
|
||||
self.assertIn("cpus: 0.35", matomo_block)
|
||||
self.assertIn('mem_limit: "402653184"', matomo_block)
|
||||
self.assertIn("start_period: 2m0s", matomo_block)
|
||||
|
||||
db_block = _extract_service_block(cfg.stdout, "db")
|
||||
self.assertIn("cpus: 0.35", db_block)
|
||||
self.assertIn('mem_limit: "335544320"', db_block)
|
||||
|
||||
bootstrap_block = _extract_service_block(cfg.stdout, "bootstrap")
|
||||
self.assertIn("MATOMO_INSTALLER_STEP_TIMEOUT_S:", bootstrap_block)
|
||||
self.assertIn("MATOMO_INSTALLER_STEP_DEADLINE_S:", bootstrap_block)
|
||||
self.assertIn("MATOMO_INSTALLER_READY_TIMEOUT_S:", bootstrap_block)
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
unittest.main()
|
||||
|
||||
@@ -1,6 +1,10 @@
|
||||
import unittest
|
||||
|
||||
from matomo_bootstrap.installers.web import _click_next_with_wait, _count_locator
|
||||
from matomo_bootstrap.installers.web import (
|
||||
_click_next_with_wait,
|
||||
_count_locator,
|
||||
_wait_for_superuser_login_field,
|
||||
)
|
||||
|
||||
|
||||
class _FlakyLocator:
|
||||
@@ -24,6 +28,8 @@ class _StaticLocator:
|
||||
def count(self) -> int:
|
||||
if self._selector == "#login-0":
|
||||
return 1 if self._page.login_visible else 0
|
||||
if self._selector == "form#generalsetupform":
|
||||
return 1 if getattr(self._page, "form_visible", False) else 0
|
||||
if self._selector == "#siteName-0":
|
||||
return 0
|
||||
return 0
|
||||
@@ -75,6 +81,7 @@ class _NoNextButLoginAppearsPage:
|
||||
def __init__(self):
|
||||
self.url = "http://matomo/index.php?action=setupSuperUser&module=Installation"
|
||||
self.login_visible = False
|
||||
self.form_visible = False
|
||||
self._wait_calls = 0
|
||||
|
||||
def locator(self, selector: str):
|
||||
@@ -125,6 +132,98 @@ class _NoNextButNamedLoginAppearsPage:
|
||||
self.login_visible = True
|
||||
|
||||
|
||||
class _NoNextButSuperuserFormContainerAppearsPage:
|
||||
def __init__(self):
|
||||
self.url = "http://matomo/index.php?action=setupSuperUser&module=Installation"
|
||||
self.login_visible = False
|
||||
self.form_visible = False
|
||||
self._wait_calls = 0
|
||||
|
||||
def locator(self, selector: str):
|
||||
return _StaticLocator(self, selector)
|
||||
|
||||
def get_by_role(self, role: str, name: str):
|
||||
return _RoleLocator(0)
|
||||
|
||||
def get_by_text(self, *_args, **_kwargs):
|
||||
return _RoleLocator(0)
|
||||
|
||||
def title(self) -> str:
|
||||
return "setupSuperUser"
|
||||
|
||||
def wait_for_load_state(self, *_args, **_kwargs):
|
||||
return None
|
||||
|
||||
def wait_for_timeout(self, *_args, **_kwargs):
|
||||
self._wait_calls += 1
|
||||
if self._wait_calls >= 1:
|
||||
self.form_visible = True
|
||||
|
||||
|
||||
class _DelayedSuperuserLoginPage:
|
||||
def __init__(self, *, reveal_after_wait_calls: int | None):
|
||||
self.url = "http://matomo/index.php?action=setupSuperUser&module=Installation"
|
||||
self.login_visible = False
|
||||
self.form_visible = False
|
||||
self._wait_calls = 0
|
||||
self._reveal_after_wait_calls = reveal_after_wait_calls
|
||||
|
||||
def locator(self, selector: str):
|
||||
return _StaticLocator(self, selector)
|
||||
|
||||
def get_by_role(self, role: str, name: str):
|
||||
return _RoleLocator(0)
|
||||
|
||||
def get_by_text(self, *_args, **_kwargs):
|
||||
return _RoleLocator(0)
|
||||
|
||||
def title(self) -> str:
|
||||
return "setupSuperUser"
|
||||
|
||||
def wait_for_load_state(self, *_args, **_kwargs):
|
||||
return None
|
||||
|
||||
def wait_for_timeout(self, *_args, **_kwargs):
|
||||
self._wait_calls += 1
|
||||
if (
|
||||
self._reveal_after_wait_calls is not None
|
||||
and self._wait_calls >= self._reveal_after_wait_calls
|
||||
):
|
||||
self.login_visible = True
|
||||
|
||||
|
||||
class _DelayedSuperuserFormContainerPage:
|
||||
def __init__(self, *, reveal_after_wait_calls: int | None):
|
||||
self.url = "http://matomo/index.php?action=setupSuperUser&module=Installation"
|
||||
self.login_visible = False
|
||||
self.form_visible = False
|
||||
self._wait_calls = 0
|
||||
self._reveal_after_wait_calls = reveal_after_wait_calls
|
||||
|
||||
def locator(self, selector: str):
|
||||
return _StaticLocator(self, selector)
|
||||
|
||||
def get_by_role(self, role: str, name: str):
|
||||
return _RoleLocator(0)
|
||||
|
||||
def get_by_text(self, *_args, **_kwargs):
|
||||
return _RoleLocator(0)
|
||||
|
||||
def title(self) -> str:
|
||||
return "setupSuperUser"
|
||||
|
||||
def wait_for_load_state(self, *_args, **_kwargs):
|
||||
return None
|
||||
|
||||
def wait_for_timeout(self, *_args, **_kwargs):
|
||||
self._wait_calls += 1
|
||||
if (
|
||||
self._reveal_after_wait_calls is not None
|
||||
and self._wait_calls >= self._reveal_after_wait_calls
|
||||
):
|
||||
self.form_visible = True
|
||||
|
||||
|
||||
class TestWebInstallerLocatorCountIntegration(unittest.TestCase):
|
||||
def test_retries_transient_navigation_error(self) -> None:
|
||||
locator = _FlakyLocator(
|
||||
@@ -168,6 +267,49 @@ class TestWebInstallerLocatorCountIntegration(unittest.TestCase):
|
||||
self.assertEqual(step, "Installation:setupSuperUser")
|
||||
self.assertTrue(page.login_visible)
|
||||
|
||||
def test_click_next_wait_treats_superuser_form_container_as_progress(self) -> None:
|
||||
page = _NoNextButSuperuserFormContainerAppearsPage()
|
||||
|
||||
step = _click_next_with_wait(page, timeout_s=1)
|
||||
|
||||
self.assertEqual(step, "Installation:setupSuperUser")
|
||||
self.assertTrue(page.form_visible)
|
||||
|
||||
def test_wait_for_superuser_login_field_allows_delayed_form(self) -> None:
|
||||
page = _DelayedSuperuserLoginPage(reveal_after_wait_calls=4)
|
||||
|
||||
visible = _wait_for_superuser_login_field(
|
||||
page,
|
||||
timeout_s=1.0,
|
||||
poll_interval_ms=1,
|
||||
)
|
||||
|
||||
self.assertTrue(visible)
|
||||
self.assertTrue(page.login_visible)
|
||||
|
||||
def test_wait_for_superuser_login_field_allows_delayed_form_container(self) -> None:
|
||||
page = _DelayedSuperuserFormContainerPage(reveal_after_wait_calls=4)
|
||||
|
||||
visible = _wait_for_superuser_login_field(
|
||||
page,
|
||||
timeout_s=1.0,
|
||||
poll_interval_ms=1,
|
||||
)
|
||||
|
||||
self.assertTrue(visible)
|
||||
self.assertTrue(page.form_visible)
|
||||
|
||||
def test_wait_for_superuser_login_field_times_out_when_absent(self) -> None:
|
||||
page = _DelayedSuperuserLoginPage(reveal_after_wait_calls=None)
|
||||
|
||||
visible = _wait_for_superuser_login_field(
|
||||
page,
|
||||
timeout_s=0.01,
|
||||
poll_interval_ms=1,
|
||||
)
|
||||
|
||||
self.assertFalse(visible)
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
unittest.main()
|
||||
|
||||
Reference in New Issue
Block a user