minor cleanups

fabric8io · Apr 1, 2021 · e642d5c · e642d5c
1 parent ae80f8c
commit e642d5c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/.../main/java/io/fabric8/kubernetes/client/dsl/internal/RawCustomResourceOperationsImpl.java b/.../main/java/io/fabric8/kubernetes/client/dsl/internal/RawCustomResourceOperationsImpl.java
@@ -1085,9 +1085,7 @@ private String getPatchDiff(String namespace, String customResourceName, Map<Str
 
     JsonNode newone = JsonDiff.asJson(PatchUtils.patchMapper().valueToTree(oldObject), PatchUtils.patchMapper().valueToTree(customResource));
 
-    String patch = objectMapper.writeValueAsString(newone);
-
-    return patch;
+    return objectMapper.writeValueAsString(newone);
   }
 
   private DeleteOptions fetchDeleteOptions(boolean cascading, String propagationPolicy) {

diff --git a/...t/java/io/fabric8/kubernetes/client/dsl/internal/RawCustomResourceOperationsImplTest.java b/...t/java/io/fabric8/kubernetes/client/dsl/internal/RawCustomResourceOperationsImplTest.java
@@ -666,7 +666,7 @@ void testEditCR() throws IOException {
     Map<String, Object> res = rawCustomResourceOperations.edit(jsonString);
 
     // Then
-    assertNotEquals(res, null);
+    assertNotEquals(null, res);
   }
 
   private void assertRequestCaptured(ArgumentCaptor<Request> captor, String url, String method) {