Store ID in a decklist_slot, update things accordingly.

Author: plural

app/models/decklist.rb

@@ -27,7 +27,10 @@ def card_slots
   end
 
   def num_cards
-    decklist_cards.map(&:quantity).sum
+    decklist_cards
+      # Exclude identity
+      .reject { |c| c.card_id == identity_card_id }
+      .map(&:quantity).sum
   end
 
   # TODO(plural): Extract this out to share between public and private decklists.
@@ -37,6 +40,8 @@ def num_cards
   def influence_spent
     qty = decklist_cards.each_with_object({}) { |c, h| h[c.card_id] = c.quantity }
     cards
+      # Exclude identity
+      .reject { |c| c.id == identity_card.id }
       .filter { |c| c.faction_id != identity_card.faction_id }
       .map { |c| c.influence_cost.nil? ? 0 : (c.influence_cost * qty[c.id]) }
       .sum

app/resources/decklist_resource.rb

@@ -50,13 +50,5 @@ class DecklistResource < ApplicationResource
     end
   end
 
-  # The rubocop warning is disabled because this relationship won't work
-  # without it because there is no identity_card table.
-  belongs_to :identity_card, resource: CardResource, foreign_key: :identity_card_id do # rubocop:disable Rails/RedundantForeignKey
-    link do |decklist|
-      format('%<url>s/%<id>s', url: Rails.application.routes.url_helpers.cards_url, id: decklist.identity_card_id)
-    end
-  end
-
   many_to_many :cards
 end

lib/tasks/import_decklists.rake

@@ -65,6 +65,7 @@ namespace :import_decklists do # rubocop:disable Metrics/BlockLength
 
       # To allow overwriting, clear out the existing cards.
       d.decklist_cards.delete_all
+      d.decklist_cards << d.decklist_cards.build(card_id: d.identity_card_id, quantity: 1)
       decklist['cards'].each do |printing_id, quantity|
         card = cards_by_id[printing_to_card[printing_id]]
         # Do not write identity cards to the decklist_cards table.

spec/acceptance/decklists_spec.rb

@@ -16,7 +16,6 @@
 
     * Side
     * Faction
-    * Identity Card
     * Cards
 
   EXPLANATION

spec/resources/decklist/reads_spec.rb

@@ -76,15 +76,6 @@ def check_included_for_id(decklist_id, include_value, resource_type, id)
       end
     end
 
-    describe 'include identity card' do
-      let!(:decklist) { Decklist.find('11111111-1111-1111-1111-111111111111') }
-      let!(:card) { Card.find('asa_group_security_through_vigilance') }
-
-      it 'works' do
-        check_included_for_id(decklist.id, 'identity_card', 'cards', card.id)
-      end
-    end
-
     describe 'include cards' do
       let!(:decklist) { Decklist.find('22222222-2222-2222-2222-222222222222') }
       let!(:card) { Card.find('the_class_act') }