central-cli 0.6.3

data/spec/central/cli/app/docker_helper_spec.rb

@@ -0,0 +1,102 @@
+require_relative '../../../spec_helper'
+require 'central/cli/apps/docker_helper'
+
+describe Central::Cli::Apps::DockerHelper do
+  let(:subject) do
+    Class.new { include Central::Cli::Apps::DockerHelper }.new
+  end
+
+  let(:services_with_valid_hooks) do
+    {
+      'test_service' => {
+        'build' => '.',
+        'image' => 'test_service',
+        'hooks' => {
+          'pre_build' => [
+            { 'cmd' => 'echo PREBUILD1', 'name' => 'hook1' },
+            { 'cmd' => 'echo PREBUILD2', 'name' => 'hook2' }
+          ]
+        }
+      }
+    }
+  end
+
+  let(:services_with_invalid_hook) do
+    {
+      'test_service' => {
+        'build' => '.',
+        'image' => 'test_service',
+        'hooks' => {
+          'pre_build' => [
+            { 'cmd' => 'echo PREBUILD1', 'name' => 'hook1' },
+            { 'cmd' => 'some_non_existing_command', 'name' => 'failing hook' }
+          ]
+        }
+      }
+    }
+  end
+
+  let(:services_with_no_hook) do
+    {
+      'test_service' => {
+        'build' => '.',
+        'image' => 'test_service'
+      }
+    }
+  end
+
+  describe '#validate_image_name' do
+    context 'when image name is valid' do
+      it 'returns true' do
+        expect(subject.validate_image_name('registry.central.local/image-name:latest')).to be_truthy
+        expect(subject.validate_image_name('my-registry.com/organization/image_name:latest')).to be_truthy
+        expect(subject.validate_image_name('my-registry.com:5000/organization/image_name:latest')).to be_truthy
+        expect(subject.validate_image_name('mysql:5.1')).to be_truthy
+        expect(subject.validate_image_name('wordpress')).to be_truthy
+      end
+    end
+  end
+
+  describe '#validate_image_name' do
+    context 'when image name is invalid' do
+      it 'returns false' do
+        expect(subject.validate_image_name('registry.central.local/image-name:')).to be_falsey
+        expect(subject.validate_image_name('mysql 5.1')).to be_falsey
+        expect(subject.validate_image_name('*.mydomain.com/mysql')).to be_falsey
+      end
+    end
+  end
+
+  describe '#run_pre_build_hook' do
+    context 'when hook defined' do
+      it 'runs the hook' do
+        allow(subject).to receive(:build_docker_image)
+        allow(subject).to receive(:push_docker_image)
+        expect(subject).to receive(:system).with('echo PREBUILD1'). and_return(true)
+        expect(subject).to receive(:system).with('echo PREBUILD2'). and_return(true)
+
+        subject.process_docker_images(services_with_valid_hooks)
+      end
+
+      it 'fails to run the hook' do
+        allow(subject).to receive(:build_docker_image)
+        allow(subject).to receive(:push_docker_image)
+        expect(subject).to receive(:system).with('echo PREBUILD1'). and_return(true)
+        expect(subject).to receive(:system).with('some_non_existing_command'). and_return(false)
+
+        expect do
+          subject.process_docker_images(services_with_invalid_hook)
+        end.to raise_error(SystemExit)
+      end
+    end
+    context 'when no hook defined' do
+      it 'runs no hooks' do
+        allow(subject).to receive(:build_docker_image)
+        allow(subject).to receive(:push_docker_image)
+        expect(subject).not_to receive(:run_pre_build_hook)
+
+        subject.process_docker_images(services_with_no_hook)
+      end
+    end
+  end
+end

data/spec/central/cli/app/scale_spec.rb

@@ -0,0 +1,49 @@
+require_relative '../../../spec_helper'
+require 'central/cli/apps/scale_command'
+
+describe Central::Cli::Apps::ScaleCommand do
+  include FixturesHelpers
+  include ClientHelpers
+
+  let(:subject) do
+    described_class.new(File.basename($PROGRAM_NAME))
+  end
+
+  let(:central_yml) do
+    fixture('wordpress-scaled.yml')
+  end
+
+  let(:central_yml_no_instances) do
+    fixture('wordpress.yml')
+  end
+
+  describe '#execute' do
+    before(:each) do
+      allow(subject).to receive(:current_dir).and_return('central-test')
+      allow(File).to receive(:exists?).and_return(true)
+      allow(File).to receive(:read).with("#{Dir.getwd}/central.yml").and_return(central_yml)
+    end
+
+    context 'when service already contains instances property' do
+      it 'aborts execution' do
+        expect do
+          subject.run(['wordpress', 3])
+        end.to raise_error(SystemExit)
+      end
+    end
+
+    context 'when service not found in YML' do
+      it 'aborts execution' do
+        expect do
+          subject.run(['mysql', 3])
+        end.to raise_error(SystemExit)
+      end
+    end
+
+    it 'scales given service' do
+      allow(File).to receive(:read).with("#{Dir.getwd}/central.yml").and_return(central_yml_no_instances)
+      expect(subject).to receive(:scale_service).with('1234567', 'central-test-wordpress', 3)
+      subject.run(['wordpress', 3])
+    end
+  end
+end

data/spec/central/cli/common_spec.rb

@@ -0,0 +1,117 @@
+require_relative '../../spec_helper'
+require 'central/cli/common'
+
+describe Central::Cli::Common do
+  let(:masters) do
+  end
+
+  let(:subject) do
+    Class.new do
+      include Central::Cli::Common
+    end.new
+  end
+
+  before(:each) do
+    allow(subject).to receive(:settings).and_return('current_server' => 'default',
+                                                    'servers' => [{}])
+    allow(subject).to receive(:save_settings)
+  end
+
+  describe '#current_stack' do
+    it 'returns nil by default' do
+      allow(subject).to receive(:settings).and_return('current_server' => 'alias',
+                                                      'servers' => [
+                                                        { 'name' => 'some_master', 'url' => 'some_master' },
+                                                        { 'name' => 'alias', 'url' => 'someurl' }
+                                                      ])
+      expect(subject.current_stack).to eq(nil)
+    end
+
+    it 'returns stack from env' do
+      expect(ENV).to receive(:[]).with('CENTRAL_STACK').and_return('foo')
+      expect(subject.current_stack).to eq('foo')
+    end
+
+    it 'returns stack from json' do
+      allow(subject).to receive(:settings).and_return('current_server' => 'alias',
+                                                      'servers' => [
+                                                        { 'name' => 'some_master', 'url' => 'some_master' },
+                                                        { 'name' => 'alias', 'url' => 'someurl', 'stack' => 'foo_stack' }
+                                                      ])
+      expect(subject.current_stack).to eq('foo_stack')
+    end
+
+    it 'returns nil if settings are not present' do
+      allow(subject).to receive(:current_master).and_raise(ArgumentError)
+      expect(subject.current_stack).to be_nil
+    end
+  end
+
+  describe '#api_url' do
+    it 'raises error by default' do
+      expect do
+        subject.api_url
+      end.to raise_error(ArgumentError)
+    end
+
+    it 'return url from env' do
+      expect(ENV).to receive(:[]).with('CENTRAL_URL').and_return('https://domain.com')
+      expect(subject.api_url).to eq('https://domain.com')
+    end
+  end
+
+  describe '#require_token' do
+    it 'raises error by default' do
+      expect do
+        subject.require_token
+      end.to raise_error(ArgumentError)
+    end
+
+    it 'return token from env' do
+      expect(ENV).to receive(:[]).with('CENTRAL_TOKEN').and_return('secret_token')
+      expect(subject.require_token).to eq('secret_token')
+    end
+  end
+
+  describe '#current_master' do
+    it 'return correct master info' do
+      allow(subject).to receive(:settings).and_return('current_server' => 'alias',
+                                                      'servers' => [
+                                                        { 'name' => 'some_master', 'url' => 'some_master' },
+                                                        { 'name' => 'alias', 'url' => 'someurl' }
+                                                      ])
+
+      expect(subject.current_master['url']).to eq('someurl')
+      expect(subject.current_master['name']).to eq('alias')
+    end
+  end
+
+  describe '#settings' do
+    it 'migrates old settings' do
+      RSpec::Mocks.space.proxy_for(subject).reset
+
+      allow(File).to receive(:exists?).and_return(true)
+      allow(File).to receive(:read).and_return('{
+        "server": {
+          "url": "https://master.domain.com:8443",
+          "stack": "my-stack",
+          "token": "central-token"
+        }
+      }')
+      expected_settings = {
+        'current_server' => 'default',
+        'servers' => [
+          {
+            'url' => 'https://master.domain.com:8443',
+            'stack' => 'my-stack',
+            'token' => 'central-token',
+            'name' => 'default'
+          }
+        ]
+      }
+      expect(File).to receive(:write).with(subject.settings_filename, JSON.pretty_generate(expected_settings))
+
+      subject.settings
+    end
+  end
+end

data/spec/central/cli/login_command_spec.rb

@@ -0,0 +1,31 @@
+require_relative '../../spec_helper'
+require 'central/cli/login_command'
+
+describe Central::Cli::LoginCommand do
+  describe '#register' do
+    let(:settings) do
+      { 'current_server' => 'alias',
+        'servers' => [
+          { 'name' => 'some_master', 'url' => 'some_master' },
+          { 'name' => 'alias', 'url' => 'someurl', 'token' => '123456' }
+        ]
+      }
+    end
+
+    let(:auth_client) do
+      double(Central::Client)
+    end
+
+    let(:subject) { described_class.new(File.basename($PROGRAM_NAME)) }
+
+    it 'asks email and password' do
+      allow(subject).to receive(:settings).and_return(settings)
+      expect(subject).to receive(:ask).once.with('Email: ').and_return('john.doe@acme.io')
+      allow(subject).to receive(:ask).once.with('Password: ').and_return('secret')
+      allow(Central::Client).to receive(:new).and_return(auth_client)
+      allow(auth_client).to receive(:post)
+      allow(subject).to receive(:request_server_info).and_return(true)
+      subject.run(['http://localhost:8443'])
+    end
+  end
+end

data/spec/central/cli/master/current_command_spec.rb

@@ -0,0 +1,55 @@
+require_relative '../../../spec_helper'
+require 'central/cli/master/current_command'
+
+describe Central::Cli::Master::CurrentCommand do
+  let(:settings) do
+    { 'current_server' => 'alias',
+      'servers' => [
+        { 'name' => 'some_master', 'url' => 'some_master' },
+        { 'name' => 'alias', 'url' => 'someurl', 'token' => '123456' }
+      ]
+    }
+  end
+
+  let(:subject) { described_class.new(File.basename($PROGRAM_NAME)) }
+
+  describe '#execute' do
+    it 'puts master name and URL' do
+      allow(subject).to receive(:settings).and_return(settings)
+
+      expect do
+        subject.run([])
+      end.to output(/alias.*someurl/).to_stdout
+    end
+
+    it 'only outputs name if name-flag is set' do
+      allow(subject).to receive(:settings).and_return(settings)
+
+      expect do
+        subject.run(['--name'])
+      end.to output("alias\n").to_stdout
+    end
+
+    it 'does not raise error when logged in' do
+      allow(subject).to receive(:settings).and_return(settings)
+
+      expect do
+        subject.run([])
+      end.to_not raise_error
+    end
+
+    it 'raises error when not logged in' do
+      allow(subject).to receive(:settings).and_return(
+        'current_server' => nil,
+        'servers' => [
+          { 'name' => 'some_master', 'url' => 'some_master' },
+          { 'name' => 'alias', 'url' => 'someurl', 'token' => '123456' }
+        ]
+      )
+
+      expect do
+        subject.run([])
+      end.to raise_error(ArgumentError)
+    end
+  end
+end

data/spec/central/cli/master/use_command_spec.rb

@@ -0,0 +1,37 @@
+require_relative '../../../spec_helper'
+require 'central/cli/master/use_command'
+
+describe Central::Cli::Master::UseCommand do
+  let(:subject) do
+    described_class.new(File.basename($PROGRAM_NAME))
+  end
+
+  let(:client) { spy(:client) }
+
+  let(:valid_settings) do
+    { 'current_server' => 'alias',
+      'servers' => [
+        { 'name' => 'some_master', 'url' => 'some_master' },
+        { 'name' => 'alias', 'url' => 'someurl', 'token' => '123456' }
+      ]
+    }
+  end
+
+  describe '#use' do
+    it 'should update current master' do
+      allow(subject).to receive(:client).and_return(client)
+      allow(subject).to receive(:settings).and_return(valid_settings)
+      expect(subject).to receive(:current_master=).with('some_master')
+      subject.run(['some_master'])
+    end
+
+    it 'should fetch stack list from master' do
+      allow(subject).to receive(:require_token).and_return('token')
+      allow(subject).to receive(:client).and_return(client)
+      allow(subject).to receive(:settings).and_return(valid_settings)
+      expect(subject).to receive(:current_master=).with('some_master')
+      expect(client).to receive(:get).with('stacks')
+      subject.run(['some_master'])
+    end
+  end
+end

data/spec/central/cli/master/users/invite_command_spec.rb

@@ -0,0 +1,34 @@
+require_relative '../../../../spec_helper'
+require 'central/cli/master/users_command'
+require 'central/cli/master/users/invite_command'
+
+describe Central::Cli::Master::Users::InviteCommand do
+  let(:subject) do
+    described_class.new(File.basename($PROGRAM_NAME))
+  end
+
+  let(:valid_settings) do
+    { 'current_server' => 'alias',
+      'servers' => [
+        { 'name' => 'some_master', 'url' => 'some_master' },
+        { 'name' => 'alias', 'url' => 'someurl', 'token' => '123456' }
+      ]
+    }
+  end
+
+  let(:client) { spy(:client) }
+
+  describe '#invite' do
+    before(:each) do
+      allow(subject).to receive(:client).and_return(client)
+      allow(subject).to receive(:settings).and_return(valid_settings)
+    end
+
+    it 'makes invitation request for all given users' do
+      expect(client).to receive(:post).with('users', email: 'john@example.org').once
+      expect(client).to receive(:post).with('users', email: 'jane@example.org').once
+
+      subject.run(['john@example.org', 'jane@example.org'])
+    end
+  end
+end

data/spec/central/cli/master/users/remove_command_spec.rb

@@ -0,0 +1,26 @@
+require_relative '../../../../spec_helper'
+require 'central/cli/master/users_command'
+require 'central/cli/master/users/remove_command'
+
+describe Central::Cli::Master::Users::RemoveCommand do
+  include ClientHelpers
+
+  describe '#execute' do
+    it 'requires api url' do
+      expect(subject).to receive(:require_api_url).once
+      subject.run(['john@domain.com'])
+    end
+
+    it 'requires token' do
+      expect(subject).to receive(:require_token).and_return(token)
+      subject.run(['john@domain.com'])
+    end
+
+    it 'sends email to master' do
+      expect(client).to receive(:delete).with(
+        'users/john@domain.com'
+      )
+      subject.run(['john@domain.com'])
+    end
+  end
+end