Add some ordering tests

Author: joshsmith

test/lib/code_corps/services/markdown_renderer_test.exs

@@ -7,6 +7,7 @@ defmodule CodeCorps.Services.MarkdownRendererServiceTest do
 
   @valid_attrs %{
     title: "Test task",
+    task_list_id: 1,
     task_type: "issue",
     markdown: "A **strong** body",
     status: "open"

test/models/task_test.exs

@@ -6,7 +6,7 @@ defmodule CodeCorps.TaskTest do
   @valid_attrs %{
     title: "Test task",
     task_type: "issue",
-    markdown: "A test task",
+    markdown: "A test task"
   }
   @invalid_attrs %{
     task_type: "nonexistent"
@@ -27,9 +27,11 @@ defmodule CodeCorps.TaskTest do
     test "renders body html from markdown" do
       user = insert(:user)
       project = insert(:project)
+      task_list = insert(:task_list)
       changes = Map.merge(@valid_attrs, %{
         markdown: "A **strong** body",
         project_id: project.id,
+        task_list_id: task_list.id,
         user_id: user.id
       })
       changeset = Task.changeset(%Task{}, changes)
@@ -120,6 +122,36 @@ defmodule CodeCorps.TaskTest do
       assert result_a.order == result_b.order
     end
 
+    test "repositions tasks correctly" do
+      user = insert(:user)
+      project = insert(:project)
+      task_list = insert(:task_list, name: "Task List", project: project)
+
+      task_1 = insert(:task, project: project, user: user, task_list: task_list, order: 1000)
+      task_2 = insert(:task, project: project, user: user, task_list: task_list, order: 2000)
+
+      task_list_2 = insert(:task_list, name: "Task List 2", project: project)
+      task_3 = insert(:task, project: project, user: user, task_list: task_list_2)
+
+      {:ok, task_1_result} =
+        task_1
+        |> Task.update_changeset(%{position: 0})
+        |> Repo.update
+
+      {:ok, task_2_result} =
+        task_2
+        |> Task.update_changeset(%{position: 1})
+        |> Repo.update
+
+      {:ok, task_3_result} =
+        task_3
+        |> Task.update_changeset(%{position: 2, task_list_id: task_list.id})
+        |> Repo.update
+
+      assert task_1_result.order < task_2_result.order
+      assert task_2_result.order < task_3_result.order
+    end
+
     test "sets state to 'published'" do
       changeset = Task.create_changeset(%Task{}, %{})
       assert changeset |> get_change(:state) == "published"

web/models/task.ex

@@ -7,16 +7,18 @@ defmodule CodeCorps.Task do
     field :body, :string
     field :markdown, :string
     field :number, :integer, read_after_writes: true
-    field :task_type, :string
+    field :order, :integer
     field :state, :string
     field :status, :string, default: "open"
+    field :task_type, :string
     field :title, :string
+
     field :position, :integer, virtual: true
-    field :order, :integer
 
     belongs_to :project, CodeCorps.Project
-    belongs_to :user, CodeCorps.User
     belongs_to :task_list, CodeCorps.TaskList
+    belongs_to :user, CodeCorps.User
+
     has_many :comments, CodeCorps.Comment
 
     timestamps()
@@ -25,7 +27,7 @@ defmodule CodeCorps.Task do
   def changeset(struct, params \\ %{}) do
     struct
     |> cast(params, [:title, :markdown, :task_type, :task_list_id, :position])
-    |> validate_required([:title, :markdown, :task_type])
+    |> validate_required([:title, :markdown, :task_list_id, :task_type])
     |> validate_inclusion(:task_type, task_types)
     |> assoc_constraint(:task_list)
     |> apply_position()
@@ -37,7 +39,7 @@ defmodule CodeCorps.Task do
     struct
     |> changeset(params)
     |> cast(params, [:project_id, :user_id])
-    |> validate_required([:project_id, :user_id, :task_list_id])
+    |> validate_required([:project_id, :user_id])
     |> assoc_constraint(:project)
     |> assoc_constraint(:user)
     |> put_change(:state, "published")