diff --git a/lib/heroku/kensa/check.rb b/lib/heroku/kensa/check.rb index 1188bd1..917106d 100644 --- a/lib/heroku/kensa/check.rb +++ b/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 diff --git a/test/provision_response_check_test.rb b/test/provision_response_check_test.rb index 8906201..935d249 100644 --- a/test/provision_response_check_test.rb +++ b/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