couchbase-shell icon indicating copy to clipboard operation
couchbase-shell copied to clipboard
verified publisher icon couchbaselabs

Improve testing framework

Open Westwooo opened this issue 1 year ago • 3 comments

Westwooo avatar Apr 30 '24 10:04 Westwooo

It might be worth having a google to see how others have approach the "setup suite" issue. When I looked it seems like there are crates out there that can help a bit.

chvck avatar May 01 '24 07:05 chvck

FWIW I think integration tests should also remain in the tests directory, and unit tests alongside the code. Another thing to consider is whether we can add some unit tests for trickier cases to repro, see the mocking approach in https://github.com/couchbaselabs/couchbase-rs/pull/155. Out of scope for this PR though.

chvck avatar May 01 '24 07:05 chvck

FWIW I think integration tests should also remain in the tests directory, and unit tests alongside the code. Another thing to consider is whether we can add some unit tests for trickier cases to repro, see the mocking approach in couchbaselabs/couchbase-rs#155. Out of scope for this PR though.

I agree, have moved everything back.

Westwooo avatar May 01 '24 11:05 Westwooo