[Merged by Bors] - feat: test mqtt connector

[EstebanBorai]

Test message consumption from MQTT using Cloud and Connectors

[galibey]

Since this connector already has integration tests written in Rust, I would propose choosing this approach for Cloud integration tests as well. It will give us more stability (no static sleeps) and better maintainability.

[EstebanBorai]

Since this connector already has integration tests written in Rust, I would propose choosing this approach for Cloud integration tests as well. It will give us more stability (no static sleeps) and better maintainability.

I really like the idea of using only Rust but the goal with these is to perform tests against CLI just as the user will do.

[galibey]

Since this connector already has integration tests written in Rust, I would propose choosing this approach for Cloud integration tests as well. It will give us more stability (no static sleeps) and better maintainability.

I really like the idea of using only Rust but the goal with these is to perform tests against CLI just as the user will do.

Yes, and this all can be done by Rust. For instance -

mqtt-connector/integration-tests/src/main.rs

Line 312 in cd17072

let output = Command::new("cdk").arg("deploy").arg("list").output()?;

[sehz]

Since this connector already has integration tests written in Rust, I would propose choosing this approach for Cloud integration tests as well. It will give us more stability (no static sleeps) and better maintainability.

I really like the idea of using only Rust but the goal with these is to perform tests against CLI just as the user will do.

Yes, and this all can be done by Rust. For instance -

mqtt-connector/integration-tests/src/main.rs

Line 312 in cd17072

let output = Command::new("cdk").arg("deploy").arg("list").output()?;

While I agree rust based CLI test will be better, we dont have good tooling yet. Fluvio repo uses BATS still and should be ok to use until we have rust replacement. We should have issue to replace it and make progress on this

[EstebanBorai]

bors r+

[bors]

Pull request successfully merged into main.

Build succeeded:

• Done