(#7013) Handle rendering modes out in the application layer.

We no longer establish the rendering mode in the actions; they just default to
"nothing", and let that flow on out to the application layer.  That lets the
facade we put before the face determine the default behaviour.

This is mostly a no-op down in the CLI side, but it makes it much easier to
integrate into MCollective, HTTP-API, and for other non-CLI users of Faces.

Reviewed-By: Max Martin <max@puppetlabs.com>

1 files changed, 10 insertions, 3 deletions

diff --git a/spec/unit/application/face_base_spec.rb b/spec/unit/application/face_base_spec.rb
index 6a4647246..6dfde4367 100755
--- a/spec/unit/application/face_base_spec.rb
+++ b/spec/unit/application/face_base_spec.rb
@@ -211,11 +211,18 @@ describe Puppet::Application::FaceBase do
   end
 
   describe "#render" do
-    it "should just return a String" do
-      app.render("hello").should == "hello"
+    before :each do
+      app.face   = Puppet::Face[:basetest, '0.0.1']
+      app.action = app.face.get_action(:foo)
+    end
+
+    ["hello", 1, 1.0].each do |input|
+      it "should just return a #{input.class.name}" do
+        app.render(input).should == input
+      end
     end
 
-    [1, 1.000, [1, 2], ["one"], [{ 1 => 1 }]].each do |input|
+    [[1, 2], ["one"], [{ 1 => 1 }]].each do |input|
       it "should render #{input.class} using the 'pp' library" do
         app.render(input).should == input.pretty_inspect
       end