@@ -16,7 +16,7 @@ describe ClientSummarizer do
 
   describe "Ncr::WorkOrder" do
     it "builds summary" do
-      wo = create(:ncr_work_order, amount: 123)
+      create(:ncr_work_order, amount: 123)
       summarizer = ClientSummarizer.new(client_namespace: "Ncr")
       summary = summarizer.run
       expect(summary.total).to eq(123)
describe ClientSummarizer do
  describe "reports initializer params" do
    it "defaults to current year" do
      summarizer = ClientSummarizer.new(client_namespace: "test")
      now = Time.zone.now
      expect(summarizer.fiscal_year).to eq(ClientSummarizer.which_fiscal_year(now.year, now.month))
    end

    it "uses namespace exactly as passed" do
      summarizer = ClientSummarizer.new(client_namespace: "nosuchclient")
      summary = summarizer.run
      expect(summary.total).to eq(0)
      expect(summary.client_namespace).to eq("nosuchclient")
    end
  end

  describe "Ncr::WorkOrder" do
    it "builds summary" do
      wo = create(:ncr_work_order, amount: 123)
      summarizer = ClientSummarizer.new(client_namespace: "Ncr")
      summary = summarizer.run
      expect(summary.total).to eq(123)
      expect(summary.status(:pending)).to eq(1)
      expect(summary.subtotal(:pending)).to eq(123)
    end
  end

  describe "Gsa18f::Procurement" do
    it "builds summary" do
      procurement = create(:gsa18f_procurement, cost_per_unit: 18.50, quantity: 20)
      summarizer = ClientSummarizer.new(client_namespace: "Gsa18f")
      summary = summarizer.run
      expect(summary.total).to eq(370)
      expect(summary.status(:pending)).to eq(1)
      expect(summary.subtotal(:pending)).to eq(370)
    end
  end
end

why throw away the variable assignment? and why in this PR?