Test fix

DataBiosphere · Sep 12, 2024 · 8a91e0b · 8a91e0b
1 parent ac47f4b
commit 8a91e0b
Showing 1 changed file with 12 additions and 18 deletions.
diff --git a/service/src/main/java/bio/terra/tanagra/db/CohortDao.java b/service/src/main/java/bio/terra/tanagra/db/CohortDao.java
@@ -10,7 +10,6 @@
 import bio.terra.tanagra.service.artifact.model.Cohort;
 import bio.terra.tanagra.service.artifact.model.CohortRevision;
 import bio.terra.tanagra.service.artifact.model.Criteria;
-import jakarta.annotation.Nullable;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collections;
@@ -798,22 +797,17 @@ MapSqlParameterSource buildRevisionParam(
       boolean isMostRecent,
       boolean isEditable,
       String userEmail,
-      long recordsCount,
-      @Nullable String reviewId) {
-    MapSqlParameterSource params =
-        new MapSqlParameterSource()
-            .addValue("cohort_id", cohortId)
-            .addValue("id", id)
-            .addValue("version", version)
-            .addValue("is_most_recent", isMostRecent)
-            .addValue("is_editable", isEditable)
-            .addValue("created_by", userEmail)
-            .addValue("last_modified_by", userEmail)
-            .addValue("records_count", recordsCount);
-
-    if (reviewId != null) {
-      params.addValue("review_id", reviewId);
-    }
-    return params;
+      Long recordsCount,
+      String reviewId) {
+    return new MapSqlParameterSource()
+        .addValue("cohort_id", cohortId)
+        .addValue("id", id)
+        .addValue("version", version)
+        .addValue("is_most_recent", isMostRecent)
+        .addValue("is_editable", isEditable)
+        .addValue("created_by", userEmail)
+        .addValue("last_modified_by", userEmail)
+        .addValue("records_count", recordsCount)
+        .addValue("review_id", reviewId);
   }
 }