Remove unused AreaOfLaw::MONEY_CLAIMS constant
Extract commonly-used AreaOfLaw names into constants

This is more unwieldy, but it reduces the chances of making a mistake in
the name of an area of law, and provides an incentive to refactor the
code to make it simpler.
Fix spelling mistake
Remove nonsensical ability to omit area_of_law argument

This method doesn’t work without an area of law, so it doesn’t make
sense for it to be optional.
Add unit tests around Court#invalid_councils

There are currently no tests covering this, but it provides important
feedback in the admin UI.
Replace Court#set_area_councils_list implementation

There’s no need to manage the court_council_links ourselves; Active
Record does that automatically when we update the councils collection.
Use better names for #set_area_councils_list parameters
Check that councils are filtered by area of law
Replace Court#area_councils implementation
Use a better name for #area_councils parameter
Make council names deterministic in court spec

The lists of councils are meant to come back in alphabetical order,
which can cause problems when that doesn’t match creation order (e.g.
“Council 9”, “Council 10”), so it’s better to just set the names
explicitly.
Add class methods for getting named areas of law

These areas are referred to explicitly in various places, so it’s
better to have one place that’s responsible for finding them.
Use AreaOfLaw class methods in Councils concern
Pass AreaOfLaw instances into area_councils methods directly
Define all the per-area getters/setters dynamically

This kind of metaprogramming is undesirable, but it at least exposes the
duplication between all these methods and avoids them getting out of
sync.
Make council list methods attr_accessible while defining them
Add simple acceptance test for editing councils
Merge pull request #219 from ministryofjustice/clean-up

Clean up the court/council management code