Policyfile can normalize run list to an invalid value

[danielsdeleo]

I had a policyfile with the following:

run_list "recipe[cia_infra:base]"

That is incorrect because it should have two colons instead of one (i.e., it should be recipe[cia_infra::base]). If you run chef install with this, it normalizes the run list item to "recipe[cia_infra:base::default]" which the server will reject, with a curt error message:

Error: Failed to upload policy to policy group uwotmate
Reason: (Net::HTTPServerException) HTTP 400 Bad Request: Field 'run_list' is not a valid run list

Probably the best thing to do is to copy the server's validation and run that against the normalized run list, though this specific problem can be fixed by ensuring the cookbook name doesn't contain a colon.

[danielsdeleo]

Fixed by: #635