fix #758 (#759)

remove trailing slash for multiple params

Author: jeromevdl

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

@@ -123,13 +123,15 @@ public BaseProvider withTransformation(Class<? extends Transformer> transformerC
      */
     @Override
     public Map<String, String> getMultiple(String path) {
+        // remove trailing whitespace
+        String pathWithoutTrailingSlash = path.replaceAll("\\/+$", "");
         try {
-            return (Map<String, String>) cacheManager.getIfNotExpired(path, now()).orElseGet(() -> {
-                Map<String, String> params = getMultipleValues(path);
+            return (Map<String, String>) cacheManager.getIfNotExpired(pathWithoutTrailingSlash, now()).orElseGet(() -> {
+                Map<String, String> params = getMultipleValues(pathWithoutTrailingSlash);
 
-                cacheManager.putInCache(path, params);
+                cacheManager.putInCache(pathWithoutTrailingSlash, params);
 
-                params.forEach((k, v) -> cacheManager.putInCache(path + "/" + k, v));
+                params.forEach((k, v) -> cacheManager.putInCache(pathWithoutTrailingSlash + "/" + k, v));
 
                 return params;
             });

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

@@ -103,6 +103,29 @@ public void getMultiple() {
         assertThat(paramByPathCaptor.getValue().recursive()).isFalse();
     }
 
+    @Test
+    public void getMultipleWithTrailingSlash() {
+        List<Parameter> parameters = new ArrayList<>();
+        parameters.add(Parameter.builder().name("/prod/app1/key1").value("foo1").build());
+        parameters.add(Parameter.builder().name("/prod/app1/key2").value("foo2").build());
+        parameters.add(Parameter.builder().name("/prod/app1/key3").value("foo3").build());
+        GetParametersByPathResponse response = GetParametersByPathResponse.builder().parameters(parameters).build();
+        when(client.getParametersByPath(paramByPathCaptor.capture())).thenReturn(response);
+
+        Map<String, String> params = provider.getMultiple("/prod/app1/");
+        assertThat(params).contains(
+                MapEntry.entry("key1", "foo1"),
+                MapEntry.entry("key2", "foo2"),
+                MapEntry.entry("key3", "foo3"));
+        assertThat(provider.get("/prod/app1/key1")).isEqualTo("foo1");
+        assertThat(provider.get("/prod/app1/key2")).isEqualTo("foo2");
+        assertThat(provider.get("/prod/app1/key3")).isEqualTo("foo3");
+
+        assertThat(paramByPathCaptor.getValue().path()).isEqualTo("/prod/app1");
+        assertThat(paramByPathCaptor.getValue().withDecryption()).isFalse();
+        assertThat(paramByPathCaptor.getValue().recursive()).isFalse();
+    }
+
     @Test
     public void getMultiple_cached_shouldNotCallSSM() {
         List<Parameter> parameters = new ArrayList<>();