Make the proof API check constraints.

The proof API now uses charmworldlib to check the charm constraints specified
in a bundle (deployer file).

Errors returned from parse_constraints are returned as errors.  If
check_constraints indicates the constraints are comma-separated then a
deprecation warning is returned.

The previous API did not allow for warnings, so that mechanism was added.

There are a lot of drive-by cleanups too, mainly in the test.  Sorry for the
noise they introduce but I think the code is more consistent and the tests are
improved.

https://codereview.appspot.com/69980054/

R=makyo.

Approved by Juju Gui Bot.