Skip to content

test(verify): run the script the export produces - #8358

Open
kz930 wants to merge 1 commit into
apache:mainfrom
kz930:feat/verify-run-generated-script
Open

test(verify): run the script the export produces#8358
kz930 wants to merge 1 commit into
apache:mainfrom
kz930:feat/verify-run-generated-script

Conversation

@kz930

@kz930 kz930 commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this PR?

The other side of the comparison. StandaloneRunner writes the script the
operator's generator emits, binds its inputs to the files the fixture wrote,
runs it, and reads the frames it leaves behind.

The script is kept where it ran, so an operator whose two answers differ can
be opened as generated rather than described second-hand.

HarnessSpec covers all three pieces on one operator whose answer is short
enough to state in full.

Any related issues, documentation, discussions?

Part of #8325, 4 of 20; that issue lists the set in order.

How was this PR tested?

The tests in this change cover it. The whole set is exercised together once the last piece lands: every operator run through the engine and through its generated script, and the two answers compared.

Was this PR authored or co-authored using generative AI tooling?

Generated-by: Claude Code (Opus 5)

The other side of the comparison. `StandaloneRunner` writes the script the
operator's generator emits, binds its inputs to the files the fixture wrote,
runs it, and reads the frames it leaves behind.

The script is kept where it ran, so an operator whose two answers differ can
be opened as generated rather than described second-hand.

`HarnessSpec` covers all three pieces on one operator whose answer is short
enough to state in full.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added feature platform Non-amber Scala service paths labels Sep 2, 2026
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Automated Reviewer Suggestions

Based on the git blame history of the changed files, we recommend the following reviewers:

  • No candidates found from git blame history.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature platform Non-amber Scala service paths

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant