Singularize Activity Log retention period unit

trento-project · Jun 24, 2024 · 3ad5d8f · 3ad5d8f
1 parent e3db6a0
commit 3ad5d8f
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 11 deletions.
diff --git a/lib/trento/activity_logging/retention_period_unit.ex b/lib/trento/activity_logging/retention_period_unit.ex
@@ -3,5 +3,5 @@ defmodule Trento.ActivityLog.RetentionPeriodUnit do
   Type that represents the possible retention period units.
   """
 
-  use Trento.Support.Enum, values: [:days, :weeks, :months, :years]
+  use Trento.Support.Enum, values: [:day, :week, :month, :year]
 end
diff --git a/lib/trento/activity_logging/retention_time.ex b/lib/trento/activity_logging/retention_time.ex
@@ -32,7 +32,7 @@ defmodule Trento.ActivityLog.RetentionTime do
   def default do
     %RetentionTime{
       value: 1,
-      unit: RetentionPeriodUnit.months()
+      unit: RetentionPeriodUnit.month()
     }
   end
 end
diff --git a/test/trento/activity_log_test.exs b/test/trento/activity_log_test.exs
@@ -41,7 +41,7 @@ defmodule Trento.ActivityLogTest do
   describe "changing activity log settings" do
     test "should not be able to change retention time if no activity log settings were previously saved" do
       assert {:error, :activity_log_settings_not_configured} ==
-               ActivityLog.change_retention_period(42, RetentionPeriodUnit.days())
+               ActivityLog.change_retention_period(42, RetentionPeriodUnit.day())
     end
 
     @validation_scenarios [
@@ -104,22 +104,22 @@ defmodule Trento.ActivityLogTest do
       %{
         name: "days",
         value: 1,
-        unit: RetentionPeriodUnit.days()
+        unit: RetentionPeriodUnit.day()
       },
       %{
         name: "weeks",
         value: 3,
-        unit: RetentionPeriodUnit.weeks()
+        unit: RetentionPeriodUnit.week()
       },
       %{
         name: "months",
         value: 5,
-        unit: RetentionPeriodUnit.months()
+        unit: RetentionPeriodUnit.month()
       },
       %{
         name: "years",
         value: 7,
-        unit: RetentionPeriodUnit.years()
+        unit: RetentionPeriodUnit.year()
       }
     ]
 
@@ -130,7 +130,7 @@ defmodule Trento.ActivityLogTest do
         insert(:activity_log_settings,
           retention_time: %{
             value: 92,
-            unit: RetentionPeriodUnit.years()
+            unit: RetentionPeriodUnit.year()
           }
         )
 
@@ -155,7 +155,7 @@ defmodule Trento.ActivityLogTest do
 
     test "should successfully handle unchanging retention periods" do
       initial_retention_period = 42
-      initial_retention_period_unit = RetentionPeriodUnit.days()
+      initial_retention_period_unit = RetentionPeriodUnit.day()
 
       insert(:activity_log_settings,
         retention_time: %{

diff --git a/test/trento/release_test.exs b/test/trento/release_test.exs
@@ -17,14 +17,14 @@ defmodule Trento.ReleaseTest do
       assert %ActivityLogSettings{
                retention_time: %RetentionTime{
                  value: 1,
-                 unit: RetentionPeriodUnit.months()
+                 unit: RetentionPeriodUnit.month()
                }
              } = Trento.Repo.one(ActivityLogSettings.base_query())
     end
 
     test "should not change previously saved retention time" do
       value = 3
-      unit = RetentionPeriodUnit.weeks()
+      unit = RetentionPeriodUnit.week()
 
       insert(:activity_log_settings,
         retention_time: %{