bfi_player_search 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bb3485213ce0966d67e48e710b21139558dc9354
-  data.tar.gz: 932e718ec4dccb5e6a8f3d571d8d72f727002370
+  metadata.gz: 1720bc250f6881e64c64249df35ba6d4442b06c6
+  data.tar.gz: 0b62081df2424d1ce0074915fdbec9b3560d5beb
 SHA512:
-  metadata.gz: 43b6362b34288678f810c0bae5be57427531788d4e046fc80dc7cbc57e150e1c839af345739552ecd07bca42f9cf796539949e3a7712f49237617d629e20d64d
-  data.tar.gz: cc5089280d6dc96eca3bfe0e7ee6b7a2d16e33b83ff15b341b982a92bbe7769a7f3d851c881511bdab8fa3d126058fc58e651f53a19e230d2ab61bf1ed329d57
+  metadata.gz: d2c042c104b1343826837e349d89612a5ce90abd3784f4f2d2a624082cec1518432d8f4b232bd72fe357a4d1448997b1e6b60c4f129f9f517249af9f3d25b084
+  data.tar.gz: d804ed0470879024ab4c5c7e0225cde7ab6a4582df0424e2764e4073c1b6ed56c3abfbbd0a2e4b3ca08bdd89b522f341ad22b8c3e82f0ec490f2db9e13849039

data/lib/bfi_player_search/result_parser.rb

@@ -34,6 +34,16 @@ module BFIPlayerSearch
       convert_to_url(path)
     end
 
+    def running_time_in_minutes
+      match = fragment.css('.metrics span').map { |n| %r{(\d+) mins}.match(n.content) }.compact.first
+      match && match[1].to_i
+    end
+
+    def director
+      match = fragment.css('.metrics span').map { |n| %r{Director\. (.*)\Z}.match(n.content) }.compact.first
+      match && match[1].strip
+    end
+
     private
     attr_reader :fragment
 

data/lib/bfi_player_search/search.rb

@@ -14,7 +14,9 @@ module BFIPlayerSearch
           :image_url => rp.image_url,
           :year => rp.year,
           :certificate => rp.certificate,
-          :free => rp.free?
+          :free => rp.free?,
+          :running_time_in_minutes => rp.running_time_in_minutes,
+          :director => rp.director
         }
       end
 

data/spec/feature/search_spec.rb

@@ -6,12 +6,16 @@ describe 'A search' do
   end
 
   context 'with some results', :vcr do
-    subject { BFIPlayerSearch::Search.new.search('girl') }
+    subject { BFIPlayerSearch::Search.new.search('wild tales') }
 
     it { expect(subject).to_not be_empty }
-    it { expect(subject.first[:title]).to_not be_empty }
+    it { expect(subject.first[:title]).to eq 'Wild Tales' }
     it { expect(subject.first[:url]).to match(%r{^http://player.bfi.org.uk/film/watch-.*}) }
     it { expect(subject.first[:image_url]).to match(%r{^http://player.bfi.org.uk//media/images/stills/film/.*\.jpg}) }
+    it { expect(subject.first[:free]).to be_falsey }
+    it { expect(subject.first[:certificate]).to eq('15') }
+    it { expect(subject.first[:running_time_in_minutes]).to eq(122) }
+    it { expect(subject.first[:director]).to eq('Damian Szifron') }
   end
 
   context 'with unrecognised page format returned' do

data/spec/lib/bfi_player_search/result_parser_spec.rb

@@ -94,4 +94,48 @@ describe BFIPlayerSearch::ResultParser do
       it { expect(subject).to be_free }
     end
   end
+
+  describe '#running_time_in_minutes' do
+    context 'no running time given' do
+      it { expect(subject.running_time_in_minutes).to be_nil }
+    end
+
+    context 'running time given' do
+      let(:fragment) {
+        Nokogiri::HTML::DocumentFragment.parse("<article class='film'>
+          <div class='film-preview'>
+            <p class='film-info metrics'>
+              <span>95 mins</span>
+              <span>United Kingdom</span>
+              <span>Director. Andrew Haigh</span>
+            </p>
+          </div>
+        </article>")
+      }
+
+      it { expect(subject.running_time_in_minutes).to eq(95) }
+    end
+  end
+
+  describe '#director' do
+    context 'no director given' do
+      it { expect(subject.director).to be_nil }
+    end
+
+    context 'director given' do
+      let(:fragment) {
+        Nokogiri::HTML::DocumentFragment.parse("<article class='film'>
+          <div class='film-preview'>
+            <p class='film-info metrics'>
+              <span>95 mins</span>
+              <span>United Kingdom</span>
+              <span>Director. Andrew Haigh</span>
+            </p>
+          </div>
+        </article>")
+      }
+
+      it { expect(subject.director).to eq('Andrew Haigh') }
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bfi_player_search
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Joel Chippindale