Byron/trash-rs

Tests don't remove created files.

Aberta

#66 aberto em 2 de mai. de 2023

 (1 comentário) (0 reação) (0 responsável)Rust (40 forks)auto 404
enhancementhelp wanted

Métricas do repositório

Stars
 (247 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Running cargo test doesn't remove the files/folders created during tests as shown below. image Of course, these are mostly 0 bytes files but I think it is a little bit messy to leave them there, I can't think of any reason for not deleting them, but maybe I am missing something.

Proposed solution

As far as I could see, all tests are more or less of the form:

fn test_delete_file() {
    create_things_to_be_deleted();
    attempt_to_delete_using_trash();
    assert_deletion();
}

What I propose is adding one more block to the tests where we attempt to delete the created files like so.

fn test_delete_file() {
    create_things_to_be_deleted();
    attempt_to_delete_using_trash();
    delete_from_trash_manually();
    assert_deletion();
}

Guia do colaborador