Prevent from creating cards when the limit has been exceeded

Author: jorgemanrubia

app/controllers/cards_controller.rb

@@ -4,6 +4,7 @@ class CardsController < ApplicationController
   before_action :set_board, only: %i[ create ]
   before_action :set_card, only: %i[ show edit update destroy ]
   before_action :ensure_permission_to_administer_card, only: %i[ destroy ]
+  before_action :ensure_can_create_cards, only: %i[ create ]
 
   def index
     set_page_and_extract_portion_from @filter.cards
@@ -42,6 +43,10 @@ def ensure_permission_to_administer_card
       head :forbidden unless Current.user.can_administer_card?(@card)
     end
 
+    def ensure_can_create_cards
+      head :forbidden if Current.account.card_limit_exceeded?
+    end
+
     def card_params
       params.expect(card: [ :status, :title, :description, :image, tag_ids: [] ])
     end

test/controllers/cards_controller_test.rb

@@ -35,6 +35,17 @@ class CardsControllerTest < ActionDispatch::IntegrationTest
     assert_redirected_to draft
   end
 
+  test "cannot create cards when card limit exceeded" do
+    logout_and_sign_in_as :mike
+    accounts(:initech).update_column(:cards_count, 1001)
+
+    assert_no_difference -> { Card.count } do
+      post board_cards_path(boards(:miltons_wish_list), script_name: accounts(:initech).slug)
+    end
+
+    assert_response :forbidden
+  end
+
   test "show" do
     get card_path(cards(:logo))
     assert_response :success

test/fixtures/accesses.yml

@@ -21,3 +21,9 @@ private_kevin:
   account: 37s_uuid
   board: private_uuid
   user: kevin_uuid
+
+miltons_wish_list_mike:
+  id: <%= ActiveRecord::FixtureSet.identify("miltons_wish_list_mike", :uuid) %>
+  account: initech_uuid
+  board: miltons_wish_list_uuid
+  user: mike_uuid