You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kvrocks.apache.org by GitBox <gi...@apache.org> on 2022/08/28 20:25:21 UTC

[GitHub] [incubator-kvrocks] torwig opened a new issue, #798: Unit tests don't remove test directory

torwig opened a new issue, #798:
URL: https://github.com/apache/incubator-kvrocks/issues/798

   ### Search before asking
   
   - [X] I had searched in the [issues](https://github.com/apache/incubator-kvrocks/issues) and found no similar issues.
   
   
   ### Version
   
   Kvrocks version: the latest unstable e4ae784a271b4d815eb3a015e27c6f6e296b3036
   OS: Fedora 35
   
   
   ### Minimal reproduce step
   
   Build and run unit tests by executing `./unittest`
   
   
   ### What did you expect to see?
   
   Deletion of the `testdb` folder at the end of unit tests.
   
   
   ### What did you see instead?
   
   Directory `testdb` exists, because `rmdir` function (https://github.com/apache/incubator-kvrocks/blob/unstable/tests/cppunit/test_base.h#L41) is unable to remove a non-empty directory.
   
   
   
   ### Anything Else?
   
   Directory `compactdb` also exists after unit tests have finished.
   If there were a function to traverse the content of a directory removing files and folders recursively, it would be applied to `compactdb` as well.
   
   ### Are you willing to submit a PR?
   
   - [ ] I'm willing to submit a PR!


-- 
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: issues-unsubscribe@kvrocks.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-kvrocks] git-hulk commented on issue #798: Unit tests don't remove test directory

Posted by GitBox <gi...@apache.org>.
git-hulk commented on issue #798:
URL: https://github.com/apache/incubator-kvrocks/issues/798#issuecomment-1229654183

   We didn't have the recursive delete directory yet.


-- 
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: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-kvrocks] tisonkun closed issue #798: Unit tests don't remove test directory

Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun closed issue #798: Unit tests don't remove test directory
URL: https://github.com/apache/incubator-kvrocks/issues/798


-- 
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: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org