fix code smells

dhis2 · Dec 12, 2024 · e5d89c9 · e5d89c9
1 parent d99550b
commit e5d89c9
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 14 deletions.
diff --git a/app/src/main/java/org/dhis2/utils/customviews/DateDialog.java b/app/src/main/java/org/dhis2/utils/customviews/DateDialog.java
@@ -130,15 +130,13 @@ private void manageButtonPeriods(boolean next) {
                     activity,
                     fromToSelector,
                     true,
-                    () -> {
-                        disposable.add(new RxDateDialog((ActivityGlobalAbstract) requireActivity(), Period.WEEKLY)
-                                .createForFilter().show()
-                                .subscribe(
-                                        periods -> fromToSelector.onFromToSelected(new ArrayList<>()),
-                                        Timber::d
-                                )
-                        );
-                    }
+                    () -> disposable.add(new RxDateDialog((ActivityGlobalAbstract) requireActivity(), Period.WEEKLY)
+                            .createForFilter().show()
+                            .subscribe(
+                                    periods -> fromToSelector.onFromToSelected(new ArrayList<>()),
+                                    Timber::d
+                            )
+                    )
             );
             dismiss();
         }

diff --git a/app/src/test/java/org/dhis2/utils/DateUtilsTest.java b/app/src/test/java/org/dhis2/utils/DateUtilsTest.java
@@ -6,8 +6,6 @@
 import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
 
-import android.util.Log;
-
 import org.dhis2.R;
 import org.dhis2.bindings.StringExtensionsKt;
 import org.dhis2.commons.date.DateUtils;
@@ -644,7 +642,7 @@ public void isInputPeriodDateInsideFutureOpenDayConfiguration() throws ParseExce
                 DateUtils.oldUiDateFormat().parse("2022-11-05")
         );
 
-        assertEquals(true, DateUtils.getInstance().isInsideFutureInputPeriod(inputPeriod.endPeriodDate(), 5));
+        assertTrue(DateUtils.getInstance().isInsideFutureInputPeriod(inputPeriod.endPeriodDate(), 5));
     }
 
     @Test
@@ -659,7 +657,7 @@ public void isInputPeriodDateOutsideFutureOpenDayConfiguration() throws ParseExc
                 DateUtils.oldUiDateFormat().parse("2022-11-15")
         );
 
-        assertEquals(false, DateUtils.getInstance().isInsideFutureInputPeriod(inputPeriod.endPeriodDate(), 5));
+        assertFalse(DateUtils.getInstance().isInsideFutureInputPeriod(inputPeriod.endPeriodDate(), 5));
     }
 
     @Test
@@ -673,7 +671,7 @@ public void isFutureInputPeriodsNotConfigured() {
                 new Date()
         );
 
-        assertEquals(false, DateUtils.getInstance().isInsideFutureInputPeriod(inputPeriod.endPeriodDate(), 0));
+        assertFalse(DateUtils.getInstance().isInsideFutureInputPeriod(inputPeriod.endPeriodDate(), 0));
     }
 
     @Test