apache / hudi-rs

A native Rust library for Apache Hudi, with bindings into Python
https://hudi.apache.org/
Apache License 2.0
153 stars 32 forks source link

test: obtain fixture artifact path programmatically #137

Open prabodh1194 opened 2 months ago

prabodh1194 commented 2 months ago

Description

I am setting up my dev environment in RustRover IDE and wanted to run the pytest from the IDE's command runner. However, the default setup was giving me some issue related to the .zip table artifact. To work around this, I have colocated a python script with the artifact so that I can deterministically get the correct path of the corresponding zip file.

How are the changes test-covered

codecov[bot] commented 2 months ago

Codecov Report

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

Project coverage is 87.77%. Comparing base (01ef2fc) to head (f5f50ed).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #137 +/- ## ======================================= Coverage 87.77% 87.77% ======================================= Files 14 14 Lines 728 728 ======================================= Hits 639 639 Misses 89 89 ```

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

prabodh1194 commented 2 months ago

@KnightChess should I go ahead & add a pre-commit hook for ruff format ?

prabodh1194 commented 2 months ago

fair enough. thank you.

xushiyan commented 2 months ago

@prabodh1194 I created https://github.com/apache/hudi-rs/issues/141 as an extended work for this. If you're interested, feel free to pick it up.

prabodh1194 commented 2 months ago

thanks. lemme try.