Skip to content

refactor: migrate tests/admin_test.go to testcore.NewEnv pattern#9675

Merged
chaptersix merged 2 commits intotemporalio:mainfrom
chaptersix:admin-test-env
Mar 26, 2026
Merged

refactor: migrate tests/admin_test.go to testcore.NewEnv pattern#9675
chaptersix merged 2 commits intotemporalio:mainfrom
chaptersix:admin-test-env

Conversation

@chaptersix
Copy link
Contributor

Summary

  • Replace suite-based AdminTestSuite with t.Run subtests using testcore.NewEnv() for better parallelization and simpler test infrastructure
  • Convert rebuildMutableStateWorkflowHelper to accept *testcore.TestEnv instead of *AdminTestSuite
  • Pass EnableChasm via testcore.WithDynamicConfig instead of OverrideDynamicConfig

Test plan

  • go build compiles cleanly
  • make lint passes
  • go test -tags disable_grpc_modules,test_dep -run TestAdminTestSuite ./tests/... passes

Replace suite-based AdminTestSuite with t.Run subtests using
testcore.NewEnv() for better parallelization and simpler test
infrastructure.
@chaptersix chaptersix marked this pull request as ready for review March 26, 2026 00:55
@chaptersix chaptersix requested review from a team as code owners March 26, 2026 00:55
Copy link
Contributor

@stephanos stephanos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@chaptersix chaptersix merged commit 7a8d234 into temporalio:main Mar 26, 2026
46 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.

2 participants