kontena-cli 1.4.0.pre6 → 1.4.0.pre7

data/spec/kontena/cli/stacks/yaml/service_extender_spec.rb

@@ -29,21 +29,30 @@ describe Kontena::Cli::Stacks::YAML::ServiceExtender do
 
     context 'environment variables' do
       it 'inherites env vars from upper level' do
-        from = { 'environment' => ['FOO=bar'] }
+        from = { 'env' => ['FOO=bar'] }
         to = {}
         result = described_class.new(to).extend_from(from)
         expect(result['environment']).to eq(['FOO=bar'])
       end
 
       it 'overrides values' do
-        from = { 'environment' => ['FOO=bar'] }
+        from = { 'env' => ['FOO=bar'] }
         to = { 'environment' => ['FOO=baz'] }
         result = described_class.new(to).extend_from(from)
         expect(result['environment']).to eq(['FOO=baz'])
       end
 
       it 'combines variables' do
-        from = { 'environment' => ['FOO=bar'] }
+        from = { 'env' => ['FOO=bar'] }
+        to = { 'environment' => ['BAR=baz'] }
+        result = described_class.new(to).extend_from(from)
+        expect(result['environment'].include?('BAR=baz')).to be_truthy
+        expect(result['environment'].include?('FOO=bar')).to be_truthy
+        expect(result['environment'].size).to eq 2
+      end
+
+      it 'combines and overrides variables' do
+        from = { 'env' => ['FOO=bar', 'BAR=buz'] }
         to = { 'environment' => ['BAR=baz'] }
         result = described_class.new(to).extend_from(from)
         expect(result['environment'].include?('BAR=baz')).to be_truthy

data/spec/kontena/cli/stacks/yaml/stack_file_loader/file_loader_spec.rb

@@ -0,0 +1,47 @@
+require 'kontena/cli/stacks/yaml/stack_file_loader'
+
+describe Kontena::Cli::Stacks::YAML::FileLoader do
+  include FixturesHelpers
+
+  describe '#match' do
+    it 'returns false when file does not exist' do
+      expect(File).to receive(:exist?).and_return(false)
+      expect(described_class.match?('foofoo')).to be_falsey
+    end
+
+    it 'returns true when file exists' do
+      expect(File).to receive(:exist?).and_return(true)
+      expect(described_class.match?('foofoo')).to be_truthy
+    end
+  end
+
+  describe '#with_context' do
+    it 'absolutizes paths' do
+      some_file = Dir.glob('*').first
+      expect(described_class.with_context(some_file)).to eq File.absolute_path(some_file)
+    end
+  end
+
+  describe 'instance methods' do
+    let(:subject) { described_class.new(fixture_path('kontena_v3.yml')) }
+
+    describe '#read_content' do
+      it 'reads the file' do
+        expect(File).to receive(:read).with(fixture_path('kontena_v3.yml')).and_call_original
+        expect(subject.read_content).to match /^stack:/
+      end
+    end
+
+    describe '#origin' do
+      it 'returns "file"' do
+        expect(subject.origin).to eq 'file'
+      end
+    end
+
+    describe '#registry' do
+      it 'returns "file://"' do
+        expect(subject.registry).to eq 'file://'
+      end
+    end
+  end
+end

data/spec/kontena/cli/stacks/yaml/stack_file_loader/registry_loader_spec.rb

@@ -0,0 +1,53 @@
+require 'kontena/stacks_cache'
+require 'yaml'
+
+describe Kontena::Cli::Stacks::YAML::RegistryLoader do
+  include FixturesHelpers
+
+  before do
+    allow(File).to receive(:exist?).and_return(false)
+  end
+
+  describe '#match' do
+    it 'returns false when input is not a registry path' do
+      expect(described_class.match?('foofoo')).to be_falsey
+    end
+
+    it 'returns true when file exists' do
+      expect(described_class.match?('user/stack')).to be_truthy
+      expect(described_class.match?('user/stack:1.0.0')).to be_truthy
+    end
+  end
+
+  describe 'instance methods' do
+    let(:subject) { described_class.new('user/stack') }
+
+    before do
+      allow(Kontena::StacksCache).to receive(:pull).with('user/stack').and_return(
+        fixture('kontena_v3.yml')
+      )
+    end
+
+    describe '#read_content' do
+      it 'reads the file' do
+        allow(Kontena::Cli::Config).to receive(:current_account).and_return(double(stacks_url: 'foo'))
+        expect(subject.read_content).to match /^stack:/
+      end
+    end
+
+    describe '#origin' do
+      it 'returns "uri"' do
+        expect(subject.origin).to eq 'registry'
+      end
+    end
+
+    describe '#registry' do
+      it 'returns "file://"' do
+        allow(Kontena::Cli::Config).to receive(:current_account).and_return(double(stacks_url: 'foo'))
+        expect(subject.registry).to eq 'foo'
+      end
+    end
+  end
+end
+
+

data/spec/kontena/cli/stacks/yaml/stack_file_loader/uri_loader_spec.rb

@@ -0,0 +1,53 @@
+require 'kontena/cli/stacks/yaml/stack_file_loader'
+require 'yaml'
+
+describe Kontena::Cli::Stacks::YAML::UriLoader do
+  include FixturesHelpers
+
+  before do
+    allow(File).to receive(:exist?).and_return(false)
+  end
+
+  describe '#match' do
+    it 'returns false when input is not an url' do
+      expect(described_class.match?('foofoo')).to be_falsey
+    end
+
+    it 'returns true when file exists' do
+      expect(described_class.match?('http://foofoo')).to be_truthy
+    end
+  end
+
+  describe 'instance methods' do
+    let(:subject) { described_class.new('http://foofoo/foo.yml') }
+
+    before do
+      stub_request(:get, 'http://foofoo/foo.yml').to_return(
+        status: 200,
+        headers: {
+          'Content-Type' => 'application/yml',
+        },
+        body: fixture('kontena_v3.yml')
+      )
+    end
+
+    describe '#read_content' do
+      it 'reads the file' do
+        expect(subject.read_content).to match /^stack:/
+      end
+    end
+
+    describe '#origin' do
+      it 'returns "uri"' do
+        expect(subject.origin).to eq 'uri'
+      end
+    end
+
+    describe '#registry' do
+      it 'returns "file://"' do
+        expect(subject.registry).to eq 'file://'
+      end
+    end
+  end
+end
+

data/spec/kontena/cli/stacks/yaml/stack_file_loader_spec.rb

@@ -0,0 +1,104 @@
+require 'kontena/cli/stacks/yaml/stack_file_loader'
+
+describe Kontena::Cli::Stacks::YAML::StackFileLoader do
+  include FixturesHelpers
+
+  before do
+    allow(File).to receive(:exist?).and_return(false)
+    [:exist?, :read].each do |meth|
+      ['docker-compose_v2.yml', 'kontena_v3.yml'].each do |file|
+        allow(File).to receive(meth)
+          .with(fixture_path(file))
+          .and_call_original
+      end
+    end
+  end
+
+  describe '#for' do
+    it 'returns a loader for a file' do
+      expect(described_class.for(fixture_path('kontena_v3.yml')).origin).to eq 'file'
+    end
+
+    it 'returns a loader for an url' do
+      expect(described_class.for('http://foofoo.com').origin).to eq 'uri'
+    end
+
+    it 'returns a loader for a registry path' do
+      expect(described_class.for('foo/foo:1.2.3').origin).to eq 'registry'
+    end
+
+    it 'raises if nothing matches' do
+      expect{described_class.for('this-should-not-exist').origin}.to raise_error(RuntimeError)
+    end
+  end
+
+  context 'instance methods' do
+    let(:subject) { described_class.for(fixture_path('kontena_v3.yml')) }
+
+    describe '#reader' do
+      it 'returns a YAML::Reader' do
+        expect(subject.reader).to be_a Kontena::Cli::Stacks::YAML::Reader
+        expect(subject.reader.loader).to eq subject
+      end
+    end
+
+    describe '#content' do
+      it 'returns the raw content' do
+        expect(File).to receive(:read).with(fixture_path('kontena_v3.yml')).and_call_original
+        expect(subject.content).to match /^stack:/
+      end
+    end
+
+    describe '#stack_name' do
+      it 'returns an accessor to stack string components' do
+        expect(subject.stack_name.user).to eq 'user'
+        expect(subject.stack_name.stack).to eq 'stackname'
+        expect(subject.stack_name.version).to eq '0.1.1'
+        expect(subject.stack_name.to_s).to eq 'user/stackname:0.1.1'
+      end
+    end
+
+    describe '#yaml' do
+      it 'returns a yaml from the file content' do
+        expect(subject.yaml['stack']).to eq 'user/stackname'
+      end
+    end
+
+    describe '#dependencies' do
+      before do
+        [:exist?, :read].each do |meth|
+          Dir.glob(File.join(File.dirname(fixture_path('stack-with-dependencies.yml')), 'stack-with-dep*.yml')).each do |file|
+            puts file
+            allow(File).to receive(meth)
+              .with(file)
+              .and_call_original
+          end
+        end
+      end
+
+      let(:subject) { described_class.for(fixture_path('stack-with-dependencies.yml')) }
+
+      it 'returns an array of hashes' do
+        expect(subject.dependencies).to match array_including(
+          hash_including(
+            'name' => 'dep_1',
+            'stack' => /dep-1.yml$/,
+            'variables' => {},
+            'depends' => array_including(
+              hash_including(
+                'name' => 'dep_1',
+                'stack' => /-1-1.yml$/,
+                'variables' => { 'dep_var' => 2 }
+              )
+            )
+          ),
+          hash_including(
+            'name' => 'dep_2',
+            'stack' => /dep-2.yml$/,
+            'variables' => { 'dep_var' => 1 }
+          )
+        )
+      end
+    end
+  end
+end

data/spec/kontena/cli/stacks/yaml/validator_v3_spec.rb

@@ -335,6 +335,25 @@ describe Kontena::Cli::Stacks::YAML::ValidatorV3 do
         end
       end
     end
+
+    context 'certificates' do
+      it 'validates certificates is array' do
+        result = subject.validate_options('certificates' => 'kontena.io')
+        expect(result.errors.key?('certificates')).to be_truthy
+
+        result = subject.validate_options('certificates' => [])
+        expect(result.errors.key?('certificates')).to be_falsey
+      end
+
+      it 'validates certificates has all needed keys' do
+        result = subject.validate_options('certificates' => [{}])
+        expect(result.errors.key?('certificates')).to be_truthy
+        expect(result.errors['certificates'].has_key?('subject')).to be_truthy
+        expect(result.errors['certificates'].has_key?('name')).to be_truthy
+        expect(result.errors['certificates'].has_key?('type')).to be_truthy
+      end
+
+    end
   end
 
   describe '#validate' do

data/spec/kontena/plugin_manager/cleaner_spec.rb

@@ -0,0 +1,20 @@
+require 'kontena/plugin_manager'
+
+describe Kontena::PluginManager::Cleaner do
+  let(:subject) { described_class.new('foo') }
+  let(:command) { double(handle_options: anything) }
+
+  before(:each) do
+    allow(subject).to receive(:command).and_return(command)
+  end
+
+  it 'returns true if the rubygems cleanup command exits with 0' do
+    expect(command).to receive(:execute).and_raise(Gem::SystemExitException, 0)
+    expect(subject.cleanup).to be_truthy
+  end
+
+  it 'raises the exception if the rubygems cleanup command exits with non-zero' do
+    expect(command).to receive(:execute).and_raise(Gem::SystemExitException, 1)
+    expect{subject.cleanup}.to raise_error(Gem::SystemExitException)
+  end
+end

data/spec/kontena/plugin_manager/common_spec.rb

@@ -0,0 +1,39 @@
+require 'kontena/plugin_manager'
+
+describe Kontena::PluginManager::Common do
+  let(:subject) { described_class }
+
+  context '#prefix' do
+    it 'converts vagrant to kontena-plugin-vagrant' do
+      expect(subject.prefix('vagrant')).to eq 'kontena-plugin-vagrant'
+    end
+
+    it 'returns the same if the string is already prefixed' do
+      expect(subject.prefix('kontena-plugin-vagrant')).to eq 'kontena-plugin-vagrant'
+    end
+  end
+
+  context '#installed' do
+    before(:each) do
+      allow(subject).to receive(:plugins).and_return([double(name: 'kontena-plugin-foo'), double(name: 'kontena-plugin-bar')])
+    end
+
+    it 'returns an installed spec by name' do
+      expect(subject.installed('bar').name).to eq 'kontena-plugin-bar'
+    end
+
+    it 'returns nothing if not found' do
+      expect(subject.installed('baz')).to be_nil
+    end
+
+    context '#installed?' do
+      it 'returns true if the plugin is installed' do
+        expect(subject.installed?('bar')).to be_truthy
+      end
+
+      it 'returns false if the plugin is not installed' do
+        expect(subject.installed?('baz')).to be_falsey
+      end
+    end
+  end
+end

data/spec/kontena/plugin_manager/installer_spec.rb

@@ -0,0 +1,50 @@
+require 'kontena/plugin_manager'
+
+describe Kontena::PluginManager::Installer do
+  context 'default version' do
+    let(:subject) { described_class.new('foo') }
+    let(:command) { double }
+
+    before(:each) do
+      allow(subject).to receive(:command).and_return(command)
+    end
+
+    context '#install' do
+      it 'runs the installer' do
+        expect(command).to receive(:install).with('kontena-plugin-foo', Gem::Requirement.default).and_return(true)
+        expect(command).to receive(:installed_gems).and_return([])
+        subject.install
+      end
+    end
+  end
+
+  context 'specific version' do
+    let(:subject) { described_class.new('foo', version: '0.1.2') }
+    let(:command) { double }
+
+    before(:each) do
+      allow(subject).to receive(:command).and_return(command)
+    end
+
+    context '#install' do
+      it 'runs the installer' do
+        version = double
+        expect(Gem::Requirement).to receive(:new).with('0.1.2').and_return(version)
+        expect(command).to receive(:install).with('kontena-plugin-foo', version).and_return(true)
+        allow(command).to receive(:installed_gems).and_return([])
+        subject.install
+      end
+    end
+  end
+
+  context 'pre-release version' do
+    let(:subject) { described_class.new('foo', pre: true) }
+
+    context '#command' do
+      it 'receives new with prerelease true' do
+        expect(Gem::DependencyInstaller).to receive(:new).with(hash_including(prerelease: true)).and_return(true)
+        subject.command
+      end
+    end
+  end
+end

data/spec/kontena/plugin_manager/loader_spec.rb

@@ -0,0 +1,5 @@
+require 'kontena/plugin_manager'
+
+describe Kontena::PluginManager::Loader do
+  let(:subject) { described_class.new }
+end

data/spec/kontena/plugin_manager/rubygems_client_spec.rb

@@ -1,4 +1,4 @@
-require 'kontena/plugin_manager/rubygems_client'
+require 'kontena/plugin_manager'
 require 'json'
 
 describe Kontena::PluginManager::RubygemsClient do
@@ -11,36 +11,22 @@ describe Kontena::PluginManager::RubygemsClient do
 
   context '#search' do
     it 'searches rubygems and returns a hash' do
-      expect(client)
-        .to receive(:get)
-        .with(
-          hash_including(
-            path: "/api/v1/search.json?query=foofoo",
-            headers: hash_including(
-              'Content-Type' => 'application/json',
-              'Accept' => 'application/json'
-            )
-          )
-        )
-        .and_return(double(body: JSON.dump(foo: 'bar')))
+      expect(subject).to receive(:client).and_return(client)
+      expect(client).to receive(:request) do |req|
+        expect(req.path).to eq '/api/v1/search.json?query=foofoo'
+        expect(req['Accept']).to eq 'application/json'
+      end.and_return(double(body: JSON.dump(foo: 'bar'), code: "200"))
       expect(subject.search('foofoo')['foo']).to eq 'bar'
     end
   end
 
   context '#versions' do
     it 'fetches version list from rubygems and returns an array of Gem::Versions' do
-      expect(client)
-        .to receive(:get)
-        .with(
-          hash_including(
-            path: "/api/v1/versions/foofoo.json",
-            headers: hash_including(
-              'Content-Type' => 'application/json',
-              'Accept' => 'application/json'
-            )
-          )
-        )
-        .and_return(double(body: JSON.dump([{'number' => '0.1.0'}, {'number' => '0.2.0'}])))
+      expect(subject).to receive(:client).and_return(client)
+      expect(client).to receive(:request) do |req|
+        expect(req.path).to eq '/api/v1/versions/foofoo.json'
+        expect(req['Accept']).to eq 'application/json'
+      end.and_return(double(body: JSON.dump([{'number' => '0.1.0'}, {'number' => '0.2.0'}]), code: "200"))
       versions = subject.versions('foofoo')
       expect(versions.first).to be_kind_of Gem::Version
       expect(versions.first.to_s).to eq '0.2.0'