🐛 (model) fix unique index

Author: an-lee

app/models/mixin_group.rb

@@ -17,7 +17,7 @@
 #
 # Indexes
 #
-#  index_mixin_groups_on_conversation_id  (conversation_id)
+#  index_mixin_groups_on_conversation_id  (conversation_id) UNIQUE
 #  index_mixin_groups_on_creator_id       (creator_id)
 #
 class MixinGroup < ApplicationRecord

app/models/multisig_account.rb

@@ -16,7 +16,7 @@
 #
 # Indexes
 #
-#  index_multisig_accounts_on_account_hash  (account_hash)
+#  index_multisig_accounts_on_account_hash  (account_hash) UNIQUE
 #  index_multisig_accounts_on_creator_id    (creator_id)
 #  index_multisig_accounts_on_member_uuids  (member_uuids)
 #

app/models/multisig_payment.rb

@@ -21,9 +21,11 @@
 #
 # Indexes
 #
+#  index_multisig_payments_on_code_id              (code_id) UNIQUE
 #  index_multisig_payments_on_creator_id           (creator_id)
 #  index_multisig_payments_on_multisig_account_id  (multisig_account_id)
 #  index_multisig_payments_on_receivers            (receivers)
+#  index_multisig_payments_on_trace_id             (trace_id) UNIQUE
 #
 class MultisigPayment < ApplicationRecord
   belongs_to :creator, class_name: 'User', foreign_key: :creator_id, inverse_of: :created_payments

app/models/multisig_transaction.rb

@@ -10,7 +10,7 @@
 #  raw_transaction     :string
 #  receiver_uuids      :uuid             is an Array
 #  sender_uuids        :uuid             is an Array
-#  signer_uuids        :uuid             default("{}"), is an Array
+#  signer_uuids        :uuid             default([]), is an Array
 #  status              :string
 #  threshold           :integer
 #  transaction_hash    :string
@@ -25,7 +25,7 @@
 #  index_multisig_transactions_on_multisig_account_id  (multisig_account_id)
 #  index_multisig_transactions_on_receiver_uuids       (receiver_uuids)
 #  index_multisig_transactions_on_sender_uuids         (sender_uuids)
-#  index_multisig_transactions_on_transaction_hash     (transaction_hash)
+#  index_multisig_transactions_on_transaction_hash     (transaction_hash) UNIQUE
 #  index_multisig_transactions_on_user_id              (user_id)
 #
 class MultisigTransaction < ApplicationRecord

db/migrate/20200324081819_add_index_to_mixin_group_account_payment_transaction.rb

@@ -0,0 +1,13 @@
+class AddIndexToMixinGroupAccountPaymentTransaction < ActiveRecord::Migration[6.0]
+  def change
+    remove_index :mixin_groups, :conversation_id
+    remove_index :multisig_accounts, :account_hash
+    remove_index :multisig_transactions, :transaction_hash
+
+    add_index :mixin_groups, :conversation_id, unique: true
+    add_index :multisig_accounts, :account_hash, unique: true
+    add_index :multisig_payments, :code_id, unique: true
+    add_index :multisig_payments, :trace_id, unique: true
+    add_index :multisig_transactions, :transaction_hash, unique: true
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2020_03_13_073024) do
+ActiveRecord::Schema.define(version: 2020_03_24_081819) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "pgcrypto"
@@ -48,7 +48,7 @@
     t.json "data"
     t.datetime "created_at", precision: 6, null: false
     t.datetime "updated_at", precision: 6, null: false
-    t.index ["conversation_id"], name: "index_mixin_groups_on_conversation_id"
+    t.index ["conversation_id"], name: "index_mixin_groups_on_conversation_id", unique: true
     t.index ["creator_id"], name: "index_mixin_groups_on_creator_id"
   end
 
@@ -87,7 +87,7 @@
     t.string "account_hash"
     t.datetime "created_at", precision: 6, null: false
     t.datetime "updated_at", precision: 6, null: false
-    t.index ["account_hash"], name: "index_multisig_accounts_on_account_hash"
+    t.index ["account_hash"], name: "index_multisig_accounts_on_account_hash", unique: true
     t.index ["creator_id"], name: "index_multisig_accounts_on_creator_id"
     t.index ["member_uuids"], name: "index_multisig_accounts_on_member_uuids"
   end
@@ -106,9 +106,11 @@
     t.json "data"
     t.datetime "created_at", precision: 6, null: false
     t.datetime "updated_at", precision: 6, null: false
+    t.index ["code_id"], name: "index_multisig_payments_on_code_id", unique: true
     t.index ["creator_id"], name: "index_multisig_payments_on_creator_id"
     t.index ["multisig_account_id"], name: "index_multisig_payments_on_multisig_account_id"
     t.index ["receivers"], name: "index_multisig_payments_on_receivers"
+    t.index ["trace_id"], name: "index_multisig_payments_on_trace_id", unique: true
   end
 
   create_table "multisig_transactions", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|
@@ -129,7 +131,7 @@
     t.index ["multisig_account_id"], name: "index_multisig_transactions_on_multisig_account_id"
     t.index ["receiver_uuids"], name: "index_multisig_transactions_on_receiver_uuids"
     t.index ["sender_uuids"], name: "index_multisig_transactions_on_sender_uuids"
-    t.index ["transaction_hash"], name: "index_multisig_transactions_on_transaction_hash"
+    t.index ["transaction_hash"], name: "index_multisig_transactions_on_transaction_hash", unique: true
     t.index ["user_id"], name: "index_multisig_transactions_on_user_id"
   end
 

test/fixtures/mixin_groups.yml

@@ -15,7 +15,7 @@
 #
 # Indexes
 #
-#  index_mixin_groups_on_conversation_id  (conversation_id)
+#  index_mixin_groups_on_conversation_id  (conversation_id) UNIQUE
 #  index_mixin_groups_on_creator_id       (creator_id)
 #
 

test/fixtures/multisig_accounts.yml

@@ -14,7 +14,7 @@
 #
 # Indexes
 #
-#  index_multisig_accounts_on_account_hash  (account_hash)
+#  index_multisig_accounts_on_account_hash  (account_hash) UNIQUE
 #  index_multisig_accounts_on_creator_id    (creator_id)
 #  index_multisig_accounts_on_member_uuids  (member_uuids)
 #

test/fixtures/multisig_payments.yml

@@ -19,9 +19,11 @@
 #
 # Indexes
 #
+#  index_multisig_payments_on_code_id              (code_id) UNIQUE
 #  index_multisig_payments_on_creator_id           (creator_id)
 #  index_multisig_payments_on_multisig_account_id  (multisig_account_id)
 #  index_multisig_payments_on_receivers            (receivers)
+#  index_multisig_payments_on_trace_id             (trace_id) UNIQUE
 #
 
 # This model initially had no columns defined. If you add columns to the

test/fixtures/multisig_transactions.yml

@@ -8,7 +8,7 @@
 #  raw_transaction     :string
 #  receiver_uuids      :uuid             is an Array
 #  sender_uuids        :uuid             is an Array
-#  signer_uuids        :uuid             default("{}"), is an Array
+#  signer_uuids        :uuid             default([]), is an Array
 #  status              :string
 #  threshold           :integer
 #  transaction_hash    :string
@@ -23,7 +23,7 @@
 #  index_multisig_transactions_on_multisig_account_id  (multisig_account_id)
 #  index_multisig_transactions_on_receiver_uuids       (receiver_uuids)
 #  index_multisig_transactions_on_sender_uuids         (sender_uuids)
-#  index_multisig_transactions_on_transaction_hash     (transaction_hash)
+#  index_multisig_transactions_on_transaction_hash     (transaction_hash) UNIQUE
 #  index_multisig_transactions_on_user_id              (user_id)
 #