feat: add pagination to tags #195

Author: hernanvicente

app/controllers/tags_controller.rb

@@ -1,9 +1,11 @@
 
 class TagsController < ApplicationController
+  include Pagy::Backend
+
   before_action :set_tag, only: [ :show, :edit, :update, :destroy ]
 
   def index
-    @tags = Tag.includes(:cognates, :reverse_cognates).references(:tag)
+    @pagy, @tags = pagy(Tag.includes(:cognates, :reverse_cognates).references(:tag))
   end
 
   def new

app/views/tags/index.html.erb

@@ -28,6 +28,9 @@
             </div>
           </div>
         </div>
+        <div class="card-footer d-flex justify-content-end">
+          <%== pagy_bootstrap_nav(@pagy) %>
+        </div>
       </div>
     </div>
   </div>

spec/factories/tags.rb

@@ -14,7 +14,9 @@
 #
 FactoryBot.define do
   factory :tag do
-    name { Faker::ProgrammingLanguage.name }
+    sequence(:name) do |n|
+      Faker::ProgrammingLanguage.name + n.to_s
+    end
 
     trait :english do
       after(:build) do |tag|

spec/views/tags/index.html.erb_spec.rb

@@ -0,0 +1,52 @@
+require "rails_helper"
+
+RSpec.describe "tags/index", type: :view do
+  context "when there are no tags" do
+    before(:each) do
+      assign(:pagy, Pagy.new(count: 0))
+      assign(:tags, [])
+    end
+
+    it "renders no tags" do
+      render
+      assert_select "table>tbody>tr", count: 0
+    end
+  end
+
+  context "when there are tags but only one page" do
+    before(:each) do
+      assign(:pagy, Pagy.new(count: 1))
+      assign(:tags, [ create(:tag, name: "Tag 1") ])
+    end
+
+    it "renders a list of tags" do
+      render
+      assert_select "table>tbody>tr", count: 1
+    end
+
+    it "renders pagination nav without page links" do
+      render
+      assert_dom "nav.pagy-bootstrap .page-link", text: "1", count: 1
+    end
+  end
+
+  context "when there are multiple pages of tags" do
+    before(:each) do
+      # Simulate being on page 2 with 10 items per page and 25 total items
+      pagy = Pagy.new(count: 25, page: 2, items: 10)
+      assign(:pagy, pagy)
+      assign(:tags, create_list(:tag, 10))
+    end
+
+    it "renders the current page of tags" do
+      render
+      assert_select "table>tbody>tr", count: 10
+    end
+
+    it "renders pagination with multiple pages" do
+      render
+      assert_select "nav.pagy-bootstrap .page-item", minimum: 2
+      assert_dom "nav.pagy-bootstrap .page-item.active", text: "2", count: 1
+    end
+  end
+end