test(cli): golden-test the apps get panel and drop stale fixtures

[jonoirwinrsa]

Stacked on #95 — base is fix/cli-drop-dead-instance-section. Merging that will retarget this to main. Stacked rather than parallel because both touch apps/testdata.

Why

#95 turned up four golden files under apps/testdata that no test referenced. They were never compared, and -update never touched them.

Three of them describe the detail panel apps get prints on success. So the main output of the command had no golden coverage at all — only the loading, error and not-found states were checked. That's also why the removed section in #95 sat there unnoticed: the file depicting it was inert.

They were orphaned for a real reason. The panel reaches the terminal through tea.Println for scrollback rather than through View(), so once details load View() returns empty and the harness has nothing to compare. Re-attaching ViewGolden to those steps would have golden-tested an empty string.

What

Renders formatAppDetailsTable directly, over the same four fixtures the goldens were built from: GPU, CPU-only, unparseable numeric fields, and empty hardware fields.

The three existing goldens pass unmodified. Their content still matches the current renderer byte for byte — the only discrepancy was trailing whitespace, which the harness strips via normalizeView and a direct call doesn't. Trimming the same way makes them pass as-is.

Only get_success_gpu.golden is newly generated, because #95 deleted it. Worth eyeballing: it is identical to the old file minus the section that PR removes, which is a decent end-to-end check on that change.

Also deletes internal/ui/commands/testdata/logs_initial_loading.golden — byte-identical to the referenced logs_initial.golden, left behind when the step was renamed. Confirmed with diff before removing.

After this, no golden file anywhere in the repo is unreferenced. I checked every testdata directory, not just this one.

Testing

go build, go vet, go test ./... clean.

On golden regeneration: I ran -update scoped to the single new GPU fixture. The other three were never regenerated — git status showed them unmodified throughout, which is the point.

The plain-text path (formatAppDetailsSimple) still has only assert.Contains coverage via the existing Test_formatAppDetailsSimple. I left that alone rather than converting it — happy to golden it too if you'd prefer the symmetry.