brightbox-cli 1.2.2 → 1.3.0

data/spec/commands/rules/update_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox rules" do
 
   describe "update" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["rules", "update"] }
+      let(:argv) { %w(rules update) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/servers/activate_console_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox servers" do
 
   describe "activate_console" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["servers", "activate_console"] }
+      let(:argv) { %w(servers activate_console) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/servers/create_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox servers" do
 
   describe "create" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["servers", "create"] }
+      let(:argv) { %w(servers create) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/servers/destroy_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox servers" do
 
   describe "destroy" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["servers", "destroy"] }
+      let(:argv) { %w(servers destroy) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/servers/list_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox servers" do
 
   describe "list" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["servers", "list"] }
+      let(:argv) { %w(servers list) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/servers/locking_spec.rb

@@ -0,0 +1,73 @@
+require "spec_helper"
+
+describe "brightbox servers" do
+  let(:output) { FauxIO.new { Brightbox.run(argv) } }
+  let(:stdout) { output.stdout }
+  let(:stderr) { output.stderr }
+
+  let(:resource_class) { Brightbox::Server }
+  let(:resource_id) { "srv-12345" }
+  let(:resource) { double(resource_class, :to_s => resource_id) }
+
+  let(:resource_ids) { [resource_id] }
+  let(:results) { [resource] }
+
+  before do
+    config = config_from_contents(USER_APP_CONFIG_CONTENTS)
+    cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
+  end
+
+  describe "lock" do
+    context "when resource is known" do
+      let(:argv) { %w(servers lock srv-12345) }
+
+      before do
+        expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
+        expect(resource).to receive(:lock!)
+      end
+
+      it "outputs notice to STDERR" do
+        expect(stderr).to match("Locking #{resource_id}")
+      end
+    end
+
+    context "when resource is unknown" do
+      let(:argv) { %w(servers lock srv-12345) }
+
+      before do
+        expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)
+      end
+
+      it "outputs error to STDERR" do
+        expect(stderr).to match("Couldn't find #{resource_id}")
+      end
+    end
+  end
+
+  describe "unlock" do
+    context "when resource is known" do
+      let(:argv) { %w(servers unlock srv-12345) }
+
+      before do
+        expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
+        expect(resource).to receive(:unlock!)
+      end
+
+      it "outputs notice to STDERR" do
+        expect(stderr).to match("Unlocking #{resource_id}")
+      end
+    end
+
+    context "when resource is unknown" do
+      let(:argv) { %w(servers unlock srv-12345) }
+
+      before do
+        expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)
+      end
+
+      it "outputs error to STDERR" do
+        expect(stderr).to match("Couldn't find #{resource_id}")
+      end
+    end
+  end
+end

data/spec/commands/servers/reboot_spec.rb

@@ -0,0 +1,55 @@
+require "spec_helper"
+
+describe "brightbox servers" do
+
+  let(:output) { FauxIO.new { Brightbox.run(argv) } }
+  let(:stdout) { output.stdout }
+  let(:stderr) { output.stderr }
+
+  let(:resource_class) { Brightbox::Server }
+  let(:resource_id) { "srv-12345" }
+  let(:resource) { double(resource_class, :to_s => resource_id) }
+
+  let(:resource_ids) { [resource_id] }
+  let(:results) { [resource] }
+
+  before do
+    config = config_from_contents(USER_APP_CONFIG_CONTENTS)
+    cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
+  end
+
+  describe "reboot" do
+    context "when resource is known" do
+      let(:argv) { %w(servers reboot srv-12345) }
+
+      before do
+        expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
+        expect(resource).to receive(:reboot).with(false)
+      end
+
+      it "outputs notice to STDERR" do
+        expect(stderr).to match("Sending reboot to #{resource_id}")
+      end
+    end
+
+    context "when resource is unknown" do
+      let(:argv) { %w(servers reboot srv-12345) }
+
+      before do
+        expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)
+      end
+
+      it "outputs error to STDERR" do
+        expect(stderr).to match("ERROR: Couldn't find #{resource_id}")
+      end
+    end
+
+    context "when no arguments are passed" do
+      let(:argv) { %w(servers reboot) }
+
+      it "outputs error to STDERR" do
+        expect(stderr).to match("ERROR: You must specify servers to reboot")
+      end
+    end
+  end
+end

data/spec/commands/servers/reset_spec.rb

@@ -0,0 +1,55 @@
+require "spec_helper"
+
+describe "brightbox servers" do
+
+  let(:output) { FauxIO.new { Brightbox.run(argv) } }
+  let(:stdout) { output.stdout }
+  let(:stderr) { output.stderr }
+
+  let(:resource_class) { Brightbox::Server }
+  let(:resource_id) { "srv-12345" }
+  let(:resource) { double(resource_class, :to_s => resource_id) }
+
+  let(:resource_ids) { [resource_id] }
+  let(:results) { [resource] }
+
+  before do
+    config = config_from_contents(USER_APP_CONFIG_CONTENTS)
+    cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
+  end
+
+  describe "reset" do
+    context "when resource is known" do
+      let(:argv) { %w(servers reset srv-12345) }
+
+      before do
+        expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
+        expect(resource).to receive(:reboot).with(true)
+      end
+
+      it "outputs notice to STDERR" do
+        expect(stderr).to match("Sending reset to #{resource_id}")
+      end
+    end
+
+    context "when resource is unknown" do
+      let(:argv) { %w(servers reset srv-12345) }
+
+      before do
+        expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)
+      end
+
+      it "outputs error to STDERR" do
+        expect(stderr).to match("ERROR: Couldn't find #{resource_id}")
+      end
+    end
+
+    context "when no arguments are passed" do
+      let(:argv) { %w(servers reset) }
+
+      it "outputs error to STDERR" do
+        expect(stderr).to match("ERROR: You must specify servers to reset")
+      end
+    end
+  end
+end

data/spec/commands/servers/show_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox servers" do
 
   describe "show" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["servers", "show"] }
+      let(:argv) { %w(servers show) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/servers/shutdown_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox servers" do
 
   describe "shutdown" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["servers", "shutdown"] }
+      let(:argv) { %w(servers shutdown) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/servers/snapshot_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox servers" do
 
   describe "snapshot" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["servers", "snapshot"] }
+      let(:argv) { %w(servers snapshot) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/servers/start_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox servers" do
 
   describe "start" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["servers", "start"] }
+      let(:argv) { %w(servers start) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/servers/stop_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox servers" do
 
   describe "stop" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["servers", "stop"] }
+      let(:argv) { %w(servers stop) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/servers/update_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox servers" do
 
   describe "update" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["servers", "update"] }
+      let(:argv) { %w(servers update) }
 
       it "does not error" do
         expect { output }.to_not raise_error

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

@@ -14,7 +14,7 @@ describe "brightbox database-servers" do
     end
 
     context "without arguments", :vcr do
-      let(:argv) { %w{sql instances create } }
+      let(:argv) { %w(sql instances create) }
       let(:expected_password) { "477wwwj48yifrnuk" }
 
       it "reports the new admin username" do
@@ -27,7 +27,7 @@ describe "brightbox database-servers" do
     end
 
     context "--allow-access=10.0.0.0", :vcr do
-      let(:argv) { %w{sql instances create --allow-access=10.0.0.0} }
+      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
@@ -37,8 +37,8 @@ describe "brightbox database-servers" do
     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} } }
+      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

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

@@ -0,0 +1,73 @@
+require "spec_helper"
+
+describe "brightbox sql instances" do
+  let(:output) { FauxIO.new { Brightbox.run(argv) } }
+  let(:stdout) { output.stdout }
+  let(:stderr) { output.stderr }
+
+  let(:resource_class) { Brightbox::DatabaseServer }
+  let(:resource_id) { "dbs-12345" }
+  let(:resource) { double(resource_class, :to_s => resource_id) }
+
+  let(:resource_ids) { [resource_id] }
+  let(:results) { [resource] }
+
+  before do
+    config = config_from_contents(USER_APP_CONFIG_CONTENTS)
+    cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
+  end
+
+  describe "lock" do
+    context "when resource is known" do
+      let(:argv) { %w(sql instances lock dbs-12345) }
+
+      before do
+        expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
+        expect(resource).to receive(:lock!)
+      end
+
+      it "outputs notice to STDERR" do
+        expect(stderr).to match("Locking #{resource_id}")
+      end
+    end
+
+    context "when resource is unknown" do
+      let(:argv) { %w(sql instances lock dbs-12345) }
+
+      before do
+        expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)
+      end
+
+      it "outputs error to STDERR" do
+        expect(stderr).to match("Couldn't find #{resource_id}")
+      end
+    end
+  end
+
+  describe "unlock" do
+    context "when resource is known" do
+      let(:argv) { %w(sql instances unlock dbs-12345) }
+
+      before do
+        expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
+        expect(resource).to receive(:unlock!)
+      end
+
+      it "outputs notice to STDERR" do
+        expect(stderr).to match("Unlocking #{resource_id}")
+      end
+    end
+
+    context "when resource is unknown" do
+      let(:argv) { %w(sql instances unlock dbs-12345) }
+
+      before do
+        expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)
+      end
+
+      it "outputs error to STDERR" do
+        expect(stderr).to match("Couldn't find #{resource_id}")
+      end
+    end
+  end
+end

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

@@ -14,7 +14,7 @@ describe "brightbox database-servers" do
     end
 
     context "when database server active", :vcr do
-      let(:argv) { %w{sql instances snapshot dbs-12345} }
+      let(:argv) { %w(sql instances snapshot dbs-12345) }
       let(:expected_args) { { :allow_access => ["10.0.0.0"] } }
 
       it "correctly sends API parameters" do
@@ -24,7 +24,7 @@ describe "brightbox database-servers" do
     end
 
     context "when database server can not be snapshotted", :vcr do
-      let(:argv) { %w{sql instances snapshot dbs-12345} }
+      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

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

@@ -14,7 +14,7 @@ describe "brightbox database-snapshots" do
     end
 
     context "when resources are available", :vcr do
-      let(:argv) { %w{sql snapshots list} }
+      let(:argv) { %w(sql snapshots list) }
 
       it "does not output to stderr" do
         expect(stderr).to be_empty

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

@@ -0,0 +1,73 @@
+require "spec_helper"
+
+describe "brightbox sql snapshots" do
+  let(:output) { FauxIO.new { Brightbox.run(argv) } }
+  let(:stdout) { output.stdout }
+  let(:stderr) { output.stderr }
+
+  let(:resource_class) { Brightbox::DatabaseSnapshot }
+  let(:resource_id) { "dbi-12345" }
+  let(:resource) { double(resource_class, :to_s => resource_id) }
+
+  let(:resource_ids) { [resource_id] }
+  let(:results) { [resource] }
+
+  before do
+    config = config_from_contents(USER_APP_CONFIG_CONTENTS)
+    cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
+  end
+
+  describe "lock" do
+    context "when resource is known" do
+      let(:argv) { %w(sql snapshots lock dbi-12345) }
+
+      before do
+        expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
+        expect(resource).to receive(:lock!)
+      end
+
+      it "outputs notice to STDERR" do
+        expect(stderr).to match("Locking #{resource_id}")
+      end
+    end
+
+    context "when resource is unknown" do
+      let(:argv) { %w(sql snapshots lock dbi-12345) }
+
+      before do
+        expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)
+      end
+
+      it "outputs error to STDERR" do
+        expect(stderr).to match("Couldn't find #{resource_id}")
+      end
+    end
+  end
+
+  describe "unlock" do
+    context "when resource is known" do
+      let(:argv) { %w(sql snapshots unlock dbi-12345) }
+
+      before do
+        expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
+        expect(resource).to receive(:unlock!)
+      end
+
+      it "outputs notice to STDERR" do
+        expect(stderr).to match("Unlocking #{resource_id}")
+      end
+    end
+
+    context "when resource is unknown" do
+      let(:argv) { %w(sql snapshots unlock dbi-12345) }
+
+      before do
+        expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)
+      end
+
+      it "outputs error to STDERR" do
+        expect(stderr).to match("Couldn't find #{resource_id}")
+      end
+    end
+  end
+end

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

@@ -14,7 +14,7 @@ describe "brightbox database-snapshots" do
     end
 
     context "when resource exists", :vcr do
-      let(:argv) { %w{sql snapshots show dbi-12345} }
+      let(:argv) { %w(sql snapshots show dbi-12345) }
 
       it "does not output to stderr" do
         expect(stderr).to be_empty

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

@@ -8,7 +8,7 @@ describe "brightbox database-types" do
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { %w{sql types list} }
+      let(:argv) { %w(sql types list) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/types/list_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox types" do
 
   describe "list" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["types", "list"] }
+      let(:argv) { %w(types list) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/users/list_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox users" do
 
   describe "list" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["users", "list"] }
+      let(:argv) { %w(users list) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/users/show_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox users" do
 
   describe "show" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["users", "show"] }
+      let(:argv) { %w(users show) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/users/update_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox users" do
 
   describe "update" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["users", "update"] }
+      let(:argv) { %w(users update) }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/zones/list_spec.rb

@@ -3,12 +3,12 @@ require "spec_helper"
 describe "brightbox zones" do
 
   describe "list" do
-    let(:output) { FauxIO.new { Brightbox::run(argv) } }
+    let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { ["zones", "list"] }
+      let(:argv) { %w(zones list) }
 
       it "does not error" do
         expect { output }.to_not raise_error