pogo 2.31.2

data/spec/heroku/command/sharing_spec.rb

@@ -0,0 +1,59 @@
+require "spec_helper"
+require "heroku/command/sharing"
+
+module Heroku::Command
+  describe Sharing do
+
+    before(:each) do
+      stub_core
+      api.post_app("name" => "myapp")
+    end
+
+    after(:each) do
+      api.delete_app("myapp")
+    end
+
+    context("list") do
+
+      it "lists collaborators" do
+        api.post_collaborator("myapp", "collaborator@example.com")
+        stderr, stdout = execute("sharing")
+        stderr.should == ""
+        stdout.should == <<-STDOUT
+=== myapp Collaborators
+collaborator@example.com
+email@example.com
+
+STDOUT
+      end
+
+    end
+
+    it "adds collaborators with default access to view only" do
+      stderr, stdout = execute("sharing:add collaborator@example.com")
+      stderr.should == ""
+      stdout.should == <<-STDOUT
+Adding collaborator@example.com to myapp collaborators... done
+STDOUT
+    end
+
+    it "removes collaborators" do
+      api.post_collaborator("myapp", "collaborator@example.com")
+      stderr, stdout = execute("sharing:remove collaborator@example.com")
+      stderr.should == ""
+      stdout.should == <<-STDOUT
+Removing collaborator@example.com from myapp collaborators... done
+STDOUT
+    end
+
+    it "transfers ownership" do
+      api.post_collaborator("myapp", "collaborator@example.com")
+      stderr, stdout = execute("sharing:transfer collaborator@example.com")
+      stderr.should == ""
+      stdout.should == <<-STDOUT
+Transferring myapp to collaborator@example.com... done
+STDOUT
+    end
+  end
+
+end

data/spec/heroku/command/ssl_spec.rb

@@ -0,0 +1,32 @@
+require "spec_helper"
+require "heroku/command/ssl"
+
+module Heroku::Command
+  describe Ssl do
+
+    it "adds ssl certificates to domains" do
+      File.should_receive(:exists?).with('.git').and_return(false)
+      File.should_receive(:exists?).with('my.crt').and_return(true)
+      File.should_receive(:read).with('my.crt').and_return('crt contents')
+      File.should_receive(:exists?).with('my.key').and_return(true)
+      File.should_receive(:read).with('my.key').and_return('key contents')
+      expires_at = Time.now + 60 * 60 * 24 * 365
+      stub_core.add_ssl('myapp', 'crt contents', 'key contents').returns({"domain" => "example.com", "expires_at" => expires_at})
+      stderr, stdout = execute("ssl:add my.crt my.key")
+      stderr.should == ""
+      stdout.should == <<-STDOUT
+Added certificate to example.com, expiring at #{expires_at}
+STDOUT
+    end
+
+    it "removes certificates" do
+      stub_core.remove_ssl('myapp', 'example.com')
+      stderr, stdout = execute("ssl:remove example.com")
+      stderr.should == ""
+      stdout.should == <<-STDOUT
+Removed certificate from example.com
+STDOUT
+    end
+
+  end
+end

data/spec/heroku/command/stack_spec.rb

@@ -0,0 +1,46 @@
+require "spec_helper"
+require "heroku/command/stack"
+
+module Heroku::Command
+  describe Stack do
+    describe "index" do
+      before(:each) do
+        stub_core
+        api.post_app("name" => "myapp", "stack" => "bamboo-mri-1.9.2")
+      end
+
+      after(:each) do
+        api.delete_app("myapp")
+      end
+
+      it "index should provide list" do
+        stderr, stdout = execute("stack")
+        stderr.should == ""
+        stdout.should == <<-STDOUT
+=== myapp Available Stacks
+  aspen-mri-1.8.6
+  bamboo-ree-1.8.7
+  cedar (beta)
+* bamboo-mri-1.9.2
+
+STDOUT
+      end
+
+      it "migrate should succeed" do
+        stderr, stdout = execute("stack:migrate bamboo-ree-1.8.7")
+        stderr.should == ""
+        stdout.should == <<-STDOUT
+-----> Preparing to migrate myapp
+       bamboo-mri-1.9.2 -> bamboo-ree-1.8.7
+
+       NOTE: Additional details here
+
+       -----> Migration prepared.
+       Run 'git push heroku master' to execute migration.
+STDOUT
+      end
+
+
+    end
+  end
+end

data/spec/heroku/command/status_spec.rb

@@ -0,0 +1,48 @@
+require "spec_helper"
+require "heroku/command/status"
+
+module Heroku::Command
+  describe Status do
+
+    before(:each) do
+      stub_core
+    end
+
+    it "displays status information" do
+      Excon.stub(
+        {
+          :host   => 'status.heroku.com',
+          :method => :get,
+          :path   => '/api/v3/current-status.json'
+        },
+        {
+          :body   => Heroku::API::OkJson.encode({"status"=>{"Production"=>"red", "Development"=>"red"}, "issues"=>[{"created_at"=>"2012-06-07T15:55:51Z", "id"=>372, "resolved"=>false, "title"=>"HTTP Routing Errors", "updated_at"=>"2012-06-07T16:14:37Z", "href"=>"https://status.heroku.com/api/v3/issues/372", "updates"=>[{"contents"=>"The number of applications seeing H99 errors is continuing to decrease as we continue to work toward a full resolution of the HTTP routing issues. The API is back online now as well. ", "created_at"=>"2012-06-07T17:47:26Z", "id"=>1088, "incident_id"=>372, "status_dev"=>"red", "status_prod"=>"red", "update_type"=>"update", "updated_at"=>"2012-06-07T17:47:26Z"}, {"contents"=>"Our engineers are continuing to work toward a full resolution of the HTTP routing issues. The API is currently in maintenance mode intentionally as we restore application operations. ", "created_at"=>"2012-06-07T17:16:40Z", "id"=>1086, "incident_id"=>372, "status_dev"=>"red", "status_prod"=>"red", "update_type"=>"update", "updated_at"=>"2012-06-07T17:26:55Z"}, {"contents"=>"Most applications are back online at this time. Our engineers are working on getting the remaining apps back online. ", "created_at"=>"2012-06-07T16:50:21Z", "id"=>1085, "incident_id"=>372, "status_dev"=>"red", "status_prod"=>"red", "update_type"=>"update", "updated_at"=>"2012-06-07T16:50:21Z"}, {"contents"=>"Our routing engineers have pushed out a patch to our routing tier. The platform is recovering and applications are coming back online. Our engineers are continuing to fully restore service.", "created_at"=>"2012-06-07T16:36:37Z", "id"=>1084, "incident_id"=>372, "status_dev"=>"red", "status_prod"=>"red", "update_type"=>"update", "updated_at"=>"2012-06-07T16:36:37Z"}, {"contents"=>"We have identified an issue with our routers that is causing errors on HTTP requests to applications. Engineers are working to resolve the issue.\r\n", "created_at"=>"2012-06-07T16:15:25Z", "id"=>1083, "incident_id"=>372, "status_dev"=>"red", "status_prod"=>"red", "update_type"=>"update", "updated_at"=>"2012-06-07T16:15:28Z"}, {"contents"=>"We have confirmed widespread errors on the platform. Our engineers are continuing to investigate.\r\n", "created_at"=>"2012-06-07T15:58:56Z", "id"=>1082, "incident_id"=>372, "status_dev"=>"red", "status_prod"=>"red", "update_type"=>"update", "updated_at"=>"2012-06-07T15:58:58Z"}, {"contents"=>"Our automated systems have detected potential platform errors.  We are investigating.\r\n", "created_at"=>"2012-06-07T15:55:51Z", "id"=>1081, "incident_id"=>372, "status_dev"=>"yellow", "status_prod"=>"yellow", "update_type"=>"issue", "updated_at"=>"2012-06-07T15:55:55Z"}]}]}),
+          :status => 200
+        }
+      )
+
+      Heroku::Command::Status.any_instance.should_receive(:time_ago).and_return('2012/09/11 09:34:56 (~ 3h ago)', '2012/09/11 12:33:56 (~ 1m ago)', '2012/09/11 12:29:56 (~ 5m ago)', '2012/09/11 12:24:56 (~ 10m ago)', '2012/09/11 12:04:56 (~ 30m ago)', '2012/09/11 11:34:56 (~ 1h ago)', '2012/09/11 10:34:56 (~ 2h ago)', '2012/09/11 09:34:56 (~ 3h ago)')
+
+      stderr, stdout = execute("status")
+      stderr.should == ''
+      stdout.should == <<-STDOUT
+=== Heroku Status
+Development: red
+Production:  red
+
+=== HTTP Routing Errors  2012/09/11 09:34:56 (~ 3h+)
+2012/09/11 12:33:56 (~ 1m ago)   update  The number of applications seeing H99 errors is continuing to decrease as we continue to work toward a full resolution of the HTTP routing issues. The API is back online now as well.
+2012/09/11 12:29:56 (~ 5m ago)   update  Our engineers are continuing to work toward a full resolution of the HTTP routing issues. The API is currently in maintenance mode intentionally as we restore application operations.
+2012/09/11 12:24:56 (~ 10m ago)  update  Most applications are back online at this time. Our engineers are working on getting the remaining apps back online.
+2012/09/11 12:04:56 (~ 30m ago)  update  Our routing engineers have pushed out a patch to our routing tier. The platform is recovering and applications are coming back online. Our engineers are continuing to fully restore service.
+2012/09/11 11:34:56 (~ 1h ago)   update  We have identified an issue with our routers that is causing errors on HTTP requests to applications. Engineers are working to resolve the issue.
+2012/09/11 10:34:56 (~ 2h ago)   update  We have confirmed widespread errors on the platform. Our engineers are continuing to investigate.
+2012/09/11 09:34:56 (~ 3h ago)   issue   Our automated systems have detected potential platform errors.  We are investigating.
+
+STDOUT
+
+      Excon.stubs.shift
+    end
+
+  end
+end

data/spec/heroku/command/version_spec.rb

@@ -0,0 +1,16 @@
+require "spec_helper"
+require "heroku/command/version"
+
+module Heroku::Command
+  describe Version do
+
+    it "shows version info" do
+      stderr, stdout = execute("version")
+      stderr.should == ""
+      stdout.should == <<-STDOUT
+#{Heroku.user_agent}
+STDOUT
+    end
+
+  end
+end

data/spec/heroku/command_spec.rb

@@ -0,0 +1,211 @@
+require "spec_helper"
+require "heroku/command"
+require 'json' #FOR WEBMOCK
+
+class FakeResponse
+
+  attr_accessor :body, :headers
+
+  def initialize(attributes)
+    self.body, self.headers = attributes[:body], attributes[:headers]
+  end
+
+  def to_s
+    body
+  end
+
+end
+
+describe Heroku::Command do
+  before {
+    Heroku::Command.load
+    stub_core # setup fake auth
+  }
+
+  describe "when the command requires confirmation" do
+
+    let(:response_that_requires_confirmation) do
+      {:status => 423,
+       :headers => { :x_confirmation_required => 'my_addon' },
+       :body => 'terms of service required'}
+    end
+
+    context "when the app is unknown" do
+      context "and the user includes --confirm APP" do
+        it "should set --app to APP and not ask for confirmation" do
+          stub_request(:post, %r{apps/XXX/addons/my_addon$}).
+            with(:body => {:confirm => "XXX"})
+          run "addons:add my_addon --confirm XXX"
+        end
+      end
+
+      context "and the user includes --confirm APP --app APP2" do
+        it "should warn that the app and confirm do not match and not continue" do
+          capture_stderr do
+            run "addons:add my_addon --confirm APP --app APP2"
+          end.should == " !    Mismatch between --app and --confirm\n"
+        end
+      end
+    end
+
+    context "and the app is known" do
+      before do
+        any_instance_of(Heroku::Command::Base) do |base|
+          stub(base).app.returns("myapp")
+        end
+      end
+
+      context "and the user includes --confirm WRONGAPP" do
+        it "should not allow include the option" do
+          stub_request(:post, %r{apps/myapp/addons/my_addon$}).
+            with(:body => "")
+          run "addons:add my_addon --confirm XXX"
+        end
+      end
+
+      context "and the user includes --confirm APP" do
+        it "should set --app to APP and not ask for confirmation" do
+          stub_request(:post, %r{apps/myapp/addons/my_addon$}).
+            with(:body => {:confirm => 'myapp'})
+
+          run "addons:add my_addon --confirm myapp"
+        end
+      end
+
+      context "and the user didn't include a confirm flag" do
+        it "should ask the user for confirmation" do
+          stub(Heroku::Command).confirm_command.returns(true)
+          stub_request(:post, %r{apps/myapp/addons/my_addon$}).
+            to_return(response_that_requires_confirmation).then.
+            to_return({:status => 200})
+
+          run "addons:add my_addon"
+        end
+
+        it "should not continue if the confirmation does not match" do
+          Heroku::Command.stub(:current_options).and_return(:confirm => 'not_myapp')
+
+          lambda do
+            Heroku::Command.confirm_command('myapp')
+          end.should raise_error(Heroku::Command::CommandFailed)
+        end
+
+        it "should not continue if the user doesn't confirm" do
+          stub(Heroku::Command).confirm_command.returns(false)
+          stub_request(:post, %r{apps/myapp/addons/my_addon$}).
+            to_return(response_that_requires_confirmation).then.
+            to_raise(Heroku::Command::CommandFailed)
+
+          run "addons:add my_addon"
+        end
+      end
+    end
+  end
+
+  describe "parsing errors" do
+    it "extracts error messages from response when available in XML" do
+      Heroku::Command.extract_error('<errors><error>Invalid app name</error></errors>').should == 'Invalid app name'
+    end
+
+    it "extracts error messages from response when available in JSON" do
+      Heroku::Command.extract_error("{\"error\":\"Invalid app name\"}").should == 'Invalid app name'
+    end
+
+    it "extracts error messages from response when available in plain text" do
+      response = FakeResponse.new(:body => "Invalid app name", :headers => { :content_type => "text/plain; charset=UTF8" })
+      Heroku::Command.extract_error(response).should == 'Invalid app name'
+    end
+
+    it "shows Internal Server Error when the response doesn't contain a XML or JSON" do
+      Heroku::Command.extract_error('<h1>HTTP 500</h1>').should == "Internal server error.\nRun 'heroku status' to check for known platform issues."
+    end
+
+    it "shows Internal Server Error when the response is not plain text" do
+      response = FakeResponse.new(:body => "Foobar", :headers => { :content_type => "application/xml" })
+      Heroku::Command.extract_error(response).should == "Internal server error.\nRun 'heroku status' to check for known platform issues."
+    end
+
+    it "allows a block to redefine the default error" do
+      Heroku::Command.extract_error("Foobar") { "Ok!" }.should == 'Ok!'
+    end
+
+    it "doesn't format the response if set to raw" do
+      Heroku::Command.extract_error("Foobar", :raw => true) { "Ok!" }.should == 'Ok!'
+    end
+
+    it "handles a nil body in parse_error_xml" do
+      lambda { Heroku::Command.parse_error_xml(nil) }.should_not raise_error
+    end
+
+    it "handles a nil body in parse_error_json" do
+      lambda { Heroku::Command.parse_error_json(nil) }.should_not raise_error
+    end
+  end
+
+  it "correctly resolves commands" do
+    class Heroku::Command::Test; end
+    class Heroku::Command::Test::Multiple; end
+
+    require "heroku/command/help"
+    require "heroku/command/apps"
+
+    Heroku::Command.parse("unknown").should be_nil
+    Heroku::Command.parse("list").should include(:klass => Heroku::Command::Apps, :method => :index)
+    Heroku::Command.parse("apps").should include(:klass => Heroku::Command::Apps, :method => :index)
+    Heroku::Command.parse("apps:create").should include(:klass => Heroku::Command::Apps, :method => :create)
+  end
+
+  context "help" do
+    it "works as a prefix" do
+      heroku("help ps:scale").should =~ /scale processes by/
+    end
+
+    it "works as an option" do
+      heroku("ps:scale -h").should =~ /scale processes by/
+      heroku("ps:scale --help").should =~ /scale processes by/
+    end
+  end
+
+  context "when no commands match" do
+
+    it "displays the version if --version is used" do
+      heroku("--version").should == <<-STDOUT
+#{Heroku.user_agent}
+STDOUT
+    end
+
+    it "suggests similar commands if there are any" do
+      original_stderr, original_stdout = $stderr, $stdout
+      $stderr = captured_stderr = StringIO.new
+      $stdout = captured_stdout = StringIO.new
+      begin
+        execute("aps")
+      rescue SystemExit
+      end
+      captured_stderr.string.should == <<-STDERR
+ !    `aps` is not a heroku command.
+ !    Perhaps you meant `apps` or `ps`.
+ !    See `heroku help` for a list of available commands.
+STDERR
+      captured_stdout.string.should == ""
+      $stderr, $stdout = original_stderr, original_stdout
+    end
+
+    it "does not suggest similar commands if there are none" do
+      original_stderr, original_stdout = $stderr, $stdout
+      $stderr = captured_stderr = StringIO.new
+      $stdout = captured_stdout = StringIO.new
+      begin
+        execute("sandwich")
+      rescue SystemExit
+      end
+      captured_stderr.string.should == <<-STDERR
+ !    `sandwich` is not a heroku command.
+ !    See `heroku help` for a list of available commands.
+STDERR
+      captured_stdout.string.should == ""
+      $stderr, $stdout = original_stderr, original_stdout
+    end
+
+  end
+end

data/spec/heroku/helpers/heroku_postgresql_spec.rb

@@ -0,0 +1,109 @@
+require "spec_helper"
+require "heroku/helpers/heroku_postgresql"
+
+include Heroku::Helpers::HerokuPostgresql
+
+describe Heroku::Helpers::HerokuPostgresql do
+
+  before do
+    subject.stub(:app_config_vars) { app_config_vars }
+  end
+
+  let(:app_config_vars) do
+    {
+      "DATABASE_URL"                => "postgres://default",
+      "HEROKU_POSTGRESQL_BLACK_URL" => "postgres://black",
+      "HEROKU_POSTGRESQL_IVORY_URL" => "postgres://default",
+      "SHARED_DATABASE_URL"         => "postgres://shared"
+    }
+  end
+
+  it "resolves DATABASE" do
+    subject.hpg_resolve('DATABASE').should == [
+      "HEROKU_POSTGRESQL_IVORY (DATABASE_URL)",
+      "postgres://default"
+    ]
+  end
+
+  it "resolves SHARED_DATABASE" do
+    subject.hpg_resolve('SHARED_DATABASE').should == [
+      "SHARED_DATABASE",
+      "postgres://shared"
+    ]
+  end
+
+  it "resolves default using NAME" do
+    subject.hpg_resolve("IVORY").should == [
+      "HEROKU_POSTGRESQL_IVORY (DATABASE_URL)",
+      "postgres://default"
+    ]
+  end
+
+  it "resolves non-default using NAME" do
+    subject.hpg_resolve("BLACK").should == [
+      "HEROKU_POSTGRESQL_BLACK",
+      "postgres://black"
+    ]
+  end
+
+  it "resolves default using NAME_URL" do
+    subject.hpg_resolve("IVORY_URL").should == [
+      "HEROKU_POSTGRESQL_IVORY (DATABASE_URL)",
+      "postgres://default"
+    ]
+  end
+
+  it "resolves non-default using NAME_URL" do
+    subject.hpg_resolve("BLACK_URL").should == [
+      "HEROKU_POSTGRESQL_BLACK",
+      "postgres://black"
+    ]
+  end
+
+  it "resolves default using lowercase" do
+    subject.hpg_resolve("ivory").should == [
+      "HEROKU_POSTGRESQL_IVORY (DATABASE_URL)",
+      "postgres://default"
+    ]
+  end
+
+  it "resolves non-default using lowercase" do
+    subject.hpg_resolve("black").should == [
+      "HEROKU_POSTGRESQL_BLACK",
+      "postgres://black"
+    ]
+  end
+
+  it "throws an error if it doesnt exist" do
+    subject.should_receive(:error).with("Unknown database: VIOLET. Valid options are: DATABASE, HEROKU_POSTGRESQL_BLACK, HEROKU_POSTGRESQL_IVORY, SHARED_DATABASE")
+    subject.hpg_resolve("violet")
+  end
+
+  context "default" do
+
+    it "errors if there is no default" do
+      subject.should_receive(:error).with("Unknown database. Valid options are: DATABASE, HEROKU_POSTGRESQL_BLACK, HEROKU_POSTGRESQL_IVORY, SHARED_DATABASE")
+      subject.hpg_resolve(nil)
+    end
+
+    it "uses the default if nothing specified" do
+      subject.hpg_resolve(nil, "DATABASE_URL").should == [
+        "HEROKU_POSTGRESQL_IVORY (DATABASE_URL)",
+        "postgres://default"
+      ]
+    end
+
+  end
+
+  context "uri" do
+
+    it "returns the uri directly" do
+      subject.hpg_resolve('postgres://uri', nil).should == [
+        nil,
+        'postgres://uri'
+      ]
+    end
+
+  end
+
+end