Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing calculated value in paternity calculator #2058

Conversation

erkde
Copy link
Contributor

@erkde erkde commented Nov 5, 2015

Adds tests to the adoption, maternity and paternity smart answers to ensure that questions with interpolated values are rendered correctly.

Adds a test helper method #question_title reads the value of a question as a String, forcing the interpolation to occur, so it can be asserted with a has_content? matcher in integration tests.

Updates the paternity calculator, adding the missing calculated value that is interpolated into the question title.

Verify step

Example URL

  • Shows a question "Was the employee (or will they be) on your payroll on 12 July 2015?" within the previous answers without breaking the page.

screen shot 2015-11-05 at 4 16 09 pm

…nsure that questions with interpolated values are rendered correctly.

Add test helper method #question_title reads the value of a question as a String, forcing the interpolation to occur, so it can be asserted with a has_content? matcher in integration tests.
@erkde erkde force-pushed the add_test_for_interpolation_in_question_titles_for_maternity_paternity_and_adoption branch from 6d18e5f to 7f5c483 Compare November 5, 2015 16:44
@chrisroos
Copy link
Contributor

Looks good to me, @erik-eide!

@chrisroos chrisroos self-assigned this Nov 6, 2015
@chrisroos chrisroos added the LGTM label Nov 6, 2015
erkde added a commit that referenced this pull request Nov 6, 2015
…question_titles_for_maternity_paternity_and_adoption

Add missing calculated value in paternity calculator
@erkde erkde merged commit e1e61e3 into master Nov 6, 2015
@erkde erkde deleted the add_test_for_interpolation_in_question_titles_for_maternity_paternity_and_adoption branch November 6, 2015 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants