Merge Groups into Course

Author: angelsl

lib/cadet/course/course.ex

@@ -7,10 +7,11 @@ defmodule Cadet.Course do
 
   import Ecto.Query
 
-  alias Cadet.Accounts.User
+  alias Cadet.{Accounts, Accounts.User}
   alias Cadet.Course.{Category, Group, Material, MaterialUpload, Sourcecast, SourcecastUpload}
 
   @upload_file_roles ~w(admin staff)a
+  @get_overviews_role ~w(staff admin)a
 
   @doc """
   Get a group based on the group name or create one if it doesn't exist
@@ -49,6 +50,33 @@ defmodule Cadet.Course do
     |> Repo.insert_or_update()
   end
 
+  @doc """
+  Returns a list of groups containing information on the each group's id, avenger name and group name
+  """
+  @type group_overview :: %{id: integer, avenger_name: String.t(), name: String.t()}
+
+  @spec get_group_overviews(%User{}) :: [group_overview]
+  def get_group_overviews(_user = %User{role: role}) do
+    if role in @get_overviews_role do
+      overviews =
+        Group
+        |> Repo.all()
+        |> Enum.map(fn group_info -> get_group_info(group_info) end)
+
+      {:ok, overviews}
+    else
+      {:error, {:unauthorized, "Unauthorized"}}
+    end
+  end
+
+  defp get_group_info(group_info) do
+    %{
+      id: group_info.id,
+      avenger_name: Accounts.get_user(group_info.leader_id).name,
+      name: group_info.name
+    }
+  end
+
   # @doc """
   # Reassign a student to a discussion group
   # This will un-assign student from the current discussion group

lib/cadet/course/groups.ex

@@ -3,11 +3,11 @@ defmodule CadetWeb.GroupController do
 
   use PhoenixSwagger
 
-  alias Cadet.Course.Groups
+  alias Cadet.Course
 
   def index(conn, _) do
     user = conn.assigns.current_user
-    result = Groups.get_group_overviews(user)
+    result = Course.get_group_overviews(user)
 
     case result do
       {:ok, groups} ->

lib/cadet_web/controllers/group_controller.ex

@@ -1,13 +1,12 @@
 defmodule Cadet.Course.GroupsTest do
   use Cadet.DataCase
 
-  alias Cadet.Accounts
-  alias Cadet.Accounts.User
-  alias Cadet.Course.Groups
+  alias Cadet.{Accounts, Accounts.User}
+  alias Cadet.Course
 
   test "get group overviews" do
     group = insert(:group)
-    {:ok, result} = Groups.get_group_overviews(%User{role: :staff})
+    {:ok, result} = Course.get_group_overviews(%User{role: :staff})
     avenger_name = Accounts.get_user(group.leader_id).name
     group_name = group.name
     group_id = group.id