Update with regard to code review

 - swap initialisation order of cookbook_pathname and cookbook_name
 - require that cookbooks have a name attribute in their metadata
 - excise side effect of cookbook_version()
Move openldap test cookbooks, add metadata

This is to test the metadata name attribute fix [CHEF-3307].
Test cookbook name being read from metadata

No explicit tests added, but existing tests and test data have been modifed to
ensure correct functionality when using a cookbook whose name and pathname do
not match.
Fix tests, slight manifest hack