Skip to content

test: parallelize workflow registry and source tests#1810

Closed
vishalg0wda wants to merge 3 commits into
mainfrom
test/parallelize-workflow-tests
Closed

test: parallelize workflow registry and source tests#1810
vishalg0wda wants to merge 3 commits into
mainfrom
test/parallelize-workflow-tests

Conversation

@vishalg0wda

Copy link
Copy Markdown
Member

Summary

  • Add t.Parallel() to TestFrozenWorkflowLockWithRegistryInput integration test
  • Add t.Parallel() to TestWorkflowSourceHasRemoteInputs unit test (including subtests)

Test plan

  • Unit tests pass with parallel execution

Add t.Parallel() to TestFrozenWorkflowLockWithRegistryInput and
TestWorkflowSourceHasRemoteInputs (including subtests) for faster
test execution.
Add t.Parallel() to:
- TestStability
- TestMultiFileStability
- TestFallbackCodeSamplesWorkflow

Note: TestWorkflowWithEnvVar cannot be parallelized due to t.Setenv usage
Note: TestSpecWorkflows explicitly requires serial execution
Add t.Parallel() to unit tests in:
- internal/github
- internal/markdown
- internal/patches (diff_test.go, pregeneration_test.go)
- internal/run (merge_test.go, remote_test.go)
- internal/schemas
- internal/utils
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