Allow negative cost_per_unit for geofences

teslamate-org · Oct 3, 2020 · 7a0609b · 7a0609b
1 parent 35add49
commit 7a0609b
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 4 deletions.
diff --git a/lib/teslamate/locations/geo_fence.ex b/lib/teslamate/locations/geo_fence.ex
@@ -34,7 +34,6 @@ defmodule TeslaMate.Locations.GeoFence do
     ])
     |> validate_required([:name, :latitude, :longitude, :radius])
     |> validate_number(:radius, greater_than: 0, less_than: 5000)
-    |> validate_number(:cost_per_unit, greater_than_or_equal_to: 0)
     |> validate_number(:session_fee, greater_than_or_equal_to: 0)
   end
 end
diff --git a/lib/teslamate_web/live/geofence_live/form.html.leex b/lib/teslamate_web/live/geofence_live/form.html.leex
@@ -68,7 +68,7 @@
         </p>
         <p class="control is-expanded">
           <%= text_input f, :cost_per_unit, class: "input",
-                             type: :number, inputmode: :decimal, min: 0.0, step: 0.0001,
+                             type: :number, inputmode: :decimal, step: 0.0001,
                              placeholder: "0.00" %>
         </p>
         <%= if @show_errors do %>

diff --git a/test/teslamate/locations/geofences_test.exs b/test/teslamate/locations/geofences_test.exs
@@ -29,7 +29,6 @@ defmodule TeslaMate.LocationsGeofencesTest do
     longitude: nil,
     radius: nil,
     billing_type: :per_hour,
-    cost_per_unit: -0.01,
     session_fee: -0.01
   }
 
@@ -67,7 +66,6 @@ defmodule TeslaMate.LocationsGeofencesTest do
                name: ["can't be blank"],
                radius: ["can't be blank"],
                billing_type: ["is invalid"],
-               cost_per_unit: ["must be greater than or equal to 0"],
                session_fee: ["must be greater than or equal to 0"]
              }