brightbox-cli 3.3.0 → 4.0.0.rc2

data/spec/commands/servers/reset_spec.rb

@@ -1,7 +1,6 @@
 require "spec_helper"
 
 describe "brightbox servers" do
-
   let(:output) { FauxIO.new { Brightbox.run(argv) } }
   let(:stdout) { output.stdout }
   let(:stderr) { output.stderr }
@@ -20,7 +19,7 @@ describe "brightbox servers" do
 
   describe "reset" do
     context "when resource is known" do
-      let(:argv) { %w(servers reset srv-12345) }
+      let(:argv) { %w[servers reset srv-12345] }
 
       before do
         expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
@@ -33,7 +32,7 @@ describe "brightbox servers" do
     end
 
     context "when resource is unknown" do
-      let(:argv) { %w(servers reset srv-12345) }
+      let(:argv) { %w[servers reset srv-12345] }
 
       before do
         expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)
@@ -45,7 +44,7 @@ describe "brightbox servers" do
     end
 
     context "when no arguments are passed" do
-      let(:argv) { %w(servers reset) }
+      let(:argv) { %w[servers reset] }
 
       it "outputs error to STDERR" do
         expect(stderr).to match("ERROR: You must specify servers to reset")

data/spec/commands/servers/show_spec.rb

@@ -1,14 +1,13 @@
 require "spec_helper"
 
 describe "brightbox servers" do
-
   describe "show" do
     let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { %w(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

@@ -1,14 +1,13 @@
 require "spec_helper"
 
 describe "brightbox servers" do
-
   describe "shutdown" do
     let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { %w(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

@@ -1,14 +1,13 @@
 require "spec_helper"
 
 describe "brightbox servers" do
-
   describe "snapshot" do
     let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { %w(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

@@ -1,14 +1,13 @@
 require "spec_helper"
 
 describe "brightbox servers" do
-
   describe "start" do
     let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { %w(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

@@ -1,14 +1,13 @@
 require "spec_helper"
 
 describe "brightbox servers" do
-
   describe "stop" do
     let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { %w(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

@@ -1,14 +1,13 @@
 require "spec_helper"
 
 describe "brightbox servers" do
-
   describe "update" do
     let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { %w(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

@@ -6,154 +6,137 @@ describe "brightbox sql instances" do
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
+    let(:sql_instance) { Brightbox::DatabaseServer.get(new_id) }
+    let(:new_id) { stdout.match(/id: (.*?)\n/)[1] }
+
     before do
-      config = config_from_contents(USER_APP_CONFIG_CONTENTS)
+      config_from_contents(API_CLIENT_CONFIG_CONTENTS)
+      Brightbox.config.reauthenticate
+    end
 
-      # Setup in the VCR recordings
-      cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
+    after do
+      sql_instance.wait_for { ready? }
+      sql_instance.destroy
     end
 
     context "without arguments", vcr: true do
-      let(:argv) { %w(sql instances create) }
-      let(:expected_password) { "477wwwj48yifrnuk" }
+      let(:argv) { %w[sql instances create] }
 
       it "reports the new admin username" do
-        expect(stdout).to include("admin_username: admin")
+        username = sql_instance.admin_username
+        expect(stdout).to include("admin_username: #{username}")
       end
 
       it "reports the new admin password" do
+        expected_password = sql_instance.admin_password
         expect(stdout).to include("admin_password: #{expected_password}")
       end
     end
 
     context "--allow-access=10.0.0.0", vcr: true 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
         expect(Brightbox::DatabaseServer).to receive(:create).with(expected_args).and_call_original
-        expect(stderr).to eql("")
+        expect(stderr).to eq("")
       end
     end
 
     context "--allow-access=srv-12345,grp-12345", vcr: true 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) { ["sql", "instances", "create", "--allow-access", "#{server.id},#{group.id}"] }
+      let(:expected_args) { { :allow_access => [server.id, group.id] } }
+      let(:group) { Brightbox::ServerGroup.all.first }
+      let(:server) { Brightbox::Server.create flavor_id: "2gb.nbs", image_id: "img-linux" }
+
+      before do
+        server.fog_model.wait_for { ready? }
+      end
 
       it "correctly sends API parameters" do
         expect(Brightbox::DatabaseServer).to receive(:create).with(expected_args).and_call_original
-        expect(stderr).to eql("")
+        expect(stderr).to eq("")
+      end
+
+      after do
+        server.destroy
       end
     end
 
     context "--engine=mysql", vcr: true do
-      let(:argv) { %w(sql instances create --engine=mysql) }
+      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("")
+        expect(stderr).to eq("")
+        expect(stdout).to include("version: 8.0")
       end
     end
 
-    context "--engine=mysql --engine-version=5.6", vcr: true do
-      let(:argv) { %w(sql instances create --engine=mysql --engine-version=5.6) }
-      let(:expected_args) { { :database_engine => "mysql", :database_version => "5.6" } }
+    context "--engine=mysql --engine-version=8.0", vcr: true do
+      let(:argv) { %w[sql instances create --engine=mysql --engine-version=8.0] }
+      let(:expected_args) { { :database_engine => "mysql", :database_version => "8.0" } }
 
       it "correctly sends API parameters" do
         expect(Brightbox::DatabaseServer).to receive(:create).with(expected_args).and_call_original
-        expect(stderr).to eql("")
+        expect(stderr).to eq("")
+        expect(stdout).to include("version: 8.0")
       end
     end
 
-    context "--maintenance-weekday=5 --maintenance_hour=11" do
-      let(:argv) { %w(sql instances create --maintenance-weekday=5 --maintenance-hour=11) }
+    context "--maintenance-weekday=5 --maintenance_hour=11", vcr: true do
+      let(:argv) { %w[sql instances create --maintenance-weekday=5 --maintenance-hour=11] }
       let(:expected_args) { { :maintenance_weekday => "5", :maintenance_hour => "11" } }
 
-      before do
-        stub_request(:post, "http://api.brightbox.dev/token").to_return(
-          :status => 200,
-          :body => '{"access_token":"44320b29286077c44f14c4efdfed70f63f4a8361","token_type":"Bearer","refresh_token":"759b2b28c228948a0ba5d07a89f39f9e268a95c0","scope":"infrastructure orbit","expires_in":7200}')
-
-        stub_request(:post, "http://api.brightbox.dev/1.0/database_servers?account_id=acc-12345")
-          .with(:body => "{\"name\":null,\"description\":null,\"maintenance_weekday\":\"5\",\"maintenance_hour\":\"11\"}")
-      end
-
       it "correctly sends API parameters" do
         expect(Brightbox::DatabaseServer).to receive(:create).with(expected_args).and_call_original
-        expect(stderr).to eql("")
+        expect(stderr).to eq("")
       end
     end
 
-    context "--maintenance-weekday=thursday" do
-      let(:argv) { %w(sql instances create --maintenance-weekday=thursday) }
+    context "--maintenance-weekday=thursday", vcr: true do
+      let(:argv) { %w[sql instances create --maintenance-weekday=thursday] }
       let(:expected_args) { { :maintenance_weekday => "4" } }
 
-      before do
-        stub_request(:post, "http://api.brightbox.dev/token").to_return(
-          :status => 200,
-          :body => '{"access_token":"44320b29286077c44f14c4efdfed70f63f4a8361","token_type":"Bearer","refresh_token":"759b2b28c228948a0ba5d07a89f39f9e268a95c0","scope":"infrastructure orbit","expires_in":7200}')
-
-        stub_request(:post, "http://api.brightbox.dev/1.0/database_servers?account_id=acc-12345")
-          .with(:body => "{\"name\":null,\"description\":null,\"maintenance_weekday\":\"4\",\"maintenance_hour\":null}")
-      end
-
       it "correctly sends API parameters" do
         expect(Brightbox::DatabaseServer).to receive(:create).with(expected_args).and_call_original
-        expect(stderr).to eql("")
+        expect(stderr).to eq("")
       end
     end
 
-    context "--snapshot=dbi-1493j" do
-      let(:argv) { ["sql", "instances", "create", "--snapshot=dbi-1493j"] }
-      let(:expected_args) { { :snapshot_id => "dbi-1493j" } }
+    context "--snapshot=dbi-12345", vcr: true do
+      let(:argv) { ["sql", "instances", "create", "--snapshot=#{sql_snapshot.id}"] }
+      let(:expected_args) { { :snapshot_id => sql_snapshot.id } }
+      let(:source) { Brightbox::DatabaseServer.create({}) }
+      let(:sql_snapshot) { source.snapshot(true) }
 
-      let(:json_response) do
-        <<-EOS
-        {
-          "id":"dbs-12345"
-        }
-        EOS
+      before do
+        source.fog_model.wait_for { ready? }
+        sql_snapshot.wait_for { ready? }
       end
 
-      before do
-        stub_request(:post, "http://api.brightbox.dev/1.0/database_servers?account_id=acc-12345")
-          .with(:headers => { "Content-Type" => "application/json" },
-                :body => hash_including("snapshot" => "dbi-1493j"))
-          .and_return(:status => 202, :body => json_response)
+      after do
+        source.destroy
+        sql_snapshot.destroy
       end
 
       it "includes schedule fields in response" do
         expect(Brightbox::DatabaseServer).to receive(:create).with(expected_args).and_call_original
-        expect(stdout).to include("id: dbs-12345")
-        expect(stderr).to eql("")
+        expect(stdout).to include("id: #{sql_instance.id}")
+        expect(stderr).to eq("")
       end
     end
 
-    context "--snapshots-schedule='0 12 * * 4'" do
+    context "--snapshots-schedule='0 12 * * 4'", vcr: true do
       let(:argv) { ["sql", "instances", "create", "--snapshots-schedule=0 12 * * 4"] }
       let(:expected_args) { { :snapshots_schedule => "0 12 * * 4" } }
 
-      let(:json_response) do
-        <<-EOS
-        {
-          "id":"dbs-12345",
-          "snapshots_schedule":"0 12 * * 4",
-          "snapshots_schedule_next_at":"2016-07-07T12:00:00Z"
-        }
-        EOS
-      end
-
-      before do
-        stub_request(:post, "http://api.brightbox.dev/1.0/database_servers?account_id=acc-12345")
-          .and_return(:status => 202, :body => json_response)
-      end
-
       it "includes schedule fields in response" do
         expect(Brightbox::DatabaseServer).to receive(:create).with(expected_args).and_call_original
         expect(stdout).to include("snapshots_schedule: 0 12 * * 4")
-        expect(stdout).to include("snapshots_schedule_next_at: 2016-07-07T12:00Z")
-        expect(stderr).to eql("")
+        expect(stdout).to match(/snapshots_schedule_next_at: 20\d{2}-\d{2}-\d{2}T12:\d{2}Z/)
+        expect(stderr).to eq("")
       end
     end
   end

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

@@ -19,7 +19,7 @@ describe "brightbox sql instances" do
 
   describe "lock" do
     context "when resource is known" do
-      let(:argv) { %w(sql instances lock dbs-12345) }
+      let(:argv) { %w[sql instances lock dbs-12345] }
 
       before do
         expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
@@ -32,7 +32,7 @@ describe "brightbox sql instances" do
     end
 
     context "when resource is unknown" do
-      let(:argv) { %w(sql instances lock dbs-12345) }
+      let(:argv) { %w[sql instances lock dbs-12345] }
 
       before do
         expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)
@@ -46,7 +46,7 @@ describe "brightbox sql instances" do
 
   describe "unlock" do
     context "when resource is known" do
-      let(:argv) { %w(sql instances unlock dbs-12345) }
+      let(:argv) { %w[sql instances unlock dbs-12345] }
 
       before do
         expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
@@ -59,7 +59,7 @@ describe "brightbox sql instances" do
     end
 
     context "when resource is unknown" do
-      let(:argv) { %w(sql instances unlock dbs-12345) }
+      let(:argv) { %w[sql instances unlock dbs-12345] }
 
       before do
         expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)

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

@@ -1,6 +1,6 @@
 require "spec_helper"
 
-describe "brghtbox sql instances" do
+describe "brightbox sql instances" do
   describe "show" do
     let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
@@ -12,14 +12,14 @@ describe "brghtbox sql instances" do
     end
 
     context "when id does not exist" do
-      let(:argv) { %w(sql instances show dbs-12345) }
+      let(:argv) { %w[sql instances show dbs-12345] }
       let(:json_response) do
         "{\"error_name\":\"missing_resource\",\"errors\":[\"Resource not found using supplied identifier\"]}"
       end
 
       before do
-        stub_request(:get, "http://api.brightbox.dev/1.0/database_servers/dbs-12345?account_id=acc-12345").
-          to_return(:status => 404, :body => json_response, :headers => { "Content-Type" => "" })
+        stub_request(:get, "http://api.brightbox.localhost/1.0/database_servers/dbs-12345?account_id=acc-12345")
+          .to_return(:status => 404, :body => json_response, :headers => { "Content-Type" => "" })
       end
 
       it "reports error" do
@@ -29,7 +29,7 @@ describe "brghtbox sql instances" do
     end
 
     context "when id exists" do
-      let(:argv) { %w(sql instances show dbs-12345) }
+      let(:argv) { %w[sql instances show dbs-12345] }
 
       let(:json_response) do
         <<-EOS
@@ -61,8 +61,8 @@ describe "brghtbox sql instances" do
       end
 
       before do
-        stub_request(:get, "http://api.brightbox.dev/1.0/database_servers/dbs-12345?account_id=acc-12345").
-          to_return(:status => 200, :body => json_response, :headers => { "Content-Type" => "" })
+        stub_request(:get, "http://api.brightbox.localhost/1.0/database_servers/dbs-12345?account_id=acc-12345")
+          .to_return(:status => 200, :body => json_response, :headers => { "Content-Type" => "" })
       end
 
       it "simplifies the maintenance window" do

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

@@ -14,7 +14,7 @@ describe "brightbox sql instances" do
     end
 
     context "when database server active", vcr: true 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 sql instances" do
     end
 
     context "when database server can not be snapshotted", vcr: true 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/instances/update_spec.rb

@@ -13,11 +13,11 @@ describe "brightbox sql instances" do
     end
 
     context "--maintenance-weekday=Monday" do
-      let(:argv) { %w(sql instances update --maintenance-weekday=Monday dbs-12345) }
+      let(:argv) { %w[sql instances update --maintenance-weekday=Monday dbs-12345] }
       let(:expected_args) { { :maintenance_weekday => "1" } }
 
       before do
-        stub_request(:get, "http://api.brightbox.dev/1.0/database_servers/dbs-12345?account_id=acc-12345")
+        stub_request(:get, "http://api.brightbox.localhost/1.0/database_servers/dbs-12345?account_id=acc-12345")
           .to_return(:status => 200, :body => '{"id":"dbs-12345","maintenance_weekday":0, "maintenance_hour":6}')
           .to_return(:status => 200,
                      :body => '{
@@ -25,7 +25,7 @@ describe "brightbox sql instances" do
                        "maintenance_weekday":1,
                        "maintenance_hour":6}')
 
-        stub_request(:put, "http://api.brightbox.dev/1.0/database_servers/dbs-12345?account_id=acc-12345")
+        stub_request(:put, "http://api.brightbox.localhost/1.0/database_servers/dbs-12345?account_id=acc-12345")
           .with(:body => "{\"maintenance_weekday\":\"1\"}")
       end
 
@@ -41,11 +41,11 @@ describe "brightbox sql instances" do
       let(:expected_args) { { :snapshots_schedule => "0 12 * * 4" } }
 
       before do
-        stub_request(:get, "http://api.brightbox.dev/1.0/database_servers/dbs-12345?account_id=acc-12345")
+        stub_request(:get, "http://api.brightbox.localhost/1.0/database_servers/dbs-12345?account_id=acc-12345")
           .to_return(:status => 200, :body => '{"id":"dbs-12345","snapshots_schedule":null,"snapshots_schedule_next_at":null}')
           .to_return(:status => 200, :body => '{"id":"dbs-12345","snapshots_schedule":"34 12 * * 4","snapshots_schedule_next_at":"2016-07-07T12:34:56Z"}')
 
-        stub_request(:put, "http://api.brightbox.dev/1.0/database_servers/dbs-12345?account_id=acc-12345")
+        stub_request(:put, "http://api.brightbox.localhost/1.0/database_servers/dbs-12345?account_id=acc-12345")
           .to_return(:status => 200, :body => '{"id":"dbs-12345","snapshots_schedule":"34 12 * * 4","snapshots_schedule_next_at":"2016-07-07T12:34:56Z"}')
       end
 
@@ -60,15 +60,15 @@ describe "brightbox sql instances" do
 
     context "--remove-snapshots-schedule" do
       let(:dbs) { Brightbox::DatabaseServer.find("dbs-12345") }
-      let(:argv) { %w(sql instances update --remove-snapshots-schedule dbs-12345) }
+      let(:argv) { %w[sql instances update --remove-snapshots-schedule dbs-12345] }
       let(:expected_args) { { :snapshots_schedule => nil } }
 
       before do
-        stub_request(:get, "http://api.brightbox.dev/1.0/database_servers/dbs-12345?account_id=acc-12345")
+        stub_request(:get, "http://api.brightbox.localhost/1.0/database_servers/dbs-12345?account_id=acc-12345")
           .to_return(:status => 200, :body => '{"id":"dbs-12345","snapshots_schedule":"34 12 * * 4","snapshots_schedule_next_at":"2016-07-07T12:34:56Z"}')
           .to_return(:status => 200, :body => '{"id":"dbs-12345","snapshots_schedule":null,"snapshots_schedule_next_at":null}')
 
-        stub_request(:put, "http://api.brightbox.dev/1.0/database_servers/dbs-12345?account_id=acc-12345")
+        stub_request(:put, "http://api.brightbox.localhost/1.0/database_servers/dbs-12345?account_id=acc-12345")
           .to_return(:status => 200, :body => '{"id":"dbs-12345","snapshots_schedule":null,"snapshots_schedule_next_at":null}')
       end
 

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

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

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

@@ -19,7 +19,7 @@ describe "brightbox sql snapshots" do
 
   describe "lock" do
     context "when resource is known" do
-      let(:argv) { %w(sql snapshots lock dbi-12345) }
+      let(:argv) { %w[sql snapshots lock dbi-12345] }
 
       before do
         expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
@@ -32,7 +32,7 @@ describe "brightbox sql snapshots" do
     end
 
     context "when resource is unknown" do
-      let(:argv) { %w(sql snapshots lock dbi-12345) }
+      let(:argv) { %w[sql snapshots lock dbi-12345] }
 
       before do
         expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)
@@ -46,7 +46,7 @@ describe "brightbox sql snapshots" do
 
   describe "unlock" do
     context "when resource is known" do
-      let(:argv) { %w(sql snapshots unlock dbi-12345) }
+      let(:argv) { %w[sql snapshots unlock dbi-12345] }
 
       before do
         expect(resource_class).to receive(:find_or_call).with(resource_ids).and_return(results)
@@ -59,7 +59,7 @@ describe "brightbox sql snapshots" do
     end
 
     context "when resource is unknown" do
-      let(:argv) { %w(sql snapshots unlock dbi-12345) }
+      let(:argv) { %w[sql snapshots unlock dbi-12345] }
 
       before do
         expect(resource_class).to receive(:find).with(resource_id).and_raise(Brightbox::Api::NotFound)

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

@@ -14,7 +14,7 @@ describe "brightbox sql snapshots" do
     end
 
     context "when resource exists", vcr: true 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 eql("")

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

@@ -1,14 +1,13 @@
 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) }
+      let(:argv) { %w[sql types list] }
 
       it "does not error" do
         expect { output }.to_not raise_error

data/spec/commands/types/list_spec.rb

@@ -1,14 +1,13 @@
 require "spec_helper"
 
 describe "brightbox 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(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

@@ -1,14 +1,13 @@
 require "spec_helper"
 
 describe "brightbox users" do
-
   describe "list" do
     let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { %w(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

@@ -1,14 +1,13 @@
 require "spec_helper"
 
 describe "brightbox users" do
-
   describe "show" do
     let(:output) { FauxIO.new { Brightbox.run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }
 
     context "" do
-      let(:argv) { %w(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

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