heroku-api 0.2.6 → 0.2.7

data/.travis.yml

@@ -1,9 +1,4 @@
-script: bundle exec rake
-
-rvm:
-  - 1.8.7
-  - 1.9.2
-  - 1.9.3
+language: ruby
 
 notifications:
   email: false
@@ -12,3 +7,10 @@ notifications:
     on_failure: always
     urls:
       - http://dx-helper.herokuapp.com/travis
+
+rvm:
+  - 1.8.7
+  - 1.9.2
+  - 1.9.3
+
+script: bundle exec rake

data/README.md

@@ -29,7 +29,7 @@ For additional details about any of the commands, see the [API docs](http://api-
 
 ### Add-ons
 
-    heroku.delete('app', 'addon')     # remove 'addon' add-on from an 'app' app
+    heroku.delete_addon('app', 'addon')     # remove 'addon' add-on from an 'app' app
     heroku.get_addons                 # see a listing of all available add-ons
     heroku.get_addons('app')          # see listing of installed add-ons for 'app' app
     heroku.post_addon('app', 'addon') # add 'addon' add-on to 'app' app
@@ -47,60 +47,60 @@ For additional details about any of the commands, see the [API docs](http://api-
 
 ### Collaborators
 
-    delete_collaborator('app', 'email@example.com') # remove 'email@example.com' collaborator from 'app' app
-    get_collaborators('app')                        # list collaborators for 'app' app
-    post_collaborator('app', 'email@example.com')   # add 'email@example.com' collaborator to 'app' app
+    heroku.delete_collaborator('app', 'email@example.com') # remove 'email@example.com' collaborator from 'app' app
+    heroku.get_collaborators('app')                        # list collaborators for 'app' app
+    heroku.post_collaborator('app', 'email@example.com')   # add 'email@example.com' collaborator to 'app' app
 
 ### Config Variables
 
-    delete_config_var('app', 'KEY')           # remove 'KEY' key from 'app' app
-    get_config_vars('app')                    # get list of config vars for 'app' app
-    put_config_vars('app', 'KEY' => 'value')  # set 'KEY' key to 'value' for 'app' app
+    heroku.delete_config_var('app', 'KEY')           # remove 'KEY' key from 'app' app
+    heroku.get_config_vars('app')                    # get list of config vars for 'app' app
+    heroku.put_config_vars('app', 'KEY' => 'value')  # set 'KEY' key to 'value' for 'app' app
 
 ### Domains
 
-    delete_domain('app', 'example.com') # remove the 'example.com' domain from the 'app' app
-    get_domains('app')                  # list configured domains for the 'app' app
-    post_domains('app', 'example.com')  # add 'example.com' domain to the 'app' app
+    heroku.delete_domain('app', 'example.com') # remove the 'example.com' domain from the 'app' app
+    heroku.get_domains('app')                  # list configured domains for the 'app' app
+    heroku.post_domains('app', 'example.com')  # add 'example.com' domain to the 'app' app
 
 ### Keys
 
-    delete_key('user@hostname.local') # remove the 'user@hostname.local' key
-    delete_keys                       # remove all keys
-    get_keys                          # list configured keys
-    post_key('key data')              # add key defined by 'key data'
+    heroku.delete_key('user@hostname.local') # remove the 'user@hostname.local' key
+    heroku.delete_keys                       # remove all keys
+    heroku.get_keys                          # list configured keys
+    heroku.post_key('key data')              # add key defined by 'key data'
 
 ### Logs
 
-    get_logs('app') # return logs information for 'app' app
+    heroku.get_logs('app') # return logs information for 'app' app
 
 ### Processes
 
-    get_ps('app')                             # list current processes for 'app' app
-    post_ps('app', 'command')                 # run 'command' command in context of 'app' app
-    post_ps_restart('app')                    # restart all processes for 'app' app
-    post_ps_scale('app', 'type', 'quantity')  # scale 'type' type processes to 'quantity' for 'app' app
-    post_ps_stop('app', 'ps' => 'web.1')      # stop 'web.1' process for 'app' app
-    post_ps_stop('app', 'type' => 'web')      # stop all 'web' processes for 'app' app
-    put_dynos('app', 'dynos')                 # set number of dynos for bamboo app 'app' to 'dynos'
-    put_workers('app', 'workers')             # set number of workers for bamboo app 'app' to 'workers'
+    heroku.get_ps('app')                             # list current processes for 'app' app
+    heroku.post_ps('app', 'command')                 # run 'command' command in context of 'app' app
+    heroku.post_ps_restart('app')                    # restart all processes for 'app' app
+    heroku.post_ps_scale('app', 'type', 'quantity')  # scale 'type' type processes to 'quantity' for 'app' app
+    heroku.post_ps_stop('app', 'ps' => 'web.1')      # stop 'web.1' process for 'app' app
+    heroku.post_ps_stop('app', 'type' => 'web')      # stop all 'web' processes for 'app' app
+    heroku.put_dynos('app', 'dynos')                 # set number of dynos for bamboo app 'app' to 'dynos'
+    heroku.put_workers('app', 'workers')             # set number of workers for bamboo app 'app' to 'workers'
 
-    post_ps_restart('app', 'ps' => 'web.1')   # restart 'web.1' process for 'app' app
+    heroku.post_ps_restart('app', 'ps' => 'web.1')   # restart 'web.1' process for 'app' app
 
 ### Releases
 
-    get_releases('app')       # list of releases for 'app' app
-    get_release('app', 'v#')  # get details of 'v#' release for 'app' app
-    post_release('app', 'v#') # rollback 'app' app to 'v#' release
+    heroku.get_releases('app')       # list of releases for 'app' app
+    heroku.get_release('app', 'v#')  # get details of 'v#' release for 'app' app
+    heroku.post_release('app', 'v#') # rollback 'app' app to 'v#' release
 
 ### Stacks
 
-    get_stack('app')          # list available stacks
-    put_stack('app', 'stack') # migrate 'app' app to 'stack' stack
+    heroku.get_stack('app')          # list available stacks
+    heroku.put_stack('app', 'stack') # migrate 'app' app to 'stack' stack
 
 ### User
 
-    get_user                  # list user info
+    heroku.get_user                  # list user info
 
 Mock
 ----

data/changelog.txt

@@ -1,3 +1,11 @@
+0.2.7 07/05/2012
+================
+
+doc fixes (fixes delete_addon and improves consistency)
+post_collaborator returns 201 when users does not yet exist
+use json (instead of okjson) when available
+update excon dependency
+
 0.2.6 06/21/2012
 ================
 

data/heroku-api.gemspec

@@ -16,7 +16,7 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
-  s.add_runtime_dependency 'excon', '~>0.14.0'
+  s.add_runtime_dependency 'excon', '~>0.14.3'
 
   s.add_development_dependency 'minitest'
   s.add_development_dependency 'rake'

data/lib/heroku/api.rb

@@ -9,8 +9,6 @@ unless $LOAD_PATH.include?(__LIB_DIR__)
   $LOAD_PATH.unshift(__LIB_DIR__)
 end
 
-require "heroku/api/vendor/okjson"
-
 require "heroku/api/errors"
 require "heroku/api/mock"
 require "heroku/api/version"
@@ -34,6 +32,36 @@ srand
 module Heroku
   class API
 
+    begin
+
+      require('json')
+
+      def self.json_decode(json)
+        JSON.parse(json)
+      rescue
+        json
+      end
+
+      def self.json_encode(object)
+        JSON.dump(object)
+      end
+
+    rescue LoadError
+
+      require('heroku/api/vendor/okjson')
+
+      def self.json_decode(json)
+        Heroku::API::OkJson.decode(json)
+      rescue
+        json
+      end
+
+      def self.json_encode(object)
+        Heroku::API::OkJson.encode(object)
+      end
+
+    end
+
     def initialize(options={})
       @api_key = options.delete(:api_key) || ENV['HEROKU_API_KEY']
       user_pass = ":#{@api_key}"
@@ -81,9 +109,7 @@ module Heroku
           response.body = Zlib::GzipReader.new(StringIO.new(response.body)).read
         end
         begin
-          response.body = Heroku::API::OkJson.decode(response.body)
-        rescue
-          # leave non-JSON body as is
+          response.body = Heroku::API.json_decode(response.body)
         end
       end
 

data/lib/heroku/api/collaborators.rb

@@ -22,7 +22,7 @@ module Heroku
     # POST /apps/:app/collaborators
     def post_collaborator(app, email)
       request(
-        :expects  => 200,
+        :expects  => [200, 201],
         :method   => :post,
         :path     => "/apps/#{app}/collaborators",
         :query    => {'collaborator[email]' => email}

data/lib/heroku/api/config_vars.rb

@@ -22,7 +22,7 @@ module Heroku
     # PUT /apps/:app/config_vars
     def put_config_vars(app, vars)
       request(
-        :body     => Heroku::API::OkJson.encode(vars),
+        :body     => Heroku::API.json_encode(vars),
         :expects  => 200,
         :method   => :put,
         :path     => "/apps/#{app}/config_vars"

data/lib/heroku/api/mock.rb

@@ -72,7 +72,7 @@ module Heroku
       def self.get_mock_addon(mock_data, addon)
         @addons ||= begin
           data = File.read("#{File.dirname(__FILE__)}/mock/cache/get_addons.json")
-          Heroku::API::OkJson.decode(data)
+          Heroku::API.json_decode(data)
         end
         @addons.detect {|addon_data| addon_data['name'] == addon}
       end
@@ -106,7 +106,7 @@ module Heroku
       def self.get_mock_feature(mock_data, feature)
         @features ||= begin
           data = File.read("#{File.dirname(__FILE__)}/mock/cache/get_features.json")
-          Heroku::API::OkJson.decode(data)
+          Heroku::API.json_decode(data)
         end
         @features.detect {|feature_data| feature_data['name'] == feature}
       end
@@ -139,7 +139,7 @@ module Heroku
 
         parsed = params.dup
         begin # try to JSON decode
-          parsed[:body] &&= Heroku::API::OkJson.decode(parsed[:body])
+          parsed[:body] &&= Heroku::API.json_decode(parsed[:body])
         rescue # else leave as is
         end
 

data/lib/heroku/api/mock/addons.rb

@@ -14,7 +14,7 @@ module Heroku
               # addon is currently installed
               remove_mock_app_addon(mock_data, app, addon)
               {
-                :body   => Heroku::API::OkJson.encode({
+                :body   => Heroku::API.json_encode({
                   "message" => nil,
                   "price"   => get_mock_addon_price(mock_data, addon),
                   "status"  => 'Uninstalled'
@@ -24,14 +24,14 @@ module Heroku
             else
               # addon is not currently installed
               {
-                :body   => Heroku::API::OkJson.encode({'error' => "The add-on #{addon} is not installed for this app. Did you mean:\n\t#{addon_names.join("\n\t")}"}),
+                :body   => Heroku::API.json_encode({'error' => "The add-on #{addon} is not installed for this app. Did you mean:\n\t#{addon_names.join("\n\t")}"}),
                 :status => 422
               }
             end
           else
             # addon does not exist
             {
-              :body   => Heroku::API::OkJson.encode({'error' => "Could not find add-on #{addon}. Did you mean:\n\t#{addon_names.join("\n\t")}"}),
+              :body   => Heroku::API.json_encode({'error' => "Could not find add-on #{addon}. Did you mean:\n\t#{addon_names.join("\n\t")}"}),
               :status => 422
             }
           end
@@ -53,7 +53,7 @@ module Heroku
         app, _ = request_params[:captures][:path]
         with_mock_app(mock_data, app) do
           {
-            :body   => Heroku::API::OkJson.encode(mock_data[:addons][app].map {|addon| addon['configured'] = true; addon}),
+            :body   => Heroku::API.json_encode(mock_data[:addons][app].map {|addon| addon['configured'] = true; addon}),
             :status => 200
           }
         end
@@ -72,7 +72,7 @@ module Heroku
                 # addon is not currently installed
                 add_mock_app_addon(mock_data, app, addon)
                 {
-                  :body   => Heroku::API::OkJson.encode({
+                  :body   => Heroku::API.json_encode({
                     "message" => nil,
                     "price"   => get_mock_addon_price(mock_data, addon),
                     "status"  => 'Installed'
@@ -82,21 +82,21 @@ module Heroku
               else
                 # addon is currently installed
                 {
-                  :body   => Heroku::API::OkJson.encode({'error' => "Add-on already installed."}),
+                  :body   => Heroku::API.json_encode({'error' => "Add-on already installed."}),
                   :status => 422
                 }
               end
             else
               # addon of same type exists
               {
-                :body   => Heroku::API::OkJson.encode({'error' => "#{app_addon_type_data['name']} add-on already added.\nTo upgrade, use addons:upgrade instead.\n"}),
+                :body   => Heroku::API.json_encode({'error' => "#{app_addon_type_data['name']} add-on already added.\nTo upgrade, use addons:upgrade instead.\n"}),
                 :status => 422
               }
             end
           else
             # addon does not exist
             {
-              :body   => Heroku::API::OkJson.encode({'error' => "Add-on not found."}),
+              :body   => Heroku::API.json_encode({'error' => "Add-on not found."}),
               :status => 404
             }
           end
@@ -117,7 +117,7 @@ module Heroku
                 mock_data[:addons][app].delete(app_addon_data)
                 add_mock_app_addon(mock_data, app, addon)
                 {
-                  :body   => Heroku::API::OkJson.encode({
+                  :body   => Heroku::API.json_encode({
                     "message" => 'Plan upgraded',
                     "price"   => get_mock_addon_price(mock_data, addon),
                     "status"  => 'Updated'
@@ -127,21 +127,21 @@ module Heroku
               else
                 # addon is currently installed
                 {
-                  :body   => Heroku::API::OkJson.encode({'error' => "Add-on already installed."}),
+                  :body   => Heroku::API.json_encode({'error' => "Add-on already installed."}),
                   :status => 422
                 }
               end
             else
               # addon of same type not installed
               {
-                :body   => Heroku::API::OkJson.encode({'error' => "Can't upgrade, no #{addon.split(':').join(' ')} add-on has been added.\nTo add, use addons:add instead.\n"}),
+                :body   => Heroku::API.json_encode({'error' => "Can't upgrade, no #{addon.split(':').join(' ')} add-on has been added.\nTo add, use addons:add instead.\n"}),
                 :status => 422
               }
             end
           else
             # addon does not exist
             {
-              :body   => Heroku::API::OkJson.encode({'error' => "Add-on not found."}),
+              :body   => Heroku::API.json_encode({'error' => "Add-on not found."}),
               :status => 404
             }
           end

data/lib/heroku/api/mock/apps.rb

@@ -15,7 +15,7 @@ module Heroku
           mock_data[:ps].delete(app)
           mock_data[:releases].delete(app)
           {
-            :body   => Heroku::API::OkJson.encode({}),
+            :body   => Heroku::API.json_encode({}),
             :status => 200
           }
         end
@@ -25,7 +25,7 @@ module Heroku
       Excon.stub(:expects => 200, :method => :get, :path => '/apps') do |params|
         request_params, mock_data = parse_stub_params(params)
         {
-          :body   => Heroku::API::OkJson.encode(mock_data[:apps]),
+          :body   => Heroku::API.json_encode(mock_data[:apps]),
           :status => 200
         }
       end
@@ -36,7 +36,7 @@ module Heroku
         app, _ = request_params[:captures][:path]
         with_mock_app(mock_data, app) do |app_data|
           {
-            :body   => Heroku::API::OkJson.encode(app_data),
+            :body   => Heroku::API.json_encode(app_data),
             :status => 200
           }
         end
@@ -49,7 +49,7 @@ module Heroku
 
         if get_mock_app(mock_data, app)
           {
-            :body => Heroku::API::OkJson.encode('error' => 'Name is already taken'),
+            :body => Heroku::API.json_encode('error' => 'Name is already taken'),
             :status => 422
           }
         else
@@ -112,7 +112,7 @@ module Heroku
           end
 
           {
-            :body   => Heroku::API::OkJson.encode(app_data),
+            :body   => Heroku::API.json_encode(app_data),
             :status => 202
           }
         end
@@ -157,12 +157,12 @@ module Heroku
           end
           if email && !collaborator
             {
-              :body   => Heroku::API::OkJson.encode('error' => 'Only existing collaborators can receive ownership for an app'),
+              :body   => Heroku::API.json_encode('error' => 'Only existing collaborators can receive ownership for an app'),
               :status => 422
             }
           else
             {
-              :body   => Heroku::API::OkJson.encode('name' => app_data['name']),
+              :body   => Heroku::API.json_encode('name' => app_data['name']),
               :status => 200
             }
           end
@@ -176,7 +176,7 @@ module Heroku
 
         with_mock_app(mock_data, app) do |app_data|
           {
-            :body   => Heroku::API::OkJson.encode({}),
+            :body   => Heroku::API.json_encode({}),
             :status => 201
           }
         end

data/lib/heroku/api/mock/collaborators.rb

@@ -25,14 +25,14 @@ module Heroku
         app, _ = request_params[:captures][:path]
         with_mock_app(mock_data, app) do
           {
-            :body   => Heroku::API::OkJson.encode(mock_data[:collaborators][app]),
+            :body   => Heroku::API.json_encode(mock_data[:collaborators][app]),
             :status => 200
           }
         end
       end
 
       # stub POST /apps/:app/collaborators
-      Excon.stub(:expects => 200, :method => :post, :path => %r{^/apps/([^/]+)/collaborators}) do |params|
+      Excon.stub(:expects => [200, 201], :method => :post, :path => %r{^/apps/([^/]+)/collaborators}) do |params|
         request_params, mock_data = parse_stub_params(params)
         app, _ = request_params[:captures][:path]
         email = request_params[:query]['collaborator[email]']
@@ -40,7 +40,7 @@ module Heroku
           mock_data[:collaborators][app] |= [{'access' => 'edit', 'email' => email, 'name' => nil}]
           {
             :body   => "#{email} added as a collaborator on #{app}.",
-            :status => 200
+            :status => 201
           }
           # Existing user response
           #{

data/lib/heroku/api/mock/config_vars.rb

@@ -10,7 +10,7 @@ module Heroku
           mock_data[:config_vars][app].delete(key)
           add_mock_release(mock_data, app, {'descr' => "Config remove #{key}"})
           {
-            :body   => Heroku::API::OkJson.encode(mock_data[:config_vars][app]),
+            :body   => Heroku::API.json_encode(mock_data[:config_vars][app]),
             :status => 200
           }
         end
@@ -22,7 +22,7 @@ module Heroku
         app, _ = request_params[:captures][:path]
         with_mock_app(mock_data, app) do
           {
-            :body   => Heroku::API::OkJson.encode(mock_data[:config_vars][app]),
+            :body   => Heroku::API.json_encode(mock_data[:config_vars][app]),
             :status => 200
           }
         end
@@ -37,7 +37,7 @@ module Heroku
           mock_data[:config_vars][app].merge!(new_config_vars)
           add_mock_release(mock_data, app, {'descr' => "Config add #{new_config_vars.keys.join(', ')}"})
           {
-            :body   => Heroku::API::OkJson.encode(mock_data[:config_vars][app]),
+            :body   => Heroku::API.json_encode(mock_data[:config_vars][app]),
             :status => 200
           }
         end

data/lib/heroku/api/mock/domains.rb

@@ -42,7 +42,7 @@ module Heroku
         app, _ = request_params[:captures][:path]
         with_mock_app(mock_data, app) do |app_data|
           {
-            :body   => Heroku::API::OkJson.encode(mock_data[:domains][app]),
+            :body   => Heroku::API.json_encode(mock_data[:domains][app]),
             :status => 200
           }
         end
@@ -66,7 +66,7 @@ module Heroku
             }
           end
           {
-            :body   => Heroku::API::OkJson.encode('domain' => domain),
+            :body   => Heroku::API.json_encode('domain' => domain),
             :status => 201
           }
         end

data/lib/heroku/api/mock/features.rb

@@ -16,20 +16,20 @@ module Heroku
             when 'app'
               mock_data[:features][:app][app].delete(feature_data)
               {
-                :body   => Heroku::API::OkJson.encode(feature_data.merge('enabled' => false)),
+                :body   => Heroku::API.json_encode(feature_data.merge('enabled' => false)),
                 :status => 200
               }
             when 'user'
               mock_data[:features][:user].delete(feature_data.merge('enabled' => false))
               {
-                :body   => Heroku::API::OkJson.encode(feature_data),
+                :body   => Heroku::API.json_encode(feature_data),
                 :status => 200
               }
             end
           else
             # feature does not exist
             {
-              :body   => Heroku::API::OkJson.encode({'error' => "Feature not found."}),
+              :body   => Heroku::API.json_encode({'error' => "Feature not found."}),
               :status => 404
             }
           end
@@ -56,13 +56,13 @@ module Heroku
         feature, _ = request_params[:captures][:path]
         if feature_data = get_mock_feature(mock_data, feature)
           {
-            :body   => Heroku::API::OkJson.encode(feature_data),
+            :body   => Heroku::API.json_encode(feature_data),
             :status => 200
           }
         else
           # feature does not exist
           {
-            :body   => Heroku::API::OkJson.encode({'error' => "Feature not found."}),
+            :body   => Heroku::API.json_encode({'error' => "Feature not found."}),
             :status => 404
           }
         end
@@ -82,7 +82,7 @@ module Heroku
             when 'app'
               mock_data[:features][:app][app] << feature_data
               {
-                :body   => Heroku::API::OkJson.encode(feature_data),
+                :body   => Heroku::API.json_encode(feature_data),
                 :status => 200
               }
             when 'user'
@@ -95,7 +95,7 @@ module Heroku
           else
             # feature does not exist
             {
-              :body   => Heroku::API::OkJson.encode({'error' => "Feature not found."}),
+              :body   => Heroku::API.json_encode({'error' => "Feature not found."}),
               :status => 404
             }
           end

data/lib/heroku/api/mock/keys.rb

@@ -26,7 +26,7 @@ module Heroku
       Excon.stub(:expects => 200, :method => :get, :path => %r{^/user/keys}) do |params|
         request_params, mock_data = parse_stub_params(params)
         {
-          :body   => Heroku::API::OkJson.encode(mock_data[:keys]),
+          :body   => Heroku::API.json_encode(mock_data[:keys]),
           :status => 200
         }
       end

data/lib/heroku/api/mock/processes.rb

@@ -8,7 +8,7 @@ module Heroku
       app, _ = request_params[:captures][:path]
       with_mock_app(mock_data, app) do |app_data|
         {
-          :body   => Heroku::API::OkJson.encode(get_mock_processes(mock_data, app)),
+          :body   => Heroku::API.json_encode(get_mock_processes(mock_data, app)),
           :status => 200
         }
       end
@@ -43,7 +43,7 @@ module Heroku
         }
         mock_data[:ps][app] << data
         {
-          :body   => Heroku::API::OkJson.encode(data),
+          :body   => Heroku::API.json_encode(data),
           :status => 200,
         }
       end
@@ -117,13 +117,13 @@ module Heroku
             }
           else
             {
-              :body   => Heroku::API::OkJson.encode('error' => "No such type as #{type}") ,
+              :body   => Heroku::API.json_encode('error' => "No such type as #{type}") ,
               :status => 422
             }
           end
         else
           {
-            :body   => Heroku::API::OkJson.encode('error' => "That feature is not available on this app's stack"),
+            :body   => Heroku::API.json_encode('error' => "That feature is not available on this app's stack"),
             :status => 422
           }
         end
@@ -139,7 +139,7 @@ module Heroku
         type  = request_params[:query].has_key?('type') && request_params[:query]['type']
         if !ps && !type
           {
-            :body   => Heroku::API::OkJson.encode({'error' => 'Missing process argument'}),
+            :body   => Heroku::API.json_encode({'error' => 'Missing process argument'}),
             :status => 422
           }
         else
@@ -160,12 +160,12 @@ module Heroku
         unless app_data['stack'] == 'cedar'
           app_data['dynos'] = dynos
           {
-            :body   => Heroku::API::OkJson.encode({'name' => app, 'dynos' => dynos}),
+            :body   => Heroku::API.json_encode({'name' => app, 'dynos' => dynos}),
             :status => 200
           }
         else
           {
-            :body   => Heroku::API::OkJson.encode({'error' => "For Cedar apps, use `heroku scale web=#{dynos}`"}),
+            :body   => Heroku::API.json_encode({'error' => "For Cedar apps, use `heroku scale web=#{dynos}`"}),
             :status => 422
           }
         end
@@ -181,12 +181,12 @@ module Heroku
         unless app_data['stack'] == 'cedar'
           app_data['workers'] = workers
           {
-            :body   => Heroku::API::OkJson.encode({'name' => app, 'workers' => workers}),
+            :body   => Heroku::API.json_encode({'name' => app, 'workers' => workers}),
             :status => 200
           }
         else
           {
-            :body   => Heroku::API::OkJson.encode({'error' => "For Cedar apps, use `heroku scale worker=#{workers}`"}),
+            :body   => Heroku::API.json_encode({'error' => "For Cedar apps, use `heroku scale worker=#{workers}`"}),
             :status => 422
           }
         end

data/lib/heroku/api/mock/releases.rb

@@ -8,7 +8,7 @@ module Heroku
         app, _ = request_params[:captures][:path]
         with_mock_app(mock_data, app) do |app_data|
           {
-            :body   => Heroku::API::OkJson.encode(mock_data[:releases][app]),
+            :body   => Heroku::API.json_encode(mock_data[:releases][app]),
             :status => 200
           }
         end
@@ -22,7 +22,7 @@ module Heroku
           releases = mock_data[:releases][app]
           if release_data = (release_name == 'current' && releases.last) || releases.detect {|release| release['name'] == release_name}
             {
-              :body   => Heroku::API::OkJson.encode(release_data),
+              :body   => Heroku::API.json_encode(release_data),
               :status => 200
             }
           else
@@ -51,7 +51,7 @@ module Heroku
               }
             else
               {
-                :body   => Heroku::API::OkJson.encode({'error' => 'Cannot rollback to a release that had a different set of addons installed'}),
+                :body   => Heroku::API.json_encode({'error' => 'Cannot rollback to a release that had a different set of addons installed'}),
                 :status => 422
               }
             end

data/lib/heroku/api/mock/stacks.rb

@@ -37,7 +37,7 @@ module Heroku
           stack_data = Marshal::load(Marshal.dump(STACKS))
           stack_data.detect {|stack| stack['name'] == app_data['stack']}['current'] = true
           {
-            :body   => Heroku::API::OkJson.encode(stack_data),
+            :body   => Heroku::API.json_encode(stack_data),
             :status => 200
           }
         end
@@ -65,13 +65,13 @@ BODY
               }
             else
               {
-                :body   => Heroku::API::OkJson.encode('error' => 'Stack not found'),
+                :body   => Heroku::API.json_encode('error' => 'Stack not found'),
                 :status => 404
               }
             end
           else
             {
-              :body   => Heroku::API::OkJson.encode('error' => 'Stack migration to/from Cedar is not available. Create a new app with --stack cedar instead.'),
+              :body   => Heroku::API.json_encode('error' => 'Stack migration to/from Cedar is not available. Create a new app with --stack cedar instead.'),
               :status => 422
             }
           end

data/lib/heroku/api/version.rb

@@ -1,5 +1,5 @@
 module Heroku
   class API
-    VERSION = "0.2.6"
+    VERSION = "0.2.7"
   end
 end

data/test/test_addons.rb

@@ -43,7 +43,7 @@ class TestAddons < MiniTest::Unit::TestCase
     data = File.read("#{File.dirname(__FILE__)}/../lib/heroku/api/mock/cache/get_addons.json")
 
     assert_equal(200, response.status)
-    assert_equal(Heroku::API::OkJson.decode(data), response.body)
+    assert_equal(Heroku::API.json_decode(data), response.body)
   end
 
   def test_get_addons_with_app

data/test/test_collaborators.rb

@@ -54,7 +54,7 @@ class TestCollaborators < MiniTest::Unit::TestCase
       email_address = 'wesley@heroku.com'
       response = heroku.post_collaborator(app_data['name'], email_address)
 
-      assert_equal(200, response.status)
+      assert_equal(201, response.status)
       assert_equal(
         "#{email_address} added as a collaborator on #{app_data['name']}.",
         response.body

data/test/test_features.rb

@@ -5,7 +5,7 @@ class TestFeatures < MiniTest::Unit::TestCase
   def setup
     @feature_data ||= begin
       data = File.read("#{File.dirname(__FILE__)}/../lib/heroku/api/mock/cache/get_features.json")
-      features_data = Heroku::API::OkJson.decode(data)
+      features_data = Heroku::API.json_decode(data)
       features_data.detect {|feature| feature['name'] == 'user_env_compile'}
     end
   end
@@ -40,7 +40,7 @@ class TestFeatures < MiniTest::Unit::TestCase
       data = File.read("#{File.dirname(__FILE__)}/../lib/heroku/api/mock/cache/get_features.json")
 
       assert_equal(200, response.status)
-      assert_equal(Heroku::API::OkJson.decode(data), response.body)
+      assert_equal(Heroku::API.json_decode(data), response.body)
     end
   end
 

data/test/test_user.rb

@@ -7,7 +7,7 @@ class TestUser < MiniTest::Unit::TestCase
     data = File.read("#{File.dirname(__FILE__)}/../lib/heroku/api/mock/cache/get_user.json")
 
     assert_equal(200, response.status)
-    assert_equal(Heroku::API::OkJson.decode(data), response.body)
+    assert_equal(Heroku::API.json_decode(data), response.body)
   end
 
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: heroku-api
 version: !ruby/object:Gem::Version
-  version: 0.2.6
+  version: 0.2.7
   prerelease: 
 platform: ruby
 authors:
@@ -9,22 +9,22 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-06-21 00:00:00.000000000 Z
+date: 2012-07-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: excon
-  requirement: &70281784396440 !ruby/object:Gem::Requirement
+  requirement: &70181884166000 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.14.0
+        version: 0.14.3
   type: :runtime
   prerelease: false
-  version_requirements: *70281784396440
+  version_requirements: *70181884166000
 - !ruby/object:Gem::Dependency
   name: minitest
-  requirement: &70281784395620 !ruby/object:Gem::Requirement
+  requirement: &70181884165300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70281784395620
+  version_requirements: *70181884165300
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70281784394080 !ruby/object:Gem::Requirement
+  requirement: &70181884164420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,7 +43,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70281784394080
+  version_requirements: *70181884164420
 description: Ruby Client for the Heroku API
 email:
 - wesley@heroku.com
@@ -121,7 +121,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2230486657400776682
+      hash: 3317431445359256884
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -130,7 +130,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2230486657400776682
+      hash: 3317431445359256884
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.15