More testing

Author: scottgerring

powertools-e2e-tests/src/test/java/software/amazon/lambda/powertools/ParametersE2ET.java

@@ -48,11 +48,11 @@ public void setup() {
         functionName = infrastructure.deploy();
     }
 
-//    @AfterAll
-//    public void tearDown() {
-//        if (infrastructure != null)
-//            infrastructure.destroy();
-//    }
+    @AfterAll
+    public void tearDown() {
+        if (infrastructure != null)
+            infrastructure.destroy();
+    }
 
     @Test
     public void test_getAppConfigValue() {

powertools-parameters/src/main/java/software/amazon/lambda/powertools/parameters/AppConfigProvider.java

@@ -38,14 +38,6 @@ private EstablishedSession(String nextSessionToken, String value) {
             this.nextSessionToken = nextSessionToken;
             this.lastConfigurationValue = value;
         }
-
-        public String getNextSessionToken() {
-            return nextSessionToken;
-        }
-
-        public String getLastConfigurationValue() {
-            return lastConfigurationValue;
-        }
     }
 
     private final AppConfigDataClient client;

powertools-parameters/src/test/java/software/amazon/lambda/powertools/parameters/AppConfigProviderTest.java

@@ -14,8 +14,11 @@
 import software.amazon.awssdk.services.appconfigdata.model.StartConfigurationSessionRequest;
 import software.amazon.awssdk.services.appconfigdata.model.StartConfigurationSessionResponse;
 import software.amazon.lambda.powertools.parameters.cache.CacheManager;
+import software.amazon.lambda.powertools.parameters.transform.TransformationManager;
+
 import static org.assertj.core.api.Assertions.assertThat;
 
+import java.time.Duration;
 import java.util.Optional;
 
 import static org.mockito.ArgumentMatchers.eq;
@@ -26,9 +29,6 @@ public class AppConfigProviderTest {
     @Mock
     AppConfigDataClient client;
 
-//    @Mock
-//    GetEnvironmentResponse environment;
-
     private AppConfigProvider provider;
 
     @Captor
@@ -49,9 +49,13 @@ public class AppConfigProviderTest {
     @BeforeEach
     public void init() {
         openMocks(this);
-
-        CacheManager cacheManager = new CacheManager();
-        provider = new AppConfigProvider(cacheManager, client, environmentName, applicationName);
+        provider = AppConfigProvider.builder()
+                .withClient(client)
+                .withApplication(applicationName)
+                .withEnvironment(environmentName)
+                .withCacheManager(new CacheManager())
+                .withTransformationManager(new TransformationManager())
+                .build();
     }