picturehouse_uk 3.0.14 → 4.0.0

data/test/live/integration_test.rb

@@ -1,7 +1,9 @@
 require 'minitest/autorun'
 require 'minitest/reporters'
-reporter_options = { color: true, slow_count: 5 }
-Minitest::Reporters.use! [Minitest::Reporters::DefaultReporter.new(reporter_options)]
+
+Minitest::Reporters.use! [
+  Minitest::Reporters::DefaultReporter.new(color: true)
+]
 
 require File.expand_path('../../../lib/picturehouse_uk.rb', __FILE__)
 
@@ -29,35 +31,16 @@ describe PicturehouseUk::Cinema do
   end
 end
 
-describe PicturehouseUk::Film do
-  let(:described_class) { PicturehouseUk::Film }
-
-  describe '.at(cinema_id)' do
-    subject { described_class.at('Duke_Of_Yorks') }
-
-    it 'returns an array of films' do
-      subject.must_be_instance_of(Array)
-      subject.each do |film|
-        film.must_be_instance_of(PicturehouseUk::Film)
-      end
-    end
-
-    it 'returns a decent number of films' do
-      subject.count.must_be :>, 5
-    end
-  end
-end
-
-describe PicturehouseUk::Screening do
-  let(:described_class) { PicturehouseUk::Screening }
+describe PicturehouseUk::Performance do
+  let(:described_class) { PicturehouseUk::Performance }
 
   describe '.at(cinema_id)' do
     subject { described_class.at('Duke_Of_Yorks') }
 
     it 'returns an array of screenings' do
       subject.must_be_instance_of(Array)
-      subject.each do |screening|
-        screening.must_be_instance_of(PicturehouseUk::Screening)
+      subject.each do |performance|
+        performance.must_be_instance_of(PicturehouseUk::Performance)
       end
     end
 

data/test/support/fake_website.rb

@@ -0,0 +1,24 @@
+# Fake website for reading fixtures from disk
+class FakeWebsite
+  def home
+    read_file('../../fixtures/home.html')
+  end
+
+  def cinema(cinema_id)
+    read_file("../../fixtures/#{cinema_id}/cinema.html")
+  end
+
+  def info(cinema_id)
+    read_file("../../fixtures/#{cinema_id}/info.html")
+  end
+
+  def whats_on(cinema_id)
+    read_file("../../fixtures/#{cinema_id}/whats_on.html")
+  end
+
+  private
+
+  def read_file(filepath)
+    File.read(File.expand_path(filepath, __FILE__))
+  end
+end

data/test/test_helper.rb

@@ -1,8 +1,18 @@
+require 'codeclimate-test-reporter'
+CodeClimate::TestReporter.start
+
 require 'minitest/autorun'
 require 'minitest/reporters'
-reporter_options = { color: true, slow_count: 5 }
-Minitest::Reporters.use! [Minitest::Reporters::DefaultReporter.new(reporter_options)]
+
+Minitest::Reporters.use! [
+  Minitest::Reporters::DefaultReporter.new(color: true, slow_count: 5)
+]
+
+require 'webmock/minitest'
+WebMock.allow_net_connect!
 
 require 'webmock/minitest'
 
 require File.expand_path('../../lib/picturehouse_uk.rb', __FILE__)
+
+require_relative 'support/fake_website'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: picturehouse_uk
 version: !ruby/object:Gem::Version
-  version: 3.0.14
+  version: 4.0.0
 platform: ruby
 authors:
 - Andy Croll
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-17 00:00:00.000000000 Z
+date: 2016-02-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -25,7 +25,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
-  name: minitest-reporters
+  name: codeclimate-test-reporter
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -39,7 +39,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: minitest-reporters
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -53,7 +53,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: webmock
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,13 +67,13 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: nokogiri
+  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  type: :runtime
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -81,21 +81,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: tzinfo
+  name: cinebase
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
-  name: tzinfo-data
+  name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -108,7 +108,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: An API to pull movie information from the picturehouse.co.uk website
+description: API pulling information from picturehouse.com
 email:
 - andy@goodscary.com
 executables: []
@@ -116,49 +116,50 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
-- ".rdoc_options"
 - ".travis.yml"
 - ".yardopts"
 - CHANGELOG.md
+- COMM-LICENSE
 - Gemfile
-- LICENSE.txt
+- LICENSE
 - README.md
 - Rakefile
 - lib/picturehouse_uk.rb
 - lib/picturehouse_uk/cinema.rb
-- lib/picturehouse_uk/film.rb
 - lib/picturehouse_uk/internal/parser/address.rb
 - lib/picturehouse_uk/internal/parser/screenings.rb
 - lib/picturehouse_uk/internal/title_sanitizer.rb
 - lib/picturehouse_uk/internal/website.rb
-- lib/picturehouse_uk/screening.rb
+- lib/picturehouse_uk/performance.rb
 - lib/picturehouse_uk/version.rb
 - picturehouse_uk.gemspec
-- test/fixture_updater.rb
-- test/fixtures/cinema/Duke_Of_Yorks.html
-- test/fixtures/cinema/Dukes_At_Komedia.html
-- test/fixtures/cinema/National_Media_Museum.html
-- test/fixtures/cinema/Phoenix_Picturehouse.html
+- rake/fixture_creator.rb
+- test/fixtures/Duke_Of_Yorks/cinema.html
+- test/fixtures/Duke_Of_Yorks/info.html
+- test/fixtures/Duke_Of_Yorks/whats_on.html
+- test/fixtures/Dukes_At_Komedia/cinema.html
+- test/fixtures/Dukes_At_Komedia/info.html
+- test/fixtures/Dukes_At_Komedia/whats_on.html
+- test/fixtures/National_Media_Museum/cinema.html
+- test/fixtures/National_Media_Museum/info.html
+- test/fixtures/National_Media_Museum/whats_on.html
+- test/fixtures/Phoenix_Picturehouse/cinema.html
+- test/fixtures/Phoenix_Picturehouse/info.html
+- test/fixtures/Phoenix_Picturehouse/whats_on.html
 - test/fixtures/home.html
-- test/fixtures/info/Duke_Of_Yorks.html
-- test/fixtures/info/Dukes_At_Komedia.html
-- test/fixtures/info/Phoenix_Picturehouse.html
-- test/fixtures/whats_on/Duke_Of_Yorks.html
-- test/fixtures/whats_on/Dukes_At_Komedia.html
-- test/fixtures/whats_on/National_Media_Museum.html
-- test/fixtures/whats_on/Phoenix_Picturehouse.html
 - test/lib/picturehouse_uk/cinema_test.rb
-- test/lib/picturehouse_uk/film_test.rb
-- test/lib/picturehouse_uk/internal/parser/address_parser_test.rb
+- test/lib/picturehouse_uk/internal/parser/address_test.rb
 - test/lib/picturehouse_uk/internal/parser/screenings_test.rb
 - test/lib/picturehouse_uk/internal/title_sanitizer_test.rb
 - test/lib/picturehouse_uk/internal/website_test.rb
-- test/lib/picturehouse_uk/screening_test.rb
+- test/lib/picturehouse_uk/performance_test.rb
 - test/lib/picturehouse_uk/version_test.rb
 - test/live/integration_test.rb
+- test/support/fake_website.rb
 - test/test_helper.rb
 homepage: ''
 licenses:
+- AGPL
 - MIT
 metadata: {}
 post_install_message: 
@@ -177,32 +178,32 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: It's a scraper, but a nice one
 test_files:
-- test/fixture_updater.rb
-- test/fixtures/cinema/Duke_Of_Yorks.html
-- test/fixtures/cinema/Dukes_At_Komedia.html
-- test/fixtures/cinema/National_Media_Museum.html
-- test/fixtures/cinema/Phoenix_Picturehouse.html
+- test/fixtures/Duke_Of_Yorks/cinema.html
+- test/fixtures/Duke_Of_Yorks/info.html
+- test/fixtures/Duke_Of_Yorks/whats_on.html
+- test/fixtures/Dukes_At_Komedia/cinema.html
+- test/fixtures/Dukes_At_Komedia/info.html
+- test/fixtures/Dukes_At_Komedia/whats_on.html
+- test/fixtures/National_Media_Museum/cinema.html
+- test/fixtures/National_Media_Museum/info.html
+- test/fixtures/National_Media_Museum/whats_on.html
+- test/fixtures/Phoenix_Picturehouse/cinema.html
+- test/fixtures/Phoenix_Picturehouse/info.html
+- test/fixtures/Phoenix_Picturehouse/whats_on.html
 - test/fixtures/home.html
-- test/fixtures/info/Duke_Of_Yorks.html
-- test/fixtures/info/Dukes_At_Komedia.html
-- test/fixtures/info/Phoenix_Picturehouse.html
-- test/fixtures/whats_on/Duke_Of_Yorks.html
-- test/fixtures/whats_on/Dukes_At_Komedia.html
-- test/fixtures/whats_on/National_Media_Museum.html
-- test/fixtures/whats_on/Phoenix_Picturehouse.html
 - test/lib/picturehouse_uk/cinema_test.rb
-- test/lib/picturehouse_uk/film_test.rb
-- test/lib/picturehouse_uk/internal/parser/address_parser_test.rb
+- test/lib/picturehouse_uk/internal/parser/address_test.rb
 - test/lib/picturehouse_uk/internal/parser/screenings_test.rb
 - test/lib/picturehouse_uk/internal/title_sanitizer_test.rb
 - test/lib/picturehouse_uk/internal/website_test.rb
-- test/lib/picturehouse_uk/screening_test.rb
+- test/lib/picturehouse_uk/performance_test.rb
 - test/lib/picturehouse_uk/version_test.rb
 - test/live/integration_test.rb
+- test/support/fake_website.rb
 - test/test_helper.rb
 has_rdoc: 

data/.rdoc_options

@@ -1,16 +0,0 @@
---- !ruby/object:RDoc::Options
-encoding: UTF-8
-static_path: []
-rdoc_include:
-- .
-charset: UTF-8
-exclude: 
-hyperlink_all: false
-line_numbers: false
-main_page: 
-markup: tomdoc
-show_hash: false
-tab_width: 8
-title: 
-visibility: :protected
-webcvs: 

data/lib/picturehouse_uk/film.rb

@@ -1,59 +0,0 @@
-module PicturehouseUk
-  # A film on the Picturehouse UK website
-  class Film
-    include Comparable
-
-    # @return [String] the name of the film
-    attr_reader :name
-    # @return [String] the normalized slug derived from the film name
-    attr_reader :slug
-
-    # @param [String] name the film name
-    # @return [PicturehouseUk::Film]
-    def initialize(name)
-      @name = name
-      @slug = name.downcase.gsub(/[^0-9a-z ]/, '').gsub(/\s+/, '-')
-    end
-
-    # Films at a single cinema
-    # @param [String] cinema_id the id of the cinema
-    # @return [Array<PicturehouseUk::Film>]
-    def self.at(cinema_id)
-      screenings(cinema_id).map { |hash| new hash[:film_name] }.uniq
-    end
-
-    # Allows sort on objects
-    # @param [PicturehouseUk::Film] other another film object
-    # @return [Integer] -1, 0 or 1
-    def <=>(other)
-      slug <=> other.slug
-    end
-
-    # Check an object is the same as another object.
-    # @param [PicturehouseUk::Film] other another film
-    # @return [Boolean] True if both objects are the same exact object, or if
-    #   they are of the same type and share an equal slug
-    # @note Guided by http://woss.name/2011/01/20/equality-comparison-and-ordering-in-ruby/
-    def eql?(other)
-      self.class == other.class && self == other
-    end
-
-    # Generates hash of slug in order to allow two records of the same type and
-    # id to work with something like:
-    #
-    #   [ Film.new('AB'), Film.new('EF') ] & [ Film.new('EF'), Film.new('GH') ]
-    #   #=> [ Film.new('EF') ]
-    #
-    # @return [Integer] hash of slug
-    # @note Guided by http://woss.name/2011/01/20/equality-comparison-and-ordering-in-ruby/
-    def hash
-      slug.hash
-    end
-
-    private
-
-    def self.screenings(cinema_id)
-      PicturehouseUk::Internal::Parser::Screenings.new(cinema_id).to_a
-    end
-  end
-end

data/lib/picturehouse_uk/screening.rb

@@ -1,70 +0,0 @@
-module PicturehouseUk
-  # A single screening of a film on the Picturehouse UK website
-  class Screening
-    # @return [String] the booking URL on the cinema website
-    attr_reader :booking_url
-    # @return [String] the cinema name
-    attr_reader :cinema_name
-    # @return [String] 2d or 3d
-    attr_reader :dimension
-    # @return [String] the film name
-    attr_reader :film_name
-
-    # @param [Hash] options
-    def initialize(options)
-      @booking_url = options.fetch(:booking_url, nil)
-      @cinema_name = options.fetch(:cinema_name)
-      @cinema_id   = options.fetch(:cinema_id)
-      @dimension   = options.fetch(:dimension, '2d')
-      @film_name   = options.fetch(:film_name)
-      @time        = options.fetch(:time)
-      @variant     = options.fetch(:variant, [])
-    end
-
-    # Screenings at a single cinema
-    # @param [String] cinema_id the id of the cinema
-    # @return [Array<PicturehouseUk::Screening>]
-    def self.at(cinema_id)
-      screenings(cinema_id).map do |attributes|
-        new cinema_hash(cinema_id).merge(attributes)
-      end.uniq
-    end
-
-    # The UTC time of the screening
-    # @return [Time]
-    def showing_at
-      @showing_at ||= begin
-        if @time.utc?
-          @time
-        else
-          TZInfo::Timezone.get('Europe/London').local_to_utc(@time)
-        end
-      end
-    end
-
-    # The date of the screening
-    # @return [Date]
-    def showing_on
-      showing_at.to_date
-    end
-
-    # The kinds of screening
-    # @return <Array[String]>
-    def variant
-      @variant.map(&:downcase).sort
-    end
-
-    private
-
-    def self.cinema_hash(cinema_id)
-      {
-        cinema_id: cinema_id,
-        cinema_name: PicturehouseUk::Cinema.find(cinema_id).name
-      }
-    end
-
-    def self.screenings(cinema_id)
-      PicturehouseUk::Internal::Parser::Screenings.new(cinema_id).to_a
-    end
-  end
-end