Added Task title change tracking

Author: begedin

lib/code_corps_web/controllers/task_controller.ex

@@ -45,6 +45,7 @@ defmodule CodeCorpsWeb.TaskController do
       current_user |> track_updated(updated_task)
       current_user |> maybe_track_connected(updated_task, task)
       current_user |> maybe_track_list_move(updated_task, task)
+      current_user |> maybe_track_title_change(updated_task, task)
 
       conn |> render("show.json-api", data: updated_task)
     end
@@ -91,4 +92,14 @@ defmodule CodeCorpsWeb.TaskController do
     user_id |> SegmentTracker.track("Moved Task Between Lists", task)
   end
   defp maybe_track_list_move(%User{}, %Task{}, %Task{}), do: :nothing
+
+  @spec maybe_track_title_change(User.t, Task.t, Task.t) :: any
+  defp maybe_track_title_change(
+    %User{id: user_id},
+    %Task{title: new_title} = task,
+    %Task{title: old_title}) when new_title != old_title do
+
+    user_id |> SegmentTracker.track("Edited Task Title", task)
+  end
+  defp maybe_track_title_change(%User{}, %Task{}, %Task{}), do: :nothing
 end

test/lib/code_corps_web/controllers/task_controller_test.exs

@@ -222,6 +222,27 @@ defmodule CodeCorpsWeb.TaskControllerTest do
       refute_received {:track, ^user_id, "Moved Task Between Lists", _}
     end
 
+    @tag :authenticated
+    test "tracks title change", %{conn: conn, current_user: current_user} do
+      task = insert(:task, user: current_user)
+      attrs = @valid_attrs |> Map.put(:title, "New title")
+      conn |> request_update(task, attrs)
+
+      traits = Task |> Repo.get(task.id) |> SegmentTraitsBuilder.build
+      user_id = current_user.id
+      assert_received {:track, ^user_id, "Edited Task Title", ^traits}
+    end
+
+    @tag :authenticated
+    test "does not track title change if none took place", %{conn: conn, current_user: current_user} do
+      task = insert(:task, user: current_user)
+      attrs = @valid_attrs |> Map.delete(:title)
+      conn |> request_update(task, attrs)
+
+      user_id = current_user.id
+      refute_received {:track, ^user_id, "Edited Task Title", _}
+    end
+
     @tag :authenticated
     test "renders 422 when data is invalid", %{conn: conn, current_user: current_user} do
       task = insert(:task, user: current_user)