Merge pull request #28 from epochtalk/thread-slugs

feat: add slug column to threads, populate existing thread slugs with…

Author: akinsey

lib/epoch/thread.ex

@@ -6,6 +6,7 @@ defmodule Epoch.Thread do
     belongs_to :board, Epoch.Board, type: :binary_id
     field :locked, :boolean
     field :sticky, :boolean
+    field :slug, :string
     field :moderated, :boolean
     field :post_count, :integer
     field :created_at, :naive_datetime

priv/repo/migrations/20200730222455_thread_slugs.exs

@@ -0,0 +1,34 @@
+defmodule Epoch.Repo.Migrations.ThreadSlugs do
+  use Ecto.Migration
+  import Ecto.Query
+
+  def up do
+    # add slug column
+    alter table(:threads) do
+      add :slug, :string, size: 100
+    end
+
+    #index
+    create unique_index(:threads, [:slug])
+
+    # flush so query populating slug will work
+    flush()
+
+    # update existing threads, set slug = id
+    from(t in "threads",
+      update: [set: [slug: t.id]],
+      where: true)
+    |> Epoch.Repo.update_all([])
+
+    # modify threads after slug update, don't allow null
+    alter table(:threads) do
+      modify :slug, :string, size: 100, null: false
+    end
+  end
+
+  def down do
+    alter table(:threads) do
+      remove :slug
+    end
+  end
+end

test/epoch_test.exs

@@ -18,6 +18,7 @@ defmodule EpochTest do
   def create_thread(board \\ create_board()) do
     %Epoch.Thread{
       board: board,
+      slug: :crypto.strong_rand_bytes(5) |> Base.url_encode64 |> binary_part(0, 5),
       created_at: NaiveDateTime.truncate(NaiveDateTime.utc_now(), :second),
       updated_at: NaiveDateTime.truncate(NaiveDateTime.utc_now(), :second)
     } |> Epoch.Repo.insert!

test/epoch_test/board_test.exs

@@ -1,11 +1,16 @@
-defmodule EpochTest.BoardTest do 
+defmodule EpochTest.BoardTest do
   use Epoch.DataCase
   import Ecto.Query
   alias Epoch.Board
   alias Epoch.Repo
 
   doctest Epoch
 
+  setup do
+    Repo.delete_all(Board)
+    :ok
+  end
+
   test "board" do
     board_created = EpochTest.create_board()
     board_retrieved = from(

test/epoch_test/post_test.exs

@@ -11,6 +11,7 @@ defmodule EpochTest.PostTest do
     } |> Epoch.Repo.insert!
     t = %Epoch.Thread{
       board: b,
+      slug: :crypto.strong_rand_bytes(5) |> Base.url_encode64 |> binary_part(0, 5),
       created_at: NaiveDateTime.truncate(NaiveDateTime.utc_now(), :second),
       updated_at: NaiveDateTime.truncate(NaiveDateTime.utc_now(), :second)
     } |> Epoch.Repo.insert!