Merge pull request #66 from mmay/master

Don't die simply because config var is missing from provisioning response

lib/heroku/kensa/check.rb

@@ -2,6 +2,7 @@
 require 'socket'
 require 'timeout'
 require 'uri'
+require 'term/ansicolor'
 
 module Heroku
   module Kensa
@@ -164,6 +165,7 @@ def call!
 
 
     class ProvisionResponseCheck < Check
+      include Term::ANSIColor
 
       def call!
         response = data[:provision_response]
@@ -192,7 +194,7 @@ def call!
             difference = data['api']['config_vars'] - response['config'].keys 
             unless difference.empty?
               verb = (difference.size == 1) ? "is" : "are"
-              error "#{difference.join(', ')} #{verb} missing from the manifest"
+              print "\n\t", yellow( "#{difference.join(', ')} #{verb} missing from the manifest")
             end
             true
           end

test/provision_response_check_test.rb

@@ -67,11 +67,6 @@ def check ; ProvisionResponseCheck ; end
       assert_invalid
     end
 
-    test "asserts all vars in manifest are in response" do
-      @response["config"].delete('MYADDON_CONFIG')
-      assert_invalid
-    end
-
     test "is valid otherwise" do
       @response["config"]["MYADDON_URL"] = "http://localhost/abc" 
       assert_valid