replikativ / konserve

A clojuresque key-value/document store protocol with core.async.
Eclipse Public License 1.0
299 stars 25 forks source link

Compliance tests for failures #36

Closed alekcz closed 3 years ago

alekcz commented 4 years ago
alekcz commented 4 years ago

@whilo the current the konserve fails the test suite what are your thoughts?

whilo commented 3 years ago

I somewhat get the sense for this kind of testing, but I am still not sure whether this corruption should not be stochastic and somewhat deeper in the logic/datastructure. At least right now we are not necessarily testing the full error propagation. I am still fine with merging this though, it cannot harm.

alekcz commented 3 years ago

The main idea of the corruption is to ensure that interactions with the underlying store always fails. In doing so the we can make sure that the backend handles these errors gracefully while propagating them in adherence with the protocol.

FYI: When I last checked the filestore failed these compliance tests.