Acceptance tests should not rely on the puppetpath beaker option

Several acceptance tests rely on puppetpath for both code and configuration related paths. Historically that setting was the same as puppetconfdir, but with the code/config split, it no longer makes sense.

The acceptance tests should be updated to:

1. Use beaker's host.puppet['setting'] methods so the test doesn't depend on hard-coded beaker default
2. Query for codedir or confdir as appropriate.
3. Take into account the agent vs master application, e.g. host.puppet('master')['confdir']