Consider using cached-path crate in resources module about rust-bert

[epwalsh]

Hey @guillaume-be, are you asking me to add tests to the cached-path crate? We have already some tests here: https://github.com/epwalsh/rust-cached-path/blob/master/src/cache.rs#L433

[guillaume-be]

Hi @epwalsh nice crate and seems to align well with the current resources downloader. The check for Etag and file locks is a nice addition.

I am trying to be mindful of the dependencies pulled in the project. I see you are relying on the failure crate which has been deprecated for a few months now. Is there any way you could rely instead on the thiserror crate for your API error handling, and anyhow for dev/examples?

[epwalsh]

Hey @guillaume-be, yeup, that sounds good. Didn't realize failure has been deprecated.

[epwalsh]

Just made a PR to switch over: epwalsh/rust-cached-path#18

[guillaume-be]

@epwalsh Thank you - this looks good. I also noticed that the crate currently does not have further dependencies - which is fine on my side. Because of this I believe a good unit test and integration test coverage would be extremely helpful. Could you please add test for key functionalities of the crate (e.g. file lock, cache corruption, Etag check) in either unit or integration tests and add them to the CI?

[guillaume-be]

Sorry I missed those!