fix(config): removed setFile setter (internal)

Signed-off-by: Marc Nuri <[email protected]>
fabric8io · Nov 5, 2024 · 3fc317b · 3fc317b
1 parent 4c37d54
commit 3fc317b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 12 deletions.
diff --git a/kubernetes-client-api/src/main/java/io/fabric8/kubernetes/client/Config.java b/kubernetes-client-api/src/main/java/io/fabric8/kubernetes/client/Config.java
@@ -1499,10 +1499,6 @@ public void setAdditionalProperty(String name, Object value) {
     this.additionalProperties.put(name, value);
   }
 
-  public void setFile(File file) {
-    this.file = file;
-  }
-
   public void setAutoConfigure(boolean autoConfigure) {
     this.autoConfigure = autoConfigure;
   }

diff --git a/...i/src/test/java/io/fabric8/kubernetes/client/utils/OpenIDConnectionUtilsBehaviorTest.java b/...i/src/test/java/io/fabric8/kubernetes/client/utils/OpenIDConnectionUtilsBehaviorTest.java
@@ -484,15 +484,15 @@ void skipsInFileWhenOriginalConfigHasNoFile() {
 
     @Test
     void skipsInFileWhenOriginalConfigHasNoCurrentContext() {
-      originalConfig.setFile(kubeConfig);
+      originalConfig = new ConfigBuilder(originalConfig).withFile(kubeConfig).build();
       persistOAuthToken(originalConfig, oAuthTokenResponse, "fake.token");
       assertThat(kubeConfig).doesNotExist();
     }
 
     @Test
     void logsWarningIfReferencedFileIsMissing() {
-      originalConfig.setFile(kubeConfig);
       originalConfig = new ConfigBuilder(originalConfig)
+          .withFile(kubeConfig)
           .withCurrentContext(new NamedContextBuilder().withName("context").build()).build();
       persistOAuthToken(originalConfig, oAuthTokenResponse, "fake.token");
       assertThat(systemErr.toString())
@@ -511,9 +511,9 @@ void setUp() throws IOException {
       }
 
       @Test
-      void persistsTokenInFile() throws IOException {
-        originalConfig.setFile(kubeConfig);
+      void persistsTokenInFile() {
         originalConfig = new ConfigBuilder(originalConfig)
+            .withFile(kubeConfig)
             .withCurrentContext(new NamedContextBuilder()
                 .withName("context")
                 .withNewContext().withUser("user").endContext().build())
@@ -524,9 +524,9 @@ void persistsTokenInFile() throws IOException {
       }
 
       @Test
-      void skipsTokenInFileIfNull() throws IOException {
-        originalConfig.setFile(kubeConfig);
+      void skipsTokenInFileIfNull() {
         originalConfig = new ConfigBuilder(originalConfig)
+            .withFile(kubeConfig)
             .withCurrentContext(new NamedContextBuilder()
                 .withName("context")
                 .withNewContext().withUser("user").endContext().build())
@@ -537,9 +537,9 @@ void skipsTokenInFileIfNull() throws IOException {
       }
 
       @Test
-      void persistsOAuthTokenInFile() throws IOException {
-        originalConfig.setFile(kubeConfig);
+      void persistsOAuthTokenInFile() {
         originalConfig = new ConfigBuilder(originalConfig)
+            .withFile(kubeConfig)
             .withCurrentContext(new NamedContextBuilder()
                 .withName("context")
                 .withNewContext().withUser("user").endContext().build())