mirror of
https://github.com/priyanshujain/sanderling.git
synced 2026-10-02 11:07:10 +00:00
* 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
189 lines
6.5 KiB
Go
189 lines
6.5 KiB
Go
// Package trace records each run's steps, snapshots, and violations to disk for later inspection.
|
|
package trace
|
|
|
|
import (
|
|
"encoding/json"
|
|
"fmt"
|
|
"io"
|
|
"os"
|
|
"path/filepath"
|
|
"sync"
|
|
"time"
|
|
|
|
"github.com/priyanshujain/sanderling/internal/hierarchy"
|
|
)
|
|
|
|
type Step struct {
|
|
Index int `json:"step"`
|
|
Timestamp time.Time `json:"timestamp"`
|
|
Screen string `json:"screen,omitempty"`
|
|
Snapshots map[string]json.RawMessage `json:"snapshots,omitempty"`
|
|
// NextAction is the action chosen for the next iteration based on observing this step.
|
|
NextAction *Action `json:"next_action,omitempty"`
|
|
Exceptions []Exception `json:"exceptions,omitempty"`
|
|
Violations []string `json:"violations,omitempty"`
|
|
Hierarchy *hierarchy.Tree `json:"hierarchy,omitempty"`
|
|
Residuals map[string]json.RawMessage `json:"residuals,omitempty"`
|
|
Metrics *Metrics `json:"metrics,omitempty"`
|
|
ExtractorChanges map[string]ExtractorChange `json:"extractor_changes,omitempty"`
|
|
// Transitional marks a step whose hierarchy still showed a NavHost
|
|
// cross-fade (multiple route-level *Screen ids) after the runner's
|
|
// retry budget. The verifier is skipped for these steps so transient
|
|
// state does not poison the previous/current extractor advance.
|
|
Transitional bool `json:"transitional,omitempty"`
|
|
// SkippedVerification is set true exactly when the verifier was skipped
|
|
// for this step, so downstream tooling can tell a deliberately-skipped
|
|
// step from one that was verified and came back clean.
|
|
SkippedVerification bool `json:"skipped_verification,omitempty"`
|
|
// Witnesses records the violation witness for each property that newly
|
|
// violated at this step: the cause and the extractor values at onset.
|
|
Witnesses map[string]Witness `json:"witnesses,omitempty"`
|
|
}
|
|
|
|
// Witness is the trace-side record of a property violation: why it fired and a
|
|
// snapshot of every extractor's value at the violating step.
|
|
type Witness struct {
|
|
Reason string `json:"reason,omitempty"`
|
|
IsError bool `json:"is_error,omitempty"`
|
|
// Step is the step the failed obligation originated at: the step that
|
|
// caused the violation. For a deferred obligation (a next, an eventually)
|
|
// this is earlier than the step whose record carries the witness, which is
|
|
// where the failure was detected.
|
|
Step int `json:"step,omitempty"`
|
|
Extractors map[string]json.RawMessage `json:"extractors,omitempty"`
|
|
}
|
|
|
|
// ExtractorChange records the prev/curr JSON values of an extractor whose
|
|
// observation differed between two consecutive steps. Surfaced under
|
|
// violation rows in the replay UI as a "what changed at this step"
|
|
// breadcrumb.
|
|
type ExtractorChange struct {
|
|
Prev json.RawMessage `json:"prev"`
|
|
Curr json.RawMessage `json:"curr"`
|
|
}
|
|
|
|
type Metrics struct {
|
|
CPUPercent float64 `json:"cpu_percent"`
|
|
HeapBytes int64 `json:"heap_bytes,omitempty"`
|
|
TotalMemoryBytes int64 `json:"total_memory_bytes,omitempty"`
|
|
}
|
|
|
|
type Action struct {
|
|
Kind string `json:"kind"`
|
|
X int `json:"x,omitempty"`
|
|
Y int `json:"y,omitempty"`
|
|
FromX int `json:"from_x,omitempty"`
|
|
FromY int `json:"from_y,omitempty"`
|
|
ToX int `json:"to_x,omitempty"`
|
|
ToY int `json:"to_y,omitempty"`
|
|
Key string `json:"key,omitempty"`
|
|
Text string `json:"text,omitempty"`
|
|
DurationMillis int `json:"duration_millis,omitempty"`
|
|
Selector string `json:"selector,omitempty"`
|
|
ResolvedBounds *BoundsRecord `json:"resolved_bounds,omitempty"`
|
|
TapPoint *PointRecord `json:"tap_point,omitempty"`
|
|
}
|
|
|
|
type BoundsRecord struct {
|
|
X int `json:"x"`
|
|
Y int `json:"y"`
|
|
Width int `json:"width"`
|
|
Height int `json:"height"`
|
|
}
|
|
|
|
type PointRecord struct {
|
|
X int `json:"x"`
|
|
Y int `json:"y"`
|
|
}
|
|
|
|
type Exception struct {
|
|
Class string `json:"class"`
|
|
Message string `json:"message,omitempty"`
|
|
StackTrace string `json:"stack_trace,omitempty"`
|
|
UnixMillis int64 `json:"unix_millis,omitempty"`
|
|
}
|
|
|
|
type Meta struct {
|
|
Seed int64 `json:"seed"`
|
|
SpecPath string `json:"spec_path"`
|
|
BundleSHA256 string `json:"bundle_sha256"`
|
|
Platform string `json:"platform"`
|
|
BundleID string `json:"bundle_id"`
|
|
StartedAt time.Time `json:"started_at"`
|
|
EndedAt *time.Time `json:"ended_at,omitempty"`
|
|
SanderlingVersion string `json:"sanderling_version"`
|
|
}
|
|
|
|
type Writer struct {
|
|
directory string
|
|
mutex sync.Mutex
|
|
file io.WriteCloser
|
|
encoder *json.Encoder
|
|
}
|
|
|
|
// NewWriter ensures `directory` exists and opens trace.jsonl for append.
|
|
// meta.json is written separately via WriteMeta. Caller must Close.
|
|
func NewWriter(directory string) (*Writer, error) {
|
|
if err := os.MkdirAll(directory, 0o755); err != nil {
|
|
return nil, fmt.Errorf("mkdir: %w", err)
|
|
}
|
|
file, err := os.OpenFile(
|
|
filepath.Join(directory, "trace.jsonl"),
|
|
os.O_CREATE|os.O_WRONLY|os.O_APPEND,
|
|
0o644,
|
|
)
|
|
if err != nil {
|
|
return nil, fmt.Errorf("open trace.jsonl: %w", err)
|
|
}
|
|
encoder := json.NewEncoder(file)
|
|
return &Writer{directory: directory, file: file, encoder: encoder}, nil
|
|
}
|
|
|
|
func (w *Writer) Directory() string { return w.directory }
|
|
|
|
func (w *Writer) WriteMeta(meta Meta) error {
|
|
body, err := json.MarshalIndent(meta, "", " ")
|
|
if err != nil {
|
|
return fmt.Errorf("marshal meta: %w", err)
|
|
}
|
|
return os.WriteFile(filepath.Join(w.directory, "meta.json"), body, 0o644)
|
|
}
|
|
|
|
func (w *Writer) WriteStep(step Step) error {
|
|
w.mutex.Lock()
|
|
defer w.mutex.Unlock()
|
|
if w.file == nil {
|
|
return fmt.Errorf("trace: writer is closed")
|
|
}
|
|
return w.encoder.Encode(step)
|
|
}
|
|
|
|
// WriteScreenshot is lock-free: each call writes a distinct, uniquely-named
|
|
// file via os.WriteFile and touches no field of Writer, so concurrent calls
|
|
// never contend.
|
|
func (w *Writer) WriteScreenshot(stepIndex int, png []byte) error {
|
|
return w.writePNG(fmt.Sprintf("step-%05d.png", stepIndex), png)
|
|
}
|
|
|
|
func (w *Writer) writePNG(name string, png []byte) error {
|
|
if len(png) == 0 {
|
|
return nil
|
|
}
|
|
directory := filepath.Join(w.directory, "screenshots")
|
|
if err := os.MkdirAll(directory, 0o755); err != nil {
|
|
return fmt.Errorf("mkdir screenshots: %w", err)
|
|
}
|
|
return os.WriteFile(filepath.Join(directory, name), png, 0o644)
|
|
}
|
|
|
|
func (w *Writer) Close() error {
|
|
w.mutex.Lock()
|
|
defer w.mutex.Unlock()
|
|
if w.file == nil {
|
|
return nil
|
|
}
|
|
err := w.file.Close()
|
|
w.file = nil
|
|
return err
|
|
}
|