Merge pull request #15 from alphagov/integration-tests-for-update

Add integration-level tests for updating.
Merge pull request #16 from alphagov/flash-notice-on-update

Show flash notice on update
Merge pull request #17 from alphagov/separate-new-edit-views

Split the new and edit views, and share the form using a partial
Return a 404 on need view if need doesn't exist.

This incurs an extra GET request, but means we don't redirect to a
non-existent URL. If the API adapters don't already get around the
duplicate request with caching, we should be able to fix that if it's an
issue.

Also, once we have separate view and edit actions, this won't be an
issue any more.
Changed impact phrasing

Phrasing and ordering of 'impact' now consistent with the Maslow templates being used in the transition workshops
Update need.rb
Merge pull request #19 from bmwelby/patch-1

Changed impact phrasing
Fix mass test breakage from pull request #19.

(Merge commit 7b826109920f806569c000397a8465966e16c8c2.)
Remove tests that duplicate Need unit tests.
Don't crash if Need API returns a 422
Add integration test for handling Need API 422s
Add unit test for need not saving
Not displaying specific save errors
Remove unnecessary test data
Make test more readable
Call to_json in the correct place in the return data
Handle 422s from the API on update requests too.
Move 422 creation test; update to stub model layer.

The test was in the "GET index" context, and was stubbing out responses
at the HTTP level, which is more the domain of integration tests.
Merge branch 'fix-404-bug'
Render edit form, not create form, on edit failure.