more sonar

Author: mkleene

sdk/src/main/java/io/opentdf/platform/sdk/Config.java

@@ -1,6 +1,5 @@
 package io.opentdf.platform.sdk;
 
-import io.opentdf.platform.policy.Algorithm;
 import io.opentdf.platform.policy.KeyAccessServer;
 import io.opentdf.platform.policy.SimpleKasKey;
 import io.opentdf.platform.policy.Value;

sdk/src/test/java/io/opentdf/platform/sdk/AutoconfigureTest.java

@@ -662,7 +662,7 @@ public void cancel() {
             var wrapper = new Object() {
                 int i = 0;
             };
-            List<KeySplitStep> plan = reasoner.getSplits(tc.getDefaults(), () -> String.valueOf(wrapper.i++ + 1), () -> Optional.empty());
+            List<KeySplitStep> plan = reasoner.getSplits(tc.getDefaults(), () -> String.valueOf(wrapper.i++ + 1), Optional::empty);
             assertThat(plan)
                     .as(tc.name)
                     .hasSameElementsAs(tc.getPlan());
@@ -1024,11 +1024,11 @@ void testUsingDefaultKasesWhenNothingElseProvided() {
                 splitGen,
                 Optional::empty);
 
-        assertThat(splits).hasSize(2);
-        assertThat(splits).asList()
-                .containsExactly(
-                        new KeySplitStep("https://example.org/kas1", "0", null),
-                        new KeySplitStep("https://example.org/kas2", "1", null)
+        assertThat(splits)
+                .hasSize(2)
+                .asList().containsExactly(
+                    new KeySplitStep("https://example.org/kas1", "0", null),
+                    new KeySplitStep("https://example.org/kas2", "1", null)
                 );
     }
 

sdk/src/test/java/io/opentdf/platform/sdk/TestUtil.java

@@ -15,7 +15,7 @@ public ResponseMessage<T> execute() {
 
             @Override
             public void cancel() {
-
+                // in tests we don't need to preserve server resources, so no-op
             }
         };
     }