Skip to content

chip ingress smoke test: verify PublishBatch via dedicated callback#22606

Draft
pkcll wants to merge 1 commit into
infoplat-3436-chipingress-publishBatchfrom
infoplat-3436-chipingress-publishBatch-smoke-test
Draft

chip ingress smoke test: verify PublishBatch via dedicated callback#22606
pkcll wants to merge 1 commit into
infoplat-3436-chipingress-publishBatchfrom
infoplat-3436-chipingress-publishBatch-smoke-test

Conversation

@pkcll
Copy link
Copy Markdown
Collaborator

@pkcll pkcll commented May 21, 2026

Summary

Add PublishBatchFunc to the CHiP test sink so the smoke test can directly observe PublishBatch RPCs without relying on grpc.Method(ctx) inspection.

Changes

  • chip-testsink/server.go — Add PublishBatchFn type and PublishBatchFunc field to Config. When set, PublishBatch delegates to it; otherwise falls back to per-event PublishFunc.
  • chip_testsink_helpers.go — Add ChipTestSinkOpt functional option and WithPublishBatchFunc helper. StartChipTestSink now accepts variadic opts.
  • chip_ingress_batching_test.go — Use PublishBatchFunc to count batch RPCs and total batched events directly. Removes grpc.Method string matching.

Testing

Builds cleanly via go build ./... in system-tests/tests.

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