palantir / atlasdb

Transactional Distributed Database Layer
https://palantir.github.io/atlasdb/
Apache License 2.0
52 stars 9 forks source link

Write tests for unwrapping KVS #3801

Open gmaretic opened 5 years ago

gmaretic commented 5 years ago

When creating a TM, we wrap the KVS in many layers. To access any methods on the underlying KVS that require casting, we have to repeatedly call getDelegates. We should have tests verifying that this behaviour is not broken.

This can either take form of unit tests or an ete test that unwraps the KVS provided by the TM.

felixdesouza commented 5 years ago

To access any methods on the underlying KVS that require casting..

Err, what? xD. This is terrifying.