Skip to content

Commit df4d22c

Browse files
committed
Update minimum wage calculator variable holding first response to be more communicative about its purpose.
1 parent dd3ad1f commit df4d22c

File tree

5 files changed

+12
-12
lines changed

5 files changed

+12
-12
lines changed

lib/smart_answer/calculators/minimum_wage_calculator.rb

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,14 @@ class MinimumWageCalculator
44
attr_accessor :age, :pay_frequency, :basic_hours, :basic_pay, :is_apprentice,
55
:overtime_hours, :overtime_hourly_rate, :accommodation_cost
66

7-
attr_reader :check, :date
7+
attr_reader :date, :what_to_check
88

99
def initialize(params = {})
1010
@age = params[:age]
1111
@date = (params[:date].nil? ? Date.today : params[:date])
1212
@basic_hours = params[:basic_hours].to_f
1313
@basic_pay = params[:basic_pay].to_f
14-
@check = params[:check]
14+
@what_to_check = params[:what_to_check]
1515
@is_apprentice = params[:is_apprentice]
1616
@pay_frequency = params[:pay_frequency] || 7
1717
@overtime_hours = params[:overtime_hours].to_i || 0

lib/smart_answer_flows/am-i-getting-minimum-wage/outcomes/current_payment_above.govspeak.erb

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
<% content_for :body do %>
22
$C
3-
<% if calculator.check == 'current_payment_april_2016' && calculator.eligible_for_living_wage? %>
3+
<% if calculator.what_to_check == 'current_payment_april_2016' && calculator.eligible_for_living_wage? %>
44
<% if calculator.living_wage_or_above? %>
55
You will be getting the national living wage
66
<% else %>
@@ -11,13 +11,13 @@
1111
<% end %>
1212
$C
1313

14-
<% if calculator.check == 'current_payment_april_2016' && calculator.eligible_for_living_wage? %>
14+
<% if calculator.what_to_check == 'current_payment_april_2016' && calculator.eligible_for_living_wage? %>
1515
The national living wage per hour for your age will be | Your actual pay
1616
<% else %>
1717
The National Minimum Wage per hour for your age | Your actual pay
1818
<% end %>
1919
- | -
20-
<% if calculator.check == 'current_payment_april_2016' && calculator.eligible_for_living_wage? %>
20+
<% if calculator.what_to_check == 'current_payment_april_2016' && calculator.eligible_for_living_wage? %>
2121
<%= format_money(calculator.national_living_wage_rate) %> | <%= format_money(calculator.total_hourly_rate) %>
2222
<% else %>
2323
<%= format_money(calculator.minimum_hourly_rate) %> | <%= format_money(calculator.total_hourly_rate) %>

lib/smart_answer_flows/shared_logic/minimum_wage.rb

+2-2
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
option "current_payment_april_2016" if self.flow_name == 'am-i-getting-minimum-wage'
66

77
calculate :calculator do |response|
8-
Calculators::MinimumWageCalculator.new(check: response)
8+
Calculators::MinimumWageCalculator.new(what_to_check: response)
99
end
1010

1111
permitted_next_nodes = [
@@ -96,7 +96,7 @@
9696
# Q3
9797
value_question :how_old_are_you?, parse: Integer do
9898
precalculate :age_title do
99-
if calculator.check == 'current_payment_april_2016'
99+
if calculator.what_to_check == 'current_payment_april_2016'
100100
PhraseList.new(:how_old_are_you_april_2016)
101101
else
102102
PhraseList.new(:how_old_are_you)

test/data/am-i-getting-minimum-wage-files.yml

+3-3
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,14 @@ lib/smart_answer_flows/locales/en/am-i-getting-minimum-wage.yml: ae4dcf74d09e349
44
test/data/am-i-getting-minimum-wage-questions-and-responses.yml: 9448744e58e3e268ba6911a58c3a1e6e
55
test/data/am-i-getting-minimum-wage-responses-and-expected-results.yml: fc0c41b148a393cf887ba11754a361ec
66
lib/smart_answer_flows/am-i-getting-minimum-wage/am_i_getting_minimum_wage.govspeak.erb: e3bad2e005bb806999594e9f87f5e5ae
7-
lib/smart_answer_flows/am-i-getting-minimum-wage/outcomes/current_payment_above.govspeak.erb: 6974ca63f1a321313c1719c289049edf
7+
lib/smart_answer_flows/am-i-getting-minimum-wage/outcomes/current_payment_above.govspeak.erb: 9b82c9fc163ecff3b9d55e6352afbddd
88
lib/smart_answer_flows/am-i-getting-minimum-wage/outcomes/current_payment_below.govspeak.erb: c5df501d4653304260fb0a95111ff111
99
lib/smart_answer_flows/am-i-getting-minimum-wage/outcomes/does_not_apply_to_historical_apprentices.govspeak.erb: bd846fa5585f32e995c74bb714f6d600
1010
lib/smart_answer_flows/am-i-getting-minimum-wage/outcomes/past_payment_above.govspeak.erb: 83b499f49b224ef3faf022f3f5df0ac1
1111
lib/smart_answer_flows/am-i-getting-minimum-wage/outcomes/past_payment_below.govspeak.erb: b53b6378246a28ae3b8706d54d09bd75
1212
lib/smart_answer_flows/am-i-getting-minimum-wage/outcomes/under_school_leaving_age.govspeak.erb: 44bf68757d1ef90249d19f5d88ad4bc2
1313
lib/smart_answer_flows/am-i-getting-minimum-wage/outcomes/under_school_leaving_age_past.govspeak.erb: 2182c103915b753894722760c9a56cf3
1414
lib/smart_answer_flows/shared/minimum_wage/_acas_information.govspeak.erb: 0d2f7c4d01e3c928701e7d649663707e
15-
lib/smart_answer_flows/shared_logic/minimum_wage.rb: eb55e4750f2d2d852c2a1a5007c62289
16-
lib/smart_answer/calculators/minimum_wage_calculator.rb: 5a10cc40f2abe6e303adfd47893d6ac0
15+
lib/smart_answer_flows/shared_logic/minimum_wage.rb: 92717bc9411a427bc422e9b528e0db36
16+
lib/smart_answer/calculators/minimum_wage_calculator.rb: f1091e755f1889d259dc511607cf59bc
1717
lib/data/minimum_wage_data.yml: 3feed11191d68237289203a2c659ecf9

test/data/minimum-wage-calculator-employers-files.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,6 @@ lib/smart_answer_flows/minimum-wage-calculator-employers/outcomes/past_payment_b
1212
lib/smart_answer_flows/minimum-wage-calculator-employers/outcomes/under_school_leaving_age.govspeak.erb: 13de5d10bbbb015df597f78c66da1b67
1313
lib/smart_answer_flows/minimum-wage-calculator-employers/outcomes/under_school_leaving_age_past.govspeak.erb: 9c56f76e04984f07627efbc1f42b2b63
1414
lib/smart_answer_flows/shared/minimum_wage/_acas_information.govspeak.erb: 0d2f7c4d01e3c928701e7d649663707e
15-
lib/smart_answer_flows/shared_logic/minimum_wage.rb: eb55e4750f2d2d852c2a1a5007c62289
16-
lib/smart_answer/calculators/minimum_wage_calculator.rb: 5a10cc40f2abe6e303adfd47893d6ac0
15+
lib/smart_answer_flows/shared_logic/minimum_wage.rb: 92717bc9411a427bc422e9b528e0db36
16+
lib/smart_answer/calculators/minimum_wage_calculator.rb: f1091e755f1889d259dc511607cf59bc
1717
lib/data/minimum_wage_data.yml: 3feed11191d68237289203a2c659ecf9

0 commit comments

Comments
 (0)