Skip to content

test: cover pool marshaler initialization#38

Merged
siyul-park merged 1 commit into
mainfrom
automation/test-gap-detection-pool-marshaler
May 24, 2026
Merged

test: cover pool marshaler initialization#38
siyul-park merged 1 commit into
mainfrom
automation/test-gap-detection-pool-marshaler

Conversation

@siyul-park

Copy link
Copy Markdown
Owner

Summary

  • add focused pool tests for marshaler initialization behavior in interp/pool.go
  • verify interpreters from the same pool share one default marshaler instance
  • verify a later WithMarshaler(custom) option still overrides the pool default

Changed paths tested

  • interp/pool.go

Added tests

  • TestNewPool/shares default marshaler within pool
  • TestNewPool/custom marshaler overrides pool default

Validation

  • go test ./interp -run 'TestNewPool|TestWithMarshaler|TestInterpreter_Marshal|TestHostObject'
  • go test ./interp
  • make lint

Skipped targets

  • interp/host.go, interp/marshal.go, interp/interp_test.go: already covered by the recent commit's added tests; no additional uncovered changed branch found that stayed narrow and high-confidence.
  • docs changes: no executable behavior to regression-test.

@siyul-park siyul-park marked this pull request as ready for review May 24, 2026 08:27
@siyul-park siyul-park merged commit 43ee667 into main May 24, 2026
5 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.

1 participant