change topic uid column to string type (#111)

Author: dcollie2

app/models/topic.rb

@@ -6,7 +6,7 @@
 #  description :text
 #  state       :integer          default("active"), not null
 #  title       :string           not null
-#  uid         :uuid             not null
+#  uid         :string
 #  created_at  :datetime         not null
 #  updated_at  :datetime         not null
 #  language_id :bigint
@@ -22,14 +22,15 @@
 class Topic < ApplicationRecord
   include Searcheable
 
+  STATES = %i[active archived].freeze
+  CONTENT_TYPES = %w[image/jpeg image/png image/svg+xml image/webp image/avif image/gif video/mp4].freeze
+
   belongs_to :language
   belongs_to :provider
   has_many_attached :documents
 
   validates :title, :language_id, :provider_id, presence: true
-  validates :documents, content_type: %w[image/jpeg image/png image/svg+xml image/webp image/avif image/gif video/mp4], size: { less_than: 10.megabytes }
-
-  STATES = %i[active archived].freeze
+  validates :documents, content_type: CONTENT_TYPES, size: { less_than: 10.megabytes }
 
   enum :state, STATES.map.with_index.to_h
 

app/views/topics/_list.html.erb

@@ -3,7 +3,7 @@
     <tr>
       <td class="text-bold-500"><%= topic.title %></td>
       <td class="text-bold-500"><%= topic.description.present? && topic.description.truncate(25, omission: "...") %></td>
-      <td class="text-bold-500"><%= topic.uid.truncate(10, omission: "...") %></td>
+      <td class="text-bold-500"><%= topic.uid.truncate(10, omission: "...") if topic.uid %></td>
       <td class="text-bold-500"><%= topic.language.name %></td>
       <td class="text-bold-500"><%= topic.provider.name %></td>
       <td class="text-bold-500"><%= topic.state %></td>

db/migrate/20250308200109_change_topics_uid.rb

@@ -0,0 +1,11 @@
+class ChangeTopicsUid < ActiveRecord::Migration[8.0]
+  def up
+    remove_column :topics, :uid
+    add_column :topics, :uid, :string
+  end
+
+  def down
+    remove_column :topics, :uid
+    add_column :topics, :uid, :uuid, default: -> { "gen_random_uuid()" }, null: false
+  end
+end

db/schema.rb

@@ -6,7 +6,7 @@
 #  description :text
 #  state       :integer          default("active"), not null
 #  title       :string           not null
-#  uid         :uuid             not null
+#  uid         :string
 #  created_at  :datetime         not null
 #  updated_at  :datetime         not null
 #  language_id :bigint
@@ -23,9 +23,8 @@
   factory :topic do
     association :provider
     association :language
-    title { "topic" }
-    description { "details" }
-    uid { SecureRandom.uuid }
+    title { "topic title" }
+    description { "many topic details" }
     state { 0 }
 
     trait :archived do

spec/factories/topics.rb

@@ -6,7 +6,7 @@
 #  description :text
 #  state       :integer          default("active"), not null
 #  title       :string           not null
-#  uid         :uuid             not null
+#  uid         :string
 #  created_at  :datetime         not null
 #  updated_at  :datetime         not null
 #  language_id :bigint

spec/models/topic_spec.rb

@@ -17,8 +17,8 @@
 
       expect(response).to redirect_to(topics_url)
       topic = Topic.last
-      expect(topic.title).to eq("topic")
-      expect(topic.description).to eq("details")
+      expect(topic.title).to eq("topic title")
+      expect(topic.description).to eq("many topic details")
       expect(topic.state).to eq("active")
     end