Merge branch 'main' into dependabot/gradle/org.jsoup-jsoup-1.17.1

codeforamerica · Nov 27, 2023 · 11a3f75 · 11a3f75
2 parents 1991bd3 + 44f3df3
commit 11a3f75
Show file tree

Hide file tree

Showing 4 changed files with 47 additions and 16 deletions.
diff --git a/.github/workflows/publish-snapshot.yml b/.github/workflows/publish-snapshot.yml
@@ -32,7 +32,7 @@ jobs:
           server-id: github
           settings-path: ${{ github.workspace }}
       - name: Create jar and publish to Sonatype
-        uses: gradle/gradle-build-action@v2.9.0
+        uses: gradle/gradle-build-action@v2.10.0
         with:
           arguments: jar publish
         env:

diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -71,7 +71,7 @@ jobs:
           echo "signing.gnupg.passphrase=${{ secrets.PLATFORM_ROBOT_GPG_PASSPHRASE }}" >> ~/.gradle/gradle.properties
           echo "no-tty" >> ~/.gnupg/gpg.conf >> ~/.gradle/gradle.properties
       - name: Publish package
-        uses: gradle/gradle-build-action@v2.9.0
+        uses: gradle/gradle-build-action@v2.10.0
         with:
           arguments: publishToSonatype closeAndReleaseSonatypeStagingRepository
         env:

diff --git a/src/main/java/formflow/library/pdf/DatabaseFieldPreparer.java b/src/main/java/formflow/library/pdf/DatabaseFieldPreparer.java
@@ -17,20 +17,23 @@ public class DatabaseFieldPreparer implements DefaultSubmissionFieldPreparer {
   public Map<String, SubmissionField> prepareSubmissionFields(Submission submission, PdfMap pdfMap) {
     Map<String, SubmissionField> databaseFields = new HashMap<>();
     Map<String, Object> dbFields = pdfMap.getDbFields();
-
-    dbFields.forEach((fieldName, value) -> {
-      switch (fieldName) {
-        case "submittedAt" ->
-            databaseFields.put(fieldName, new DatabaseField(fieldName, formatDateWithNoTime(submission.getSubmittedAt())));
-        case "submissionId" -> databaseFields.put(fieldName, new DatabaseField(fieldName, submission.getId().toString()));
-        case "createdAt" ->
-            databaseFields.put(fieldName, new DatabaseField(fieldName, formatDateWithNoTime(submission.getCreatedAt())));
-        case "updatedAt" ->
-            databaseFields.put(fieldName, new DatabaseField(fieldName, formatDateWithNoTime(submission.getUpdatedAt())));
-        case "flow" -> databaseFields.put(fieldName, new DatabaseField(fieldName, submission.getFlow()));
-        default -> log.error("Unable to map unknown database field: {}", fieldName);
-      }
-    });
+
+    if (dbFields != null) {
+      dbFields.forEach((fieldName, value) -> {
+        switch (fieldName) {
+          case "submittedAt" ->
+              databaseFields.put(fieldName, new DatabaseField(fieldName, formatDateWithNoTime(submission.getSubmittedAt())));
+          case "submissionId" -> databaseFields.put(fieldName, new DatabaseField(fieldName, submission.getId().toString()));
+          case "createdAt" ->
+              databaseFields.put(fieldName, new DatabaseField(fieldName, formatDateWithNoTime(submission.getCreatedAt())));
+          case "updatedAt" ->
+              databaseFields.put(fieldName, new DatabaseField(fieldName, formatDateWithNoTime(submission.getUpdatedAt())));
+          case "flow" -> databaseFields.put(fieldName, new DatabaseField(fieldName, submission.getFlow()));
+          default -> log.error("Unable to map unknown database field: {}", fieldName);
+        }
+      });
+    }
+
     return databaseFields;
   }
 

diff --git a/src/test/java/formflow/library/pdf/DatabaseFieldPreparerTest.java b/src/test/java/formflow/library/pdf/DatabaseFieldPreparerTest.java
@@ -1,6 +1,7 @@
 package formflow.library.pdf;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 
 import formflow.library.data.Submission;
 import java.util.List;
@@ -59,4 +60,31 @@ void shouldNotCreateDbFieldsForItemsNotPresentInPdfMap() {
         Map.entry("updatedAt", new DatabaseField("updatedAt", "09/15/2020"))
     );
   }
+
+  @Test
+  void DbFieldsShouldNotBeRequired_shouldNotThrowErrorIfDbFieldsIsNull() {
+    DatabaseFieldPreparer dataBaseFieldPreparer = new DatabaseFieldPreparer();
+    PdfMap pdfMap = new PdfMap();
+    pdfMap.setFlow("flow1");
+    // Assert Does Not throw for a completely missing dbFields Map
+    pdfMapConfiguration = new PdfMapConfiguration(List.of(pdfMap));
+    assertDoesNotThrow(() -> dataBaseFieldPreparer.prepareSubmissionFields(submission,
+        pdfMapConfiguration.getPdfMap(submission.getFlow())));
+    assertThat(dataBaseFieldPreparer.prepareSubmissionFields(submission,
+        pdfMapConfiguration.getPdfMap(submission.getFlow()))).isEmpty();
+
+    // Assert does not throw for a null dbFields Map value
+    pdfMap.setDbFields(null);
+    assertDoesNotThrow(() -> dataBaseFieldPreparer.prepareSubmissionFields(submission,
+        pdfMapConfiguration.getPdfMap(submission.getFlow())));
+    assertThat(dataBaseFieldPreparer.prepareSubmissionFields(submission,
+        pdfMapConfiguration.getPdfMap(submission.getFlow()))).isEmpty();
+
+    // Assert does not throw for an empty dbFields Map value
+    pdfMap.setDbFields(Map.of());
+    assertDoesNotThrow(() -> dataBaseFieldPreparer.prepareSubmissionFields(submission,
+        pdfMapConfiguration.getPdfMap(submission.getFlow())));
+    assertThat(dataBaseFieldPreparer.prepareSubmissionFields(submission,
+        pdfMapConfiguration.getPdfMap(submission.getFlow()))).isEmpty();
+  }
 }