#13159 Automatically (Soft-)Delete Samples & Pathogen Tests with Negative Test Results for COVID-19 - rename config property

Author: Levente Gal

sormas-api/src/main/java/de/symeda/sormas/api/ConfigFacade.java

@@ -162,5 +162,5 @@ public interface ConfigFacade {
 
 	boolean isAnyCaseClassificationCalculationEnabled();
 
-	Integer getNegaiveCovidSamplesMaxAgeDays();
+	Integer getNegaiveCovidTestsMaxAgeDays();
 }

sormas-backend/src/main/java/de/symeda/sormas/backend/common/ConfigFacadeEjb.java

@@ -187,7 +187,7 @@ public class ConfigFacadeEjb implements ConfigFacade {
 	public static final int DEFAULT_DOCUMENT_UPLOAD_SIZE_LIMIT_MB = 20;
 	public static final String IMPORT_FILE_SIZE_LIMIT_MB = "importFileSizeLimitMb";
 	public static final int DEFAULT_IMPOR_FILE_SIZE_LIMIT_MB = 20;
-	public static final String NEGAIVE_COVID_SAMPLES_MAX_AGE_DAYS = "negaiveCovidSamplesMaxAgeDays";
+	public static final String NEGATIVE_COVID_TESTS_MAX_AGE_DAYS = "negativeCovidTestsMaxAgeDays";
 
 	private final Logger logger = LoggerFactory.getLogger(getClass());
 
@@ -840,8 +840,8 @@ public void resetRequestContext() {
 	}
 
 	@Override
-	public Integer getNegaiveCovidSamplesMaxAgeDays() {
-		return parseProperty(NEGAIVE_COVID_SAMPLES_MAX_AGE_DAYS, null, Integer::parseInt);
+	public Integer getNegaiveCovidTestsMaxAgeDays() {
+		return parseProperty(NEGATIVE_COVID_TESTS_MAX_AGE_DAYS, null, Integer::parseInt);
 	}
 
 	@LocalBean

sormas-backend/src/main/java/de/symeda/sormas/backend/sample/SampleService.java

@@ -1286,7 +1286,7 @@ public List<DiseaseVariant> getAssociatedDiseaseVariants(String sampleUuid) {
 	}
 
 	public void cleanupOldCovidSamples() {
-		final Integer maxAgeDays = configFacade.getNegaiveCovidSamplesMaxAgeDays();
+		final Integer maxAgeDays = configFacade.getNegaiveCovidTestsMaxAgeDays();
 		if (maxAgeDays == null) {
 			return;
 		}

sormas-backend/src/test/java/de/symeda/sormas/backend/sample/DeleteOldPathogenTestsAndSamplesTest.java

@@ -54,7 +54,7 @@ public void init() {
 		user = creator.createUser(rdcf).toReference();
 		caze = creator.createCase(user, creator.createPerson().toReference(), rdcf);
 
-		MockProducer.getProperties().setProperty(ConfigFacadeEjb.NEGAIVE_COVID_SAMPLES_MAX_AGE_DAYS, String.valueOf(negativeCovidTestMaxAge));
+		MockProducer.getProperties().setProperty(ConfigFacadeEjb.NEGATIVE_COVID_TESTS_MAX_AGE_DAYS, String.valueOf(negativeCovidTestMaxAge));
 	}
 
 	@Test
@@ -242,7 +242,7 @@ public void testDeletionReferenceDate() {
 
 	@Test
 	public void testNotConfigured() {
-		MockProducer.getProperties().remove(ConfigFacadeEjb.NEGAIVE_COVID_SAMPLES_MAX_AGE_DAYS);
+		MockProducer.getProperties().remove(ConfigFacadeEjb.NEGATIVE_COVID_TESTS_MAX_AGE_DAYS);
 
 		SampleDto sample = creator.createSample(caze.toReference(), user, rdcf.facility, s -> {
 			s.setPathogenTestResult(PathogenTestResultType.NEGATIVE);

sormas-base/setup/sormas.properties

@@ -143,7 +143,7 @@ app.url=
 # Number of days after which negative CORONAVIRUS pathogen tests and their samples are soft deleted
 # default: not set, meaning no delete
 # possible values: any integer representing the number of days
-# negativeCovidSamplesMaxAgeDays
+# negativeCovidTestsMaxAgeDays
 
 # The similarity threshold after which two names are identified as similar enough to consider them for duplicate detection.
 # The default value should work for most servers. If you need to change it, please change it carefully as slightly higher or lower values already lead to significant differences.