fix(core): Proper retry count in KazooRetry

[ceache]

Make sure the number of attempts matches the max_retry parameter. Add unit tests to that effect.

Why is this needed?

So that users can precisely control the number of retries

Proposed Changes

• Move the location of the attempt increase to before the check against max_try so we don't end up calling max_tries + 1 times the function.

Does this PR introduce any breaking change?

No, it shouldn't.

[codecov[bot]]

Codecov Report

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

Project coverage is 97.01%. Comparing base (2fb93a8) to head (73309b2). Report is 2 commits behind head on master.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #748 +/- ## ========================================== + Coverage 96.81% 97.01% +0.19% ========================================== Files 27 27 Lines 3549 3549 ========================================== + Hits 3436 3443 +7 + Misses 113 106 -7 ```

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