feat: add pagination to topics #195

Author: hernanvicente

app/controllers/topics_controller.rb

@@ -1,8 +1,10 @@
 class TopicsController < ApplicationController
+  include Pagy::Backend
+
   before_action :set_topic, only: [ :show, :edit, :tags, :update, :destroy, :archive ]
 
   def index
-    @topics = scope.search_with_params(search_params)
+    @pagy, @topics = pagy(scope.search_with_params(search_params))
     @available_providers = other_available_providers
     @languages = scope.map(&:language).uniq.sort_by(&:name)
   end

app/views/topics/index.html.erb

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

spec/factories/topics.rb

@@ -25,7 +25,7 @@
   factory :topic do
     association :provider
     association :language
-    title { "topic title" }
+    sequence(:title) { |n| "topic title #{10}" }
     description { "many topic details" }
     published_at { DateTime.new(2023, 1, 1) }
     state { 0 }

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

@@ -0,0 +1,64 @@
+require "rails_helper"
+
+RSpec.describe "topics/index", type: :view do
+  before(:each) do
+    def view.search_params = {}
+    def view.topics_title = "Topics"
+
+    # Allow the original render method to work normally
+    allow(view).to receive(:render).and_call_original
+    # But stub specifically the search partial render
+    allow(view).to receive(:render).with("search", any_args).and_return("")
+
+    assign(:available_providers, [])
+  end
+
+  context "when there are no topics" do
+    before(:each) do
+      assign(:pagy, Pagy.new(count: 0))
+      assign(:topics, [])
+    end
+
+    it "renders no topics" do
+      render
+      assert_select "table>tbody>tr", count: 0
+    end
+  end
+
+  context "when there are topics but only one page" do
+    before(:each) do
+      assign(:pagy, Pagy.new(count: 1))
+      assign(:topics, [ create(:topic, title: "Topic 1") ])
+    end
+
+    it "renders a list of topics" do
+      render
+      assert_select "table>tbody>tr", count: 1
+    end
+
+    it "renders pagination nav with unique item" do
+      render
+      assert_dom "nav.pagy-bootstrap .page-item", text: "1", count: 1
+    end
+  end
+
+  context "when there are multiple pages of topics" 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(:topics, create_list(:topic, 10))
+    end
+
+    it "renders the current page of topics" 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