Remove unused mock in specs

It looks like it was expected that the price and amount columns would be formatted, but this is not currently the case. So I cleaned this up to be less ambiguous.

If any of these columns were to be configured for formatting, this line could be added back to test for unformatted output.
This commit is contained in:
David Cook
2023-02-21 13:05:34 +11:00
committed by Filipe
parent 12de3ae584
commit 96eaec908d
3 changed files with 1 additions and 4 deletions

View File

@@ -118,7 +118,6 @@ describe Reporting::Reports::OrdersAndFulfillment::OrderCycleCustomerTotals do
end
it "shows the correct payment fee amount for the order" do
allow(report).to receive(:raw_render?).and_return(true)
expect(report.rows.last.pay_fee_price).to eq completed_payment.adjustment.amount
end
end

View File

@@ -49,7 +49,6 @@ module Reporting
double(name: "taxon2")]
allow(variant).to receive_message_chain(:product, :group_buy_unit_size).and_return(21)
allow(subject).to receive(:query_result).and_return [variant]
allow(subject).to receive(:raw_render?).and_return(true)
expect(subject.table_rows).to eq([[
"Supplier",

View File

@@ -124,7 +124,7 @@ module Reporting
end
it "get correct data" do
allow(subject).to receive(:raw_render?).and_return(true)
allow(subject).to receive(:unformatted_render?).and_return(true)
@expected_table_rows = [
[5, "My Hub"],
[12, "My Other Hub"],
@@ -156,7 +156,6 @@ module Reporting
{ group_by: :customer, header: true }
]
allow(subject).to receive(:display_header_row?).and_return(true)
allow(subject).to receive(:raw_render?).and_return(true)
@expected_rows = [
{ header: "Hub 1" },
{ header: "Abby" },