Skip to content

feat(report): click a trend point to open that run's report - #250

Merged
shossain786 merged 1 commit into
masterfrom
feat/165-trend-point-opens-run
Nov 12, 2026
Merged

shossain786 merged 1 commit into
masterfrom
feat/165-trend-point-opens-run

Conversation

@shossain786

Copy link
Copy Markdown
Contributor

Closes #165. This is the remaining idea from the issue; the flaky trend line (#185) and hiding the empty flaky chart (#243) already landed.

What

  • New opt-in flakiness.keepReports: N (default 0, so nothing changes unless it's set).
  • After each run, the HTML report is copied into the history directory next to its metrics JSON as selenium-boot-report-<timestamp>.html. Copies beyond N are deleted, oldest first.
  • On the trend charts, a point whose report was kept opens it in a new tab. Its tooltip says "Click to open this run" and the cursor becomes a pointer. Points without a kept report behave as before.
  • The link is relative from the report to the history directory, so historyDir can point outside target/. When the two are on different roots, the link is a file: URI. A kept copy sits in the history directory, so its own links are plain file names.
  • Documented in reporting/dashboard-trends.md.

Why opt-in

Reports embed their screenshots as Base64, so keeping every one by default could grow the history directory quickly.

Verification

  • mvn clean verify: green.
  • HtmlReportGeneratorTest: 37/37, including 4 new tests:
    • only runs with a kept report get a link
    • each copy gets local links, and the oldest copies are pruned
    • nothing is kept by default
    • the link base is relative for target/metrics-history and for ../.selenium-boot-history
  • Browser check on a real generated report served over HTTP, with history in a sibling directory:
    • hovering a kept run shows the tooltip footer
    • clicking it requested ../hist/selenium-boot-report-20261110-100000.html (200)
    • clicking the current run's point did nothing

Not covered

  • A real end-to-end run with keepReports set in selenium-boot.yml. The YAML binding is the same bean setter as historyRuns, and ExecutionMetrics.lastHistoryName() is set during export.

🤖 Generated with Claude Code

New opt-in flakiness.keepReports (default 0): after each run, a copy of
the HTML report is saved in the history directory next to its metrics
JSON as selenium-boot-report-<timestamp>.html, and the oldest copies
beyond the limit are deleted. Trend-chart points whose report was kept
open it in a new tab and show "Click to open this run" in the tooltip.

The link is relative from the report to the history directory (a file:
URI when they are on different roots); kept copies link their siblings
by file name.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@shossain786
shossain786 merged commit 82e6039 into master Nov 12, 2026
2 checks passed
@shossain786
shossain786 deleted the feat/165-trend-point-opens-run branch November 12, 2026 02:47
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.

Report: extend the trend charts (flaky count, link to a past run)

1 participant