Merge pull request #1381 from sarveshkaushal/master

[Code cleanup] making nested class static

Author: k8s-ci-robot

extended/src/main/java/io/kubernetes/client/extended/event/legacy/EventAggregator.java

@@ -88,7 +88,7 @@ record = this.spammingCache.get(aggregatedKey, AggregatedRecord::new);
     return new MutablePair<>(aggregatedEvent, aggregatedKey);
   }
 
-  private class AggregatedRecord {
+  private static class AggregatedRecord {
     private Set<String> localKeys = new HashSet<>();
     private DateTime lastTimestamp;
 

extended/src/main/java/io/kubernetes/client/extended/event/legacy/EventLogger.java

@@ -67,7 +67,7 @@ public void updateState(CoreV1Event event) {
     this.eventCache.put(key, log);
   }
 
-  private class EventLog {
+  private static class EventLog {
     private Integer count;
     private DateTime firstTimestamp;
     private String name;

extended/src/main/java/io/kubernetes/client/extended/workqueue/DefaultDelayingQueue.java

@@ -125,7 +125,7 @@ private void insert(
   }
 
   // WaitForEntry holds the data to add and the time it should be added.
-  private class WaitForEntry<T> implements Delayed {
+  private static class WaitForEntry<T> implements Delayed {
 
     private WaitForEntry(T data, Temporal readyAtMillis) {
       this.data = data;

extended/src/test/java/io/kubernetes/client/extended/controller/ControllerManagerTest.java

@@ -35,7 +35,7 @@ public void testControllerStartShutdown() {
     assertTrue(dummy2.stopped);
   }
 
-  class DummyController implements Controller {
+  static class DummyController implements Controller {
 
     private boolean started;
     private boolean stopped;

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

@@ -91,7 +91,7 @@ public KubernetesInformerConfigurer kubernetesInformerConfigurer(
                   apiVersion = "v1",
                   resourcePlural = "configmaps")),
     })
-    class TestSharedInformerFactory extends SharedInformerFactory {}
+    static class TestSharedInformerFactory extends SharedInformerFactory {}
   }
 
   @Autowired private SharedInformerFactory informerFactory;

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

@@ -122,7 +122,7 @@ public void testHandlerSendingLargeData() throws IOException {
     assertArrayEquals(output, mockWebSocket.data);
   }
 
-  private class MockWebSocket implements WebSocket {
+  private static class MockWebSocket implements WebSocket {
     byte[] data;
     private boolean closed = false;
 

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

@@ -221,7 +221,7 @@ public void testNullNamespace() {
     assertNull(config.getNamespace());
   }
 
-  class FakeAuthenticator implements Authenticator {
+  static class FakeAuthenticator implements Authenticator {
     public String token;
     public String refresh;
     public boolean expired;