feat: add create / update threshold for movie watch statuses

lib/media_server/watches.ex

@@ -78,10 +78,21 @@ defmodule MediaServer.Watches do
 
     case movie do
       nil ->
-        create_movie(attrs)
+
+        if ((attrs.current_time / attrs.duration) * 100) < 90 do
+          create_movie(attrs)
+        else
+          nil
+        end
 
       _ ->
-        update_movie(movie, attrs)
+        if ((attrs.current_time / attrs.duration) * 100) < 90 do
+          update_movie(movie, attrs)
+        else
+          delete_movie(movie)
+
+          nil
+        end
     end
   end
 

test/media_server/watches_test.exs

@@ -84,16 +84,31 @@ defmodule MediaServer.WatchesTest do
       movie_fixture(%{user_id: user.id})
       update_attrs = %{
         movie_id: 42,
-        current_time: 62,
+        current_time: 89,
+        duration: 100,
         user_id: user.id
       }
 
       {:ok, %Movie{} = movie} = Watches.update_or_create_movie(update_attrs)
       assert movie.movie_id == 42
-      assert movie.current_time == 62
+      assert movie.current_time == 89
+      assert movie.duration == 100
       assert movie.user_id == user.id
     end
 
+    test "update_or_create_movie/2 deletes movie" do
+      user = AccountsFixtures.user_fixture()
+      movie_fixture(%{user_id: user.id})
+      update_attrs = %{
+        movie_id: 42,
+        current_time: 90,
+        duration: 100,
+        user_id: user.id
+      }
+
+      refute Watches.update_or_create_movie(update_attrs)
+    end
+
     test "update_or_create_movie/2 creates movie" do
       user = AccountsFixtures.user_fixture()
       movie_fixture(%{user_id: user.id})

test/media_server_web/live/watch_movie_live_test.exs

@@ -27,7 +27,7 @@ defmodule MediaServerWeb.WatchMovieLiveTest do
       assert html_response(conn, 200)
     end
 
-    test "it has watch status", %{conn: conn, user: user} do
+    test "has watch status", %{conn: conn, user: user} do
 
       conn =
         post(conn, Routes.user_session_path(conn, :create), %{
@@ -40,12 +40,33 @@ defmodule MediaServerWeb.WatchMovieLiveTest do
 
       render_hook(view, :movie_destroyed, %{
         movie_id: movie["id"],
-        current_time: 39,
-        duration: 78,
+        current_time: 89,
+        duration: 100,
         user_id: user.id
       })
 
       assert WatchesFixtures.get_movie_watch()
     end
+
+    test "no watch status", %{conn: conn, user: user} do
+
+      conn =
+        post(conn, Routes.user_session_path(conn, :create), %{
+          "user" => %{"email" => user.email, "password" => AccountsFixtures.valid_user_password()}
+        })
+
+      movie = MoviesFixtures.get_movie()
+
+      {:ok, view, _html} = live(conn, Routes.watch_movie_show_path(conn, :show, movie["id"]))
+
+      render_hook(view, :movie_destroyed, %{
+        movie_id: movie["id"],
+        current_time: 90,
+        duration: 100,
+        user_id: user.id
+      })
+
+      refute WatchesFixtures.get_movie_watch()
+    end
   end
 end