kontena-cli 1.0.6 → 1.1.0.pre1

data/spec/kontena/cli/nodes/health_command_spec.rb

@@ -0,0 +1,206 @@
+require 'kontena/cli/nodes/health_command'
+
+describe Kontena::Cli::Nodes::HealthCommand do
+  include ClientHelpers
+  include OutputHelpers
+
+  before do
+    allow(subject).to receive(:health_icon) {|health| health.inspect }
+  end
+
+  context "for an online node" do
+    let :node do
+      {
+        "connected" => true,
+        "name" => "node-4",
+        "node_number" => 4,
+        "initial_member" => false,
+      }
+    end
+
+    let :grid_health do
+      :ok
+    end
+
+    describe '#node_health' do
+      it "returns ok" do
+        expect(subject.node_health(node, grid_health)).to eq(:ok)
+      end
+    end
+
+    describe '#show_node_health' do
+      it "returns true" do
+        expect{subject.show_node_health(node)}.to return_and_output true,
+          ":ok Node is online"
+      end
+    end
+  end
+
+  context "for an offline node" do
+    let :node do
+      {
+        "connected" => false,
+        "name" => "node-4",
+        "node_number" => 4,
+        "initial_member" => false,
+      }
+    end
+
+    let :grid_health do
+      :ok
+    end
+
+    describe '#node_health' do
+      it "returns offline" do
+        expect(subject.node_health(node, grid_health)).to eq(:offline)
+      end
+    end
+
+    describe '#show_node_health' do
+      it "returns false" do
+        expect{subject.show_node_health(node)}.to return_and_output false,
+          ":offline Node is offline"
+      end
+    end
+  end
+
+  context "for an online initial node in an ok grid" do
+    let :node do
+      {
+        "connected" => true,
+        "name" => "node-1",
+        "node_number" => 1,
+        "initial_member" => true,
+      }
+    end
+
+    let :grid_health do
+      :ok
+    end
+
+    describe '#node_health' do
+      it "returns ok" do
+        expect(subject.node_health(node, grid_health)).to eq(:ok)
+      end
+    end
+
+    describe '#show_node_health' do
+      it "returns true" do
+        expect{subject.show_node_health(node)}.to return_and_output true,
+          ":ok Node is online"
+      end
+    end
+  end
+
+  context "for an online initial node in a warning grid" do
+    let :node do
+      {
+        "connected" => true,
+        "name" => "node-1",
+        "node_number" => 1,
+        "initial_member" => true,
+      }
+    end
+
+    let :grid_health do
+      :warning
+    end
+
+    describe '#node_health' do
+      it "returns ok" do
+        expect(subject.node_health(node, grid_health)).to eq(:warning)
+      end
+    end
+
+    describe '#show_node_health' do
+      it "returns true" do
+        expect{subject.show_node_health(node)}.to return_and_output true,
+          ":ok Node is online"
+      end
+    end
+  end
+
+  context "for an online initial node in an error grid" do
+    let :node do
+      {
+        "connected" => true,
+        "name" => "node-1",
+        "node_number" => 1,
+        "initial_member" => true,
+      }
+    end
+
+    let :grid_health do
+      :error
+    end
+
+    describe '#node_health' do
+      it "returns ok" do
+        expect(subject.node_health(node, grid_health)).to eq(:error)
+      end
+    end
+
+    describe '#show_node_health' do
+      it "returns true" do
+        expect{subject.show_node_health(node)}.to return_and_output true,
+          ":ok Node is online"
+      end
+    end
+  end
+
+  context "for an offline initial node in a warning grid" do
+    let :node do
+      {
+        "connected" => false,
+        "name" => "node-1",
+        "node_number" => 1,
+        "initial_member" => true,
+      }
+    end
+
+    let :grid_health do
+      :warning
+    end
+
+    describe '#node_health' do
+      it "returns offline" do
+        expect(subject.node_health(node, grid_health)).to eq(:offline)
+      end
+    end
+
+    describe '#show_node_health' do
+      it "returns false" do
+        expect{subject.show_node_health(node)}.to return_and_output false,
+          ":offline Node is offline"
+      end
+    end
+  end
+
+  context "for an offline initial node in a error grid" do
+    let :node do
+      {
+        "connected" => false,
+        "name" => "node-1",
+        "node_number" => 1,
+        "initial_member" => true,
+      }
+    end
+
+    let :grid_health do
+      :error
+    end
+
+    describe '#node_health' do
+      it "returns offline" do
+        expect(subject.node_health(node, grid_health)).to eq(:offline)
+      end
+    end
+
+    describe '#show_node_health' do
+      it "returns false" do
+        expect{subject.show_node_health(node)}.to return_and_output false,
+          ":offline Node is offline"
+      end
+    end
+  end
+end

data/spec/kontena/cli/nodes/list_command_spec.rb

@@ -0,0 +1,205 @@
+describe Kontena::Cli::Nodes::ListCommand do
+  include ClientHelpers
+  include OutputHelpers
+
+  before do
+    allow(subject).to receive(:health_icon) {|health| health.inspect }
+  end
+
+  describe '#show_grid_nodes' do
+    context "For a initial_size=3 grid" do
+      let :grid do
+        {
+          "id" => "test",
+          "name" => "test",
+          "initial_size" => 3,
+          "node_count" => 1,
+        }
+      end
+
+      context "with a single node" do
+        let :grid_nodes do
+          { "nodes" => [
+            {
+              "connected" => true,
+              "name" => "node-1",
+              "node_number" => 1,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+          ] }
+        end
+
+        it "outputs node with error" do
+          expect{subject.show_grid_nodes(grid, grid_nodes['nodes'])}.to output_table [
+            [':error node-1', '1.1-dev', 'online', '1 / 3', '-'],
+          ]
+        end
+      end
+
+      context "with a single online node" do
+        let :grid_nodes do
+          { "nodes" => [
+            {
+              "connected" => true,
+              "name" => "node-1",
+              "node_number" => 1,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+            {
+              "connected" => false,
+              "name" => "node-2",
+              "node_number" => 2,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+          ] }
+        end
+
+        it "outputs online node with error" do
+          expect{subject.show_grid_nodes(grid, grid_nodes['nodes'])}.to output_table [
+            [':error node-1', '1.1-dev', 'online', '1 / 3', '-'],
+            [':offline node-2', '1.1-dev', 'offline', '2 / 3', '-'],
+          ]
+        end
+      end
+
+      context "with two online nodes" do
+        let :grid_nodes do
+          { "nodes" => [
+            {
+              "connected" => true,
+              "name" => "node-1",
+              "node_number" => 1,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+            {
+              "connected" => true,
+              "name" => "node-2",
+              "node_number" => 2,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+          ] }
+        end
+
+        it "outputs both nodes with warning" do
+          expect{subject.show_grid_nodes(grid, grid_nodes['nodes'])}.to output_table [
+            [':warning node-1', '1.1-dev', 'online', '1 / 3', '-'],
+            [':warning node-2', '1.1-dev', 'online', '2 / 3', '-'],
+          ]
+        end
+      end
+
+      context "with two online nodes and one offline node" do
+        let :grid_nodes do
+          { "nodes" => [
+            {
+              "connected" => true,
+              "name" => "node-1",
+              "node_number" => 1,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+            {
+              "connected" => true,
+              "name" => "node-2",
+              "node_number" => 2,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+            {
+              "connected" => false,
+              "name" => "node-3",
+              "node_number" => 3,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+          ] }
+        end
+
+        it "outputs two nodes with warning and one offline" do
+          expect{subject.show_grid_nodes(grid, grid_nodes['nodes'])}.to output_table [
+            [':warning node-1', '1.1-dev', 'online',  '1 / 3', '-'],
+            [':warning node-2', '1.1-dev', 'online',  '2 / 3', '-'],
+            [':offline node-3', '1.1-dev', 'offline', '3 / 3', '-'],
+          ]
+        end
+      end
+
+      context "with two online initial nodes and one non-initial node" do
+        let :grid_nodes do
+          { "nodes" => [
+            {
+              "connected" => true,
+              "name" => "node-1",
+              "node_number" => 1,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+            {
+              "connected" => true,
+              "name" => "node-2",
+              "node_number" => 2,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+            {
+              "connected" => true,
+              "name" => "node-4",
+              "node_number" => 4,
+              "initial_member" => false,
+              'agent_version' => '1.1-dev',
+            },
+          ] }
+        end
+
+        it "outputs two nodes with warning and one online" do
+          expect{subject.show_grid_nodes(grid, grid_nodes['nodes'])}.to output_table [
+            [':warning node-1', '1.1-dev', 'online',  '1 / 3', '-'],
+            [':warning node-2', '1.1-dev', 'online',  '2 / 3', '-'],
+            [':ok node-4', '1.1-dev', 'online', '-', '-'],
+          ]
+        end
+      end
+
+      context "with three online initial nodes" do
+        let :grid_nodes do
+          { "nodes" => [
+            {
+              "connected" => true,
+              "name" => "node-1",
+              "node_number" => 1,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+            {
+              "connected" => true,
+              "name" => "node-2",
+              "node_number" => 2,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+            {
+              "connected" => true,
+              "name" => "node-3",
+              "node_number" => 3,
+              "initial_member" => true,
+              'agent_version' => '1.1-dev',
+            },
+          ] }
+        end
+
+        it "outputs three nodes with ok" do
+          expect{subject.show_grid_nodes(grid, grid_nodes['nodes'])}.to output_table [
+            [':ok node-1', '1.1-dev', 'online', '1 / 3', '-'],
+            [':ok node-2', '1.1-dev', 'online', '2 / 3', '-'],
+            [':ok node-3', '1.1-dev', 'online', '3 / 3', '-'],
+          ]
+        end
+      end
+    end
+  end
+end

data/spec/kontena/cli/vault/export_spec.rb

@@ -0,0 +1,32 @@
+require_relative "../../../spec_helper"
+require 'kontena/cli/vault/export_command'
+
+describe Kontena::Cli::Vault::ExportCommand do
+
+  include RequirementsHelper
+
+  let(:subject) do
+    described_class.new(File.basename($0))
+  end
+
+  expect_to_require_current_master
+
+  before(:each) do
+    allow(Kontena::Cli::Config.instance).to receive(:current_master).and_return(Kontena::Cli::Config::Server.new)
+  end
+
+  it 'goes through the list of vault keys and outputs a yaml' do
+    expect(Kontena).to receive(:run).with(/^vault ls/, returning: :result).and_return(['foo', 'bar'])
+    expect(Kontena).to receive(:run).with(/^vault read.*bar/, returning: :result).and_return('barbar')
+    expect(Kontena).to receive(:run).with(/^vault read.*foo/, returning: :result).and_return('foofoo')
+    expect{subject.run([])}.to output(/bar: barbar\nfoo: foofoo/).to_stdout
+  end
+
+  it 'goes through the list of vault keys and outputs a json' do
+    expect(Kontena).to receive(:run).with(/^vault ls/, returning: :result).and_return(['foo', 'bar'])
+    expect(Kontena).to receive(:run).with(/^vault read.*bar/, returning: :result).and_return('barbar')
+    expect(Kontena).to receive(:run).with(/^vault read.*foo/, returning: :result).and_return('foofoo')
+    expect{subject.run(['--json'])}.to output(/\"bar\":\"barbar\",\"foo\":\"foofoo\"/).to_stdout
+  end
+
+end

data/spec/kontena/cli/vault/import_spec.rb

@@ -0,0 +1,69 @@
+require_relative "../../../spec_helper"
+require 'kontena/cli/vault/import_command'
+
+
+describe Kontena::Cli::Vault::ImportCommand do
+
+  include RequirementsHelper
+
+  let(:subject) do
+    described_class.new(File.basename($0))
+  end
+
+  before(:each) do
+    allow(Kontena::Cli::Config.instance).to receive(:current_master).and_return(Kontena::Cli::Config::Server.new)
+    allow(Kontena::Cli::Config.instance).to receive(:current_grid).and_return("foofoo")
+  end
+
+  expect_to_require_current_master
+
+  it 'asks for confirmation' do
+    expect(subject).to receive(:confirm).and_raise(RuntimeError, 'confirm')
+    expect(File).to receive(:read).with('foo.yml').and_return("foo: bar\n")
+    expect{subject.run(['foo.yml'])}.to raise_error(RuntimeError, 'confirm')
+  end
+
+  it 'dies if the yml contains something odd' do
+    expect(File).to receive(:read).with('foo.yml').and_return({foo: 'bar', bar: { foo: ["bar"] }}.to_yaml)
+    expect(subject).to receive(:exit_with_error).with(/Invalid value/).and_raise(RuntimeError, 'invalid')
+    expect{subject.run(['foo.yml'])}.to raise_error(RuntimeError, 'invalid')
+  end
+
+  it 'runs vault write for kv-pairs in yaml' do
+    expect(File).to receive(:read).with('foo.yml').and_return("foo: bar\nbar: foo\n")
+    expect(Kontena).to receive(:run).with(/vault update.*foo bar/).and_return(0)
+    expect(Kontena).to receive(:run).with(/vault update.*bar foo/).and_return(0)
+    subject.run(['--force', 'foo.yml'])
+  end
+
+  it 'runs vault rm for kv-pairs with null value in yaml' do
+    expect(File).to receive(:read).with('foo.yml').and_return("foo: bar\nbar: null\n")
+    expect(Kontena).to receive(:run).with(/vault update.*foo bar/).and_return(0)
+    expect(Kontena).to receive(:run).with(/vault rm.*bar/).and_return(0)
+    subject.run(['--force', 'foo.yml'])
+  end
+
+  it 'runs vault rm for kv-pairs with empty value in yaml when --empty-is-null' do
+    expect(File).to receive(:read).with('foo.yml').and_return("foo: bar\nbar:\n")
+    expect(Kontena).to receive(:run).with(/vault update.*foo bar/).and_return(0)
+    expect(Kontena).to receive(:run).with(/vault rm.*bar/).and_return(0)
+    subject.run(['--force', '--empty-is-null', 'foo.yml'])
+  end
+
+  it 'does not run vault rm for kv-pairs with empty value in yaml when no --empty-is-null' do
+    expect(File).to receive(:read).with('foo.yml').and_return("foo: bar\nbar: \"\"\n")
+    expect(Kontena).to receive(:run).with(/vault update.*foo bar/).and_return(0)
+    expect(Kontena).to receive(:run).with(/vault update.*bar ''/).and_return(0)
+    expect(Kontena).not_to receive(:run).with(/vault rm.*bar/)
+    subject.run(['--force', 'foo.yml'])
+  end
+
+  it 'doesnt vault rm for kv-pairs with null value in yaml when --skip-null used' do
+    expect(File).to receive(:read).with('foo.yml').and_return("foo: bar\nbar: null\n")
+    expect(Kontena).to receive(:run).with(/vault update.*foo bar/).and_return(0)
+    expect(Kontena).not_to receive(:run).with(/vault rm.*bar/)
+    subject.run(['--force', '--skip-null', 'foo.yml'])
+  end
+
+end
+