dry-dock 0.1.0

data/spec/drydock/container_config_spec.rb

@@ -0,0 +1,230 @@
+
+# Tests taken from specs at https://github.com/docker/docker/blob/master/runconfig/compare_test.go
+RSpec.describe Drydock::ContainerConfig do
+  
+  describe '#==' do
+
+    let(:ports1) { {"1111/tcp" => {}, "2222/tcp" => {}} }
+    let(:ports2) { {"3333/tcp" => {}, "4444/tcp" => {}} }
+    let(:ports3) { {"1111/tcp" => {}, "2222/tcp" => {}, "5555/tcp" => {}} }
+
+    let(:volumes1) { {"/test1" => {}} }
+    let(:volumes2) { {"/test2" => {}} }
+    let(:volumes3) { {"/test1" => {}, "/test3" => {}} }
+
+    let(:envs1) { ["ENV1=value1", "ENV2=value2"] }
+    let(:envs2) { ["ENV1=value1", "ENV3=value3"] }
+
+    let(:entrypoint1) { ['/bin/sh', '-c'] }
+    let(:entrypoint2) { ['/bin/sh', '-d'] }
+    let(:entrypoint3) { ['/bin/sh', '-c', 'echo'] }
+
+    let(:cmd1) { ['/bin/sh', '-c'] }
+    let(:cmd2) { ['/bin/sh', '-d'] }
+    let(:cmd3) { ['/bin/sh', '-c', 'echo'] }
+
+    let(:labels1) { {"LABEL1" => "value1", "LABEL2" => "value2"} }
+    let(:labels2) { {"LABEL1" => "value1", "LABEL2" => "value3"} }
+    let(:labels3) { {"LABEL1" => "value1", "LABEL2" => "value2", "LABEL3" => "value3"} }
+
+    context 'empty configs' do
+      it_behaves_like 'the same configs' do
+        let(:config1) { {} }
+        let(:config2) { {} }
+      end
+    end
+
+    context 'different hostnames, domain names, or image' do
+      it_behaves_like 'the same configs' do
+        let(:config1) {
+          {
+            Hostname:   "host1",
+            Domainname: "domain1",
+            Image:      "image1",
+            User:       "user"
+          }
+        }
+        let(:config2) {
+          {
+            Hostname:   "host2",
+            Domainname: "domain2",
+            Image:      "image2",
+            User:       "user"
+          }
+        }
+      end
+    end
+
+    context 'only OpenStdin' do
+      it_behaves_like 'the same configs' do
+        let(:config1) { {OpenStdin: false} }
+        let(:config2) { {OpenStdin: false} }
+      end
+    end
+
+    context 'only Env' do
+      it_behaves_like 'the same configs' do
+        let(:config1) { {Env: envs1.dup} }
+        let(:config2) { {Env: envs1.dup} }
+      end
+    end
+
+    context 'only Cmd' do
+      it_behaves_like 'the same configs' do
+        let(:config1) { {Cmd: cmd1.dup} }
+        let(:config2) { {Cmd: cmd1.dup} }
+      end
+    end
+
+    context 'only Labels' do
+      it_behaves_like 'the same configs' do
+        let(:config1) { {Labels: labels1.dup} }
+        let(:config2) { {Labels: labels1.dup} }
+      end
+    end
+
+    context 'only ExposedPorts' do
+      it_behaves_like 'the same configs' do
+        let(:config1) { {ExposedPorts: ports1.dup} }
+        let(:config2) { {ExposedPorts: ports1.dup} }
+      end
+    end
+
+    context 'only Entrypoint' do
+      it_behaves_like 'the same configs' do
+        let(:config1) { {Entrypoint: entrypoint1.dup} }
+        let(:config2) { {Entrypoint: entrypoint1.dup} }
+      end
+    end
+
+    context 'only Volumes' do
+      it_behaves_like 'the same configs' do
+        let(:config1) { {Volumes: volumes1.dup} }
+        let(:config2) { {Volumes: volumes1.dup} }
+      end
+    end
+
+
+
+    context 'with nil' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {} }
+        let(:config2) { nil }
+      end
+    end
+
+    context 'different users' do
+      it_behaves_like 'different configs' do
+        let(:config1) {
+          {
+            Hostname:   "host1",
+            Domainname: "domain1",
+            Image:      "image1",
+            User:       "user1"
+          }
+        }
+        let(:config2) {
+          {
+            Hostname:   "host1",
+            Domainname: "domain1",
+            Image:      "image1",
+            User:       "user2"
+          }
+        }
+      end
+    end
+
+    context 'differing OpenStdin' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {OpenStdin: false} }
+        let(:config2) { {OpenStdin: true } }
+      end
+    end
+
+    context 'differing OpenStdin, reversed' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {OpenStdin: true } }
+        let(:config2) { {OpenStdin: false} }
+      end
+    end
+
+    context 'differing Env' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {Env: envs1} }
+        let(:config2) { {Env: envs2} }
+      end
+    end
+
+    context 'differing Cmd' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {Cmd: cmd1} }
+        let(:config2) { {Cmd: cmd2} }
+      end
+    end
+
+    context 'differing Cmd parts' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {Cmd: cmd1} }
+        let(:config2) { {Cmd: cmd3} }
+      end
+    end
+
+    context 'differing Labels' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {Labels: labels1} }
+        let(:config2) { {Labels: labels2} }
+      end
+    end
+
+    context 'differing Labels' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {Labels: labels1} }
+        let(:config2) { {Labels: labels3} }
+      end
+    end
+
+    context 'differing ExposedPorts' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {ExposedPorts: ports1} }
+        let(:config2) { {ExposedPorts: ports2} }
+      end
+    end
+
+    context 'differing number of ExposedPorts' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {ExposedPorts: ports1} }
+        let(:config2) { {ExposedPorts: ports3} }
+      end
+    end
+
+    context 'differing Entrypoint' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {Entrypoint: entrypoint1} }
+        let(:config2) { {Entrypoint: entrypoint2} }
+      end
+    end
+
+    context 'differing Entrypoint parts' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {Entrypoint: entrypoint1} }
+        let(:config2) { {Entrypoint: entrypoint3} }
+      end
+    end
+
+    context 'differing Volumes' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {Volumes: volumes1} }
+        let(:config2) { {Volumes: volumes2} }
+      end
+    end
+
+    context 'differing number of Volumes' do
+      it_behaves_like 'different configs' do
+        let(:config1) { {Volumes: volumes1} }
+        let(:config2) { {Volumes: volumes3} }
+      end
+    end
+
+  end
+
+end

data/spec/drydock/docker_api_patch_spec.rb

@@ -0,0 +1,103 @@
+
+RSpec.describe Docker::Container do
+  
+  describe '#archive_put' do
+    let(:tar) { Docker::Util.create_tar('/some_file' => 'contents_123') }
+    let(:container) { described_class.create('Image' => 'alpine:latest', 'Cmd' => ['/bin/sh']) }
+    let(:image) { container.commit }
+    let(:run_container) { image.run(['/bin/cat', '/some_file']).tap(&:wait) }
+    let(:output) { run_container.streaming_logs(stdout: true, stderr: true) }
+
+    after(:each) { container.remove }
+
+    context 'when given a tar stream' do
+      after do
+        run_container.remove
+        image.remove
+      end
+
+      it 'has a file in the container' do
+        container.archive_put('/') do |output|
+          output.write(tar)
+        end
+        expect(output).to start_with('contents_123')
+      end
+    end
+  end
+
+  describe '#archive_get' do
+    let(:container) { described_class.create('Image' => 'alpine:latest', 'Cmd' => ['/bin/touch', '/real_file']) }
+
+    after(:each) { container.remove }
+
+    context 'when the file does not exist' do
+      it 'is not found' do
+        container.start
+        container.wait
+        expect { container.archive_get('/not_a_file') { |c| c } }.to raise_error(Docker::Error::NotFoundError)
+      end
+    end
+
+    context 'when the file is found' do
+      it 'yields in chunks' do
+        container.start
+        container.wait
+
+        chunks = StringIO.new
+        container.archive_get('/real_file') { |c| chunks << c }
+        expect(chunks.string).to start_with("real_file\0\0")
+      end
+    end
+  end
+
+  describe '#archive_head' do
+    let(:container) { described_class.create('Image' => 'alpine:latest', 'Cmd' => ['/bin/touch', '/real_file']) }
+
+    before(:each) { container.tap(&:start).tap(&:wait) }
+    after(:each) { container.remove }
+
+    context 'when the file does not exist' do
+      it 'is not found' do
+        expect(container.archive_head('/not_a_file')).to eq(nil)
+      end
+    end
+
+    context 'when the file is found' do
+      it 'returns the stat results' do
+        stat = container.archive_head('/real_file')
+        expect(stat).to be_a(Docker::ContainerPathStat)
+        expect(stat.name).to eq('real_file')
+        expect(stat.size).to eq(0)
+
+        expect(stat.mode.file_mode).to eq(0644)
+        expect(stat.mode.regular?).to be_truthy
+        expect(stat.mode.directory?).to be_falsey
+
+        expect(stat.mode.flags).to be_empty
+        expect(stat.mode.short_flags).to be_empty
+        expect(stat.mode.to_s).to eq('')
+
+        expect(stat.mtime).not_to be_nil
+      end
+    end
+
+    context 'when a link is found' do
+      it 'returns the stat results' do
+        stat = container.archive_head('/etc/mtab')
+        expect(stat).to be_a(Docker::ContainerPathStat)
+        expect(stat).to respond_to(:link?)
+
+        expect(stat.mode.link?).to be_truthy
+        expect(stat.link?).to be_truthy
+        expect(stat.link_target).to eq('/proc/mounts')
+
+        expect(stat.mode.flags).to eq([:link])
+        expect(stat.mode.short_flags).to eq(['L'])
+        expect(stat.mode.to_s).to eq('L')
+
+        expect(stat.mtime).not_to be_nil
+      end
+    end
+  end
+
+end

data/spec/drydock/drydock_spec.rb

@@ -0,0 +1,25 @@
+
+RSpec.describe Drydock do
+
+  describe '.build' do
+
+    let(:opts) { Hash.new }
+
+    context 'with an empty script' do
+      let(:script) { "" }
+      it_behaves_like 'a Drydockfile'
+    end
+
+    context 'from the scratch image' do
+      let(:script) { "from 'scratch'" }
+      it_behaves_like 'a Drydockfile'
+    end
+
+    context 'from an Alpine Linux image' do
+      let(:script) { "from 'gliderlabs/alpine'" }
+      it_behaves_like 'a Drydockfile'
+    end
+
+  end
+
+end

data/spec/drydock/file_manager_spec.rb

@@ -0,0 +1,53 @@
+
+RSpec.describe Drydock::FileManager do
+
+  describe '.find' do
+
+    let(:ignore_none) do
+      double('Ignore Nothing').tap do |file|
+        allow(file).to receive(:match?).and_return(false)
+      end
+    end
+
+    context 'current directory with default options' do
+      subject { described_class.find('.', ignore_none) }
+
+      it { is_expected.to respond_to(:each) }
+      it { is_expected.to include('bin/drydock') }
+      it { is_expected.to have_at_least(30).items }
+    end
+
+    context 'a subdirectory' do
+      context 'with default options' do
+        subject { described_class.find('spec/assets', ignore_none) }
+        it { is_expected.to include('MANIFEST') }
+      end
+
+      context 'with prepend path' do
+        subject { described_class.find('spec/assets', ignore_none, prepend_path: true) }
+        it { is_expected.to include('spec/assets/MANIFEST') }
+      end
+
+      context 'with trailing slashes with prepend path' do
+        subject { described_class.find('spec/assets/', ignore_none, prepend_path: true) }
+        it { is_expected.to include('spec/assets/MANIFEST') }
+      end
+    end
+
+    context 'an absolute path' do
+      context 'with default options' do
+        let(:abs_path) { File.expand_path('spec/assets') }
+        subject { described_class.find(abs_path, ignore_none) }
+        it { is_expected.to include('MANIFEST') }
+      end
+
+      context 'with prepend path' do
+        let(:abs_path) { File.expand_path('spec/assets') }
+        subject { described_class.find(abs_path, ignore_none, prepend_path: true) }
+        it { is_expected.to include(abs_path + '/MANIFEST') }
+      end
+    end
+
+  end
+
+end

data/spec/drydock/formatters_spec.rb

@@ -0,0 +1,26 @@
+
+RSpec.describe Drydock::Formatters do
+
+  describe '.number' do
+    context 'a small integer' do
+      subject { described_class.number(2) }
+      it { is_expected.to eq('2') }
+    end
+
+    context 'a large integer' do
+      subject { described_class.number(5196714) }
+      it { is_expected.to eq('5,196,714') }
+    end
+
+    context 'a large decimal' do
+      subject { described_class.number(1024.66) }
+      it { is_expected.to eq('1,024.66') }
+    end
+
+    context 'a negative decimal' do
+      subject { described_class.number(-24.52) }
+      it { is_expected.to eq('-24.52') }
+    end
+  end
+
+end

data/spec/drydock/ignorefile_definition_spec.rb

@@ -0,0 +1,123 @@
+
+RSpec.describe Drydock::IgnorefileDefinition do
+
+  describe '.new' do
+    it 'raises an error when no filename is provided' do
+      expect { described_class.new() }.to raise_error(ArgumentError)
+    end
+
+    it 'raises an error when a non-existant file is provided' do
+      expect { described_class.new('file_does_not_exist') }.not_to raise_error
+    end
+  end
+
+  context 'when given a file handle with two positive rules' do
+    let(:file) { StringIO.new(".gitignore\nvendor/ruby") }
+    let(:ifd)  { described_class.new(file) }
+
+    describe '.new' do
+      subject { ifd }
+      it 'reads the entire file' do
+        expect(file.eof?).to be(false)
+        expect(file.pos).to eq(0)
+
+        expect { subject }.not_to raise_error
+        expect(file.eof?).to be(true)
+      end
+    end
+
+    describe '#match?' do
+      context 'for a filename appearing in the ignore file' do
+        subject { ifd.match?('.gitignore') }
+        it { is_expected.to be(true) }
+      end
+
+      context 'for a directory name not exactly appearing in the ignore file' do
+        subject { ifd.match?('vendor') }
+        it { is_expected.to be(false) }
+      end
+
+      context 'for a directory name appearing in the ignore file' do
+        subject { ifd.match?('vendor/ruby') }
+        it { is_expected.to be(true) }
+      end
+
+      context 'for a regular filename NOT appearing in the ignore file' do
+        subject { ifd.match?('Gemfile') }
+        it { is_expected.to be(false) }
+      end
+
+      context 'for a dotfile NOT appearing in the ignore' do
+        subject { ifd.match?('.rspec') }
+        it { is_expected.to be(false) }
+      end
+    end
+
+    describe '#size' do
+      subject { ifd.size }
+      it { is_expected.to eq(2) }
+    end
+  end
+
+  context 'when given a file handle with a negative rule' do
+    let(:file) { StringIO.new("!.gitignore") }
+    let(:ifd)  { described_class.new(file) }
+
+    describe '#match?' do
+      context 'for a filename excluded in the ignore file' do
+        subject { ifd.match?('.gitignore') }
+        it { is_expected.to be(false) }
+      end
+
+      context 'for a filename not excluded in the ignore file' do
+        subject { ifd.match?('Gemfile') }
+        it { is_expected.to be(false) }
+      end
+    end
+
+    describe '#size' do
+      subject { ifd.size }
+      it { is_expected.to eq(1) }
+    end
+  end
+
+  context 'when given an empty file handle with automatic dotfile handling' do
+    let(:file) { StringIO.new("") }
+    let(:ifd)  { described_class.new(file, dotfiles: true) }
+
+    describe '#match?' do
+      context 'for a dotfile' do
+        subject { ifd.match?('.gitignore') }
+        it { is_expected.to eq(true) }
+      end
+    end
+
+    describe '#size' do
+      subject { ifd.size }
+      it { is_expected.to eq(0) }
+    end
+  end
+
+  context 'when given a file handle with a wildcard' do
+    let(:file) { StringIO.new("Gemfile*") }
+    let(:ifd)  { described_class.new(file) }
+
+    describe '#match?' do
+      context 'for an exact match' do
+        subject { ifd.match?('Gemfile')}
+        it { is_expected.to eq(true) }
+      end
+
+      context 'for a substring prefix match' do
+        subject { ifd.match?('Gemfile.lock') }
+        it { is_expected.to eq(true) }
+      end
+
+      context 'for a substring suffix match' do
+        subject { ifd.match?('NotGemfile') }
+        it { is_expected.to eq(false) }
+      end
+    end
+  end
+
+end

data/spec/drydock/image_repository_spec.rb

@@ -0,0 +1,54 @@
+
+RSpec.describe Drydock::ImageRepository do
+  
+  describe '.all' do
+    it 'completes without an error' do
+      expect { described_class.all }.not_to raise_error
+    end
+  end
+
+  describe '.dangling' do
+    it 'completes without an error' do
+      expect { described_class.dangling }.not_to raise_error
+    end
+  end
+
+  describe '.each' do
+    it 'returns each image' do
+      described_class.each do |image|
+        expect(image).to be_a(Docker::Image)
+      end
+    end
+  end
+
+  describe '.find_by_config' do
+    let(:cmd)       { '/bin/ls -l' }
+    let(:image)     { Docker::Image.create(fromImage: 'alpine', tag: 'latest') }
+    let(:container) { image.run(cmd).tap(&:wait) }
+    let(:run_image) { container.commit }
+
+    after(:each) do
+      container.remove
+      run_image.remove
+    end
+
+    it 'returns the latest image' do
+      expect(run_image.id).not_to be_nil
+
+      build_config = Drydock::ContainerConfig.from(
+        Cmd: cmd.to_s.split(/\s+/),
+        Tty: false,
+        Image: image.id,
+        Env: nil
+      )
+
+      possible_images = described_class.select_by_config(build_config)
+      expect(possible_images).not_to be_empty
+
+      found_image = described_class.find_by_config(build_config)
+      expect(found_image).not_to be_nil
+      expect(found_image.id).to eq(run_image.id)
+    end
+  end
+
+end

data/spec/drydock/object_caches/base_spec.rb

@@ -0,0 +1,28 @@
+
+RSpec.describe Drydock::ObjectCaches::Base do
+
+  describe '.new' do
+    it 'takes no arguments' do
+      expect { described_class.new }.not_to raise_error
+    end
+  end
+
+  describe '#fetch' do
+    it 'is not implemented' do
+      expect { described_class.new.fetch(:foo) }.to raise_error(NotImplementedError)
+    end
+  end
+
+  describe '#get' do
+    it 'is not implemented' do
+      expect { described_class.new.get(:foo) }.to raise_error(NotImplementedError)
+    end
+  end
+
+  describe '#set' do
+    it 'is not implemented' do
+      expect { described_class.new.set(:foo, :some_value) }.to raise_error(NotImplementedError)
+    end
+  end
+
+end

data/spec/drydock/object_caches/filesystem_cache_spec.rb

@@ -0,0 +1,48 @@
+
+RSpec.describe Drydock::ObjectCaches::FilesystemCache do
+  include FakeFS::SpecHelpers # mock the filesystem
+  
+  describe '.new' do
+    it 'can take zero or one argument' do
+      expect { described_class.new }.not_to raise_error
+      expect { described_class.new("/tmp/drydock-spec") }.not_to raise_error
+    end
+
+    it 'creates the cache directory' do
+      described_class.new("/tmp/drydock-spec")
+      expect(File.exist?("/tmp/drydock-spec/cache")).to eq(true)
+    end
+  end
+
+  describe '#fetch' do
+    let(:cache) { described_class.new("/tmp/drydock-spec") }
+
+    it 'uses an existing value, when one is already set' do
+      cache.set('hello_world', 'Hello, World!')
+      expect(cache.fetch('hello_world') { 'Backup Value' }).to eq('Hello, World!')
+    end
+
+    it 'invokes the writer if no value is already set' do
+      expect(cache.fetch('hello_world') { 'Backup Value' }).to eq('Backup Value')
+    end
+  end
+
+  describe '#get, #set' do
+    let(:cache) { described_class.new("/tmp/drydock-spec") }
+
+    context 'when used in value form' do
+      it 'persists the cache entry' do
+        expect { cache.set('hello_world', 'Hello, World!') }.not_to raise_error
+        expect(cache.get('hello_world')).to eq('Hello, World!')
+      end
+    end
+
+    context 'when used in block form' do
+      it 'persists the cache entry' do
+        expect { cache.set('bye_world') { |f| f.write 'Goodbye, World!' } }.not_to raise_error
+        expect(cache.get('bye_world')).to eq('Goodbye, World!')
+      end
+    end
+  end
+
+end