brightbox-cli 4.2.1 → 4.3.0

data/spec/commands/volumes/detach_spec.rb

@@ -0,0 +1,180 @@
+require "spec_helper"
+
+describe "brightbox volumes detach" do
+  include VolumeHelpers
+
+  let(:output) { FauxIO.new { Brightbox.run(argv) } }
+  let(:stdout) { output.stdout }
+  let(:stderr) { output.stderr }
+
+  before do
+    config_from_contents(API_CLIENT_CONFIG_CONTENTS)
+
+    stub_request(:post, "http://api.brightbox.localhost/token")
+      .to_return(
+        status: 200,
+        body: JSON.dump(
+          access_token: "ACCESS-TOKEN",
+          refresh_token: "REFRESH_TOKEN"
+        )
+      )
+
+    Brightbox.config.reauthenticate
+  end
+
+  context "without arguments" do
+    let(:argv) { %w[volumes detach] }
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("ERROR: You must specify volume IDs as arguments\n")
+
+      expect(stdout).to match("")
+    end
+  end
+
+  context "with one argument" do
+    let(:argv) { %w[volumes detach vol-dkl43] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/volumes/vol-dkl43")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: volume_response(
+            id: "vol-dkl43",
+            status: "attached",
+            boot: false,
+            server: {
+              id: "srv-03431"
+            }
+          )
+        )
+        .to_return(
+          status: 200,
+          body: volume_response(
+            id: "vol-dkl43",
+            status: "detached",
+            boot: false,
+            server: nil
+          )
+        )
+
+      stub_request(:post, "http://api.brightbox.localhost/1.0/volumes/vol-dkl43/detach")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 202,
+          body: volume_response(
+            id: "vol-dkl43",
+            status: "detached",
+            server: {
+              id: "srv-03431"
+            }
+          )
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to match("Detaching vol-dkl43\n")
+
+      aggregate_failures do
+        expect(stdout).to match("vol-dkl43")
+        expect(stdout).to match("detached")
+        expect(stdout).to match("false")
+      end
+    end
+  end
+
+  context "with multiple arguments" do
+    let(:argv) { %w[volumes detach vol-p3241 vol-3422f] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/volumes/vol-p3241")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: volume_response(
+            id: "vol-p3241",
+            status: "attached",
+            boot: false,
+            server: {
+              id: "srv-03431"
+            }
+          )
+        )
+        .to_return(
+          status: 200,
+          body: volume_response(
+            id: "vol-p3241",
+            status: "detached",
+            boot: false,
+            server: nil
+          )
+        )
+
+      stub_request(:post, "http://api.brightbox.localhost/1.0/volumes/vol-p3241/detach")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 202,
+          body: volume_response(
+            id: "vol-p3241",
+            status: "detached",
+            server: {
+              id: "srv-03431"
+            }
+          )
+        )
+
+        stub_request(:get, "http://api.brightbox.localhost/1.0/volumes/vol-3422f")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: volume_response(
+            id: "vol-3422f",
+            status: "attached",
+            boot: false,
+            server: {
+              id: "srv-03431"
+            }
+          )
+        )
+        .to_return(
+          status: 200,
+          body: volume_response(
+            id: "vol-3422f",
+            status: "detached",
+            boot: false,
+            server: nil
+          )
+        )
+
+      stub_request(:post, "http://api.brightbox.localhost/1.0/volumes/vol-3422f/detach")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 202,
+          body: volume_response(
+            id: "vol-3422f",
+            status: "detached",
+            server: {
+              id: "srv-03431"
+            }
+          )
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to match("Detaching vol-3422f\n")
+
+      aggregate_failures do
+        expect(stdout).to match("vol-p3241")
+        expect(stdout).to match("detached")
+        expect(stdout).to match("false")
+      end
+    end
+  end
+end

data/spec/commands/volumes/list_spec.rb

@@ -0,0 +1,99 @@
+require "spec_helper"
+
+describe "brightbox volumes list" do
+  include VolumeHelpers
+
+  let(:output) { FauxIO.new { Brightbox.run(argv) } }
+  let(:stdout) { output.stdout }
+  let(:stderr) { output.stderr }
+
+  before do
+    config_from_contents(API_CLIENT_CONFIG_CONTENTS)
+
+    stub_request(:post, "http://api.brightbox.localhost/token")
+      .to_return(
+        status: 200,
+        body: JSON.dump(
+          access_token: "ACCESS-TOKEN",
+          refresh_token: "REFRESH_TOKEN"
+        )
+      )
+
+    Brightbox.config.reauthenticate
+  end
+
+  context "without arguments" do
+    let(:argv) { %w[volumes list] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/volumes")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: volumes_response
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("")
+
+      aggregate_failures do
+        expect(stdout).to match("id.*type.*size.*status.*server.*boot")
+
+        expect(stdout).to match("vol-12345")
+        expect(stdout).to match("network")
+        expect(stdout).to match("10240")
+        expect(stdout).to match("attached")
+        expect(stdout).to match("srv-12345")
+        expect(stdout).to match("false")
+
+        expect(stdout).to match("vol-abcde")
+        expect(stdout).to match("network")
+        expect(stdout).to match("10240")
+        expect(stdout).to match("attached")
+        expect(stdout).to match("srv-12345")
+        expect(stdout).to match("true")
+      end
+    end
+  end
+
+  context "with identifier" do
+    let(:argv) { %w[volumes list vol-12345] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/volumes/vol-12345")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: volume_response(
+            id: "vol-12345",
+            storage_type: "network",
+            size: 10_240,
+            status: "attached",
+            server: {
+              id: "srv-12345"
+            }
+          )
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).not_to match("ERROR")
+
+      aggregate_failures do
+        expect(stdout).to match("id.*type.*size.*status.*server.*boot")
+
+        expect(stdout).to match("vol-12345")
+        expect(stdout).to match("network")
+        expect(stdout).to match("10240")
+        expect(stdout).to match("attached")
+        expect(stdout).to match("srv-12345")
+        expect(stdout).to match("false")
+      end
+    end
+  end
+end

data/spec/commands/volumes/resize_spec.rb

@@ -0,0 +1,96 @@
+require "spec_helper"
+
+describe "brightbox volumes resize" do
+  include VolumeHelpers
+
+  let(:output) { FauxIO.new { Brightbox.run(argv) } }
+  let(:stdout) { output.stdout }
+  let(:stderr) { output.stderr }
+
+  before do
+    config_from_contents(API_CLIENT_CONFIG_CONTENTS)
+
+    stub_request(:post, "http://api.brightbox.localhost/token")
+      .to_return(
+        status: 200,
+        body: JSON.dump(
+          access_token: "ACCESS-TOKEN",
+          refresh_token: "REFRESH_TOKEN"
+        )
+      )
+
+    Brightbox.config.reauthenticate
+  end
+
+  context "without arguments" do
+    let(:argv) { %w[volumes resize] }
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("ERROR: You must specify the volume ID as the first argument\n")
+
+      expect(stdout).to match("")
+    end
+  end
+
+  context "without size option" do
+    let(:argv) { %w[volumes resize vol-i89u9] }
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("ERROR: A 'size' option is required\n")
+
+      expect(stdout).to match("")
+    end
+  end
+
+  context "with size option" do
+    let(:argv) { %w[volumes resize --size 20480 vol-op324] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/volumes/vol-op324")
+      .with(query: hash_including(account_id: "acc-12345"))
+      .to_return(
+        status: 200,
+        body: volume_response(
+          id: "vol-op324",
+          size: 10_240
+        )
+      )
+      .to_return(
+        status: 200,
+        body: volume_response(
+          id: "vol-op324",
+          size: 20_480
+        )
+      )
+
+      stub_request(:post, "http://api.brightbox.localhost/1.0/volumes/vol-op324/resize")
+        .with(query: hash_including(account_id: "acc-12345"),
+              body: hash_including(
+                from: 10_240,
+                to: 20_480
+              ))
+        .to_return(
+          status: 202,
+          body: volume_response(
+            id: "vol-op324",
+            size: 20_480
+          )
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("Resizing vol-op324\n")
+
+      aggregate_failures do
+        expect(stdout).to match("vol-op324")
+        expect(stdout).to match("20480")
+      end
+    end
+  end
+end

data/spec/commands/volumes/show_spec.rb

@@ -0,0 +1,95 @@
+require "spec_helper"
+
+describe "brightbox volumes show" do
+  include VolumeHelpers
+
+  let(:output) { FauxIO.new { Brightbox.run(argv) } }
+  let(:stdout) { output.stdout }
+  let(:stderr) { output.stderr }
+
+  before do
+    config_from_contents(API_CLIENT_CONFIG_CONTENTS)
+
+    stub_request(:post, "http://api.brightbox.localhost/token")
+      .to_return(
+        status: 200,
+        body: JSON.dump(
+          access_token: "ACCESS-TOKEN",
+          refresh_token: "REFRESH_TOKEN"
+        )
+      )
+
+    Brightbox.config.reauthenticate
+  end
+
+  context "without arguments" do
+    let(:argv) { %w[volumes show] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/volumes")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: volumes_response
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("")
+
+      aggregate_failures do
+        expect(stdout).to match("id: vol-12345")
+        expect(stdout).to match("id: vol-abcde")
+      end
+    end
+  end
+
+  context "with identifier" do
+    let(:argv) { %w[volumes show vol-88878] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/volumes/vol-88878")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: volume_response(
+            id: "vol-88878",
+            description: "A volume for testing",
+            filesystem_type: "ext4",
+            storage_type: "network",
+            size: 10_240,
+            status: "attached",
+            server: nil
+          )
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).not_to match("ERROR")
+
+      aggregate_failures do
+        expect(stdout).to match("id: vol-88878")
+        expect(stdout).to match("status: attached")
+        expect(stdout).to match("created_at: 2023-01-01T01:01Z")
+        expect(stdout).to match("description: A volume for testing")
+        expect(stdout).to match("zone: zon-12345")
+        expect(stdout).to match("size: 10240")
+        expect(stdout).to match("type: network")
+        expect(stdout).to match("server:")
+        expect(stdout).to match("boot: false")
+        expect(stdout).to match("filesystem_label:")
+        expect(stdout).to match("filesystem_type: ext4")
+        expect(stdout).to match("delete_with_server: false")
+        expect(stdout).to match("encrypted: false")
+        expect(stdout).to match("serial: vol-12345")
+        expect(stdout).to match("source:")
+        expect(stdout).to match("image: img-12345")
+        expect(stdout).to match("locked: false")
+      end
+    end
+  end
+end

data/spec/commands/volumes/update_spec.rb

@@ -0,0 +1,188 @@
+require "spec_helper"
+
+describe "brightbox volumes update" do
+  include VolumeHelpers
+
+  let(:output) { FauxIO.new { Brightbox.run(argv) } }
+  let(:stdout) { output.stdout }
+  let(:stderr) { output.stderr }
+
+  before do
+    config_from_contents(API_CLIENT_CONFIG_CONTENTS)
+
+    stub_request(:post, "http://api.brightbox.localhost/token")
+      .to_return(
+        status: 200,
+        body: JSON.dump(
+          access_token: "ACCESS-TOKEN",
+          refresh_token: "REFRESH_TOKEN"
+        )
+      )
+
+    Brightbox.config.reauthenticate
+  end
+
+  context "without arguments" do
+    let(:argv) { %w[volumes update] }
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("ERROR: You must specify the volume ID as the first argument\n")
+
+      expect(stdout).to match("")
+    end
+  end
+
+  context "with new options" do
+    let(:argv) { %w[volumes update --name New --desc Testing --serial 12345 vol-908us] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/volumes/vol-908us")
+      .with(query: hash_including(account_id: "acc-12345"))
+      .to_return(
+        status: 200,
+        body: volume_response(
+          id: "vol-908us",
+          name: "Old",
+          description: nil,
+          delete_with_server: true,
+          serial: "vol-908us"
+        )
+      )
+      .to_return(
+        status: 200,
+        body: volume_response(
+          id: "vol-908us",
+          name: "New",
+          description: "Testing",
+          delete_with_server: true,
+          serial: "12345"
+        )
+      )
+
+      stub_request(:put, "http://api.brightbox.localhost/1.0/volumes/vol-908us")
+        .with(query: hash_including(account_id: "acc-12345"),
+              body: {
+                name: "New",
+                description: "Testing",
+                serial: "12345"
+              })
+        .to_return(
+          status: 202,
+          body: volume_response(
+            id: "vol-908us",
+            name: "New",
+            description: "Testing",
+            delete_with_server: true,
+            serial: "12345"
+          )
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("Updating vol-908us\n")
+
+      aggregate_failures do
+        expect(stdout).to match("vol-908us")
+        expect(stdout).to match("New")
+      end
+    end
+  end
+
+  context "with delete-with-server option" do
+    let(:argv) { %w[volumes update --delete-with-server vol-kl234] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/volumes/vol-kl234")
+      .with(query: hash_including(account_id: "acc-12345"))
+      .to_return(
+        status: 200,
+        body: volume_response(
+          id: "vol-kl234",
+          delete_with_server: false
+        )
+      )
+      .to_return(
+        status: 200,
+        body: volume_response(
+          id: "vol-kl234",
+          delete_with_server: true
+        )
+      )
+
+      stub_request(:put, "http://api.brightbox.localhost/1.0/volumes/vol-kl234")
+        .with(query: hash_including(account_id: "acc-12345"),
+              body: {
+                delete_with_server: true
+              })
+        .to_return(
+          status: 202,
+          body: volume_response(
+            id: "vol-kl234",
+            delete_with_server: true
+          )
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("Updating vol-kl234\n")
+
+      aggregate_failures do
+        expect(stdout).to match("vol-kl234")
+        expect(stdout).to match("false")
+      end
+    end
+  end
+
+  context "with no-delete-with-server option" do
+    let(:argv) { %w[volumes update --no-delete-with-server vol-sdj2j] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/volumes/vol-sdj2j")
+      .with(query: hash_including(account_id: "acc-12345"))
+      .to_return(
+        status: 200,
+        body: volume_response(
+          id: "vol-sdj2j",
+          delete_with_server: true
+        )
+      )
+      .to_return(
+        status: 200,
+        body: volume_response(
+          id: "vol-sdj2j",
+          delete_with_server: false
+        )
+      )
+
+      stub_request(:put, "http://api.brightbox.localhost/1.0/volumes/vol-sdj2j")
+        .with(query: hash_including(account_id: "acc-12345"),
+              body: {
+                delete_with_server: false
+              })
+        .to_return(
+          status: 202,
+          body: volume_response(
+            id: "vol-sdj2j",
+            delete_with_server: false
+          )
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("Updating vol-sdj2j\n")
+
+      aggregate_failures do
+        expect(stdout).to match("vol-sdj2j")
+        expect(stdout).to match("false")
+      end
+    end
+  end
+end