Files
pj 94d9511312 test: full test-suite refactor sweep (#61)
* chore(test): start test-suite refactor sweep

* test(ltl): pin exact multi-obligation residual AST

* test(ltl): table-test finalize Kleene connective combinations

* test(ltl): pin reduce over pending inner for bound, Or, Not

* test(ltl): marshal bounded Always steps/duration/deadline

* test(verifier): cover LTL combinator verdict transitions and within unit panic

* test(verifier): table-test DecodeAction kinds and lastAction field exposure

* test(verifier): assert WithPlatform(ios) reaches the picker host and key pool

* test(verifier): widen weighted-selection assertion to a 5x skew margin

* test(verifier): un-skip ax-find round trip with a committed tree fixture

* test(runner): pin isWDADrop to sidecar reconnect-failed message origin

* test(runner): assert PressKey/Wait trace encoding records kind-specific fields

* test(runner): cover RenderSummary unsupported-verbs surfacing branch

* test(trace): set Hierarchy in round-trip and lock lossy Tree contract

Also add a -race concurrent WriteStep test that asserts N well-formed JSONL lines, catching torn lines if the writer mutex is dropped.

* test(trace): round-trip witnesses/changes/metrics/exceptions, pin step-0 witness

* test(trace): document ViolationsAreGreppable grep contract and lock-free WriteScreenshot

* test(hierarchy): cover invalid-JSON and malformed-bounds parser paths

* test(trace): guard writer mutex via WriteStep/Close race on w.file

* test(replay): drop unfailable assets and devproxy assertions

* test(replay): cache reuses on equal mtime, reparses after append

* test(replay): violation marker falls back to detection step when attributed missing

* test(replay): corrupt meta/trace dirs return 500 with error body

* test(replay): SSE client receives runs.changed after a broadcast

* test(replay): Run coalesces creates, ignores write/chmod, closes subs on cancel

* fix(sidecar): synchronize health fixture writes and exercise healthError

* test(sidecar): cover swipe/longpress/doubletap/erase/presskey/metrics/logs translations

* test(sidecar): cover DoubleTapSelector composition and mid-gesture cancel

* test(sidecar): assert gRPC error status surfaces from action RPC

* fix(chrome): route action methods through runCtx so caller cancellation aborts CDP

* fix(chrome): route hierarchy/screenshot/waitidle/metrics through runCtx

* refactor(ios): extract pure simctl JSON parsers

* refactor(ios): add command-runner seams for EnsureSimulator

* test(ios): table-test simctl parsers and EnsureSimulator seams

* test(sidecarassets): cover placeholder build path

* test(sidecarassets): assert reuse via sentinel bytes not mtime

* test(bundler): cover properties-only spec registration

* refactor(testrun): extract prepareBundleInputs from Execute

* test(testrun): cover prepareBundleInputs aliases and missing-runtime error

* test(testrun): table-test resolveRuntimeSibling search edges

* test(testrun): exact-output tests for progressHandler line format

* fix(cmd): point bundle-check aliases at pkg/spec/src

* test(cmd): smoke-test bundle-check resolves spec aliases

* test(cmd): table-test hier-check parse and FindAll on fixture

* test(cmd): unit-test buildBrowseURL deep-link vs root

* test(cmd): drop flaky TestRun_Doctor that launched real Chromium

* test(cmd): pin pipeline error to bundle resolution on web platform

* test(replay-ui): add bun test script

* ci(replay-ui): run bun test via make web-test target

* ci(replay-ui): point bun cache key at replay-ui/bun.lock

* test(replay-ui): exercise real URL encoding and non-ok throw in getJson

* refactor(replay-ui): extract snapshot flatten/getAtPath into lib module

* test(replay-ui): pin snapshot flatten/getAtPath path round-trip

* refactor(replay-ui): extract action selector/format into lib module

* test(replay-ui): pin action selector parse and row formatting

* refactor(replay-ui): share one statusFor between panels

* refactor(replay-ui): extract run-history derivation into lib module

* test(replay-ui): pin shared statusFor precedence and ordering

* test(replay-ui): pin run-history derivation alignment

* refactor(replay-ui): export clampIndex for testing

* refactor(replay-ui): extract keyboard-nav dispatch into pure module

* refactor(replay-ui): extract metrics formatters into lib module

* test(replay-ui): pin clampIndex step boundaries

* test(replay-ui): pin keyboard-nav ownership and key routing

* test(replay-ui): pin metrics formatters and path gap handling

* refactor(sidecar): expose device-output parsers as internal for testing

* test(sidecar): table-test device-output parsers against malformed input

* test(sidecar): cover logcat parsing year inference and line skipping

* test(sidecar): pin pressKey keycode mapping and unknown-key rejection

* test(sidecar): metrics bundleId falls back to launched app and honors override

* test(sidecar): loosen deadline upper bound to tolerate slow CI scheduling

* test(web-runtime): export selector builders for unit tests

* test(web-runtime): guard sanitize cycle, function, and depth limits

* test(web-runtime): table-test selector builder quoting and escaping

* test(sidecar): collapse scalar-forwarding RPC tests into a table

* test(replay-ui): dedup step/summary fixtures into shared module

* test(ios): collapse pickSimulator point-tests into a table
2026-06-06 13:59:08 +05:30

411 lines
12 KiB
Go

package replay
import (
"context"
"encoding/json"
"io"
"io/fs"
"net/http"
"net/http/httptest"
"os"
"path/filepath"
"strings"
"testing"
"testing/fstest"
"time"
"github.com/priyanshujain/sanderling/internal/trace"
)
var testAssetsFS fs.FS = fstest.MapFS{
"index.html": &fstest.MapFile{
Data: []byte(`<!doctype html><html><body><div id="root"></div></body></html>`),
},
}
func newFixtureServer(t *testing.T) (*Server, string) {
t.Helper()
root := t.TempDir()
startedAt := time.Date(2026, 4, 17, 18, 0, 0, 0, time.UTC)
endedAt := startedAt.Add(5 * time.Second)
writeRun(t, root, "run-a", trace.Meta{
StartedAt: startedAt,
EndedAt: &endedAt,
SpecPath: "spec.ts",
Seed: 42,
Platform: "android",
BundleID: "com.example",
}, []trace.Step{
{Index: 1, Timestamp: startedAt, Screen: "Home"},
{Index: 2, Timestamp: startedAt.Add(time.Second), Screen: "Home", NextAction: &trace.Action{Kind: "tap"}},
{Index: 3, Timestamp: startedAt.Add(2 * time.Second), Screen: "Cart", Violations: []string{"propA"}},
})
writeRun(t, root, "run-b", trace.Meta{
StartedAt: startedAt.Add(time.Hour),
}, []trace.Step{
{Index: 1, Timestamp: startedAt.Add(time.Hour)},
})
screenshotsDirectory := filepath.Join(root, "run-a", "screenshots")
if err := os.MkdirAll(screenshotsDirectory, 0o755); err != nil {
t.Fatal(err)
}
pngBody := []byte{0x89, 'P', 'N', 'G', 0, 1, 2, 3}
if err := os.WriteFile(filepath.Join(screenshotsDirectory, "step-00001.png"), pngBody, 0o644); err != nil {
t.Fatal(err)
}
server, err := NewServer(ServerOptions{RunsDirectory: root, AssetsFS: testAssetsFS})
if err != nil {
t.Fatal(err)
}
return server, root
}
func TestHandleRunsList_OrdersAndCountsViolations(t *testing.T) {
server, _ := newFixtureServer(t)
recorder := httptest.NewRecorder()
request := httptest.NewRequest(http.MethodGet, "/api/runs", nil)
server.Handler().ServeHTTP(recorder, request)
if recorder.Code != http.StatusOK {
t.Fatalf("status = %d", recorder.Code)
}
var summaries []RunSummary
if err := json.Unmarshal(recorder.Body.Bytes(), &summaries); err != nil {
t.Fatal(err)
}
if len(summaries) != 2 {
t.Fatalf("len = %d", len(summaries))
}
if summaries[0].ID != "run-b" {
t.Errorf("first id = %q, want run-b (most recent)", summaries[0].ID)
}
if summaries[1].ViolationCount != 1 {
t.Errorf("run-a violation count = %d, want 1", summaries[1].ViolationCount)
}
if !summaries[0].InProgress {
t.Error("run-b should be in_progress (no ended_at)")
}
}
func TestHandleRunDetail_DecodesMetaAndStepSummaries(t *testing.T) {
server, _ := newFixtureServer(t)
recorder := httptest.NewRecorder()
request := httptest.NewRequest(http.MethodGet, "/api/runs/run-a", nil)
server.Handler().ServeHTTP(recorder, request)
if recorder.Code != http.StatusOK {
t.Fatalf("status = %d, body=%s", recorder.Code, recorder.Body.String())
}
var detail RunDetail
if err := json.Unmarshal(recorder.Body.Bytes(), &detail); err != nil {
t.Fatal(err)
}
if detail.Meta.BundleID != "com.example" {
t.Errorf("bundle = %q", detail.Meta.BundleID)
}
if len(detail.Steps) != 3 {
t.Fatalf("steps len = %d", len(detail.Steps))
}
if detail.Steps[1].ActionKind != "tap" {
t.Errorf("step 2 action = %q", detail.Steps[1].ActionKind)
}
if !detail.Steps[2].HasViolations {
t.Error("step 3 should HasViolations")
}
}
func TestHandleStep_ReturnsFullStep(t *testing.T) {
server, _ := newFixtureServer(t)
recorder := httptest.NewRecorder()
request := httptest.NewRequest(http.MethodGet, "/api/runs/run-a/steps/3", nil)
server.Handler().ServeHTTP(recorder, request)
if recorder.Code != http.StatusOK {
t.Fatalf("status = %d", recorder.Code)
}
var step trace.Step
if err := json.Unmarshal(recorder.Body.Bytes(), &step); err != nil {
t.Fatal(err)
}
if step.Index != 3 || step.Screen != "Cart" {
t.Errorf("step = %+v", step)
}
if len(step.Violations) != 1 {
t.Errorf("violations = %v", step.Violations)
}
}
func TestHandleStep_ErrorCases(t *testing.T) {
server, _ := newFixtureServer(t)
cases := []struct {
name string
path string
wantStatus int
}{
{"unknown run", "/api/runs/nope", http.StatusNotFound},
{"non-numeric step", "/api/runs/run-a/steps/abc", http.StatusBadRequest},
{"out-of-range step", "/api/runs/run-a/steps/999", http.StatusNotFound},
}
for _, testCase := range cases {
t.Run(testCase.name, func(t *testing.T) {
recorder := httptest.NewRecorder()
request := httptest.NewRequest(http.MethodGet, testCase.path, nil)
server.Handler().ServeHTTP(recorder, request)
if recorder.Code != testCase.wantStatus {
t.Errorf("status = %d, want %d, body=%s", recorder.Code, testCase.wantStatus, recorder.Body.String())
}
})
}
}
func TestServer_CorruptRunDirReturns500WithError(t *testing.T) {
root := t.TempDir()
mustWriteFile(t, filepath.Join(root, "bad-meta", "meta.json"), "{not json")
mustWriteFile(t, filepath.Join(root, "bad-trace", "meta.json"), `{"started_at":"2026-04-17T18:00:00Z"}`)
mustWriteFile(t, filepath.Join(root, "bad-trace", "trace.jsonl"), "{not json\n")
server, err := NewServer(ServerOptions{RunsDirectory: root, AssetsFS: testAssetsFS})
if err != nil {
t.Fatal(err)
}
cases := []string{
"/api/runs/bad-meta",
"/api/runs/bad-trace",
"/api/runs/bad-trace/steps/1",
}
for _, path := range cases {
t.Run(path, func(t *testing.T) {
recorder := httptest.NewRecorder()
request := httptest.NewRequest(http.MethodGet, path, nil)
server.Handler().ServeHTTP(recorder, request)
if recorder.Code != http.StatusInternalServerError {
t.Fatalf("status = %d, want 500, body=%s", recorder.Code, recorder.Body.String())
}
if strings.TrimSpace(recorder.Body.String()) == "" {
t.Error("expected a non-empty error body")
}
})
}
}
func mustWriteFile(t *testing.T, path, body string) {
t.Helper()
if err := os.MkdirAll(filepath.Dir(path), 0o755); err != nil {
t.Fatal(err)
}
if err := os.WriteFile(path, []byte(body), 0o644); err != nil {
t.Fatal(err)
}
}
func TestScreenshot_ServesWhitelistedPNG(t *testing.T) {
server, _ := newFixtureServer(t)
recorder := httptest.NewRecorder()
request := httptest.NewRequest(http.MethodGet, "/api/runs/run-a/screenshots/step-00001.png", nil)
server.Handler().ServeHTTP(recorder, request)
if recorder.Code != http.StatusOK {
t.Fatalf("status = %d", recorder.Code)
}
if recorder.Body.Len() == 0 {
t.Error("empty body")
}
}
func TestScreenshot_RejectsTraversalAndBadNames(t *testing.T) {
server, _ := newFixtureServer(t)
cases := []string{
"/api/runs/run-a/screenshots/../meta.json",
"/api/runs/run-a/screenshots/..%2Fmeta.json",
"/api/runs/run-a/screenshots/step-00001.txt",
"/api/runs/run-a/screenshots/.png",
}
for _, path := range cases {
t.Run(path, func(t *testing.T) {
recorder := httptest.NewRecorder()
request := httptest.NewRequest(http.MethodGet, path, nil)
server.Handler().ServeHTTP(recorder, request)
if recorder.Code == http.StatusOK {
t.Errorf("status = %d (should not be 200) body=%s", recorder.Code, recorder.Body.String())
}
})
}
}
func TestSSE_ReturnsWhenContextCanceled(t *testing.T) {
server, _ := newFixtureServer(t)
httpServer := httptest.NewServer(server.Handler())
defer httpServer.Close()
context, cancel := context.WithCancel(context.Background())
request, err := http.NewRequestWithContext(context, http.MethodGet, httpServer.URL+"/api/events", nil)
if err != nil {
t.Fatal(err)
}
response, err := http.DefaultClient.Do(request)
if err != nil {
t.Fatal(err)
}
defer response.Body.Close()
if response.StatusCode != http.StatusOK {
t.Fatalf("status = %d", response.StatusCode)
}
done := make(chan struct{})
go func() {
_, _ = io.Copy(io.Discard, response.Body)
close(done)
}()
cancel()
select {
case <-done:
case <-time.After(2 * time.Second):
t.Fatal("SSE handler did not exit after context cancel")
}
}
func TestSSE_DeliversRunsChangedAfterBroadcast(t *testing.T) {
server, _ := newFixtureServer(t)
httpServer := httptest.NewServer(server.Handler())
defer httpServer.Close()
ctx, cancel := context.WithCancel(context.Background())
defer cancel()
request, err := http.NewRequestWithContext(ctx, http.MethodGet, httpServer.URL+"/api/events", nil)
if err != nil {
t.Fatal(err)
}
response, err := http.DefaultClient.Do(request)
if err != nil {
t.Fatal(err)
}
defer response.Body.Close()
events := make(chan string, 1)
go func() {
buffer := make([]byte, 256)
for {
n, err := response.Body.Read(buffer)
if n > 0 {
events <- string(buffer[:n])
}
if err != nil {
return
}
}
}()
deadline := time.Now().Add(2 * time.Second)
for {
server.Watcher().broadcast()
select {
case got := <-events:
if strings.Contains(got, ": ping") {
continue
}
if !strings.Contains(got, "event: runs.changed") {
t.Fatalf("event = %q, want runs.changed", got)
}
return
case <-time.After(50 * time.Millisecond):
if time.Now().After(deadline) {
t.Fatal("client never received runs.changed after broadcast")
}
}
}
}
func TestDevProxy_ForwardsRequestBodyUnchanged(t *testing.T) {
received := make(chan string, 1)
upstream := httptest.NewServer(http.HandlerFunc(func(responseWriter http.ResponseWriter, request *http.Request) {
body, err := io.ReadAll(request.Body)
if err != nil {
t.Errorf("upstream read: %v", err)
}
received <- string(body)
responseWriter.WriteHeader(http.StatusOK)
}))
defer upstream.Close()
root := t.TempDir()
server, err := NewServer(ServerOptions{RunsDirectory: root, DevTarget: upstream.URL})
if err != nil {
t.Fatal(err)
}
httpServer := httptest.NewServer(server.Handler())
defer httpServer.Close()
body := strings.NewReader("hello world")
request, err := http.NewRequest(http.MethodPost, httpServer.URL+"/anything", body)
if err != nil {
t.Fatal(err)
}
response, err := http.DefaultClient.Do(request)
if err != nil {
t.Fatal(err)
}
response.Body.Close()
if response.StatusCode != http.StatusOK {
t.Errorf("status = %d", response.StatusCode)
}
select {
case got := <-received:
if got != "hello world" {
t.Errorf("upstream body = %q, want %q", got, "hello world")
}
case <-time.After(2 * time.Second):
t.Fatal("upstream never received the request")
}
}
func TestAssets_FallbackToIndexHTML(t *testing.T) {
server, _ := newFixtureServer(t)
recorder := httptest.NewRecorder()
request := httptest.NewRequest(http.MethodGet, "/runs/some-id-that-only-the-spa-knows", nil)
server.Handler().ServeHTTP(recorder, request)
if recorder.Code != http.StatusOK {
t.Fatalf("status = %d", recorder.Code)
}
body := recorder.Body.String()
if !strings.Contains(body, "<div id=\"root\"></div>") {
t.Errorf("expected SPA shell with #root, got %q", body)
}
}
func TestAssets_API404DoesNotFallThrough(t *testing.T) {
server, _ := newFixtureServer(t)
recorder := httptest.NewRecorder()
request := httptest.NewRequest(http.MethodGet, "/api/wat", nil)
server.Handler().ServeHTTP(recorder, request)
if recorder.Code != http.StatusNotFound {
t.Errorf("status = %d, want 404", recorder.Code)
}
}
func TestResolveRunsDirectory(t *testing.T) {
root := t.TempDir()
writeRun(t, root, "r1", trace.Meta{StartedAt: time.Now().UTC()}, nil)
runsDirectory, deepLink, err := ResolveRunsDirectory("")
if err != nil || runsDirectory != "./runs" || deepLink != "" {
t.Errorf("default = (%q, %q, %v)", runsDirectory, deepLink, err)
}
runsDirectory, deepLink, err = ResolveRunsDirectory(root)
if err != nil || runsDirectory != root || deepLink != "" {
t.Errorf("multi-run dir = (%q, %q, %v)", runsDirectory, deepLink, err)
}
singleRun := filepath.Join(root, "r1")
runsDirectory, deepLink, err = ResolveRunsDirectory(singleRun)
if err != nil {
t.Fatal(err)
}
if filepath.Clean(runsDirectory) != filepath.Clean(root) || deepLink != "r1" {
t.Errorf("single-run dir = (%q, %q)", runsDirectory, deepLink)
}
}
func TestDevProxy_RejectsInvalidTarget(t *testing.T) {
if _, err := newDevProxy(":://bad-url"); err == nil {
t.Error("expected parse error for invalid URL")
}
}