weibangpeng opened a new pull request, #38979: URL: https://github.com/apache/beam/pull/38979
## Summary Increases unit test coverage for the Go SDK harness package from **43% → 50.7%**, addressing issue #21386. ## Changes Adds ~30 test functions across 7 files (1 new, 6 modified): - `harness_test.go` — `TestFail`, `TestControl_MetStoreToString`, `TestControl_GetPlanOrResponse` (5 scenarios) - `logging_test.go` — `TestConvertSeverity` (6 cases) - `monitoring_test.go` — `TestMonitoring_NilStore`, `TestGetNextShortID` - `worker_status_test.go` — `TestIsAliveAndShutdown`, `TestMemoryUsage`, `TestGoroutineDump`, `TestBuildInfo`, `TestActiveProcessBundleStates`, `TestCacheStats` - `datamgr_test.go` — ScopedDataManager cached-port paths, elementsChan lifecycle, terminateStreamOnError, closeInstruction - `statemgr_test.go` — ScopedStateReader construction/close/closed-path guards - **`sampler_test.go` (new)** — stateSampler lifecycle (new, stop, start via context cancel and done channel) ## Test plan All tests pass: ``` ok github.com/apache/beam/sdks/v2/go/pkg/beam/core/runtime/harness coverage: 50.7% of statements ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
