lie18uci commented on PR #5633:
URL: https://github.com/apache/texera/pull/5633#issuecomment-4687933502

   > @lie18uci thanks for the fix. Could you please add tests to guard the 
change?
   
   Thank you for the review. I added `GitVersionControlLocalFileStorageSpec`, 
which creates a temporary nested repository directory, calls `deleteRepo`, and 
verifies that the repository directory is deleted recursively.
   
   I also ran `sbt scalafmtCheckAll` locally successfully. My local targeted 
test run is still blocked by jOOQ/PostgreSQL setup because PostgreSQL is not 
running on `localhost:5432`, so I am relying on GitHub CI for the backend test 
run.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to