Skip to content

feat: add workflow harness integration tests#345

Merged
marccampbell merged 3 commits into
mainfrom
feat/workflow-harness-tests
Jun 4, 2026
Merged

feat: add workflow harness integration tests#345
marccampbell merged 3 commits into
mainfrom
feat/workflow-harness-tests

Conversation

@elasticclaw-factory

Copy link
Copy Markdown
Contributor

Add integration tests for the workflow pipeline engine using the factorytest infrastructure.

Tests cover:

  • GitHub issue pre-commit workflow (entry → pre-commit → review)
  • Run action failure with/without continue_on_error
  • Linear issue move on stage transition
  • Terminal stage deduplication

Also fixes pipeline trigger evaluation for non-[DONE] messages by calling checkPipelineMessageTriggers from the message handler.

Fixes #326

Add integration tests for the workflow pipeline engine using the
factorytest infrastructure. Tests cover:

- GitHub issue pre-commit workflow (entry → pre-commit → review)
- Run action failure with/without continue_on_error
- Linear issue move on stage transition
- Terminal stage deduplication

Also fixes pipeline trigger evaluation for non-[DONE] messages by
calling checkPipelineMessageTriggers from the message handler.

Fixes #326
@greptile-apps

greptile-apps Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "feat: add workflow harness integration t..." | Re-trigger Greptile

Comment thread pkg/hub/workflow_harness_integration_test.go
Comment thread pkg/hub/workflow_harness_integration_test.go
Comment thread pkg/hub/factorytest/fake_bridge.go Outdated
ElasticClaw Agent added 2 commits June 4, 2026 12:19
- Add 'failing' test provider that simulates run action failures
- Use failing provider in run-fails-stop and run-fails-continue tests
- Fix Linear test to check IssueStates map instead of SawAPICall()
- Fix run-fails-stop test to verify inject is suppressed on actual failure
@greptile-apps

greptile-apps Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Reviews (2): Last reviewed commit: "fix: store claw token on FakeBridge for ..." | Re-trigger Greptile

@greptile-apps

greptile-apps Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Reviews (3): Last reviewed commit: "fix: address greptile review feedback on..." | Re-trigger Greptile

@marccampbell marccampbell merged commit 7da437e into main Jun 4, 2026
11 checks passed
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.

Add workflow harness integration tests

1 participant