Skip to content

Vv/test automation - #264

Open
vishnuv688 wants to merge 56 commits into
mainfrom
vv/test-automation
Open

Vv/test automation#264
vishnuv688 wants to merge 56 commits into
mainfrom
vv/test-automation

Conversation

@vishnuv688

@vishnuv688 vishnuv688 commented Jul 27, 2026

Copy link
Copy Markdown
Member

What & why

Type of change

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Polish (an improvement to an existing feature)
  • Breaking change (existing behavior changes for users)
  • Documentation
  • Internal (build, CI, dependencies, tooling)

Packages touched

  • shared (types and contracts)
  • core (framework-agnostic capture/reporting)
  • service (WebdriverIO adapter)
  • nightwatch-devtools (Nightwatch adapter)
  • selenium-devtools (Selenium adapter)
  • backend (server)
  • app (UI)
  • script (page-injected runtime)

Notes for reviewers

Screenshots / recordings

Comment thread .github/workflows/verify.yml Fixed
@greptile-apps

greptile-apps Bot commented Jul 27, 2026

Copy link
Copy Markdown

Greptile Summary

The PR expands component and adapter test automation while updating trace capture, run-state continuity, trace parsing, and related app behavior.

  • Adds WebdriverIO browser-based UI coverage across explorer, workbench, player, panel, and tab components.
  • Adds cross-adapter trace configuration and capture support for WebdriverIO, Selenium, and Nightwatch/Cucumber examples.
  • Propagates stable run identifiers through worker connections so backend state can continue across workers.
  • Consolidates trace archive paths and improves trace reading, writing, command selection, error rendering, and compare-step resolution.
  • Removes the previous fixture-driven capture-parity verification layer and documents the replacement testing strategy.

Confidence Score: 5/5

The PR appears safe to merge because no blocking failure remains within the scope of the prior review thread.

The previously reported missing-fixture skip path no longer exists: the capture-parity suite, ready matrix, fixtures, verification script, and associated CI workflow were removed together, so the old silent-skip behavior cannot occur on the current HEAD.

Important Files Changed

Filename Overview
packages/core/src/run-id.ts Introduces shared environment-backed run identity generation and propagation.
packages/backend/src/index.ts Updates worker connection handling to preserve backend state across sockets belonging to the same run.
packages/backend/src/trace-reader.ts Extends trace archive parsing around the consolidated trace-file contract.
packages/app/src/components/workbench/compare/stepResolution.ts Resolves preserved test UIDs directly so compare commands are windowed to the selected test.
packages/nightwatch-devtools/src/cucumber-lifecycle.ts Adds pre-quit Cucumber capture handling while the scenario browser session remains available.
packages/app/wdio.conf.ts Configures the new WebdriverIO browser-component test suite.

Sequence Diagram

sequenceDiagram
  participant Runner
  participant Adapter
  participant Core
  participant Backend
  participant App
  Runner->>Adapter: Start run and browser session
  Adapter->>Core: Resolve and propagate run ID
  Adapter->>Backend: Connect worker socket with runId
  Backend->>Backend: Preserve or reset accumulated run state
  Adapter->>Core: Capture commands, assertions, and snapshots
  Core->>Backend: Stream live events or write trace.zip
  Backend->>App: Replay accumulated run data
  App->>App: Render tested explorer and workbench states
Loading

Reviews (2): Last reviewed commit: "fix(app): window the compare panel to a ..." | Re-trigger Greptile

Comment thread test/capture/capture-parity.test.ts Outdated
@greptile-apps

greptile-apps Bot commented Jul 31, 2026

Copy link
Copy Markdown

Too many files changed for review. (198 files found, 100 file limit)

Bypass the limit by tagging @greptile-apps to review.

Comment thread .github/workflows/ci.yml Fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants