brightbox-cli 1.1.0 → 1.2.0

data/spec/cassettes/brightbox_database-snapshots/show/when_resource_exists/does_not_output_to_stderr.yml

@@ -0,0 +1,93 @@
+---
+http_interactions:
+- request:
+    method: get
+    uri: http://api.brightbox.dev/1.0/database_snapshots/dbi-12345?account_id=acc-12345
+    body:
+      encoding: US-ASCII
+      string: ''
+    headers:
+      User-Agent:
+      - fog/1.19.0
+      Authorization:
+      - OAuth f83da712e6299cda953513ec07f7a754f747d727
+      Content-Type:
+      - application/json
+  response:
+    status:
+      code: 200
+      message:
+    headers:
+      Content-Type:
+      - application/json; charset=utf-8
+      X-Ua-Compatible:
+      - IE=Edge
+      Etag:
+      - "\"fac2ed3edb567b9281616c1bdc334a1a\""
+      Cache-Control:
+      - max-age=0, private, must-revalidate
+      X-Request-Id:
+      - 5514a55df99fee2c5f631f7ff8acfe3d
+      X-Runtime:
+      - '0.205251'
+      Server:
+      - WEBrick/1.3.1 (Ruby/1.9.3/2013-11-22)
+      Date:
+      - Fri, 17 Jan 2014 10:45:59 GMT
+      Content-Length:
+      - '449'
+      Connection:
+      - Keep-Alive
+    body:
+      encoding: UTF-8
+      string: "{\"id\":\"dbi-12345\",\"resource_type\":\"database_snapshot\",\"url\":\"https://api.gb1.brightbox.com/1.0/database_snapshots/dbi-12345\",\"name\":\"\",\"description\":\"\",\"status\":\"active\",\"database_version\":\"5.5\",\"size\":0,\"created_at\":\"2014-01-17T10:23:00Z\",\"updated_at\":\"2014-01-17T10:23:00Z\",\"deleted_at\":null,\"account\":{\"id\":\"acc-12345\",\"resource_type\":\"account\",\"url\":\"https://api.gb1.brightbox.com/1.0/accounts/acc-12345\",\"name\":\"VCR
+        account\",\"status\":\"active\"}}"
+    http_version:
+  recorded_at: Fri, 17 Jan 2014 10:45:59 GMT
+- request:
+    method: post
+    uri: http://api.brightbox.dev/token
+    body:
+      encoding: UTF-8
+      string: "{\"grant_type\":\"none\",\"client_id\":\"app-12345\"}"
+    headers:
+      User-Agent:
+      - fog/1.20.0
+      Authorization:
+      - Basic YXBwLTEyMzQ1Om1vY2J1aXBiaWFhNms2Yw==
+      Content-Type:
+      - application/json
+  response:
+    status:
+      code: 400
+      message:
+    headers:
+      X-Frame-Options:
+      - sameorigin
+      X-Xss-Protection:
+      - 1; mode=block
+      Content-Type:
+      - application/json;charset=utf-8
+      Cache-Control:
+      - no-store
+      Content-Length:
+      - '135'
+      X-Ua-Compatible:
+      - IE=Edge
+      X-Request-Id:
+      - 3223730ff4c4c65e052fc92b95171647
+      X-Runtime:
+      - '0.102349'
+      Server:
+      - WEBrick/1.3.1 (Ruby/1.9.3/2013-11-22)
+      Date:
+      - Tue, 18 Feb 2014 17:30:27 GMT
+      Connection:
+      - close
+    body:
+      encoding: UTF-8
+      string: "{\"error\":\"unauthorized_client\",\"error_description\":\"The authenticated
+        client is not authorized to use the access grant type provided.\"}"
+    http_version:
+  recorded_at: Tue, 18 Feb 2014 17:30:27 GMT
+recorded_with: VCR 2.5.0

data/spec/commands/accounts/list_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe "brightbox accounts" do
 
-  describe "list", :broken_1_8 do
+  describe "list" do
     let(:output) { FauxIO.new { Brightbox::run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }

data/spec/commands/cloudips/map_spec.rb

@@ -7,11 +7,30 @@ describe "brightbox cloudips" do
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
-    context "" do
-      let(:argv) { ["cloudips", "map"] }
+    before do
+      config = config_from_contents(USER_APP_CONFIG_CONTENTS)
 
-      it "does not error" do
-        expect { output }.to_not raise_error
+      # Setup in the VCR recordings
+      cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
+    end
+
+    context "when destination is a server ID", :vcr do
+      let(:argv) { %w{cloudips map cip-12345 srv-12345} }
+      let(:target) { "int-12345" }
+
+      it "passes the interface identifier to the API" do
+        expect_any_instance_of(Brightbox::CloudIP).to receive(:map).with(target).and_call_original
+        expect(stdout).to include("mapped")
+      end
+    end
+
+    context "when destination is another value", :vcr do
+      let(:target) { "res-12345" }
+      let(:argv) { %w{cloudips map cip-12345 res-12345} }
+
+      it "passes the identifier to the API" do
+        expect_any_instance_of(Brightbox::CloudIP).to receive(:map).with(target)
+        expect(stderr).to include(target)
       end
     end
   end

data/spec/commands/config/user_add_spec.rb

@@ -6,7 +6,7 @@ require "spec_helper"
 #
 describe "brightbox config" do
 
-  describe "user_add", :broken_1_8 do
+  describe "user_add" do
     let(:output) { FauxIO.new { Brightbox::run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }

data/spec/commands/sql/instances/create_spec.rb

@@ -0,0 +1,69 @@
+require "spec_helper"
+
+describe "brightbox database-servers" do
+  describe "create" do
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
+    let(:stdout) { output.stdout }
+    let(:stderr) { output.stderr }
+
+    before do
+      config = config_from_contents(USER_APP_CONFIG_CONTENTS)
+
+      # Setup in the VCR recordings
+      cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
+    end
+
+    context "without arguments", :vcr do
+      let(:argv) { %w{sql instances create } }
+      let(:expected_password) { "477wwwj48yifrnuk" }
+
+      it "reports the new admin username" do
+        expect(stdout).to include("admin_username: admin")
+      end
+
+      it "reports the new admin password" do
+        expect(stdout).to include("admin_password: #{expected_password}")
+      end
+    end
+
+    context "--allow-access=10.0.0.0", :vcr do
+      let(:argv) { %w{sql instances create --allow-access=10.0.0.0} }
+      let(:expected_args) { { :allow_access => ["10.0.0.0"] } }
+
+      it "correctly sends API parameters" do
+        expect(Brightbox::DatabaseServer).to receive(:create).with(expected_args).and_call_original
+        expect(stderr).to eql("")
+      end
+    end
+
+    context "--allow-access=srv-12345,grp-12345", :vcr do
+      let(:argv) { %w{sql instances create --allow-access srv-12345,grp-12345} }
+      let(:expected_args) { { :allow_access => %w{srv-12345 grp-12345} } }
+
+      it "correctly sends API parameters" do
+        expect(Brightbox::DatabaseServer).to receive(:create).with(expected_args).and_call_original
+        expect(stderr).to eql("")
+      end
+    end
+
+    context "--engine=mysql", :vcr do
+      let(:argv) { %w(sql instances create --engine=mysql) }
+      let(:expected_args) { { :database_engine => "mysql" } }
+
+      it "correctly sends API parameters" do
+        expect(Brightbox::DatabaseServer).to receive(:create).with(expected_args).and_call_original
+        expect(stderr).to eql("")
+      end
+    end
+
+    context "--engine=mysql --engine-version=5.6", :vcr do
+      let(:argv) { %w(sql instances create --engine=mysql --engine-version=5.6) }
+      let(:expected_args) { { :database_engine => "mysql", :database_version => "5.6" } }
+
+      it "correctly sends API parameters" do
+        expect(Brightbox::DatabaseServer).to receive(:create).with(expected_args).and_call_original
+        expect(stderr).to eql("")
+      end
+    end
+  end
+end

data/spec/commands/sql/instances/snapshot_spec.rb

@@ -0,0 +1,36 @@
+require "spec_helper"
+
+describe "brightbox database-servers" do
+  describe "snapshot" do
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
+    let(:stdout) { output.stdout }
+    let(:stderr) { output.stderr }
+
+    before do
+      config = config_from_contents(USER_APP_CONFIG_CONTENTS)
+
+      # Setup in the VCR recordings
+      cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
+    end
+
+    context "when database server active", :vcr do
+      let(:argv) { %w{sql instances snapshot dbs-12345} }
+      let(:expected_args) { { :allow_access => ["10.0.0.0"] } }
+
+      it "correctly sends API parameters" do
+        expect_any_instance_of(Brightbox::DatabaseServer).to receive(:snapshot).and_call_original
+        expect(stderr).to include("Creating snapshot for dbs-12345")
+      end
+    end
+
+    context "when database server can not be snapshotted", :vcr do
+      let(:argv) { %w{sql instances snapshot dbs-12345} }
+
+      it "reports an error to the user" do
+        expect_any_instance_of(Brightbox::DatabaseServer).to receive(:snapshot).and_call_original
+        expect(stderr).to include("Creating snapshot for dbs-12345")
+        expect(stderr).to include("ERROR: Unable to action the request due to conflicting states")
+      end
+    end
+  end
+end

data/spec/commands/sql/snapshots/list_spec.rb

@@ -0,0 +1,29 @@
+require "spec_helper"
+
+describe "brightbox database-snapshots" do
+  describe "list" do
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
+    let(:stdout) { output.stdout }
+    let(:stderr) { output.stderr }
+
+    before do
+      config = config_from_contents(USER_APP_CONFIG_CONTENTS)
+
+      # Setup in the VCR recordings
+      cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
+    end
+
+    context "when resources are available", :vcr do
+      let(:argv) { %w{sql snapshots list} }
+
+      it "does not output to stderr" do
+        expect(stderr).to be_empty
+      end
+
+      it "outputs table details to stdout" do
+        expect(stdout).to_not be_empty
+        expect(stdout).to match(/dbi-\d+/)
+      end
+    end
+  end
+end

data/spec/commands/sql/snapshots/show_spec.rb

@@ -0,0 +1,29 @@
+require "spec_helper"
+
+describe "brightbox database-snapshots" do
+  describe "show" do
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
+    let(:stdout) { output.stdout }
+    let(:stderr) { output.stderr }
+
+    before do
+      config = config_from_contents(USER_APP_CONFIG_CONTENTS)
+
+      # Setup in the VCR recordings
+      cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
+    end
+
+    context "when resource exists", :vcr do
+      let(:argv) { %w{sql snapshots show dbi-12345} }
+
+      it "does not output to stderr" do
+        expect(stderr).to be_empty
+      end
+
+      it "outputs table details to stdout" do
+        expect(stdout).to_not be_empty
+        expect(stdout).to match(/dbi-12345/)
+      end
+    end
+  end
+end

data/spec/commands/sql/types/list_spec.rb

@@ -0,0 +1,18 @@
+require "spec_helper"
+
+describe "brightbox database-types" do
+
+  describe "list" do
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
+    let(:stdout) { output.stdout }
+    let(:stderr) { output.stderr }
+
+    context "" do
+      let(:argv) { %w{sql types list} }
+
+      it "does not error" do
+        expect { output }.to_not raise_error
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -29,9 +29,7 @@ RSpec.configure do |config|
   config.include TokenHelpers
   config.include PasswordPromptHelpers
 
-  if RUBY_VERSION >= "1.8.7"
-    config.filter_run_excluding :broken_1_8 => true
-  end
+  config.filter_run_excluding :vcr if RUBY_VERSION <= "1.8.7"
 
   # For each test, point to the testing endpoint to make it safer and easier to
   # record from dev endpoints. Devs can DNS api.brightbox.dev to their dev service

data/spec/support/vcr.rb

@@ -11,8 +11,8 @@ VCR.configure do |vcr|
 
   vcr.configure_rspec_metadata!
   vcr.default_cassette_options = {
-    #:record => :new_episodes,
     :record => :none,
+    #:record => :new_episodes,
     :match_requests_on => [:method, :path, :body],
     :update_content_length_header => true
   }

data/spec/unit/brightbox/api/conn_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe Brightbox::Api, ".conn" do
 
-  context "when account is not required" do
+  context "when account is not required", :vcr do
     it "returns a 'real' fog compute instance" do
       expect(Brightbox::Api.conn).to be_instance_of(Fog::Compute::Brightbox::Real)
     end

data/spec/unit/brightbox/bb_config/add_section_spec.rb

@@ -3,7 +3,7 @@ require "spec_helper"
 describe Brightbox::BBConfig do
 
   describe "#add_section" do
-    context "when first and only client", :vcr, :broken_1_8 do
+    context "when first and only client", :vcr do
       let(:config) { Brightbox::BBConfig.new }
       let(:client_alias) { "dev" }
       let(:client_id) { "app-12345" }

data/spec/unit/brightbox/bb_config/renew_tokens_spec.rb

@@ -11,7 +11,7 @@ describe Brightbox::BBConfig do
   let(:cli_id) { "cli-12345" }
   let(:cli_secret) { "qy6xxnvy4o0tgv5" }
 
-  describe "#renew_tokens", :broken_1_8 do
+  describe "#renew_tokens" do
     context "when using a user app with no tokens", :vcr do
       before do
         contents = <<-EOS

data/spec/unit/brightbox/database_server_spec.rb

@@ -0,0 +1,9 @@
+require "spec_helper"
+
+describe Brightbox::DatabaseServer do
+  subject(:database_server) do
+    Brightbox::DatabaseServer.new("dbs-12345")
+  end
+
+  it_behaves_like "a wrapped API resource"
+end

data/spec/unit/brightbox/database_snapshot_spec.rb

@@ -0,0 +1,9 @@
+require "spec_helper"
+
+describe Brightbox::DatabaseSnapshot do
+  subject(:database_snapshot) do
+    Brightbox::DatabaseSnapshot.new("dbi-12345")
+  end
+
+  it_behaves_like "a wrapped API resource"
+end

data/spec/unit/brightbox/database_type_spec.rb

@@ -0,0 +1,9 @@
+require "spec_helper"
+
+describe Brightbox::DatabaseType do
+  subject(:database_type) do
+    Brightbox::DatabaseType.new("dbt-12345")
+  end
+
+  it_behaves_like "a wrapped API resource"
+end

data/spec/unit/brightbox/firewall_policy/create_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe Brightbox::FirewallPolicy do
 
-  describe "#create", :vcr, :broken_1_8 do
+  describe "#create", :vcr do
 
     it "should create firewall policy" do
       params = { :name => "rspec_tests"}

data/spec/unit/brightbox/firewall_policy/destroy_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe Brightbox::FirewallPolicy do
 
-  describe "#destroy", :vcr, :broken_1_8 do
+  describe "#destroy", :vcr do
     it "should destroy firewall policy" do
       params = { :name => "rspec_tests"}
       @group = Brightbox::ServerGroup.create(params)