risingwavelabs / risingwave

Best-in-class stream processing, analytics, and management. Perform continuous analytics, or build event-driven applications, real-time ETL pipelines, and feature stores in minutes. Unified streaming and batch. PostgreSQL compatible.
https://go.risingwave.com/slack
Apache License 2.0
6.97k stars 575 forks source link

ci: e2e coverage for Rust #32

Closed skyzh closed 1 year ago

skyzh commented 2 years ago

Currently, e2e doesn't take Rust codebase into account when calculating coverage. In the future, we should do this.

liurenjie1024 commented 2 years ago

Is it really reasonable to include e2e test when calculating coverage?

skyzh commented 2 years ago

Our java frontend uses e2e for coverage. When we have a Rust frontend, we also need this.

skyzh commented 2 years ago

cargo llvm provides an easy way to run external tests:

https://github.com/taiki-e/cargo-llvm-cov#get-coverage-of-external-tests

We can use this to generate e2e test coverage.