Added ProjectGithubRepo tracking

Author: begedin

lib/code_corps/analytics/segment_traits_builder.ex

@@ -37,6 +37,18 @@ defmodule CodeCorps.Analytics.SegmentTraitsBuilder do
       user_id: installation.user_id
     }
   end
+  defp traits(%CodeCorps.ProjectGithubRepo{} = record) do
+    record = record |> Repo.preload([:project, :github_repo])
+    %{
+      project: record.project.title,
+      project_id: record.project_id,
+      github_repo_id: record.github_repo_id,
+      github_repo_github_account_login: record.github_repo.github_account_login,
+      github_repo_github_account_type: record.github_repo.github_account_type,
+      github_repo_github_id: record.github_repo.github_id,
+      github_repo_name: record.github_repo.name
+    }
+  end
   defp traits(%CodeCorps.ProjectSkill{} = record) do
     record = record |> Repo.preload([:project, :skill])
     %{

lib/code_corps_web/controllers/project_github_repo_controller.ex

@@ -2,7 +2,7 @@ defmodule CodeCorpsWeb.ProjectGithubRepoController do
   @moduledoc false
   use CodeCorpsWeb, :controller
 
-  alias CodeCorps.{ProjectGithubRepo, User, Helpers.Query}
+  alias CodeCorps.{Analytics.SegmentTracker, ProjectGithubRepo, User, Helpers.Query}
 
   action_fallback CodeCorpsWeb.FallbackController
   plug CodeCorpsWeb.Plug.DataToAttributes
@@ -27,6 +27,9 @@ defmodule CodeCorpsWeb.ProjectGithubRepoController do
     with %User{} = current_user <- conn |> Guardian.Plug.current_resource,
          {:ok, :authorized} <- current_user |> Policy.authorize(:create, %ProjectGithubRepo{}, params),
          {:ok, %ProjectGithubRepo{} = project_github_repo} <- create_project_repo_changeset(params) |> Repo.insert do
+
+      current_user |> track_created(project_github_repo)
+
       conn |> put_status(:created) |> render("show.json-api", data: project_github_repo)
     end
   end
@@ -36,7 +39,10 @@ defmodule CodeCorpsWeb.ProjectGithubRepoController do
     with %ProjectGithubRepo{} = project_github_repo <- ProjectGithubRepo |> Repo.get(id),
          %User{} = current_user <- conn |> Guardian.Plug.current_resource,
          {:ok, :authorized} <- current_user |> Policy.authorize(:delete, project_github_repo, params),
-         {:ok, _project_github_repo} <- project_github_repo |> Repo.delete do
+         {:ok, project_github_repo} <- project_github_repo |> Repo.delete do
+
+      current_user |> track_deleted(project_github_repo)
+
       conn |> send_resp(:no_content, "")
     end
   end
@@ -46,4 +52,14 @@ defmodule CodeCorpsWeb.ProjectGithubRepoController do
     %ProjectGithubRepo{}
     |> ProjectGithubRepo.create_changeset(params)
   end
+
+  @spec track_created(User.t, ProjectGithubRepo.t) :: any
+  defp track_created(%User{id: user_id}, %ProjectGithubRepo{} = project_github_repo) do
+    user_id |> SegmentTracker.track("Added ProjectGithubRepo", project_github_repo)
+  end
+
+  @spec track_deleted(User.t, ProjectGithubRepo.t) :: any
+  defp track_deleted(%User{id: user_id}, %ProjectGithubRepo{} = project_github_repo) do
+    user_id |> SegmentTracker.track("Removed ProjectGithubRepo", project_github_repo)
+  end
 end

test/lib/code_corps/analytics/segment_traits_builder_test.exs

@@ -20,6 +20,7 @@ defmodule CodeCorps.Analytics.SegmentTraitsBuilderTest do
 
       assert :project_skill |> insert |> SegmentTraitsBuilder.build
       assert :project_user |> insert |> SegmentTraitsBuilder.build
+      assert :project_github_repo |> insert |> SegmentTraitsBuilder.build
 
       assert :stripe_connect_account |> insert |> SegmentTraitsBuilder.build
       assert :stripe_connect_charge |> insert |> SegmentTraitsBuilder.build

test/lib/code_corps_web/controllers/project_github_repo_controller_test.exs

@@ -1,6 +1,8 @@
 defmodule CodeCorpsWeb.ProjectGithubRepoControllerTest do
   use CodeCorpsWeb.ApiCase, resource_name: :project_github_repo
 
+  alias CodeCorps.{Analytics.SegmentTraitsBuilder, ProjectGithubRepo, Repo}
+
   describe "index" do
     test "lists all entries on index", %{conn: conn} do
       [project_github_repo_1, project_github_repo_2] = insert_pair(:project_github_repo)
@@ -51,6 +53,21 @@ defmodule CodeCorpsWeb.ProjectGithubRepoControllerTest do
       assert conn |> request_create(attrs) |> json_response(201)
     end
 
+    @tag :authenticated
+    test "is being tracked", %{conn: conn, current_user: current_user} do
+      project = insert(:project)
+      insert(:project_user, project: project, user: current_user, role: "owner")
+      github_repo = insert(:github_repo)
+
+      attrs = %{project: project, github_repo: github_repo}
+      conn |> request_create(attrs)
+
+      user_id = current_user.id
+      traits = ProjectGithubRepo |> Repo.one |> SegmentTraitsBuilder.build
+
+      assert_receive({:track, ^user_id, "Added ProjectGithubRepo", ^traits})
+    end
+
     @tag :authenticated
     test "renders 422 error when data is invalid", %{conn: conn, current_user: current_user} do
       project = insert(:project)
@@ -79,6 +96,20 @@ defmodule CodeCorpsWeb.ProjectGithubRepoControllerTest do
       assert conn |> request_delete(project_github_repo) |> response(204)
     end
 
+    @tag :authenticated
+    test "is being tracked", %{conn: conn, current_user: current_user} do
+      project = insert(:project)
+      insert(:project_user, project: project, user: current_user, role: "owner")
+      project_github_repo = insert(:project_github_repo, project: project)
+
+      conn |> request_delete(project_github_repo)
+
+      user_id = current_user.id
+      traits = project_github_repo |> SegmentTraitsBuilder.build
+
+      assert_receive({:track, ^user_id, "Removed ProjectGithubRepo", ^traits})
+    end
+
     test "renders 401 when unauthenticated", %{conn: conn} do
       assert conn |> request_delete |> json_response(401)
     end