spinna 0.0.3 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0db8fb458a101ffcb7deebae97f3c721aae130ad
-  data.tar.gz: 6d9b9b2daf0069039b511e80f5b68ba3f665b927
+  metadata.gz: f36240defb27579b06167812df1cf88fabc13db1
+  data.tar.gz: 408ef8b78af8699adcd8c40956f9b9fab70bad14
 SHA512:
-  metadata.gz: 19b8427129a40acb143947a212498a8e8194880bc8ee99046f21244f09c9df61c8318640a94cda5fc97c885c46ae0524b89cf2667fc4242c1d06af93d40874b3
-  data.tar.gz: 561ab20d83b2797b94799c22a7f367348b682a4a3712499ed652155e5029fbdff166ba5222da4fdcbe1ad2b9061966d2ca2fc521069e2e013e557cbcbc908fbe
+  metadata.gz: 88f5ac62c36e40617f5ec835da4f4809a9904ae4ffb16ba413ef479f3e4002f40a4f23f035979b7088dfe6751e3ef23d9d528313e8647b61ac5a6a370a63963c
+  data.tar.gz: 9af06e83001c5f2bead99012b865b0c1ada8e16c06f1b9bf19a5742726281538c7d8f5cb010c2c7655367dca7761c14208db7ff3db6b744f2c45ca4f7834634a

data/CHANGELOG.md

@@ -1,7 +1,7 @@
 # Changelog
 All notable changes to this project will be documented in this file.
 
-## 0.0.3 - 2015-07-06
+## 0.0.5 - 2015-07-06
 
 ### Added
 - Nothing

data/lib/spinna/version.rb

@@ -1,3 +1,3 @@
 module Spinna
-  VERSION = "0.0.3"
+  VERSION = "0.0.5"
 end

data/spinna.gemspec

@@ -13,8 +13,9 @@ Gem::Specification.new do |spec|
   spec.homepage      = "https://github.com/hydrozen/spinna"
   spec.license       = "MIT"
 
-  spec.files         = `git ls-files -z`.split("\x0")
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+  spec.bindir        = "exe"
+  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
   spec.add_development_dependency "bundler", "~> 1.8"

metadata

@@ -1,12 +1,12 @@
 --- !ruby/object:Gem::Specification
 name: spinna
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.5
 platform: ruby
 authors:
 - Patrick Paul-Hus
 autorequire: 
-bindir: bin
+bindir: exe
 cert_chain: []
 date: 2015-07-06 00:00:00.000000000 Z
 dependencies:
@@ -153,7 +153,8 @@ dependencies:
 description: Picks albums from your music collection so you don’t have to.
 email:
 - hydrozen@gmail.com
-executables: []
+executables:
+- spinna
 extensions: []
 extra_rdoc_files: []
 files:
@@ -173,24 +174,6 @@ files:
 - lib/spinna/music_library.rb
 - lib/spinna/picker.rb
 - lib/spinna/version.rb
-- spec/fixtures/data_dir/config.yml
-- spec/fixtures/data_dir/config1.yml
-- spec/fixtures/data_dir/config2.yml
-- spec/fixtures/data_dir/config3.yml
-- spec/fixtures/data_dir/config4.yml
-- spec/fixtures/data_dir/history.log
-- spec/fixtures/data_dir_without_config/.gitkeep
-- spec/fixtures/data_dir_without_history/config.yml
-- spec/fixtures/source_dir/album1/.gitkeep
-- spec/fixtures/source_dir/album2/.gitkeep
-- spec/fixtures/source_dir/album3/.gitkeep
-- spec/functional/get_spec.rb
-- spec/spec_helper.rb
-- spec/unit/client_spec.rb
-- spec/unit/config_spec.rb
-- spec/unit/history_spec.rb
-- spec/unit/music_library_spec.rb
-- spec/unit/picker_spec.rb
 - spinna.gemspec
 homepage: https://github.com/hydrozen/spinna
 licenses:
@@ -216,22 +199,4 @@ rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Randomly picks albums from your music library so you don’t have to.
-test_files:
-- spec/fixtures/data_dir/config.yml
-- spec/fixtures/data_dir/config1.yml
-- spec/fixtures/data_dir/config2.yml
-- spec/fixtures/data_dir/config3.yml
-- spec/fixtures/data_dir/config4.yml
-- spec/fixtures/data_dir/history.log
-- spec/fixtures/data_dir_without_config/.gitkeep
-- spec/fixtures/data_dir_without_history/config.yml
-- spec/fixtures/source_dir/album1/.gitkeep
-- spec/fixtures/source_dir/album2/.gitkeep
-- spec/fixtures/source_dir/album3/.gitkeep
-- spec/functional/get_spec.rb
-- spec/spec_helper.rb
-- spec/unit/client_spec.rb
-- spec/unit/config_spec.rb
-- spec/unit/history_spec.rb
-- spec/unit/music_library_spec.rb
-- spec/unit/picker_spec.rb
+test_files: []

data/spec/fixtures/data_dir/config.yml

@@ -1,3 +0,0 @@
-source_dir: '/tmp'
-history_size: 10
-number_of_picks: 5

data/spec/fixtures/data_dir/config1.yml

@@ -1,2 +0,0 @@
-This is not a valid configuration file
-so it will blow up.

data/spec/fixtures/data_dir/config2.yml

@@ -1,3 +0,0 @@
-# This configuration does not have a source_dir set.
-foo: 'meh'
-bar: 'blah'

data/spec/fixtures/data_dir/config3.yml

@@ -1,4 +0,0 @@
-# This configuration does not set the history_size
-
-source_dir: '/foo'
-number_of_picks: 5

data/spec/fixtures/data_dir/config4.yml

@@ -1,4 +0,0 @@
-# This configuration does not set the number of picks.
-
-source_dir: '/foo'
-history_size: 10

data/spec/fixtures/data_dir/history.log

@@ -1,3 +0,0 @@
-album 1
-album 2
-album 3

data/spec/fixtures/data_dir_without_history/config.yml

@@ -1,3 +0,0 @@
-source_dir: '/tmp'
-history_size: 10
-number_of_picks: 5

data/spec/functional/get_spec.rb

@@ -1,99 +0,0 @@
-require 'spec_helper'
-require 'open3'
-
-describe 'spinna get' do
-  before do
-    create_fake_source_dir
-    create_fake_data_dir
-    @download_dir = create_tmp_download_dir
-    ENV['SPINNA_DATA_DIR'] = File.join(Dir.tmpdir, 'spinna')
-    @wd = Dir.getwd
-    Dir.chdir(@download_dir)
-  end
-
-  after do
-    ENV.delete('SPINNA_DATA_DIR')
-    Dir.chdir(@wd)
-  end
-
-  context 'when used without any options' do
-    before do
-      Open3.popen3("#{@wd}/exe/spinna get") do |stdin, stdout, stderr, wait_thr|
-        @stdout = stdout.read
-        @stderr = stderr.read
-        @exit_status = wait_thr.value
-      end
-    end
-
-    it 'gets the number of albums set in the config from source_dir' do
-      # It must not have output any errors.
-      @stderr.must_be_empty
-
-      # Gets the correct number of picks
-      picks = @stdout.split("\n")
-      picks.size.must_equal 2
-
-      # Exits with a successful exit code
-      @exit_status.success?.must_equal true
-
-      # It added the picks to the history log
-      history = File.read("#{ENV['SPINNA_DATA_DIR']}/history.log")
-      history.split("\n").size.must_equal 2
-    end
-  end
-
-  context 'when given a -n (--number_of_picks) option' do
-    before do
-      Open3.popen3("#{@wd}/exe/spinna get -n 3") do |stdin, stdout, stderr, wait_thr|
-        @stdout = stdout.read
-        @stderr = stderr.read
-        @exit_status = wait_thr.value
-      end
-    end
-
-    it 'gets that number of albums from source_dir' do
-      # It must not output any errors
-      @stderr.must_be_empty
-
-      # Gets the correct number of picks
-      # picks = @stdout.split("\n")
-      # picks.size.must_equal 3
-
-      # Exits with a successful exit code
-      @exit_status.success?.must_equal true
-
-      # It updates the history log
-      history = File.read("#{ENV['SPINNA_DATA_DIR']}/history.log")
-      history.split("\n").size.must_equal 3
-    end
-  end
-
-  context 'when given a -p (--pattern) option' do
-    before do
-      Open3.popen3("#{@wd}/exe/spinna get -p \"album 1\" -n 1") do |stdin, stdout, stderr, wait_thr|
-        @stdout = stdout.read
-        @stderr = stderr.read
-        @exit_status = wait_thr.value
-      end
-    end
-
-    it 'picks only albums that match the given pattern' do
-      # It must not output any errors
-      @stderr.must_be_empty
-
-      # Gets the correct number of picks
-      picks = @stdout.split("\n")
-      picks.size.must_equal 1
-
-      # The pick should match the given pattern
-      picks.first.must_equal "Copying album 1..."
-
-      # Exits with a successful exit code
-      @exit_status.success?.must_equal true
-
-      # It updates the history log
-      history = File.read("#{ENV['SPINNA_DATA_DIR']}/history.log")
-      history.split("\n").size.must_equal 1
-    end
-  end
-end

data/spec/spec_helper.rb

@@ -1,51 +0,0 @@
-# Code coverage
-require 'simplecov'
-require 'coveralls'
-
-SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
-  SimpleCov::Formatter::HTMLFormatter,
-  Coveralls::SimpleCov::Formatter
-]
-SimpleCov.start
-
-require 'minitest/autorun'
-require 'minitest/spec'
-require 'minitest/pride'
-require 'minitest-spec-context'
-require 'mocha/mini_test'
-require 'minitest/reporters'
-require 'tmpdir'
-require 'fileutils'
-
-Minitest::Reporters.use! Minitest::Reporters::DefaultReporter.new(:color => true)
-
-def create_fake_source_dir
-  path = File.join(Dir.tmpdir, 'spinna_source_dir')
-  FileUtils.rm_rf(path)
-  FileUtils.mkdir(path)
-  5.times do |i|
-    FileUtils.mkdir(File.join(path, "album #{i+1}"))
-  end
-end
-
-def create_fake_data_dir
-  path = File.join(Dir.tmpdir, 'spinna')
-  FileUtils.rm_rf(path)
-  FileUtils.mkdir(path)
-  config = {
-    'source_dir' => File.join(Dir.tmpdir, 'spinna_source_dir'),
-    'history_size' => 4,
-    'number_of_picks' => 2
-  }
-  config_path = File.join(path, 'config.yml')
-  File.open(config_path, 'w+') do |f|
-    f.write config.to_yaml
-  end
-end
-
-def create_tmp_download_dir
-  path = File.join(Dir.tmpdir, 'spinna_download_dir')
-  FileUtils.rm_rf(path)
-  FileUtils.mkdir(path)
-  path
-end

data/spec/unit/client_spec.rb

@@ -1,115 +0,0 @@
-require 'spec_helper'
-require 'spinna/client'
-
-describe Spinna::Client do
-  describe '.new' do
-    let(:config) { stub() }
-    let(:history) { stub() }
-    let(:picker) { stub() }
-    let(:music_library) { stub() }
-
-    before do
-      Spinna::Config.expects(:new).returns(config)
-      Spinna::History.expects(:new).returns(history)
-      Spinna::MusicLibrary.expects(:new).returns(music_library)
-    end
-
-    subject { Spinna::Client.new }
-
-    it 'assigns a config object to an instance variable' do
-      subject.config.must_equal config
-    end
-
-    it 'assigns a history object to an instance variable' do
-      subject.history.must_equal history
-    end
-
-    it 'assigns a music_library object to an instance variable' do
-      subject.library.must_equal music_library
-    end
-  end
-
-  describe '#get' do
-    let(:default_number_of_picks) { 2 }
-    let(:picker) { stub }
-    let(:music_library) { stub }
-    let(:picks) { ['album 1', 'album 2'] }
-
-    before do
-      Spinna::Config.stubs(:new).returns(config)
-      Spinna::History.stubs(:new).returns(history)
-      Spinna::Picker.stubs(:new).returns(picker)
-      Spinna::MusicLibrary.stubs(:new).returns(music_library)
-    end
-
-    context 'when called without arguments' do
-      let(:config) do
-        stub(
-          :number_of_picks => default_number_of_picks,
-          :source_dir => File.expand_path("../../fixtures/source_dir", __FILE__)
-        )
-      end
-      let(:history) { stub }
-      let(:picker) do
-        picker = stub
-        picker.stubs(:pick).returns(picks)
-        picker
-      end
-
-      subject { Spinna::Client.new }
-
-      it 'requests the default number of picks from the picker' do
-        picker.expects(:pick).with(default_number_of_picks, nil)
-        subject.get
-      end
-
-      it 'returns the picks from the picker' do
-        subject.get.must_equal picks
-      end
-    end
-
-    context 'when given a number_of_picks option' do
-      let(:config) do
-        stub(
-          :number_of_picks => default_number_of_picks,
-          :source_dir => File.expand_path("../../fixtures/source_dir", __FILE__)
-        )
-      end
-      let(:history) { stub }
-      let(:picker) { stub(:pick => [picks.first]) }
-
-      subject { Spinna::Client.new }
-
-      it 'requests the given number of picks from the picker' do
-        picker.expects(:pick).with(1, nil).returns([picks.first])
-        subject.get(:number_of_picks => 1)
-      end
-
-      it 'returns the picks from the picker' do
-        subject.get(:number_of_picks => 1).must_equal [picks.first]
-      end
-    end
-
-    context 'when given a pattern' do
-      let(:config) do
-        stub(
-          :number_of_picks => default_number_of_picks,
-          :source_dir => File.expand_path("../../fixtures/source_dir", __FILE__)
-        )
-      end
-      let(:history) { stub }
-      let(:picker) { stub(:pick => [picks.first]) }
-
-      subject { Spinna::Client.new }
-
-      it 'requests the picks from the picker with the correct options' do
-        picker.expects(:pick).with(1, 'foo').returns([picks.first])
-        subject.get(:number_of_picks => 1, :pattern => 'foo')
-      end
-
-      it 'returns the picks from the picker' do
-        subject.get(:number_of_picks => 1, :pattern => 'foo').must_equal [picks.first]
-      end
-    end
-  end
-end

data/spec/unit/config_spec.rb

@@ -1,147 +0,0 @@
-require 'spec_helper'
-require 'spinna/config'
-
-describe Spinna::Config do
-  describe 'DEFAULT_HISTORY_SIZE' do
-    it 'is set to 50' do
-      Spinna::Config::DEFAULT_HISTORY_SIZE.must_equal 50
-    end
-  end
-
-  describe 'DEFAULT_NUMBER_OF_PICKS' do
-    it 'is set to 10' do
-      Spinna::Config::DEFAULT_NUMBER_OF_PICKS.must_equal 10
-    end
-  end
-
-  describe '#new' do
-    let(:data_dir) { File.expand_path("../../fixtures/data_dir", __FILE__) }
-    let(:non_existing_data_dir) { File.join(Dir.home, '.nonexisting') }
-    let(:data_dir_without_config) { File.expand_path("../../fixtures/data_dir_without_config", __FILE__) }
-
-    context 'when the data_dir does not exist' do
-      let(:opts) { {:data_dir => non_existing_data_dir} }
-
-      it 'creates it and raises error because the configuration file is missing' do
-        Dir.expects(:mkdir).with(opts[:data_dir], 0700)
-        proc { Spinna::Config.new(opts) }.must_raise Spinna::ConfigFileNotFoundError
-      end
-    end
-
-    context 'when the configuration file does not exist' do
-      let(:opts) { {:data_dir => data_dir_without_config} }
-
-      subject do
-        Spinna::Config.new(opts)
-      end
-
-      before do
-        Dir.stubs(:mkdir)
-      end
-
-      it 'throws an exception' do
-        err = proc { subject }.must_raise Spinna::ConfigFileNotFoundError
-        err.message.must_match %r{Could not find the configuration file.}
-      end
-    end
-
-    context 'when the configuration file is not valid json' do
-      before do
-        YAML.stubs(:load_file).returns("&&&foo")
-      end
-
-      it 'throws an exception' do
-        opts = {
-          :data_dir => data_dir
-        }
-        err = proc { Spinna::Config.new(opts) }.must_raise Spinna::InvalidConfigFileError
-        err.message.must_match %r{Could not parse the configuration file.}
-      end
-    end
-
-    context 'when the source_dir config parameter is missing' do
-      before do
-        YAML.stubs(:load_file).returns({'foo' => 'bar'})
-      end
-
-      it 'throws an exception' do
-        opts = {:data_dir => data_dir}
-        err = proc { Spinna::Config.new(opts) }.must_raise Spinna::InvalidConfigFileError
-        err.message.must_match %r{You must set a source_dir in your configuration file.}
-      end
-    end
-
-    context 'when the source_dir does not actually exist on the filesystem' do
-      before do
-        YAML.stubs(:load_file).returns({
-          'source_dir' => '/this/dir/does/not/exist'
-        })
-      end
-
-      it 'throws an exception' do
-        opts = {
-          :data_dir => data_dir
-        }
-        proc { Spinna::Config.new(opts) }.must_raise Spinna::InvalidSourceDirError
-      end
-    end
-
-    context 'when the configuration is valid and all parameters were provided' do
-      it 'sets the source_dir' do
-        opts = {
-          :data_dir => data_dir,
-          :config_file => 'config.yml'
-        }
-        config = Spinna::Config.new(opts)
-        config.source_dir.must_equal '/tmp'
-      end
-
-      it 'sets the history_size' do
-        opts = {
-          :data_dir => data_dir,
-          :config_file => 'config.yml'
-        }
-        config = Spinna::Config.new(opts)
-        config.history_size.must_equal 10
-      end
-
-      it 'sets the number_of_picks' do
-        opts = {
-          :data_dir => data_dir,
-          :config_file => 'config.yml'
-        }
-        config = Spinna::Config.new(opts)
-        config.number_of_picks.must_equal 5
-      end
-    end
-
-    context 'when the configuration is valid but the history_size was not provided' do
-      let(:opts) { {:data_dir => data_dir} }
-
-      subject { Spinna::Config.new(opts) }
-
-      before do
-        YAML.stubs(:load_file).returns({'source_dir' => '/tmp'})
-      end
-
-      it 'uses the default value' do
-        subject.history_size.must_equal Spinna::Config::DEFAULT_HISTORY_SIZE
-      end
-    end
-
-    context 'when the configuration is valid but the number_of_picks was not provided' do
-      let(:opts) { {:data_dir => data_dir} }
-
-      subject { Spinna::Config.new(opts) }
-
-      before do
-        YAML.stubs(:load_file).returns({'source_dir' => '/tmp'})
-      end
-
-      it 'uses the default value' do
-        subject.number_of_picks.must_equal Spinna::Config::DEFAULT_NUMBER_OF_PICKS
-      end
-    end
-  end
-end
-

data/spec/unit/history_spec.rb

@@ -1,100 +0,0 @@
-require 'spec_helper'
-require 'spinna/history'
-require 'spinna/config'
-
-describe Spinna::History do
-  let(:data_dir) { File.expand_path("../../fixtures/data_dir", __FILE__) }
-  let(:data_dir_without_history) { File.expand_path("../../fixtures/data_dir_without_history", __FILE__) }
-  let(:opts) { {:data_dir => data_dir} }
-  let(:config) { stub(:data_dir => data_dir, :history_size => 10) }
-  let(:history_size) { 3 }
-
-  describe '#new' do
-    subject { Spinna::History.new(config) }
-
-    it 'saves the passed in config in an instance variable' do
-      subject.config.must_equal config
-    end
-
-    context 'when the history log file exists' do
-      it 'reads the history log from the filesystem' do
-        subject.log.size.must_equal history_size
-      end
-    end
-
-    context 'when the history log does not exist' do
-      let(:config) { stub(:data_dir => data_dir_without_history, :history_size => 10) }
-
-      subject { Spinna::History.new(config) }
-
-      it 'does not read the history log from the filesystem' do
-        File.expects(:read).never
-        subject
-      end
-
-      it 'inits the log as an empty array' do
-        subject.log.size.must_equal 0
-      end
-    end
-  end
-
-  describe '#append' do
-    subject { Spinna::History.new(config) }
-
-    context 'when the album is not in the history' do
-      it 'appends the album at the end of the history log' do
-        size_before_append = subject.log.size
-        subject.append('album 11')
-        subject.log.size.must_equal size_before_append + 1
-        subject.log.last.must_equal 'album 11'
-      end
-    end
-
-    context 'when the album is already in the history' do
-      it 'does nothing' do
-        size_before_append = subject.log.size
-        subject.append('album 1')
-        subject.log.size.must_equal size_before_append
-      end
-    end
-
-    context 'when the history log is full' do
-      let(:config) { stub(:data_dir => data_dir, :history_size => 3) }
-      it 'removes the old albums from the log to make room for the new albums' do
-        subject.append('album 4')
-        subject.log.size.must_equal 3
-        subject.log.first.must_equal 'album 2'
-        subject.log.last.must_equal 'album 4'
-      end
-    end
-  end
-
-  describe '#include?' do
-    subject { Spinna::History.new(config) }
-
-    context 'when the album is in the log' do
-      it 'returns true' do
-        subject.include?('album 1').must_equal true
-      end
-    end
-
-    context 'when the album is not in the log' do
-      it 'returns false' do
-        subject.include?('album 99').must_equal false
-      end
-    end
-  end
-
-  describe '#save' do
-    # This is used to fake writing the history.log file.
-    let(:history_log) { StringIO.new('', 'w+') }
-
-    subject { Spinna::History.new(config) }
-    it 'saves the log to the filesystem' do
-      File.expects(:open).with(subject.history_path, 'w+').yields(history_log)
-      subject.save
-      history_log.string.must_equal "album 1\nalbum 2\nalbum 3\n"
-    end
-  end
-end
-

data/spec/unit/music_library_spec.rb

@@ -1,59 +0,0 @@
-require 'spec_helper'
-require 'spinna/music_library'
-
-describe Spinna::MusicLibrary do
-  describe '.new' do
-    let(:config) { stub }
-    let(:history) { stub }
-
-    subject { Spinna::MusicLibrary.new(config, history) }
-
-    it 'assigns the config to an instance variable' do
-      subject.instance_variable_get(:@config).must_equal config
-    end
-
-    it 'assigns the history to an instance variable' do
-      subject.instance_variable_get(:@history).must_equal history
-    end
-  end
-
-  describe '.pickable_albums' do
-    let(:config) { stub_everything }
-    let(:history) { stub_everything }
-
-    before do
-      Dir.stubs(:entries).returns(['.', '..', '.hidden', 'album 1', 'album 2', 'album 3'])
-      history.stubs(:log).returns(['album 1'])
-    end
-
-    subject { Spinna::MusicLibrary.new(config, history) }
-
-    context 'when not given a pattern and some albums are pickable' do
-      it 'returns those albums as an array' do
-        subject.pickable_albums.must_equal ['album 2', 'album 3']
-      end
-    end
-
-    context 'when not given a pattern and no albums are pickable' do
-      before do
-        history.stubs(:log).returns(['album 1', 'album 2', 'album 3'])
-      end
-
-      it 'returns an empty array' do
-        subject.pickable_albums.must_equal []
-      end
-    end
-
-    context 'when given a pattern and some albums match that pattern' do
-      it 'returns those albums as an array' do
-        subject.pickable_albums('album 2').must_equal ['album 2']
-      end
-    end
-
-    context 'when given a pattern and no albums match' do
-      it 'returns an empty array' do
-        subject.pickable_albums('foobar').must_equal []
-      end
-    end
-  end
-end

data/spec/unit/picker_spec.rb

@@ -1,250 +0,0 @@
-require 'spec_helper'
-require 'spinna/picker'
-
-describe Spinna::Picker do
-  describe '.new' do
-    let(:config) { stub }
-    let(:history) { stub }
-    let(:library) { stub }
-
-    subject do
-      Spinna::Picker.new(config, history, library)
-    end
-
-    it 'assigns the config to an instance variable' do
-      subject.instance_variable_get(:@config).must_equal config
-    end
-
-    it 'assigns the history to an instance variable' do
-      subject.instance_variable_get(:@history).must_equal history
-    end
-
-    it 'assigns the library to an instance variable' do
-      subject.instance_variable_get(:@library).must_equal library
-    end
-  end
-
-  describe '#pick' do
-    let(:source_dir) { File.expand_path("../../fixtures/source_dir", __FILE__) }
-    let(:config) { stub(:source_dir => source_dir) }
-    let(:history) do
-      history = stub(
-        :include? => false,
-        :log => [],
-        :append => nil,
-        :save => nil
-      )
-    end
-    let(:library) { stub }
-
-    before do
-      FileUtils.stubs(:cp_r)
-    end
-
-    context 'when not given a number_of_picks parameter' do
-      subject do
-        picker = Spinna::Picker.new(config, history, library)
-      end
-
-      it 'raises an error' do
-        proc { subject.pick }.must_raise ArgumentError
-      end
-    end
-
-    context 'when given a number_of_picks but no pattern' do
-      context 'and there are more albums to choose from' do
-        let(:albums) { ['album 1', 'album 2', 'album 3'] }
-        let(:library) { stub(:pickable_albums => albums) }
-
-        subject do
-          Spinna::Picker.new(config, history, library).pick(1)
-        end
-
-        before do
-          albums.expects(:sample).with(1).returns(['album 1'])
-        end
-
-        it 'request the pickable albums from the library' do
-          library
-            .expects(:pickable_albums)
-            .with(nil)
-            .returns(albums)
-          capture_io { subject }
-        end
-
-        it 'returns that number of picks in an array' do
-          capture_io do
-            subject.size.must_equal 1
-          end
-        end
-
-        it 'copies the picks' do
-          FileUtils.expects(:cp_r).once
-          capture_io { subject }
-        end
-
-        it 'adds the picks to the history' do
-          history.expects(:append).once
-          capture_io { subject }
-        end
-
-        it 'saves the history' do
-          history.expects(:save).once
-          capture_io do
-            subject
-          end
-        end
-
-        it 'outputs the picks to the screen' do
-          out, err = capture_io do
-            subject
-          end
-          out.must_match %r{Copying album 1\.\.\.}
-        end
-      end
-
-      context 'and there are not enough albums to choose from' do
-        let(:albums) { ['album 1', 'album 2', 'album 3'] }
-        let(:library) { stub(:pickable_albums => albums) }
-
-        subject do
-          Spinna::Picker.new(config, history, library).pick(5)
-        end
-
-        it 'request the pickable albums from the library' do
-          library
-            .expects(:pickable_albums)
-            .with(nil)
-            .returns(albums)
-          capture_io { subject }
-        end
-
-        it 'returns all the remaining albums' do
-          capture_io do
-            subject.size.must_equal 3
-          end
-        end
-
-        it 'adds the picks to the history' do
-          history.expects(:append).times(3)
-          capture_io { subject }
-        end
-
-        it 'saves the history' do
-          history.expects(:save).once
-          capture_io { subject }
-        end
-
-        it 'outputs the picks on the screen' do
-          out, err = capture_io do
-            subject
-          end
-          lines = out.split("\n")
-          lines.size.must_equal 3
-          lines.each do |line|
-            line.must_match %r{Copying album [1-3]\.\.\.}
-          end
-        end
-      end
-    end
-
-    context 'when given a number_of_picks and a pattern' do
-      context 'and there are more albums that match than requested' do
-        let(:albums) do
-          albums = ['album 1', 'album 2', 'album 3']
-          albums.stubs(:sample).returns(picks)
-          albums
-        end
-        let(:picks) { ['album 1', 'album 2'] }
-        let(:library) { stub(:pickable_albums => albums) }
-
-        subject do
-          Spinna::Picker.new(config, history, library).pick(2, 'album')
-        end
-
-        it 'requests the pickable albums from the library' do
-          library
-            .expects(:pickable_albums)
-            .with('album')
-            .returns(albums)
-          capture_io { subject }
-        end
-
-        it 'randomly selects albums from the available ones' do
-          albums.expects(:sample).with(2).returns(picks)
-          capture_io { subject }
-        end
-
-        it 'returns the albums that match' do
-          capture_io do
-            subject.must_equal picks
-          end
-        end
-
-        it 'adds the picks to the history' do
-          history.expects(:append).times(picks.size)
-          capture_io { subject }
-        end
-
-        it 'saves the history' do
-          history.expects(:save).once
-          capture_io { subject }
-        end
-
-        it 'outputs the picks on the screen' do
-          out, err = capture_io do
-            subject
-          end
-          lines = out.split("\n")
-          lines.size.must_equal picks.size
-          lines.each do |line|
-            line.must_match %r{Copying album [1-3]\.\.\.}
-          end
-        end
-      end
-
-      context 'and there are not enough albums to choose from' do
-        let(:albums) { ['album 1', 'album 2', 'album 3'] }
-        let(:library) { stub(:pickable_albums => albums) }
-        let(:picks) { ['album 1', 'album 2', 'album 3'] }
-
-        subject do
-          Spinna::Picker.new(config, history, library).pick(5, 'album')
-        end
-
-        it 'requests the pickable_albums from the library' do
-          library
-            .expects(:pickable_albums)
-            .with('album')
-            .returns(albums)
-          capture_io { subject }
-        end
-
-        it 'returns all the pickable albums that match the pattern' do
-          capture_io { subject.must_equal picks }
-        end
-
-        it 'adds the picks to the history' do
-          history.expects(:append).times(picks.size)
-          capture_io { subject }
-        end
-
-        it 'saves the history' do
-          history.expects(:save).once
-          capture_io { subject }
-        end
-
-        it 'outputs the picks on the screen' do
-          out, err = capture_io do
-            subject
-          end
-          lines = out.split("\n")
-          lines.size.must_equal picks.size
-          lines.each do |line|
-            line.must_match %r{Copying album [1-3]\.\.\.}
-          end
-        end
-      end
-    end
-  end
-end