Test: Adds missing tests for requests controller #print_unfulfilled

Author: bsbonus

spec/requests/partners/requests_spec.rb

@@ -423,4 +423,60 @@
       end
     end
   end
+
+  describe "GET #print_unfulfilled" do
+    let(:item1) { create(:item, name: "Good item") }
+    let(:item2) { create(:item, name: "Crap item") }
+    let(:partner1) { create(:partner, organization: organization) }
+    let(:partner_user) { partner1.primary_user }
+    let!(:pending_request) { create(:request, :with_item_requests, :pending, partner: partner1, request_items: [{ item_id: item1.id, quantity: '100' }]) }
+    let!(:started_request) { create(:request, :with_item_requests, :started, partner: partner1, request_items: [{ item_id: item2.id, quantity: '50' }]) }
+    let!(:discarded_request) { create(:request, :with_item_requests, :discarded, partner: partner1, request_items: [{ item_id: item2.id, quantity: '30' }]) }
+    let!(:fulfilled_request) { create(:request, :with_item_requests, :fulfilled, partner: partner1, request_items: [{ item_id: item2.id, quantity: '20' }]) }
+
+    before do
+      partner_user.add_role(Role::ORG_ADMIN, organization)
+      sign_in(partner_user)
+      get print_unfulfilled_requests_path(format: :pdf)
+    end
+
+    it "returns a PDF file" do
+      PDF::Reader.new(StringIO.new(response.body))
+      expect(response.content_type).to eq('application/pdf')
+      expect(response.headers['Content-Disposition']).to include('inline')
+      expect(response.body.bytes[0..3]).to eq('%PDF'.bytes)
+    end
+
+    it "includes only 'pending' and 'started' requests" do
+      pdf_content = PDF::Reader.new(StringIO.new(response.body))
+      # this is a semi-lazy check, since we're ensuring 1 page for each request. In real world,
+      # it's possible that there could be more than 1 page per request if the request is long.
+
+      expect(pdf_content.page_count).to eq(2)
+    end
+
+    it "calls compute_and_render with the 2 matching requests" do
+      # Create a double for the PDF instance
+      pdf_double = double("PicklistsPdf")
+
+      # Expect PicklistsPdf.new to be called with correct args and return our double
+      expect(PicklistsPdf).to receive(:new)
+        .with(organization, kind_of(ActiveRecord::Relation))
+        .and_return(pdf_double)
+
+      # Expect compute_and_render to be called on our double and return some PDF data
+      # We don't really care about the content, the PDF model is tested elsewhere
+      expect(pdf_double).to receive(:compute_and_render)
+        .and_return("fake pdf content")
+
+      # Make the request
+      get print_unfulfilled_requests_path(format: :pdf)
+
+      # Verify the response
+      expect(response).to be_successful
+      expect(response.content_type).to eq("application/pdf")
+      expect(response.headers["Content-Disposition"]).to include("inline")
+      expect(response.body).to eq("fake pdf content")
+    end
+  end
 end