consul/nairobi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use named parameter in mappable shared example

Browse Source 
Boolean arguments are harder to read, so we added a useless variable.
This is the perfect scenario for a named parameter.
This commit is contained in:
Javi Martín
2019-09-22 21:52:13 +02:00
parent 4e72d9f3b7
commit 175163adb6
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
spec/features/management/budget_investments_spec.rb
View File

@@ -16,7 +16,7 @@ describe "Budget Investments" do
"",
"management_budget_investment_path",
{ "budget_id": "budget_id" },
management = true
management: true
context "Load" do

2
spec/shared/features/mappable.rb
View File

@@ -1,4 +1,4 @@
shared_examples "mappable" do |mappable_factory_name, mappable_association_name, mappable_new_path, mappable_edit_path, mappable_show_path, mappable_path_arguments, management = false|
shared_examples "mappable" do |mappable_factory_name, mappable_association_name, mappable_new_path, mappable_edit_path, mappable_show_path, mappable_path_arguments, management: false|
include ActionView::Helpers