Address code review feedback - improve exception handling and remove Thread.yield

Co-authored-by: brendandburns <[email protected]>

Author: Copilot

extended/src/test/java/io/kubernetes/client/extended/event/EventCorrelatorTest.java

@@ -157,7 +157,12 @@ void eventCorrelate(
     // Using a polling loop instead of a fixed sleep makes the test more robust
     long startTime = System.currentTimeMillis();
     while (System.currentTimeMillis() - startTime < 100) {
-      Thread.sleep(10);
+      try {
+        Thread.sleep(10);
+      } catch (InterruptedException e) {
+        Thread.currentThread().interrupt();
+        throw e;
+      }
     }
     OffsetDateTime now = OffsetDateTime.now();
     newEvent.setFirstTimestamp(now);

spring/src/test/java/io/kubernetes/client/spring/extended/controller/KubernetesReconcilerCreatorTest.java

@@ -198,7 +198,12 @@ void simplePodController() throws InterruptedException {
     WorkQueue<Request> workQueue = ((DefaultController) testController).getWorkQueue();
     long deadline = System.currentTimeMillis() + 2000; // 2 second timeout
     while (workQueue.length() == 0 && System.currentTimeMillis() < deadline) {
-      Thread.sleep(50);
+      try {
+        Thread.sleep(50);
+      } catch (InterruptedException e) {
+        Thread.currentThread().interrupt();
+        throw e;
+      }
     }
 
     assertThat(workQueue.length()).isEqualTo(1);

util/src/test/java/io/kubernetes/client/PortForwardTest.java

@@ -187,8 +187,6 @@ void brokenPortPassing() throws IOException, InterruptedException {
     synchronized (block) {
       t.start();
       initStarted.await();
-      // Give init() a moment to process the message
-      Thread.yield();
       handler.close();
       block.wait();
     }