codestyle

Author: Jonas-Isr

cloudplatform/connectivity-destination-service/src/main/java/com/sap/cloud/sdk/cloudplatform/connectivity/DestinationService.java

@@ -41,7 +41,6 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.RequiredArgsConstructor;
-import lombok.Setter;
 import lombok.extern.slf4j.Slf4j;
 
 /**
@@ -121,12 +120,7 @@ static ResilienceConfiguration createResilienceConfiguration(
         Try<Destination>
         tryGetDestination( @Nonnull final String destinationName, @Nonnull final DestinationOptions options )
     {
-        return Cache
-            .getOrComputeDestination(
-                this,
-                destinationName,
-                options,
-                this::loadAndParseDestination);
+        return Cache.getOrComputeDestination(this, destinationName, options, this::loadAndParseDestination);
     }
 
     Destination loadAndParseDestination( final String destName, final DestinationOptions options )
@@ -451,12 +445,14 @@ static boolean isChangeDetectionEnabled()
             return changeDetectionEnabled;
         }
 
-        static boolean isGetAllDocumentsPrepended() {
-          return getAllDocumentsPrepended;
+        static boolean isGetAllDocumentsPrepended()
+        {
+            return getAllDocumentsPrepended;
         }
 
-        public static void setGetAllDocumentsPrepended(boolean bool) {
-          getAllDocumentsPrepended = bool;
+        public static void setGetAllDocumentsPrepended( boolean bool )
+        {
+            getAllDocumentsPrepended = bool;
         }
 
         @Nonnull
@@ -856,7 +852,7 @@ private static Try<Destination> getOrComputeDestination(
             @Nonnull final DestinationService loader,
             @Nonnull final String destinationName,
             @Nonnull final DestinationOptions options,
-            @Nonnull final BiFunction<String, DestinationOptions, Destination> destinationDownloader)
+            @Nonnull final BiFunction<String, DestinationOptions, Destination> destinationDownloader )
         {
             if( !cacheEnabled ) {
                 return Try.ofSupplier(() -> destinationDownloader.apply(destinationName, options));

cloudplatform/connectivity-destination-service/src/test/java/com/sap/cloud/sdk/cloudplatform/connectivity/DestinationServiceTest.java

@@ -1930,8 +1930,7 @@ void testPrependGetAllDestinationsCall()
             .when(destinationServiceAdapter)
             .getConfigurationAsJson(eq("/v1/subaccountDestinations"), any());
 
-          DestinationService.Cache.setGetAllDocumentsPrepended(true);
-
+        DestinationService.Cache.setGetAllDocumentsPrepended(true);
 
         final DestinationOptions options =
             DestinationOptions.builder().augmentBuilder(augmenter().retrievalStrategy(ALWAYS_PROVIDER)).build();
@@ -1944,7 +1943,7 @@ void testPrependGetAllDestinationsCall()
             .getConfigurationAsJson(eq("/v1/destinations/" + destinationName), any());
         verifyNoMoreInteractions(destinationServiceAdapter);
 
-      DestinationService.Cache.setGetAllDocumentsPrepended(false);
+        DestinationService.Cache.setGetAllDocumentsPrepended(false);
     }
 
     @Test
@@ -1957,7 +1956,7 @@ void testPrependGetAllDestinationsCallWithMissingDestination()
             .when(destinationServiceAdapter)
             .getConfigurationAsJson(eq("/v1/subaccountDestinations"), any());
 
-      DestinationService.Cache.setGetAllDocumentsPrepended(true);
+        DestinationService.Cache.setGetAllDocumentsPrepended(true);
 
         final DestinationOptions options =
             DestinationOptions.builder().augmentBuilder(augmenter().retrievalStrategy(ALWAYS_PROVIDER)).build();
@@ -1970,6 +1969,6 @@ void testPrependGetAllDestinationsCallWithMissingDestination()
         verify(destinationServiceAdapter, times(1)).getConfigurationAsJson(eq("/v1/subaccountDestinations"), any());
         verifyNoMoreInteractions(destinationServiceAdapter);
 
-      DestinationService.Cache.setGetAllDocumentsPrepended(false);
+        DestinationService.Cache.setGetAllDocumentsPrepended(false);
     }
 }