Move a draft card between boards

.github/workflows/ci-checks.yml

@@ -9,7 +9,7 @@ permissions:
 jobs:
   gemfile-drift:
     name: Gemfile drift
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
 
     steps:
       - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
@@ -25,7 +25,7 @@ jobs:
 
   security:
     name: Security
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
 
     steps:
       - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
@@ -47,7 +47,7 @@ jobs:
 
   lint:
     name: Lint
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
 
     steps:
       - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
@@ -66,7 +66,7 @@ jobs:
 
   zizmor:
     name: GitHub Actions audit
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
 
     steps:
       - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1

.github/workflows/dependabot-sync-saas-lockfile.yml

@@ -13,7 +13,7 @@ permissions:
 
 jobs:
   sync:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
 
     steps:
       - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # zizmor: ignore[artipacked] -- credentials needed for git push # v7.0.1

.github/workflows/test.yml

@@ -16,7 +16,7 @@ permissions:
 jobs:
   test:
     name: Tests (${{ matrix.mode }})
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
 
     strategy:
       matrix:

db/migrate/20261102120000_remove_board_creator_drafted_index_from_cards.rb

@@ -0,0 +1,5 @@
+class RemoveBoardCreatorDraftedIndexFromCards < ActiveRecord::Migration[8.2]
+  def up
+    remove_index :cards, name: "index_cards_on_board_creator_drafted", if_exists: true
+  end
+end

db/schema_sqlite.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[8.2].define(version: 2026_10_19_150000) do
+ActiveRecord::Schema[8.2].define(version: 2026_11_02_120000) do
   create_table "accesses", id: :uuid, force: :cascade do |t|
     t.datetime "accessed_at"
     t.uuid "account_id", null: false

test/controllers/cards/boards_controller_test.rb

@@ -35,4 +35,24 @@ class Cards::BoardsControllerTest < ActionDispatch::IntegrationTest
     assert_equal card.title, json["title"]
     assert_equal new_board.id, json["board"]["id"]
   end
+
+  test "update moves a draft to a board where its creator has another draft" do
+    draft = users(:kevin).draft_new_card_in(boards(:writebook))
+    users(:kevin).draft_new_card_in(boards(:private))
+
+    put card_board_path(draft), params: { board_id: boards(:private).id }
+
+    assert_redirected_to draft
+    assert_equal boards(:private), draft.reload.board
+    assert_equal 2, boards(:private).cards.drafted.where(creator: users(:kevin)).count
+  end
+
+  test "update moves a draft to a board without a draft of its creator" do
+    draft = users(:kevin).draft_new_card_in(boards(:writebook))
+
+    put card_board_path(draft), params: { board_id: boards(:private).id }
+
+    assert_redirected_to draft
+    assert_equal boards(:private), draft.reload.board
+  end
 end

test/db/migrate/remove_board_creator_drafted_index_from_cards_test.rb

@@ -0,0 +1,67 @@
+require "test_helper"
+
+class RemoveBoardCreatorDraftedIndexFromCardsTest < ActiveSupport::TestCase
+  INDEX_NAME = "index_cards_on_board_creator_drafted"
+
+  test "does nothing when the drafted card index is absent" do
+    assert_not drafted_card_index?
+
+    assert_no_queries_match(/DROP INDEX/i, include_schema: true) { migrate_up }
+
+    assert_not drafted_card_index?
+  end
+
+  if ActiveRecord::Base.connection_db_config.adapter == "sqlite3"
+    test "removes the drafted card index when present" do
+      add_drafted_card_index
+
+      migrate_up
+
+      assert_not drafted_card_index?
+    end
+
+    test "the drafted card index rejects moving a draft onto a board with another draft of its creator" do
+      add_drafted_card_index
+      draft = kevin_drafts_on_writebook_and_private
+
+      error = assert_raises(ActiveRecord::RecordNotUnique) { draft.move_to(boards(:private)) }
+
+      assert_match(/cards\.board_id, cards\.creator_id/, error.message)
+    end
+
+    test "a draft moves to a board where its creator already has a draft once the index is gone" do
+      add_drafted_card_index
+      draft = kevin_drafts_on_writebook_and_private
+      migrate_up
+
+      draft.move_to(boards(:private))
+
+      assert_equal 2, boards(:private).cards.drafted.where(creator: users(:kevin)).count
+    end
+  end
+
+  private
+    def add_drafted_card_index
+      connection.add_index :cards, [ :board_id, :creator_id ], unique: true, where: "status = 'drafted'", name: INDEX_NAME
+    end
+
+    def migrate_up
+      migration = connection.pool.migration_context.migrations.find { it.name == "RemoveBoardCreatorDraftedIndexFromCards" }
+      flunk "Migration RemoveBoardCreatorDraftedIndexFromCards is missing" unless migration
+
+      ActiveRecord::Migration.suppress_messages { migration.migrate(:up) }
+    end
+
+    def drafted_card_index?
+      connection.index_name_exists?(:cards, INDEX_NAME)
+    end
+
+    def kevin_drafts_on_writebook_and_private
+      users(:kevin).draft_new_card_in(boards(:private))
+      users(:kevin).draft_new_card_in(boards(:writebook))
+    end
+
+    def connection
+      ActiveRecord::Base.connection
+    end
+end