eagraiclainne/app

Watch

Fork 0

test(jobs): re-record the job card goldens for the review #222

Merged

nalum merged 1 commit from test/job-card-review-goldens into main

2026-08-26 18:34:32 +00:00

nalum commented

2026-08-24 21:07:27 +00:00

Owner

The pixels for the review fixes, kept out of the code's diff.

126 re-recorded: 56 web gallery and 56 Roborazzi — every card gained an action
row and the spacing moved — plus 14 live whole-page goldens across Today, both
Jobs boards and the phone chrome, in both modes.

Two new cases rode with the code that added them, per AGENTS.md:
someone-elses-job-your-sub-job and no-date-with-points.

🤖 Generated with Claude Code

The pixels for the review fixes, kept out of the code's diff. 126 re-recorded: 56 web gallery and 56 Roborazzi — every card gained an action row and the spacing moved — plus 14 live whole-page goldens across Today, both Jobs boards and the phone chrome, in both modes. Two new cases rode with the code that added them, per AGENTS.md: `someone-elses-job-your-sub-job` and `no-date-with-points`. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

nalum added 1 commit

2026-08-24 21:07:27 +00:00

test(jobs): re-record the job card goldens for the review

check / commits (pull_request) Successful in 10s

Details

check / go (pull_request) Successful in 2m49s

Details

check / report (pull_request) Successful in 4s

Details

check / web (pull_request) Successful in 8m15s

Details

android / build (pull_request) Successful in 16m51s

Details

android / report (pull_request) Successful in 3s

Details

aea19448c7

Every job-card golden moves: the action row is new on all of them, the
spacing is the design's own again, and the shared height budget is the
measured 312px. 56 cases per surface (every fixture case on hearth, the
canonical case across all 19 families) plus the 14 live whole-page shots
that carry cards — Today, Jobs mine, Jobs everyone, and the phone chrome.

Pixels only, no code: the two cases the review added (someone else's job
with a sub-job of yours, and a meta row with no due date) rode in with the
commit that introduced them, because a new case has no image in the tree
to compare against and would be red on its own.

nalum added the

area/jobs

area/testing

labels

2026-08-24 21:07:27 +00:00

forgejo-actions commented

2026-08-24 21:17:19 +00:00

Test report

Suite	Tests	Result	Skipped
Unit	1435	✅ pass	1
Integration	130	✅ pass	—

Coverage: 27.0%

_{Updated by the check workflow · commit 96901cd266}

## Test report | Suite | Tests | Result | Skipped | | --- | --: | --- | --: | | Unit | 1435 | ✅ pass | 1 | | Integration | 130 | ✅ pass | — | **Coverage:** 27.0% <sub>Updated by the check workflow · commit 96901cd266b02def5a7b0d71418505d9efd53cd2</sub>

forgejo-actions commented

2026-08-24 21:27:17 +00:00

Android test report

Suite	Tests	Result	Skipped
Unit (debug)		❌ 1 failed

Coverage:

_{Updated by the android workflow · commit 96901cd266}

## Android test report | Suite | Tests | Result | Skipped | | --- | --: | --- | --: | | Unit (debug) | | ❌ 1 failed | | **Coverage:** <sub>Updated by the android workflow · commit 96901cd266b02def5a7b0d71418505d9efd53cd2</sub>

nalum force-pushed test/job-card-review-goldens from aea19448c7

check / commits (pull_request) Successful in 10s

Details

check / go (pull_request) Successful in 2m49s

Details

check / report (pull_request) Successful in 4s

Details

check / web (pull_request) Successful in 8m15s

Details

android / build (pull_request) Successful in 16m51s

Details

android / report (pull_request) Successful in 3s