torganiser 0.0.5 → 0.1.0

data/lib/torganiser/series.rb

@@ -1,11 +1,9 @@
 module Torganiser
-
   # Models the series information from an episode file
   class Series
-
     attr_reader :name, :year
 
-    def initialize name, year: nil
+    def initialize(name, year: nil)
       @name = name
       @year = year
     end
@@ -13,7 +11,5 @@ module Torganiser
     def display_name
       @display_name = year ? "#{name} (#{year})" : name
     end
-
   end
-
 end

data/lib/torganiser/version.rb

@@ -1,3 +1,4 @@
+# Defines gem version
 module Torganiser
-  VERSION = "0.0.5"
+  VERSION = '0.1.0'
 end

data/spec/spec_helper.rb

@@ -1,4 +1,4 @@
-require "codeclimate-test-reporter"
+require 'codeclimate-test-reporter'
 CodeClimate::TestReporter.start
 # At the top because simplecov needs to watch files being loaded
 require 'simplecov'

data/spec/torganiser/arranger_spec.rb

@@ -1,36 +1,53 @@
 require 'spec_helper'
 
+# Main module
 module Torganiser
-
   describe Arranger do
 
-    describe "for a collection" do
-      subject { Arranger.new("/my/media") }
+    describe 'for a collection' do
+      subject { Arranger.new('/my/media') }
 
-      context "when arranging a file" do
+      context 'when arranging a file' do
 
         before do
           allow(FileUtils).to receive(:mkdir_p)
           allow(FileUtils).to receive(:mv)
         end
 
-        let(:file) { "/tmp/stuff/Waffle.Cone.2007.S01E02.HDTV.x264-LOL.mp4" }
+        let(:file) { '/tmp/stuff/Waffle.Cone.2007.S01E02.HDTV.x264-LOL.mp4' }
 
-        it "creates a destination by series and season" do
+        it 'creates a destination by series and season' do
           expect(FileUtils).to receive(:mkdir_p).with(
-            "/my/media/Waffle Cone (2007)/Season 1"
+            '/my/media/Waffle Cone (2007)/Season 1'
           )
           subject.arrange file
         end
 
-        it "moves the file to the destination" do
+        it 'moves the file to the destination' do
           expect(FileUtils).to receive(:mv).with(
             file,
-            "/my/media/Waffle Cone (2007)/Season 1"
+            '/my/media/Waffle Cone (2007)/Season 1'
           )
           subject.arrange file
         end
 
+        context 'when set up to copy, rather than move' do
+
+          before do
+            allow(FileUtils).to receive(:cp)
+          end
+
+          subject { Arranger.new('/my/media', copy: true) }
+
+          it 'copies the file to the destination' do
+            expect(FileUtils).to receive(:cp).with(
+              file,
+              '/my/media/Waffle Cone (2007)/Season 1'
+            )
+            subject.arrange file
+          end
+        end
+
       end
     end
   end

data/spec/torganiser/episode_file_spec.rb

@@ -1,69 +1,72 @@
 require 'spec_helper'
 
+# Main module
 module Torganiser
-
   describe EpisodeFile do
 
     subject { EpisodeFile.new(file) }
 
-    context "when initialized with an informative filename" do
+    let(:match) do
+      {
+        name: 'Hello',
+        season: '02',
+        episode: '01'
+      }
+    end
+
+    let(:file) { 'file/path/Hello.S02E01.mp4' }
 
-      let(:file) { "file/path/Hello.S02E01.mp4"}
+    before do
+      allow(Matcher).to receive(:match).and_return match
+    end
+
+    context 'when initialized with a filename' do
 
       it 'extracts the base file name' do
         expect(subject.basename).to eq 'Hello.S02E01.mp4'
       end
 
-      it 'extracts season number' do
-        expect(subject.season).to eq 2
+      it 'matches the basename using the Matcher' do
+        expect(Matcher).to receive(:match).with('Hello.S02E01.mp4')
+        subject.series
       end
 
-      it 'extracts episode number' do
-        expect(subject.episode).to eq 1
+      it 'extracts season number as an integer' do
+        expect(subject.season).to eq 2
       end
 
-      it 'creates a series with a name and no year' do
-        expect(Series).to receive(:new).with(
-          "Hello", year: nil
-        )
-        subject.series
+      it 'extracts episode number as an integer' do
+        expect(subject.episode).to eq 1
       end
 
-      context "that contains year information" do
-
-        let(:file) { "file/path/Hello.2008.S02E01.mp4"}
+      context 'that contains year information' do
+        let(:match) do
+          {
+            name: 'Hello',
+            year: '2008'
+          }
+        end
 
         it 'creates a series with a name and year' do
           expect(Series).to receive(:new).with(
-            "Hello", year: 2008
+            'Hello', year: 2008
           )
           subject.series
         end
 
       end
 
-      context "that is a double episode" do
+      context 'that has a series name in dot-format' do
 
-        let(:file) { "file/path/Hello.2008.S02E01E02.mp4"}
-
-        it 'extracts season number' do
-          expect(subject.season).to eq 2
-        end
-
-        it 'extracts first episode number' do
-          expect(subject.episode).to eq 1
+        let(:match) do
+          {
+            name: 'Goodbye.Hello.Hamburger'
+          }
         end
 
-
-      end
-
-      context "that has a series name in dot-format" do
-
-        let(:file) { "file/path/Goodbye.Hello.Hamburger.2008.S02E01.mp4"}
-
         it 'creates a series with a name with spaces' do
           expect(Series).to receive(:new).with(
-            "Goodbye Hello Hamburger",
+            'Goodbye Hello Hamburger',
             anything
           )
           subject.series
@@ -71,96 +74,34 @@ module Torganiser
 
       end
 
-      context "in short format" do
-        let(:file) { "file/path/Hello.2014.302.hdtv-lol.mp4" }
-
-        it 'extracts season number' do
-          expect(subject.season).to eq 3
-        end
+      context 'that is dash-separated' do
 
-        it 'extracts episode number' do
-          expect(subject.episode).to eq 2
+        let(:match) do
+          {
+            name: "Wiffle's Berry - "
+          }
         end
 
-        context "with an 'x'" do
-          let(:file) { "file/path/Hello.2014.4x07.hdtv-lol.mp4" }
-
-          it 'extracts season number' do
-            expect(subject.season).to eq 4
-          end
-
-          it 'extracts episode number' do
-            expect(subject.episode).to eq 7
-          end
-
-        end
-
-        context "surrounded in square brackets" do
-          let(:file) { "file/path/Wootle [3x08] Some title here.avi" }
-
-          it 'extracts season number' do
-            expect(subject.season).to eq 3
-          end
-
-          it 'extracts episode number' do
-            expect(subject.episode).to eq 8
-          end
-
-        end
-
-      end
-
-      context "that is dash-separated" do
-        let(:file) { "Wiffle's Berry - S01E12-the title.avi" }
-
-        it 'extracts series correctly' do
+        it 'creates a series with the correct name' do
           expect(Series).to receive(:new).with(
             "Wiffle's Berry", year: nil
           )
           subject.series
         end
 
-        it 'extracts season number' do
-          expect(subject.season).to eq 1
-        end
-
-        it 'extracts episode number' do
-          expect(subject.episode).to eq 12
-        end
-      end
-
-      context "that is a special" do
-        let(:file) { "file/path/Hello.2014.s00.hdtv-lol.mp4" }
-
-        it 'extracts season number as zero' do
-          expect(subject.season).to eq 0
-        end
-
-        it 'extracts episode number as zero' do
-          expect(subject.episode).to eq 0
-        end
-
-        context "with a season specified" do
-          let(:file) { "file/path/Hello.2014.s01.special.hdtv-lol.mp4" }
-
-          it 'extracts season number' do
-            expect(subject.season).to eq 1
-          end
-
-        end
       end
 
     end
 
-    context "when initialised with a file whose name cannot be parsed" do
+    context 'when initialised with a file whose name cannot be parsed' do
 
-      let(:file) { "file/path/Hello.this-contains-no-information.mp4"}
+      let(:match) { nil }
 
       it 'blows up in a more helpful manner' do
-        expect{ subject.series }.to raise_error(/Unable to parse #{file}/) end
+        expect { subject.series }.to raise_error(/Unable to parse #{file}/)
+      end
 
     end
 
   end
-
 end

data/spec/torganiser/file_query_spec.rb

@@ -1,49 +1,49 @@
 require 'spec_helper'
 
+# Main module
 module Torganiser
-
   describe FileQuery do
 
-    context "when initialised" do
+    context 'when initialised' do
 
-      context "with no arguments" do
+      context 'with no arguments' do
 
-        it "has no directories" do
+        it 'has no directories' do
           expect(subject.directories).to be_empty
         end
 
-        it "has no media extensions" do
+        it 'has no media extensions' do
           expect(subject.extensions).to be_empty
         end
 
-        it "is empty" do
+        it 'is empty' do
           expect(subject).to be_empty
         end
 
       end
 
-      context "with a directory" do
-        subject { FileQuery.new(directories: "woot/waffle") }
+      context 'with a directory' do
+        subject { FileQuery.new(directories: 'woot/waffle') }
 
-        it "is not empty" do
+        it 'is not empty' do
           expect(subject).not_to be_empty
         end
 
-        it "adds that directory to the list of search directories" do
+        it 'adds that directory to the list of search directories' do
           expect(subject.directories).to eq ['woot/waffle']
         end
 
       end
 
-      context "with a directory and a media extension" do
+      context 'with a directory and a media extension' do
 
         subject { FileQuery.new(directories: 'mydir', extensions: 'mp4') }
 
-        it "adds that directory to the list of search directories" do
+        it 'adds that directory to the list of search directories' do
           expect(subject.directories).to eq ['mydir']
         end
 
-        it "adds that extension to the list of media extensions" do
+        it 'adds that extension to the list of media extensions' do
           expect(subject.extensions).to eq ['mp4']
         end
 
@@ -51,7 +51,7 @@ module Torganiser
 
     end
 
-    describe "search pattern" do
+    describe 'search pattern' do
 
       let(:extensions) { nil }
 
@@ -64,49 +64,49 @@ module Torganiser
         )
       end
 
-      context "with one directory" do
+      context 'with one directory' do
 
-        let(:directories) { "stuff" }
+        let(:directories) { 'stuff' }
 
-        it "includes only that directory" do
+        it 'includes only that directory' do
           expect(subject.pattern).to match(/^stuff\/\*\*/)
         end
 
       end
 
-      context "with multiple directories" do
+      context 'with multiple directories' do
 
-        let(:directories) { ["one","two"] }
+        let(:directories) { %w(one two) }
 
-        it "includes all specified directories" do
+        it 'includes all specified directories' do
           expect(subject.pattern).to match(/{one,two}\/\*\*/)
         end
 
       end
 
-      context "with no extensions" do
+      context 'with no extensions' do
 
-        it "matches all files" do
+        it 'matches all files' do
           expect(subject.pattern).to match(/\*\*\/\*/)
         end
 
       end
 
-      context "with one extension" do
+      context 'with one extension' do
 
-        let(:extensions) { "mp4" }
+        let(:extensions) { 'mp4' }
 
-        it "includes only that extension" do
+        it 'includes only that extension' do
           expect(subject.pattern).to match(/\*\*\/\*mp4/)
         end
 
       end
 
-      context "with multiple extensions" do
+      context 'with multiple extensions' do
 
-        let(:extensions) { ["mp4", "mov", "qt"] }
+        let(:extensions) { %w(mp4 mov qt) }
 
-        it "includes all specified extensions" do
+        it 'includes all specified extensions' do
           expect(subject.pattern).to match(/\*\*\/\*{mp4,mov,qt}/)
         end
 

data/spec/torganiser/match_one_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+# Main module
+module Torganiser
+  describe MatchOne do
+    it 'stringifies a set of alternatives' do
+      expect(MatchOne.new(%w(a b c)).to_s).to eq '(a|b|c)'
+    end
+  end
+end