Refactor tests to remove test fixtures and serial crate #10

New Issue

Ggg_123 · 2020-12-02T11:00:31+03:00

Ggg_123 commented

2020-12-02 11:00:31 +03:00

serial crate is required only because tests use the same test files, so they cannot read/write them at the same time.
Replacing serial with something like tempfile and moving test data from files into code will allow tests to run in parallel, making their execution faster and also prevent test data from becoming corrupted by misbehaving tests.

`serial` crate is required only because tests use the same test files, so they cannot read/write them at the same time. Replacing serial with something like [tempfile](https://crates.io/crates/tempfile) and moving test data from files into code will allow tests to run in parallel, making their execution faster and also prevent test data from becoming corrupted by misbehaving tests.

Ggg_123 self-assigned this 2020-12-02 11:00:31 +03:00

dkanus commented

2020-12-02 13:00:56 +03:00

serial was removed as a dependency, tests were rewritten to use separate temporary files.

Unnecessary files are cleaned up with auxiliary struct with the drop trait, which seems to work just fine and does not require another dependency. Maybe just keep this solution or is tempfile superior in some way?

Do we really need to move test data into the code? I feel like separating them is a cleaner approach.

`serial` was removed as a dependency, tests were rewritten to use separate temporary files. Unnecessary files are cleaned up with auxiliary struct with the `drop` trait, which seems to work just fine and does not require another dependency. Maybe just keep this solution or is `tempfile` superior in some way? Do we really need to move test data into the code? I feel like separating them is a cleaner approach.

Ggg_123 commented

2020-12-02 15:08:46 +03:00

Nvm then.

Ggg_123 closed this issue

2020-12-02 15:08:57 +03:00

Sign in to join this conversation.