Looks like in renaming faces to face we just missed some files.  They
got copied, not moved.

Paired-with: Max Martin <max@puppetlabs.com>

This has been removed in favor of 'puppet parser validate <manifest>'.

Paired-With: Jesse Wolfe

After some discussion we decided that most uses of the Puppet Face
infrastructure were about single faces on their own, not about the collection,
and so we were better referring to Puppet::Face[...] in code.

This implements that by translating names and references in the Ruby code to
the new, s-less, name.

Previously we would only emit help for a face; now we fully integrate legacy
subcommands in the sense that asking for face-level help will emit their
entire help text.

Asking for any action subsequent will raise an error: this is an annoyingly
inconsistent behaviour, but there isn't a saner definition of the change.

Reviewed-By: Matt Robinson <matt@puppetlabs.com>

We used to blink and miss the fact that we failed to load an action or face in
the past; now we test for that, and fail with a clear error message when the
user asks for something we can't deliver.

Additionally, fix a couple of tests that were silently broken because they
passed the wrong arguments, but still got some output.

Paired-With: Matt Robinson <matt@puppetlabs.com>

We now set the filename attribute of the ERB instance, which results in any
error messages showing up with the right attribution: to the file they are
defined in, rather than just '(erb)'.

Paired-With: Matt Robinson <matt@puppetlabs.com>

Rather than hard-coding the layout of help output in the code, push it out
into external templates.  At the moment overriding that would require
changing the ERB code next to puppet/faces/help.rb file on disk.

Paired-With: Matt Robinson <matt@puppetlabs.com>

We now emit the summary of actions for an individual face, in the same format
as the summary of available faces.  This moves forward through the feature set
defined for the help subcommand.

Reviewed-By: Matt Robinson <matt@puppetlabs.com>

We use a dubious, but effective, regexp match on the existing man(1) style
help string for the application to extract the summary data.  This should
properly be implemented as a summary method down in the applications
themselves...

Paired-With: Matt Robinson <matt@puppetlabs.com>

We now enumerate and print the list of legacy applications in the unadorned
help action; this allows us a path to migrating forward smoothly while still
providing a good user experience.

Paired-With: Matt Robinson <matt@puppetlabs.com>