apache / incubator-graphar

An open source, standard data file format for graph data storage and retrieval.
https://graphar.apache.org/
Apache License 2.0
217 stars 46 forks source link

feat(devinfra): Add test coverage to CI to measure the covering code of unitest (Spark) #589

Closed Elssky closed 1 month ago

Elssky commented 1 month ago

Reason for this PR

solution to https://github.com/apache/incubator-graphar/issues/189, extension of https://github.com/apache/incubator-graphar/pull/580

What changes are included in this PR?

Add Spark test coverage to CI to measure the covering code of unitest### Are these changes tested?

Are there any user-facing changes?

no

codecov-commenter commented 1 month ago

Welcome to Codecov :tada:

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered :open_umbrella:

SemyonSinchenko commented 1 month ago

May we extend it to PySpark too? The coverage is already computed during the CI but it doesn't published. I see, that the Codecov tool has a support of pytest that is used for PySpark.

The entry point of PySpark tests is the Makefile: https://github.com/apache/incubator-graphar/blob/main/pyspark/Makefile#L35

Elssky commented 1 month ago

May we extend it to PySpark too? The coverage is already computed during the CI but it doesn't published. I see, that the Codecov tool has a support of pytest that is used for PySpark.

The entry point of PySpark tests is the Makefile: https://github.com/apache/incubator-graphar/blob/main/pyspark/Makefile#L35

Sure, I'll keep track of this task👌