ahmido
22214f22d6
Applied customer/auditor safety layout changes to CustomerReviewWorkspace, EnvironmentReviewResource, EvidenceSnapshotResource, ReviewPackResource, and StoredReportResource as per Spec 372. Co-authored-by: Ahmed Darrazi <ahmed.darrazi@live.de> Reviewed-on: #443
54 lines
2.7 KiB
Markdown
54 lines
2.7 KiB
Markdown
# Browser Verification Report
|
|
|
|
Status: PASS.
|
|
|
|
## Browser Harness
|
|
|
|
- Command: `./vendor/bin/sail artisan test --compact tests/Browser/Spec372CustomerAuditorSurfaceSafetySmokeTest.php`
|
|
- Result: pass, 1 test, 38 assertions.
|
|
- Fixture: test-created owner user and `Spec372 Browser Environment` via local smoke-login route.
|
|
- Checks: page load, required visible text, section ordering via browser script, no JavaScript errors, no console logs, desktop screenshots, mobile Evidence Snapshot overflow check.
|
|
|
|
## URLs Tested
|
|
|
|
| Page | URL Pattern | Status |
|
|
|---|---|---|
|
|
| Customer Review Workspace | `/admin/reviews/workspace?environment_id={environment}` | pass |
|
|
| Environment Review View | `/admin/workspaces/{workspace}/environments/{environment}/environment-reviews/{record}` | pass |
|
|
| Review Pack View | `/admin/workspaces/{workspace}/environments/{environment}/review-packs/{record}` | pass |
|
|
| Stored Report View | `/admin/workspaces/{workspace}/environments/{environment}/stored-reports/{record}` | pass |
|
|
| Evidence Snapshot View | `/admin/workspaces/{workspace}/environments/{environment}/evidence/{record}` | pass |
|
|
|
|
## Screenshot Outputs
|
|
|
|
- `artifacts/screenshots/001-customer-review-workspace-after.png`
|
|
- `artifacts/screenshots/002-environment-review-view-after.png`
|
|
- `artifacts/screenshots/003-review-pack-view-after.png`
|
|
- `artifacts/screenshots/004-stored-report-view-after.png`
|
|
- `artifacts/screenshots/005-evidence-snapshot-view-after-or-blocked.png`
|
|
- `artifacts/screenshots/006-evidence-snapshot-view-mobile.png`
|
|
|
|
## Score Tracking
|
|
|
|
Scores are implementation-review estimates based on the Spec 372 browser smoke and visual spot-check, not a new full audit pass.
|
|
|
|
| Page | Spec 368 Score | Target | After Score | Result |
|
|
|---|---:|---:|---:|---|
|
|
| Customer Review Workspace | 4.2 | >= 4.5 | 4.6 | pass |
|
|
| Environment Review View | 3.7 | >= 4.2 | 4.3 | pass |
|
|
| Review Pack View | 4.2 | >= 4.5 | 4.5 | pass |
|
|
| Stored Report View | 4.2 | >= 4.5 | 4.5 | pass |
|
|
| Evidence Snapshot View | 0.0 blocked | >= 4.2 if reachable | 4.3 | pass, reachable |
|
|
|
|
## Evidence Snapshot Reachability
|
|
|
|
- Spec 368 result: redirected to `/admin/login`.
|
|
- Spec 372 result: reached the environment-scoped Evidence Snapshot detail with the smoke-login fixture.
|
|
- Follow-up status: no fixture blocker remains for this path in the Spec 372 smoke. Broader evidence-overview/report-list coverage remains outside this spec.
|
|
|
|
## Visual Notes
|
|
|
|
- Customer Review Workspace first viewport is decision/evidence-first and no longer exposes operation proof by default.
|
|
- Evidence Snapshot mobile screenshot has no horizontal overflow according to browser assertion.
|
|
- Filament shell density remains existing app chrome and is out of scope for this feature.
|