update

Author: mfriesen

dynamodb-documents/src/main/java/com/formkiq/stacks/dynamodb/DocumentServiceImpl.java

@@ -516,9 +516,11 @@ public boolean deleteDocument(final String siteId, final String documentId,
             .findAny().orElse(null);
       }
 
-      AttributeValueToMap transform = new AttributeValueToMap();
-      Map<String, Object> apply = transform.apply(documentRecord);
-      this.interceptor.deleteDocument(siteId, documentId, softDelete, apply);
+      if (documentRecord != null) {
+        AttributeValueToMap transform = new AttributeValueToMap();
+        Map<String, Object> apply = transform.apply(documentRecord);
+        this.interceptor.deleteDocument(siteId, documentId, softDelete, apply);
+      }
     }
 
     return deleted;

dynamodb-documents/src/test/java/com/formkiq/stacks/dynamodb/FolderIndexProcessorTest.java

@@ -658,7 +658,7 @@ public void testMove04() throws Exception {
   }
 
   private void verifyIndex(final String siteId, final FolderIndexRecord map, final String pk,
-                           final String sk, final String path, final boolean hasDates) {
+      final String sk, final String path, final boolean hasDates) {
 
     assertEquals(pk, map.pk(siteId));
     assertEquals(sk, map.sk());

lambda-api/src/integration/java/com/formkiq/stacks/api/awstest/FoldersRequestTest.java

@@ -144,38 +144,37 @@ void testGetFolders02() throws Exception {
     String siteId = UUID.randomUUID().toString();
 
     List<ApiClient> clients = getApiClients(siteId);
-    String random = UUID.randomUUID().toString();
+    final String random = UUID.randomUUID().toString();
 
-    for (ApiClient apiClient : clients) {
+    ApiClient apiClient = clients.get(0);
 
-      // when
-      for (int i = 0; i < numberOfThreads; i++) {
-        final int ii = i;
-        executorService.submit(() -> {
+    // when
+    for (int i = 0; i < numberOfThreads; i++) {
+      final int ii = i;
+      executorService.submit(() -> {
+        try {
           try {
-            try {
-              String path = "Chicago_" + random + "/sample" + ii + ".txt";
-              addDocument(apiClient, siteId, path, content.getBytes(StandardCharsets.UTF_8),
-                  "text/plain", null);
-            } catch (IOException | InterruptedException | URISyntaxException | ApiException e) {
-              throw new RuntimeException(e);
-            }
-          } finally {
-            latch.countDown();
+            String path = "Chicago_" + random + "/sample" + ii + ".txt";
+            addDocument(apiClient, siteId, path, content.getBytes(StandardCharsets.UTF_8),
+                "text/plain", null);
+          } catch (IOException | InterruptedException | URISyntaxException | ApiException e) {
+            throw new RuntimeException(e);
           }
-        });
-      }
+        } finally {
+          latch.countDown();
+        }
+      });
+    }
 
-      // then
-      latch.await();
-      executorService.shutdown();
+    // then
+    latch.await();
+    executorService.shutdown();
 
-      DocumentFoldersApi foldersApi = new DocumentFoldersApi(apiClient);
-      List<SearchResultDocument> docs = notNull(foldersApi
-          .getFolderDocuments(siteId, null, "Chicago_" + random, "200", null, null).getDocuments());
+    DocumentFoldersApi foldersApi = new DocumentFoldersApi(apiClient);
+    List<SearchResultDocument> docs = notNull(foldersApi
+        .getFolderDocuments(siteId, null, "Chicago_" + random, "200", null, null).getDocuments());
 
-      assertEquals(numberOfThreads, docs.size());
-    }
+    assertEquals(numberOfThreads, docs.size());
   }
 
   /**