arunsingh opened a new pull request, #3377:
URL: https://github.com/apache/iggy/pull/3377
## Summary
Closes #2892.
This PR introduces a reusable source connector integration-test helper and
wires the existing random source tests through it as the first Pareto-suite
coverage target.
Changes:
- Add `connectors::source_suite` with shared polling/config helpers for
source connector tests.
- Register the helper module under integration connector tests.
- Replace duplicated random source message polling with the shared helper.
- Keep connector restart orchestration in the random source test while using
the shared consumer/config path.
- Add usage documentation for future source connector tests, including what
the first suite covers and what it intentionally does not cover yet.
## Scope
First-suite coverage:
- Source eventually produces messages.
- Source continues producing messages after connector runtime restart.
- Polling is bounded by explicit retry/timeout configuration.
Intentionally not covered in this first pass:
- Forced send failure paths.
- Backend mark/delete side effects.
- Exact duplicate semantics.
- Docker-backed source-specific behavior.
## Local validation
Passed:
- `cargo fmt --all`
- `git diff --check`
- WSL/Linux focused random gate:
- `cargo test -p integration --test mod -- connectors::random`
- Result: 2 passed, 0 failed, 850 filtered out
Blocked locally / not run to completion:
- `cargo sort --no-format --workspace`
- Blocked because `cargo-sort` is not installed locally: `error: no such
command: sort`.
- `cargo clippy --all-targets --all-features -- -D warnings`
- Blocked in the Windows native environment before reaching this PR's code
because `hwlocality-sys` could not find `pkg-config`/`hwloc`.
- A later Windows-native rebuild probe timed out during clippy without
useful diagnostics; leftover build processes were stopped.
- `cargo test -p integration -- connectors::random`
- Blocked in the Windows native environment by the same
`hwlocality-sys`/`pkg-config` dependency gate.
Docker note:
- Docker is not required for this PR path because `random_source` does not
use testcontainers or external backend services.
## Notes
This PR is opened as draft while maintainer scope and CI feedback are
confirmed.
--
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]
