brightbox-cli 4.3.2 → 4.5.0.rc1

data/spec/commands/configmaps/create_spec.rb

@@ -0,0 +1,257 @@
+require "spec_helper"
+
+describe "brightbox configmaps create" do
+  let(:output) { FauxIO.new { Brightbox.run(argv) } }
+  let(:stdout) { output.stdout }
+  let(:stderr) { output.stderr }
+
+  let(:token) { SecureRandom.hex }
+
+  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: token))
+
+    Brightbox.config.reauthenticate
+  end
+
+  context "without options" do
+    let(:argv) { %w[configmaps create] }
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to match("Config map data is required as 'data' option")
+
+      expect(stdout).to eq("")
+    end
+  end
+
+  context "with 'name' and 'data'" do
+    let(:argv) { ["configmaps", "create", "--name", "tester", "--data", payload] }
+    let(:payload) { { key: "value" }.to_json }
+
+    before do
+      stub_request(:post, "http://api.brightbox.localhost/1.0/config_maps")
+        .with(headers: { "Content-Type" => "application/json" },
+              query: hash_including(account_id: "acc-12345"),
+              body: {
+                name: "tester",
+                data: {
+                  key: "value"
+                }
+              })
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-lk342",
+            name: "tester",
+            data: {
+              key: "value"
+            }
+          }.to_json
+        )
+    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.*name")
+
+        expect(stdout).to match("cfg-lk342")
+      end
+    end
+  end
+
+  context "with 'data'" do
+    let(:argv) { ["configmaps", "create", "--data", payload] }
+    let(:payload) { { key: "value" }.to_json }
+
+    before do
+      stub_request(:post, "http://api.brightbox.localhost/1.0/config_maps")
+        .with(headers: { "Content-Type" => "application/json" },
+              query: hash_including(account_id: "acc-12345"),
+              body: {
+                data: {
+                  key: "value"
+                }
+              }.to_json)
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-lk342",
+            data: {
+              key: "value"
+            }
+          }.to_json
+        )
+    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.*name")
+
+        expect(stdout).to match("cfg-lk342")
+      end
+    end
+
+    context "with invalid update" do
+      let(:payload) { "Not JSON!" }
+
+      it "does not error" do
+        expect { output }.to_not raise_error
+
+        expect(stderr).to eq("ERROR: Config map data was not valid JSON\n")
+
+        expect(stdout).to eq("")
+      end
+    end
+  end
+
+  context "with new map from 'data-file'" do
+    context "when filename is used" do
+      let(:argv) { ["configmaps", "create", "--data-file", data_file.path] }
+      let(:data_file) { Tempfile.open("config_map_test_data") }
+
+      around do |example|
+        data_file.write(payload)
+        data_file.close
+
+        example.run
+
+        data_file.unlink
+      end
+
+      context "with valid update" do
+        let(:payload) { { new_key: "new setting" }.to_json }
+
+        before do
+          stub_request(:post, "http://api.brightbox.localhost/1.0/config_maps")
+            .with(query: hash_including(account_id: "acc-12345"),
+                  body: {
+                    data: {
+                      new_key: "new setting"
+                    }
+                  })
+            .to_return(
+              status: 200,
+              body: {
+                id: "cfg-s432l",
+                name: "",
+                data: {
+                  new_key: "new setting"
+                }
+              }.to_json
+            )
+        end
+
+        it "does not error" do
+          expect { output }.to_not raise_error
+
+          expect(stderr).to eq("Creating config map\n")
+
+          aggregate_failures do
+            expect(stdout).to match("cfg-s432l")
+          end
+        end
+
+        context "with invalid update" do
+          let(:payload) { "Not JSON!" }
+
+          it "does not error" do
+            expect { output }.to_not raise_error
+
+            expect(stderr).to eq("ERROR: Config map data was not valid JSON\n")
+
+            expect(stdout).to eq("")
+          end
+        end
+      end
+    end
+  end
+
+  context "when '-' is used for STDIN" do
+    let(:argv) { ["configmaps", "create", "--data-file", "-", "cfg-stdin"] }
+
+    before do
+      stdin_data = StringIO.new
+      stdin_data.puts(payload)
+      stdin_data.rewind
+
+      $stdin = stdin_data
+    end
+
+    after do
+      $stdin = STDIN
+    end
+
+    context "with valid update" do
+      let(:payload) { { use_stdin: true }.to_json }
+
+      before do
+        stub_request(:post, "http://api.brightbox.localhost/1.0/config_maps")
+          .with(query: hash_including(account_id: "acc-12345"),
+                body: {
+                  data: {
+                    use_stdin: true
+                  }
+                })
+          .to_return(
+            status: 200,
+            body: {
+              id: "cfg-mj53s",
+              name: "",
+              data: {
+                use_stdin: true
+              }
+            }.to_json
+          )
+      end
+
+      it "does not error" do
+        expect { output }.to_not raise_error
+
+        expect(stderr).to eq("Creating config map\n")
+
+        aggregate_failures do
+          expect(stdout).to match("cfg-mj53s")
+        end
+      end
+
+      context "with invalid update" do
+        let(:payload) { "Not JSON!" }
+
+        it "does not error" do
+          expect { output }.to_not raise_error
+
+          expect(stderr).to eq("ERROR: Config map data was not valid JSON\n")
+
+          expect(stdout).to eq("")
+        end
+      end
+    end
+  end
+
+  context "with mutually exclusive data options" do
+    let(:argv) { ["configmaps", "create", "--data", payload, "--data-file", "-"] }
+    let(:payload) { { new_key: "new value" }.to_json }
+
+    context "with invalid update" do
+      it "does not error" do
+        expect { output }.to_not raise_error
+
+        expect(stderr).to eq("ERROR: Config map data can only be passed by either 'data' or 'data-file'\n")
+
+        expect(stdout).to eq("")
+      end
+    end
+  end
+end

data/spec/commands/configmaps/destroy_spec.rb

@@ -0,0 +1,156 @@
+require "spec_helper"
+
+describe "brightbox configmaps destroy" do
+  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[configmaps destroy] }
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("ERROR: You must specify config map IDs as arguments\n")
+
+      expect(stdout).to match("")
+    end
+  end
+
+  context "with one argument" do
+    let(:argv) { %w[configmaps destroy cfg-xdwe2] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-xdwe2")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-xdwe2"
+          }.to_json
+        )
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-xdwe2"
+          }.to_json
+        )
+
+      stub_request(:delete, "http://api.brightbox.localhost/1.0/config_maps/cfg-xdwe2")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 202,
+          body: {
+            id: "cfg-xdwe2"
+          }.to_json
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to match("Destroying cfg-xdwe2\n")
+
+      expect(stdout).to eq("")
+    end
+  end
+
+  context "with multiple arguments" do
+    let(:argv) { %w[configmaps destroy cfg-12ds4 cfg-vf567] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-12ds4")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-12ds4"
+          }.to_json
+        )
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-12ds4"
+          }.to_json
+        )
+
+      stub_request(:delete, "http://api.brightbox.localhost/1.0/config_maps/cfg-12ds4")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-12ds4"
+          }.to_json
+        )
+
+      stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-vf567")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-vf567"
+          }.to_json
+        )
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-vf567"
+          }.to_json
+        )
+
+      stub_request(:delete, "http://api.brightbox.localhost/1.0/config_maps/cfg-vf567")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-vf567"
+          }.to_json
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to match("Destroying cfg-12ds4\n")
+      expect(stderr).to match("Destroying cfg-vf567\n")
+
+      expect(stdout).to eq("")
+    end
+  end
+
+  context "with unknown argument" do
+    let(:argv) { %w[configmaps destroy cfg-lk234] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-lk234")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 404,
+          body: ""
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to match("Couldn't find cfg-lk234\n")
+
+      expect(stdout).to eq("")
+    end
+  end
+end

data/spec/commands/configmaps/list_spec.rb

@@ -0,0 +1,96 @@
+require "spec_helper"
+
+describe "brightbox configmaps list" do
+  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[configmaps list] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: config_maps_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.*name")
+
+        expect(stdout).to match("cfg-12345")
+        expect(stdout).to match("Test 12345")
+
+        expect(stdout).to match("cfg-abcde")
+        expect(stdout).to match("Test ABCDE")
+      end
+    end
+  end
+
+  context "with identifier" do
+    let(:argv) { %w[configmaps list cfg-312re] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-312re")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-312re",
+            name: "My config",
+            data: {
+              key: "value"
+            }
+          }.to_json
+        )
+    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.*name")
+
+        expect(stdout).to match("cfg-312re")
+        expect(stdout).to match("My config")
+      end
+    end
+  end
+
+  def config_maps_response
+    [
+      {
+        id: "cfg-12345",
+        name: "Test 12345"
+      },
+      {
+        id: "cfg-abcde",
+        name: "Test ABCDE"
+      }
+    ].to_json
+  end
+end