shoothzj opened a new pull request, #4329:
URL: https://github.com/apache/bookkeeper/pull/4329

   ### Motivation
   
   The tests under `bookkeeper-server` were failing on Windows due to path 
handling and other compatibility issues. The aim of this PR is to ensure that 
all tests pass on Windows as well as other operating systems by making the 
necessary adjustments in the test code.
   
   ### Changes
   
   - **TestBKConfiguration**: Updated the comment style from a block comment to 
a single-line comment for `setZkRetryBackoffMaxRetries`.
   - **TestTLS**:
     - Modified `getResourcePath` to use `Paths.get()` which handles file paths 
in a cross-platform manner.
     - Replaced `assertTrue` with `assertFalse` followed by `isEmpty()` to 
check if ensembles are present, improving the readability.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to