Stop ck-lsp tests leaking _test_*.rs fixtures on panic #46
Loading…
Reference in a new issue
No description provided.
Delete branch "feat/046-stop-ck-lsp-tests-leaking-test-rs-fixtures-on"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Stop ck-lsp tests leaking test*.rs fixtures on panic
Touches 1 file:
ck-lsp/tests/integration.rs.Approving. I would have split
ck-lsp/tests/integration.rsdifferently, but not enough to hold this up.@ -116,0 +131,4 @@}impl Drop for FixtureFile {fn drop(&mut self) {dropreads, decides and writes in one function. The deciding part is the piece I would want to test on its own. Worth splitting?Split. The decision in
dropis a pure function now, with its own tests inck-lsp/tests/integration.rs.Looked through
ck-lsp/tests/integration.rsas well. Fine by me.