test: added rust sdk test for source transformer

numaproj / numaflow

Kubernetes-native platform to run massively parallel data/streaming jobs

https://numaflow.numaproj.io/

Apache License 2.0

1.09k stars 111 forks source link

test: added rust sdk test for source transformer #1756

Closed shubhamdixit863 closed 3 months ago

shubhamdixit863 commented 3 months ago

What this PR does

This PR introduces a new test for the Rust SDK aimed at ensuring the functionality of the source transformer module. By integrating this test, we aim to maintain high code quality and reliability of the transformations performed by this module. Fixes -https://github.com/numaproj/numaflow-rs/issues/56

Changes Introduced

Added Rust SDK test for source transformer to verify its correctness and stability.

The added tests were executed locally to confirm their effectiveness. Results confirm that the new Rust SDK test for the source transformer runs successfully without introducing regressions.

vigith commented 3 months ago

https://github.com/numaproj/numaflow/blob/main/test/sdks-e2e/sdks_test.go

vigith commented 3 months ago

@KeranYang we need the same publish scripts for Rust SDK too.

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 56.89%. Comparing base (adbc5ca) to head (8edfee7).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #1756 +/- ## ========================================== + Coverage 56.86% 56.89% +0.03% ========================================== Files 216 216 Lines 17350 17350 ========================================== + Hits 9866 9872 +6 + Misses 6659 6643 -16 - Partials 825 835 +10 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

shubhamdixit863 commented 3 months ago

E2E should pass once we rebuild & push the image with the protobuf time conversion fix, right? @shubhamdixit863

Yes its fixed