go-zookeeper / zk

Native ZooKeeper client for Go
BSD 3-Clause "New" or "Revised" License
504 stars 130 forks source link

adding requireMinimumZkVersion test helper #133

Closed jeffbean closed 3 months ago

jeffbean commented 4 months ago

inspiration from https://github.com/go-zookeeper/zk/pull/88

cleanup up the logic. but the idea is to support the tests requiring a version of ZK based on the API support in ZK.

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.91%. Comparing base (03869bc) to head (bfdb74a).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #133 +/- ## ========================================== + Coverage 76.55% 77.91% +1.36% ========================================== Files 7 7 Lines 1322 1322 ========================================== + Hits 1012 1030 +18 + Misses 217 200 -17 + Partials 93 92 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.