cineworld_uk 2.1.6 → 3.0.0

data/test/lib/cineworld_uk/cinema_test.rb

@@ -1,21 +1,21 @@
 require_relative '../../test_helper'
+require_relative '../../support/fixture_reader'
 
 describe CineworldUk::Cinema do
-  let(:website) { Minitest::Mock.new }
+  include Support::FixtureReader
 
-  before do
-    WebMock.disable_net_connect!
-  end
+  let(:described_class) { CineworldUk::Cinema }
+  let(:api_response) { Minitest::Mock.new }
+
+  before { WebMock.disable_net_connect! }
 
   describe '.all' do
-    subject { CineworldUk::Cinema.all }
+    subject { described_class.all }
 
-    before do
-      website.expect(:cinemas, cinemas_html)
-    end
+    before { api_response.expect(:cinema_list, cinema_list_json) }
 
     it 'returns an Array of CineworldUK::Cinemas' do
-      CineworldUk::Internal::Website.stub :new, website do
+      CineworldUk::Internal::ApiResponse.stub :new, api_response do
         subject.must_be_instance_of(Array)
         subject.each do |value|
           value.must_be_instance_of(CineworldUk::Cinema)
@@ -24,89 +24,30 @@ describe CineworldUk::Cinema do
     end
 
     it 'returns the correctly sized array' do
-      CineworldUk::Internal::Website.stub :new, website do
-        subject.size.must_equal 82
-      end
-    end
-  end
-
-  describe '.find(id)' do
-    subject { CineworldUk::Cinema.find(id) }
-
-    describe 'Brighton' do
-      let(:id) { 3 }
-
-      before do
-        website.expect(:cinemas, cinemas_html)
-      end
-
-      it 'returns a cinema' do
-        CineworldUk::Internal::Website.stub :new, website do
-          subject.must_be_instance_of(CineworldUk::Cinema)
-
-          subject.id.must_equal 3
-          subject.brand.must_equal 'Cineworld'
-          subject.name.must_equal 'Brighton'
-        end
+      CineworldUk::Internal::ApiResponse.stub :new, api_response do
+        subject.size.must_equal(88)
       end
     end
   end
 
-  describe '.new' do
-    it 'removes "London - " name prefix' do
-      cinema = CineworldUk::Cinema.new 79, 'London - The O2, Greenwich'
-      cinema.id.must_equal 79
-      cinema.name.must_equal 'The O2, Greenwich'
-      cinema.slug.must_equal 'the-o2-greenwich'
-    end
-
-    it 'removes " - " and replaces it with a colon ": "' do
-      cinema = CineworldUk::Cinema.new 88, 'Glasgow - IMAX at GSC'
-      cinema.id.must_equal 88
-      cinema.name.must_equal 'Glasgow: IMAX at GSC'
-      cinema.slug.must_equal 'glasgow-imax-at-gsc'
-    end
-  end
-
   describe '#adr' do
-    subject { cinema.adr }
+    subject { described_class.new(id).adr }
 
-    describe '(brighton)' do
-      let(:cinema) { CineworldUk::Cinema.new('3', 'Brighton') }
+    before do
+      api_response.expect(:cinema_detail, cinema_detail_json(id), [id])
+    end
 
-      before do
-        website.expect(:cinema_information, information_html('brighton'), [3])
-      end
+    describe 'Brighton (3)' do
+      let(:id) { 3 }
 
       it 'returns the address hash' do
-        CineworldUk::Internal::Website.stub :new, website do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
           subject.must_equal(
-            street_address: 'Brighton Marina',
+            street_address: 'Brighton Marina Village',
             extended_address: nil,
             locality: 'Brighton',
-            region: 'East Sussex',
-            postal_code: 'BN2 5UF',
-            country: 'United Kingdom'
-          )
-        end
-      end
-    end
-
-    describe '(bristol)' do
-      let(:cinema) { CineworldUk::Cinema.new('4', 'Bristol') }
-
-      before do
-        website.expect(:cinema_information, information_html('bristol'), [4])
-      end
-
-      it 'returns the address hash' do
-        CineworldUk::Internal::Website.stub :new, website do
-          subject.must_equal(
-            street_address: 'Hengrove Leisure Park',
-            extended_address: 'Hengrove Way',
-            locality: 'Bristol',
             region: nil,
-            postal_code: 'BS14 0HR',
+            postal_code: 'BN2 5UF',
             country: 'United Kingdom'
           )
         end
@@ -115,111 +56,89 @@ describe CineworldUk::Cinema do
   end
 
   describe '#extended_address' do
-    subject { cinema.extended_address }
-
-    describe '(brighton)' do
-      let(:cinema) { CineworldUk::Cinema.new('3', 'Brighton') }
-
-      before do
-        website.expect(:cinema_information, information_html('brighton'), [3])
-      end
+    subject { described_class.new(id).extended_address }
 
-      it 'returns nil' do
-        CineworldUk::Internal::Website.stub :new, website do
-          subject.must_be_nil
-        end
-      end
+    before do
+      api_response.expect(:cinema_detail, cinema_detail_json(id), [id])
     end
 
-    describe '(bristol)' do
-      let(:cinema) { CineworldUk::Cinema.new('4', 'Bristol') }
-
-      before do
-        website.expect(:cinema_information, information_html('bristol'), [4])
-      end
+    describe 'Brighton (3)' do
+      let(:id) { 3 }
 
-      it 'returns the second line' do
-        CineworldUk::Internal::Website.stub :new, website do
-          subject.must_equal 'Hengrove Way'
+      it 'returns the address hash' do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal('')
         end
       end
     end
   end
 
-  describe '#films' do
-    subject { CineworldUk::Cinema.new('3', 'Brighton').films }
-
-    it 'calls out to Screening object' do
-      CineworldUk::Film.stub :at, [:film] do
-        subject.must_equal([:film])
-      end
-    end
-  end
-
   describe '#full_name' do
-    subject { cinema.full_name }
+    subject { described_class.new(id).full_name }
+
+    before { api_response.expect(:cinema_list, cinema_list_json) }
 
-    describe 'simple name (brighton)' do
-      let(:cinema) { CineworldUk::Cinema.new('3', 'Brighton') }
+    describe 'simple name (Brighton)' do
+      let(:id) { 3 }
 
       it 'returns the brand in the name' do
-        subject.must_equal 'Cineworld Brighton'
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal 'Cineworld Brighton'
+        end
       end
     end
 
-    describe 'complex name (glasgow imax)' do
-      let(:cinema) { CineworldUk::Cinema.new('88', 'Glasgow - IMAX at GSC') }
+    describe 'complex name (Glasgow IMAX)' do
+      let(:id) { 88 }
 
       it 'returns the brand in the name' do
-        subject.must_equal 'Cineworld Glasgow: IMAX at GSC'
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal 'Cineworld Glasgow: IMAX at GSC'
+        end
       end
     end
   end
 
   describe '#locality' do
-    subject { cinema.locality }
+    subject { described_class.new(id).locality }
 
-    describe '(brighton)' do
-      let(:cinema) { CineworldUk::Cinema.new('3', 'Brighton') }
+    before do
+      api_response.expect(:cinema_detail, cinema_detail_json(id), [id])
+    end
 
-      before do
-        website.expect(:cinema_information, information_html('brighton'), [3])
-      end
+    describe 'Brighton (3)' do
+      let(:id) { 3 }
 
       it 'returns the town/city' do
-        CineworldUk::Internal::Website.stub :new, website do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
           subject.must_equal 'Brighton'
         end
       end
     end
 
-    describe '(bristol)' do
-      let(:cinema) { CineworldUk::Cinema.new('4', 'Bristol') }
+    describe 'London - 10 (Chelsea)' do
+      let(:id) { 10 }
 
-      before do
-        website.expect(:cinema_information, information_html('bristol'), [4])
-      end
-
-      it 'returns the town/city' do
-        CineworldUk::Internal::Website.stub :new, website do
-          subject.must_equal 'Bristol'
+      it 'returns borough of London' do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal 'Chelsea'
         end
       end
     end
   end
 
   describe '#postal_code' do
-    subject { cinema.postal_code }
+    subject { described_class.new(id).postal_code }
 
-    describe '(brighton)' do
-      let(:cinema) { CineworldUk::Cinema.new('3', 'Brighton') }
+    before do
+      api_response.expect(:cinema_detail, cinema_detail_json(id), [id])
+    end
 
-      before do
-        website.expect(:cinema_information, information_html('brighton'), [3])
-      end
+    describe 'Brighton (3)' do
+      let(:id) { 3 }
 
       it 'returns the post code' do
-        CineworldUk::Internal::Website.stub :new, website do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
           subject.must_equal 'BN2 5UF'
         end
       end
@@ -227,88 +146,68 @@ describe CineworldUk::Cinema do
   end
 
   describe '#region' do
-    subject { cinema.region }
-
-    describe '(brighton)' do
-      let(:cinema) { CineworldUk::Cinema.new('3', 'Brighton') }
-
-      before do
-        website.expect(:cinema_information, information_html('brighton'), [3])
-      end
+    subject { described_class.new(id).region }
 
-      it 'returns the county' do
-        CineworldUk::Internal::Website.stub :new, website do
-          subject.must_equal 'East Sussex'
-        end
-      end
+    before do
+      api_response.expect(:cinema_detail, cinema_detail_json(id), [id])
     end
 
-    describe '(bristol)' do
-      let(:cinema) { CineworldUk::Cinema.new('4', 'Bristol') }
-
-      before do
-        website.expect(:cinema_information, information_html('bristol'), [4])
-      end
+    describe 'no region - Brighton (3)' do
+      let(:id) { 3 }
 
-      it 'returns nil' do
-        CineworldUk::Internal::Website.stub :new, website do
-          subject.must_be_nil
+      it 'returns the empty string is none exists' do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal ''
         end
       end
     end
-  end
 
-  describe '#screenings' do
-    subject { CineworldUk::Cinema.new('3', 'Brighton').screenings }
+    describe 'London - Chelsea (10)' do
+      let(:id) { 10 }
 
-    it 'calls out to Screening object' do
-      CineworldUk::Screening.stub :at, [:screening] do
-        subject.must_equal([:screening])
+      it 'returns "London"' do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal 'London'
+        end
       end
     end
   end
 
   describe '#street_address' do
-    subject { cinema.street_address }
+    subject { described_class.new(id).street_address }
 
-    describe '(brighton)' do
-      let(:cinema) { CineworldUk::Cinema.new('3', 'Brighton') }
+    before do
+      api_response.expect(:cinema_detail, cinema_detail_json(id), [id])
+    end
 
-      before do
-        website.expect(:cinema_information, information_html('brighton'), [3])
-      end
+    describe 'Brighton (3)' do
+      let(:id) { 3 }
 
-      it 'returns the street address' do
-        CineworldUk::Internal::Website.stub :new, website do
-          subject.must_equal 'Brighton Marina'
+      it 'returns the first line of the Address' do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal 'Brighton Marina Village'
         end
       end
     end
   end
 
   describe '#url' do
-    subject { cinema.url }
-
-    describe '(brighton)' do
-      let(:cinema) { CineworldUk::Cinema.new('3', 'Brighton') }
+    subject { described_class.new(id).url }
 
-      it 'returns the what on url' do
-        subject.must_equal 'http://www.cineworld.co.uk/whatson?cinema=3'
-      end
+    before do
+      api_response.expect(:cinema_detail, cinema_detail_json(id), [id])
     end
-  end
-
-  private
 
-  def read_file(filepath)
-    File.read(File.expand_path(filepath, __FILE__))
-  end
-
-  def cinemas_html
-    read_file('../../../fixtures/cinemas.html')
-  end
+    describe 'Brighton (3)' do
+      let(:id) { 3 }
 
-  def information_html(filename)
-    read_file("../../../fixtures/information/#{filename}.html")
+      it 'returns the url' do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal(
+            "http://www.cineworld.co.uk/cinemas/#{id}/information"
+          )
+        end
+      end
+    end
   end
 end

data/test/lib/cineworld_uk/internal/api_response_test.rb

@@ -0,0 +1,85 @@
+require_relative '../../../test_helper'
+require_relative '../../../support/fixture_reader'
+
+describe CineworldUk::Internal::ApiResponse do
+  include Support::FixtureReader
+
+  let(:described_class) { CineworldUk::Internal::ApiResponse }
+  let(:standard) { 'key=ios&territory=GB' }
+
+  describe '#cinema_list' do
+    subject { described_class.new.cinema_list }
+
+    before { stub_get("cinema/list?full=true&#{standard}", cinema_list_json) }
+
+    it 'returns a string' do
+      subject.class.must_equal String
+    end
+  end
+
+  describe '#cinema_detail(cinema_id, date)' do
+    subject { described_class.new.cinema_detail(3) }
+
+    before do
+      stub_get("cinema/detail?cinema=3&#{standard}", cinema_detail_json(3))
+    end
+
+    it 'returns a string' do
+      subject.class.must_equal String
+    end
+  end
+
+  describe '#dates(cinema_id)' do
+    subject { described_class.new.dates(3) }
+
+    before { stub_get("dates?cinema=3&#{standard}", dates_json(3)) }
+
+    it 'returns a string' do
+      subject.class.must_equal String
+    end
+  end
+
+  describe '#film_list' do
+    subject { described_class.new.film_list }
+
+    before { stub_get("film/list?full=true&#{standard}", film_list_json) }
+
+    it 'returns a string' do
+      subject.class.must_equal String
+    end
+  end
+
+  describe '#film_list_comingsoon' do
+    subject { described_class.new.film_list_comingsoon }
+
+    before do
+      stub_get("film/list/comingsoon?full=true&#{standard}",
+               film_list_comingsoon_json)
+    end
+
+    it 'returns a string' do
+      subject.class.must_equal String
+    end
+  end
+
+  describe '#performances(cinema_id, date)' do
+    subject { described_class.new.performances(3, Date.today + 1) }
+
+    before do
+      stub_get("performances?cinema=3&date=#{tomorrow_s}&#{standard}",
+               performances_tomorrow_json(3))
+    end
+
+    it 'returns a string' do
+      subject.class.must_equal String
+    end
+  end
+
+  private
+
+  def stub_get(site_path, response_body)
+    url      = "http://www2.cineworld.co.uk/api/#{site_path}"
+    response = { status: 200, body: response_body, headers: {} }
+    stub_request(:get, url).to_return(response)
+  end
+end

data/test/lib/cineworld_uk/internal/parser/api/cinema_address_test.rb

@@ -0,0 +1,93 @@
+require_relative '../../../../../test_helper'
+require_relative '../../../../../support/fixture_reader'
+
+describe CineworldUk::Internal::Parser::Api::CinemaAddress do
+  include Support::FixtureReader
+
+  let(:described_class) { CineworldUk::Internal::Parser::Api::CinemaAddress }
+  let(:api_response) { Minitest::Mock.new }
+
+  before { WebMock.disable_net_connect! }
+
+  describe '#to_hash' do
+    subject { described_class.new(id).to_hash }
+
+    before do
+      api_response.expect(:cinema_detail, cinema_detail_json(id), [id])
+    end
+
+    describe 'passed simple (Brighton)' do
+      let(:id) { 3 }
+
+      it 'returns address hash' do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal(street_address:   'Brighton Marina Village',
+                             extended_address: nil,
+                             locality:         'Brighton',
+                             region:           nil,
+                             postal_code:      'BN2 5UF',
+                             country:          'United Kingdom')
+        end
+      end
+    end
+
+    describe 'passed three line (NEC)' do
+      let(:id) { 96 }
+
+      it 'returns address hash' do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal(street_address:   'Resorts World',
+                             extended_address: 'Pendigo Way',
+                             locality:         'Birmingham',
+                             region:           nil,
+                             postal_code:      'B40 1PU',
+                             country:          'United Kingdom')
+        end
+      end
+    end
+
+    describe 'passed three line (Edinburgh)' do
+      let(:id) { 21 }
+
+      it 'returns address hash' do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal(street_address:   'Fountain Park',
+                             extended_address: '130/3 Dundee Street',
+                             locality:         'Edinburgh',
+                             region:           nil,
+                             postal_code:      'EH11 1AF',
+                             country:          'United Kingdom')
+        end
+      end
+    end
+
+    describe 'passed three line (Chelsea)' do
+      let(:id) { 10 }
+
+      it 'returns address hash' do
+        CineworldUk::Internal::ApiResponse.stub :new, api_response do
+          subject.must_equal(street_address:   '279 Kings Road',
+                             extended_address: nil,
+                             locality:         'Chelsea',
+                             region:           'London',
+                             postal_code:      'SW3 5EW',
+                             country:          'United Kingdom')
+        end
+      end
+    end
+
+    # describe 'passed non-existant api' do
+    #   let(:id) { 0 }
+    #
+    #   it 'returns hash of nils' do
+    #     subject.must_be_instance_of(Hash)
+    #     subject.must_equal(street_address:   nil,
+    #                        extended_address: nil,
+    #                        locality:         "not an address",
+    #                        region:           nil,
+    #                        postal_code:      "not an address",
+    #                        country:          "United Kingdom")
+    #   end
+    # end
+  end
+end

data/test/lib/cineworld_uk/internal/parser/api/film_lookup_test.rb

@@ -0,0 +1,30 @@
+require_relative '../../../../../test_helper'
+require_relative '../../../../../support/fixture_reader'
+
+describe CineworldUk::Internal::Parser::Api::FilmLookup do
+  include Support::FixtureReader
+
+  let(:described_class) { CineworldUk::Internal::Parser::Api::FilmLookup }
+  let(:api_response) { Minitest::Mock.new }
+
+  before { WebMock.disable_net_connect! }
+
+  before do
+    api_response.expect(:film_list, film_list_json)
+    api_response.expect(:film_list_comingsoon, film_list_comingsoon_json)
+  end
+
+  describe '#to_hash' do
+    subject { described_class.new.to_hash }
+
+    it 'returns a Hash of Film objects, keyed by Integers' do
+      CineworldUk::Internal::ApiResponse.stub :new, api_response do
+        subject.must_be_instance_of(Hash)
+        subject.each do |key, value|
+          key.must_be_instance_of(Fixnum)
+          value.must_be_instance_of(CineworldUk::Internal::Parser::Api::Film)
+        end
+      end
+    end
+  end
+end