mirror of
https://github.com/openfoodfoundation/openfoodnetwork
synced 2026-02-08 22:56:06 +00:00
Update params syntax in engine controller specs
This commit is contained in:
@@ -25,10 +25,12 @@ describe OrderManagement::Reports::BulkCoopController, type: :controller do
|
||||
describe "#create" do
|
||||
context "when the parameters are valid" do
|
||||
it "sends the generated report in the correct format" do
|
||||
post :create, report: {
|
||||
start_at: "2018-10-09 07:30:00",
|
||||
report_type: "bulk_coop_supplier_report"
|
||||
}, report_format: "csv"
|
||||
post :create, params: {
|
||||
report: {
|
||||
start_at: "2018-10-09 07:30:00",
|
||||
report_type: "bulk_coop_supplier_report"
|
||||
}, report_format: "csv"
|
||||
}
|
||||
|
||||
expect(response).to be_success
|
||||
expect(response.body).not_to be_blank
|
||||
@@ -38,10 +40,12 @@ describe OrderManagement::Reports::BulkCoopController, type: :controller do
|
||||
|
||||
context "when the parameters are invalid" do
|
||||
it "renders the report form with an error" do
|
||||
post :create, report: {
|
||||
start_at: "invalid_date",
|
||||
report_type: "bulk_coop_supplier_report"
|
||||
}, report_format: "csv"
|
||||
post :create, params: {
|
||||
report: {
|
||||
start_at: "invalid_date",
|
||||
report_type: "bulk_coop_supplier_report"
|
||||
}, report_format: "csv"
|
||||
}
|
||||
|
||||
expect(flash[:error]).to eq(I18n.t("invalid_filter_parameters", scope: i18n_scope))
|
||||
expect(response).to render_template(new_template_path)
|
||||
@@ -55,10 +59,12 @@ describe OrderManagement::Reports::BulkCoopController, type: :controller do
|
||||
let(:current_user) { distributor.owner }
|
||||
|
||||
it "renders the report form with an error" do
|
||||
post :create, report: {
|
||||
distributor_ids: [other_distributor.id],
|
||||
report_type: "bulk_coop_supplier_report"
|
||||
}, report_format: "csv"
|
||||
post :create, params: {
|
||||
report: {
|
||||
distributor_ids: [other_distributor.id],
|
||||
report_type: "bulk_coop_supplier_report"
|
||||
}, report_format: "csv"
|
||||
}
|
||||
|
||||
expect(flash[:error]).to eq(report_klass::Authorizer.parameter_not_allowed_error_message)
|
||||
expect(response).to render_template(new_template_path)
|
||||
@@ -72,7 +78,7 @@ describe OrderManagement::Reports::BulkCoopController, type: :controller do
|
||||
let(:current_user) { distributor.owner }
|
||||
|
||||
it "applies permissions to report" do
|
||||
post :create, report: {}, report_format: "csv"
|
||||
post :create, params: { report: {}, report_format: "csv" }
|
||||
|
||||
expect(assigns(:permissions).allowed_distributors.to_a).to eq([distributor])
|
||||
end
|
||||
|
||||
@@ -25,7 +25,9 @@ describe OrderManagement::Reports::EnterpriseFeeSummariesController, type: :cont
|
||||
describe "#create" do
|
||||
context "when the parameters are valid" do
|
||||
it "sends the generated report in the correct format" do
|
||||
post :create, report: { start_at: "2018-10-09 07:30:00" }, report_format: "csv"
|
||||
post :create, params: {
|
||||
report: { start_at: "2018-10-09 07:30:00" }, report_format: "csv"
|
||||
}
|
||||
|
||||
expect(response).to be_success
|
||||
expect(response.body).not_to be_blank
|
||||
@@ -35,7 +37,9 @@ describe OrderManagement::Reports::EnterpriseFeeSummariesController, type: :cont
|
||||
|
||||
context "when the parameters are invalid" do
|
||||
it "renders the report form with an error" do
|
||||
post :create, report: { start_at: "invalid date" }, report_format: "csv"
|
||||
post :create, params: {
|
||||
report: { start_at: "invalid date" }, report_format: "csv"
|
||||
}
|
||||
|
||||
expect(flash[:error]).to eq(I18n.t("invalid_filter_parameters", scope: i18n_scope))
|
||||
expect(response).to render_template(new_template_path)
|
||||
@@ -49,7 +53,9 @@ describe OrderManagement::Reports::EnterpriseFeeSummariesController, type: :cont
|
||||
let(:current_user) { distributor.owner }
|
||||
|
||||
it "renders the report form with an error" do
|
||||
post :create, report: { distributor_ids: [other_distributor.id] }, report_format: "csv"
|
||||
post :create, params: {
|
||||
report: { distributor_ids: [other_distributor.id] }, report_format: "csv"
|
||||
}
|
||||
|
||||
expect(flash[:error]).to eq(report_klass::Authorizer.parameter_not_allowed_error_message)
|
||||
expect(response).to render_template(new_template_path)
|
||||
@@ -66,7 +72,7 @@ describe OrderManagement::Reports::EnterpriseFeeSummariesController, type: :cont
|
||||
let(:current_user) { distributor.owner }
|
||||
|
||||
it "applies permissions to report" do
|
||||
post :create, report: {}, report_format: "csv"
|
||||
post :create, params: { report: {}, report_format: "csv" }
|
||||
|
||||
expect(assigns(:permissions).allowed_order_cycles.to_a).to eq([order_cycle])
|
||||
end
|
||||
|
||||
Reference in New Issue
Block a user