fail fast on config:add

lib/heroku/command/config.rb

@@ -36,20 +36,18 @@ def add
       # try to get the app to fail fast
       detected_app = app
 
-      display "Adding config vars:"
-      display_vars(vars, :indent => 2)
-
-      display "Restarting app...", false
+      display "Adding config vars and restarting app...", false
       heroku.add_config_vars(detected_app, vars)
-
       display " done", false
 
       begin
         release = heroku.releases(detected_app).last
         display(", #{release["name"]}", false) if release
       rescue RestClient::RequestFailed => e
       end
-      display "."
+
+      display
+      display_vars(vars, :indent => 2)
     end
 
     # config:remove KEY1 [KEY2 ...]

spec/heroku/command/config_spec.rb

@@ -35,6 +35,16 @@ module Heroku::Command
       @config.add
     end
 
+    it "doesnt show config vars when an error occurs during add" do
+      Heroku::Command.should_receive(:error).with("command failed")
+      any_instance_of(Heroku::Client) do |client|
+        stub(client).add_config_vars do
+          raise CommandFailed, "command failed"
+        end
+      end
+      run("config:add A=newvalue").should_not =~ /newvalue/
+    end
+
     it "allows config vars with = in the value" do
       @config.stub!(:args).and_return(['a=b=c'])
       @config.heroku.should_receive(:add_config_vars).with('myapp', {'a'=>'b=c'})