brightbox-cli 4.3.1 → 4.4.0

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

data/spec/commands/configmaps/show_spec.rb

@@ -0,0 +1,236 @@
+require "spec_helper"
+
+describe "brightbox configmaps show" 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 show] }
+
+    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: cfg-12345")
+        expect(stdout).to match("id: cfg-abcde")
+      end
+    end
+  end
+
+  context "with identifier" do
+    let(:argv) { %w[configmaps show cfg-lk432] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-lk432")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-lk432",
+            name: "Staging Config Example",
+            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: cfg-lk432")
+        expect(stdout).to match("name: Staging Config Example")
+      end
+    end
+  end
+
+  context "with '--data' output" do
+    context "with multiple IDs" do
+      let(:argv) { %w[configmaps show --data cfg-m543s cfg-klds4] }
+
+      it "does not error" do
+        expect { output }.to_not raise_error
+
+        expect(stderr).to eq("ERROR: You can only access data for a single config map at a time\n")
+
+        expect(stdout).to eq("")
+      end
+    end
+
+    context "without '--format'" do
+      let(:argv) { %w[configmaps show --data cfg-xd3d4] }
+
+      before do
+        stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-xd3d4")
+          .with(query: hash_including(account_id: "acc-12345"))
+          .to_return(
+            status: 200,
+            body: {
+              id: "cfg-lk432",
+              name: "Staging Config Example",
+              data: {
+                key: "value",
+                name: "key name"
+              }
+            }.to_json
+          )
+      end
+
+      it "does not error" do
+        expect { output }.to_not raise_error
+
+        expect(stderr).not_to match("ERROR")
+
+        expect(stdout).to eq(%({"key":"value","name":"key name"}\n))
+      end
+    end
+
+    context "with '--format json'" do
+      let(:argv) { %w[configmaps show --data --format json cfg-xd3d4] }
+
+      before do
+        stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-xd3d4")
+          .with(query: hash_including(account_id: "acc-12345"))
+          .to_return(
+            status: 200,
+            body: {
+              id: "cfg-lk432",
+              name: "Staging Config Example",
+              data: {
+                key: "value",
+                name: "key name"
+              }
+            }.to_json
+          )
+      end
+
+      it "does not error" do
+        expect { output }.to_not raise_error
+
+        expect(stderr).to eq("")
+
+        expect(stdout).to eq(%({"key":"value","name":"key name"}\n))
+      end
+    end
+
+    context "without '--format text'" do
+      let(:argv) { %w[configmaps show --data --format text cfg-xd3d4] }
+
+      before do
+        stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-xd3d4")
+          .with(query: hash_including(account_id: "acc-12345"))
+          .to_return(
+            status: 200,
+            body: {
+              id: "cfg-lk432",
+              name: "Staging Config Example",
+              data: {
+                key: "value",
+                name: "key name"
+              }
+            }.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("^             key: value")
+          expect(stdout).to match("^            name: key name")
+        end
+      end
+    end
+  end
+
+  context "with '--format' without 'data'" do
+    let(:argv) { %w[configmaps show --format json cfg-lk432] }
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to match("ERROR: The 'format' option can only be used with 'data'")
+
+      expect(stdout).to eq("")
+    end
+  end
+
+  context "with simple output" do
+    let(:argv) { %w[--simple configmaps show cfg-lk432] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-lk432")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-lk432",
+            name: "Staging Config Example",
+            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\tcfg-lk432$")
+        expect(stdout).to match("^name\tStaging Config Example$")
+      end
+    end
+  end
+
+  def config_maps_response
+    [
+      {
+        id: "cfg-12345",
+        name: "Test 12345",
+        data: {}
+      },
+      {
+        id: "cfg-abcde",
+        name: "Test ABCDE",
+        data: {
+          key: "value"
+        }
+      }
+    ].to_json
+  end
+end

data/spec/commands/configmaps/update_spec.rb

@@ -0,0 +1,342 @@
+require "spec_helper"
+require "tempfile"
+
+describe "brightbox configmaps update" 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 update] }
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("ERROR: You must specify the config map ID as the first argument\n")
+
+      expect(stdout).to match("")
+    end
+  end
+
+  context "with new name" do
+    let(:argv) { %w[configmaps update --name New cfg-0932s] }
+
+    before do
+      stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-0932s")
+        .with(query: hash_including(account_id: "acc-12345"))
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-0932s",
+            name: "Old"
+          }.to_json
+        )
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-0932s",
+            name: "New",
+            data: {}
+          }.to_json
+        )
+
+      stub_request(:put, "http://api.brightbox.localhost/1.0/config_maps/cfg-0932s")
+        .with(query: hash_including(account_id: "acc-12345"),
+              body: {
+                name: "New"
+              })
+        .to_return(
+          status: 200,
+          body: {
+            id: "cfg-0932s",
+            name: "New",
+            data: {}
+          }.to_json
+        )
+    end
+
+    it "does not error" do
+      expect { output }.to_not raise_error
+
+      expect(stderr).to eq("Updating cfg-0932s\n")
+
+      aggregate_failures do
+        expect(stdout).to match("cfg-0932s")
+        expect(stdout).to match("New")
+      end
+    end
+  end
+
+  context "with new map from 'data'" do
+    let(:argv) { ["configmaps", "update", "--data", payload, "cfg-25hrt"] }
+
+    context "with valid update" do
+      let(:payload) { { new_key: "new value" }.to_json }
+
+      before do
+        stub_request(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-25hrt")
+          .with(query: hash_including(account_id: "acc-12345"))
+          .to_return(
+            status: 200,
+            body: {
+              id: "cfg-25hrt",
+              name: "",
+              data: {
+                old_key: "old value"
+              }
+            }.to_json
+          )
+          .to_return(
+            status: 200,
+            body: {
+              id: "cfg-25hrt",
+              name: "",
+              data: {
+                new_key: "new value"
+              }
+            }.to_json
+          )
+
+        stub_request(:put, "http://api.brightbox.localhost/1.0/config_maps/cfg-25hrt")
+          .with(query: hash_including(account_id: "acc-12345"),
+                body: {
+                  data: {
+                    new_key: "new value"
+                  }
+                })
+          .to_return(
+            status: 200,
+            body: {
+              id: "cfg-25hrt",
+              name: "",
+              data: {
+                new_key: "new value"
+              }
+            }.to_json
+          )
+      end
+
+      it "does not error" do
+        expect { output }.to_not raise_error
+
+        expect(stderr).to eq("Updating cfg-25hrt\n")
+
+        aggregate_failures do
+          expect(stdout).to match("cfg-25hrt")
+        end
+      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", "update", "--data-file", data_file.path, "cfg-gr45a"] }
+      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(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-gr45a")
+            .with(query: hash_including(account_id: "acc-12345"))
+            .to_return(
+              status: 200,
+              body: {
+                id: "cfg-gr45a",
+                name: "",
+                data: {
+                  old_key: "old setting"
+                }
+              }.to_json
+            )
+            .to_return(
+              status: 200,
+              body: {
+                id: "cfg-gr45a",
+                name: "",
+                data: {
+                  new_key: "new setting"
+                }
+              }.to_json
+            )
+
+          stub_request(:put, "http://api.brightbox.localhost/1.0/config_maps/cfg-gr45a")
+            .with(query: hash_including(account_id: "acc-12345"),
+                  body: {
+                    data: {
+                      new_key: "new setting"
+                    }
+                  })
+            .to_return(
+              status: 200,
+              body: {
+                id: "cfg-gr45a",
+                name: "",
+                data: {
+                  new_key: "new setting"
+                }
+              }.to_json
+            )
+        end
+
+        it "does not error" do
+          expect { output }.to_not raise_error
+
+          expect(stderr).to eq("Updating cfg-gr45a\n")
+
+          aggregate_failures do
+            expect(stdout).to match("cfg-gr45a")
+          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 "when '-' is used for STDIN" do
+      let(:argv) { ["configmaps", "update", "--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(:get, "http://api.brightbox.localhost/1.0/config_maps/cfg-stdin")
+            .with(query: hash_including(account_id: "acc-12345"))
+            .to_return(
+              status: 200,
+              body: {
+                id: "cfg-stdin",
+                name: "",
+                data: {
+                  use_stdin: true
+                }
+              }.to_json
+            )
+            .to_return(
+              status: 200,
+              body: {
+                id: "cfg-stdin",
+                name: "",
+                data: {
+                  use_stdin: true
+                }
+              }.to_json
+            )
+
+          stub_request(:put, "http://api.brightbox.localhost/1.0/config_maps/cfg-stdin")
+            .with(query: hash_including(account_id: "acc-12345"),
+                  body: {
+                    data: {
+                      use_stdin: true
+                    }
+                  })
+            .to_return(
+              status: 200,
+              body: {
+                id: "cfg-stdin",
+                name: "",
+                data: {
+                  use_stdin: true
+                }
+              }.to_json
+            )
+        end
+
+        it "does not error" do
+          expect { output }.to_not raise_error
+
+          expect(stderr).to eq("Updating cfg-stdin\n")
+
+          aggregate_failures do
+            expect(stdout).to match("cfg-stdin")
+          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", "update", "--data", payload, "--data-file", "-", "cfg-25hrt"] }
+      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
+end