odeon_uk 4.0.1 → 5.1.0

data/test/lib/odeon_uk/internal/title_sanitizer_test.rb

@@ -1,121 +1,141 @@
 require_relative '../../../test_helper'
 
-describe OdeonUk::Internal::TitleSanitizer do
-  let(:described_class) { OdeonUk::Internal::TitleSanitizer }
+class OdeonUk::Internal::TitleSanitizerTest < Minitest::Test
+  def test_removes_2d
+    assert_sanitized 'Iron Man 3', 'Iron Man 3 2D'
+  end
 
-  describe '#sanitized' do
-    subject { described_class.new(title).sanitized }
+  def test_removes_3d
+    assert_sanitized 'Iron Man 3', 'Iron Man 3 3D'
+  end
 
-    describe 'with 2d in title' do
-      let(:title) { 'Iron Man 3 2D' }
+  def test_removes_autism_prefix
+    assert_sanitized 'Planes', 'Autism Friendly Screening - Planes'
+  end
 
-      it 'removes dimension' do
-        subject.must_equal('Iron Man 3')
-      end
-    end
+  def test_bolshoi
+    assert_sanitized 'Bolshoi: Spartacus', 'Bolshoi - Spartacus (Live)'
+  end
 
-    describe 'with 3d in title' do
-      let(:title) { 'Iron Man 3 3D' }
+  def test_removes_cinemagic_prefix
+    assert_sanitized 'Echo Planet', 'Cinemagic 2013 - Echo Planet'
+  end
 
-      it 'removes dimension' do
-        subject.must_equal('Iron Man 3')
-      end
-    end
+  def test_globe_on_screen
+    assert_sanitized 'Globe: Twelfth Night', 'Globe On Screen: Twelfth Night'
+  end
 
-    describe 'Austism screening' do
-      let(:title) { 'Autism Friendly Screening - Planes' }
+  def test_met_opera
+    assert_sanitized 'Met Opera: Eugene Onegin 2013', 'Met Opera - Eugene Onegin 2013'
+  end
 
-      it 'removes prefix' do
-        subject.must_equal('Planes')
-      end
-    end
+  def test_met_opera_season
+    assert_sanitized 'Met Opera: La Fanciulla del West',
+                     'Met Opera 2026-27: La Fanciulla del West'
+  end
 
-    describe 'Bolshoi screeening' do
-      let(:title) { 'Bolshoi - Spartacus (Live)' }
+  def test_nt_live
+    assert_sanitized 'National Theatre: War Horse', 'NT Live - War Horse'
+  end
 
-      it 'removes prefix' do
-        subject.must_equal('Bolshoi: Spartacus')
-      end
-    end
+  def test_national_theatre_encore
+    assert_sanitized 'National Theatre: Frankenstein',
+                     'National Theatre Live - Frankenstein (Encore Screening)'
+  end
 
-    describe 'Cinemagic screeening' do
-      let(:title) { 'Cinemagic 2013 - Echo Planet' }
+  def test_roh
+    assert_sanitized 'Royal Opera House: Carmen', 'ROH - Carmen'
+  end
 
-      it 'removes prefix' do
-        subject.must_equal('Echo Planet')
-      end
-    end
+  def test_rbo_season
+    assert_sanitized 'Royal Ballet and Opera: Swan Lake',
+                     'RBO Cinema Season 2026-27: Swan Lake'
+  end
 
-    describe 'Globe on Screen screeening' do
-      let(:title) { 'Globe On Screen: Twelfth Night' }
+  def test_short_rbo_season
+    assert_sanitized "Royal Ballet and Opera: Alice's Adventures in Wonderland",
+                     "RBO 2026-27: Alice's Adventures in Wonderland"
+  end
 
-      it 'removes prefix' do
-        subject.must_equal('Globe: Twelfth Night')
-      end
-    end
+  def test_rsc
+    assert_sanitized 'Royal Shakespeare Company: Richard II', 'Richard II - RSC Live 2013'
+  end
 
-    describe 'Met Opera screeening' do
-      let(:title) { 'Met Opera - Eugene Onegin 2013' }
+  def test_removes_ukjff_prefix
+    assert_sanitized 'From Cable Street To Brick Lane',
+                     'UKJFF - From Cable Street To Brick Lane'
+  end
 
-      it 'removes prefix' do
-        subject.must_equal('Met Opera: Eugene Onegin 2013')
-      end
-    end
+  def test_removes_singalong
+    assert_sanitized 'Frozen', 'Frozen Sing-a-long'
+  end
 
-    describe 'NT Live screeening' do
-      let(:title) { 'NT Live - War Horse' }
+  def test_html_ampersands
+    assert_sanitized 'Fast & Furious 7', 'Fast &amp; Furious 7'
+  end
 
-      it 'removes prefix' do
-        subject.must_equal('National Theatre: War Horse')
-      end
-    end
+  def test_removes_language_suffix
+    assert_sanitized 'Jailer 2', 'Jailer 2 (Telugu)'
+  end
+
+  def test_removes_encore_suffix
+    assert_sanitized 'Avengers Endgame', 'Avengers Endgame: Encore'
+  end
+
+  def test_keeps_sing_along_within_title
+    assert_sanitized 'Bebefinn Sing-Along Movie', 'Bebefinn Sing-Along Movie'
+  end
+
+  def test_removes_dubbed
+    assert_sanitized 'Arrietty', 'Arrietty (Dubbed)'
+  end
 
-    describe 'National Theatre screeening' do
-      let(:title) { 'National Theatre Live - Frankenstein (Encore Screening)' }
+  def test_removes_subbed
+    assert_sanitized 'Arrietty', 'Arrietty (Subbed)'
+  end
 
-      it 'removes prefix' do
-        subject.must_equal('National Theatre: Frankenstein')
-      end
-    end
+  def test_removes_english_dub
+    assert_sanitized 'Kung Fu Soccer', 'Kung Fu Soccer (English Dub)'
+  end
 
-    describe 'ROH screeening' do
-      let(:title) { 'Autism Friendly Screening - Planes' }
+  def test_removes_subtitle_description
+    assert_sanitized 'Kung Fu Soccer', 'Kung Fu Soccer (Mandarin with EN/CH subtitles)'
+  end
 
-      it 'removes prefix' do
-        subject.must_equal('Planes')
-      end
-    end
+  def test_removes_q_and_a
+    assert_sanitized 'NAZA', 'NAZA + Q&A'
+  end
 
-    describe 'RSC screeening' do
-      let(:title) { 'Richard II - RSC Live 2013' }
+  def test_removes_in_person_q_and_a
+    assert_sanitized "Charlie Shaw's Revenge", "Charlie Shaw's Revenge + In-Person Q&A"
+  end
 
-      it 'removes prefix' do
-        subject.must_equal('Royal Shakespeare Company: Richard II')
-      end
-    end
+  def test_removes_festival_prefix
+    assert_sanitized 'Scum Of The Earth', 'Cine-Excess 2026: Scum Of The Earth'
+  end
 
-    describe 'UK Jewish FF screeening' do
-      let(:title) { 'UKJFF - From Cable Street To Brick Lane' }
+  def test_removes_anniversary
+    assert_sanitized 'SCREAM', 'SCREAM (30th Anniversary)'
+  end
 
-      it 'removes prefix' do
-        subject.must_equal('From Cable Street To Brick Lane')
-      end
-    end
+  def test_world_tour_is_not_capitalised
+    assert_sanitized "ENHYPEN World Tour 'BLOOD SAGA'", "ENHYPEN WORLD TOUR 'BLOOD SAGA'"
+  end
 
-    describe 'singalong screeening' do
-      let(:title) { 'Frozen Sing-a-long' }
+  def test_keeps_titles_with_their_own_subtitles
+    [
+      'Harry Potter and the Deathly Hallows: Part 1',
+      'Spider-Man: Brand New Day',
+      'Hadestown: The Musical',
+      'Rammstein - Live in Mexico City',
+      'Ozzy & Black Sabbath: Back to the Beginning',
+      "BTS 'ARIRANG' IN BUENOS AIRES: LIVE VIEWING"
+    ].each { |title| assert_sanitized title, title }
+  end
 
-      it 'removes prefix' do
-        subject.must_equal('Frozen')
-      end
-    end
-
-    describe 'HTML ampersands' do
-      let(:title) { 'Fast &amp; Furious 7' }
+  private
 
-      it 'removes prefix' do
-        subject.must_equal('Fast & Furious 7')
-      end
-    end
+  def assert_sanitized(expected, title)
+    assert_equal expected, OdeonUk::Internal::TitleSanitizer.new(title).sanitized
   end
 end

data/test/lib/odeon_uk/performance_test.rb

@@ -1,172 +1,155 @@
 require_relative '../../test_helper'
 
-describe OdeonUk::Performance do
-  let(:described_class) { OdeonUk::Performance }
-  let(:api_response) { FakeApiResponse.new }
-
-  before { WebMock.disable_net_connect! }
-  after { WebMock.allow_net_connect! }
-
-  describe '.at(cinema_id)' do
-    subject { described_class.at(cinema_id) }
-
-    describe 'passed Fixnum' do
-      let(:cinema_id) { 71 }
-
-      it 'returns an array of performances' do
-        OdeonUk::Internal::ApiResponse.stub :new, api_response do
-          subject.must_be_instance_of(Array)
-          subject.each do |element|
-            element.must_be_instance_of(described_class)
-          end
-        end
-      end
-
-      it 'returns at least a sensible number' do
-        OdeonUk::Internal::ApiResponse.stub :new, api_response do
-          subject.count.must_be :>, 110
-        end
-      end
-    end
+class OdeonUk::PerformanceTest < Minitest::Test
+  DATE = Date.new(2026, 10, 24)
 
-    describe 'passed String' do
-      let(:cinema_id) { '71' }
-
-      it 'returns an array of performances' do
-        OdeonUk::Internal::ApiResponse.stub :new, api_response do
-          subject.must_be_instance_of(Array)
-          subject.each do |element|
-            element.must_be_instance_of(described_class)
-          end
-        end
-      end
-
-      it 'returns at least a sensible number' do
-        OdeonUk::Internal::ApiResponse.stub :new, api_response do
-          subject.count.must_be :>, 110
-        end
-      end
-    end
+  def setup
+    @api_response = FakeApiResponse.new
   end
 
-  describe '.new' do
-    subject { described_class.new(options) }
-
-    describe 'simple' do
-      let(:options) do
-        {
-          film_name:   'The Big Short',
-          cinema_id:   71,
-          cinema_name: 'Brighton',
-          starting_at: Time.utc(2013, 9, 12, 11, 0)
-        }
-      end
-
-      it 'sets cinema name and film name' do
-        subject.film_name.must_equal 'The Big Short'
-        subject.cinema_name.must_equal 'Brighton'
-      end
-
-      it 'booking url, dimension & varient are set to defaults' do
-        subject.booking_url.must_equal nil
-        subject.dimension.must_equal '2d'
-        subject.variant.must_equal []
-      end
-    end
+  def test_at_returns_performances
+    performances = performances_at(824)
+
+    assert(performances.all? { |p| p.instance_of?(OdeonUk::Performance) })
+    assert_equal 6, performances.count
   end
 
-  describe '#dimension' do
-    let(:options) do
-      {
-        film_name:   'Ant Man',
-        dimension:   '3d',
-        cinema_id:   71,
-        cinema_name: 'Brighton',
-        starting_at: Time.utc(2013, 9, 12, 11, 0)
-      }
-    end
+  def test_at_accepts_string_id
+    assert_equal 6, performances_at('824').count
+  end
 
-    subject { described_class.new(options).dimension }
+  def test_at_sets_cinema_details
+    performances = performances_at(824)
 
-    it 'returns 2d or 3d' do
-      subject.must_be_instance_of(String)
-      subject.must_equal '3d'
-    end
+    assert_equal ['824'], performances.map(&:cinema_id).uniq
+    assert_equal ['Brighton'], performances.map(&:cinema_name).uniq
   end
 
-  describe '#starting_at' do
-    subject { described_class.new(options).starting_at }
+  def test_at_parses_first_performance
+    performance = performances_at(824).first
 
-    describe 'with utc time' do
-      let(:options) do
-        {
-          film_name:   'Steve Jobs',
-          cinema_id:   71,
-          cinema_name: 'Brighton',
-          starting_at: Time.utc(2013, 9, 12, 11, 0)
-        }
-      end
+    assert_equal 'The Example Film', performance.film_name
+    assert_equal Time.utc(2026, 10, 24, 8, 0), performance.starting_at
+    assert_equal '2d', performance.dimension
+    assert_equal %w(audio_described), performance.variant
+    assert_equal 'https://www.odeon.co.uk/ticketing/seat-picker/?showtimeId=824-1001',
+                 performance.booking_url
+  end
 
-      it 'returns UTC time' do
-        subject.must_be_instance_of Time
-        subject.must_equal Time.utc(2013, 9, 12, 11, 0)
-      end
-    end
+  def test_at_parses_3d_imax
+    performance = performances_at(824)[1]
 
-    describe 'with non-utc time' do
-      let(:options) do
-        {
-          film_name:   'Steve Jobs',
-          cinema_id:   71,
-          cinema_name: 'Brighton',
-          starting_at: Time.new(2013, 9, 12, 11, 0)
-        }
-      end
-
-      it 'returns UTC time' do
-        subject.must_be_instance_of Time
-        subject.must_equal Time.utc(2013, 9, 12, 10, 0)
-        subject.utc?.must_equal(true)
-      end
-    end
+    assert_equal '3d', performance.dimension
+    assert_equal %w(imax), performance.variant
   end
 
-  describe '#showing_on' do
-    let(:options) do
-      {
-        film_name:   'The Hunger Games',
-        cinema_id:   71,
-        cinema_name: 'Brighton',
-        starting_at: Time.utc(2013, 9, 12, 11, 0)
-      }
-    end
+  def test_at_returns_language_and_subtitles
+    performances = performances_at(824)
+    tamil = performances[3]
+    korean = performances[2]
+
+    assert_equal 'Imaginary Picture', tamil.film_name
+    assert_equal 2025, tamil.film_year
+    assert_equal 'Tamil', tamil.language
+    assert tamil.subtitled?
+    assert_equal 'Korean', korean.language
+    assert_equal %w(arts subtitled), korean.variant
+  end
 
-    subject { described_class.new(options).showing_on }
+  def test_at_treats_open_captions_as_subtitled
+    kids = performances_at(824)[4]
 
-    it 'returns date of showing' do
-      subject.must_be_instance_of(Date)
-      subject.must_equal Date.new(2013, 9, 12)
-    end
+    assert_nil kids.language
+    assert_equal %w(kids subtitled), kids.variant
+    assert kids.subtitled?
   end
 
-  describe '#variant' do
-    subject { described_class.new(options).variant }
+  def test_at_keeps_after_midnight_on_next_utc_day
+    assert_equal Time.utc(2026, 10, 24, 23, 15), performances_at(824).last.starting_at
+  end
 
-    let(:options) do
-      {
-        film_name:   'Carol',
-        cinema_id:   71,
-        cinema_name: 'Brighton',
-        starting_at: Time.utc(2013, 9, 12, 11, 0),
-        variant:     %w(Subtitles Senior)
-      }
-    end
+  def test_at_requests_each_date_in_range
+    to = DATE + 2
+
+    with_fake_api { OdeonUk::Performance.at(824, from: DATE, to: to) }
+
+    assert_equal (DATE..to).to_a, @api_response.requested_dates
+  end
 
-    it 'is an alphabetically ordered array of lower-cased strings' do
-      subject.must_be_instance_of Array
-      subject.each { |element| element.must_be_instance_of String }
-      subject.must_equal %w(senior subtitles)
+  def test_at_rejects_range_ending_before_start
+    assert_raises(ArgumentError) do
+      OdeonUk::Performance.at(824, from: DATE, to: DATE - 1)
     end
   end
 
+  def test_new_sets_names
+    performance = OdeonUk::Performance.new(options)
+
+    assert_equal 'The Big Short', performance.film_name
+    assert_equal 'Brighton', performance.cinema_name
+  end
+
+  def test_new_sets_defaults
+    performance = OdeonUk::Performance.new(options)
+
+    assert_nil performance.booking_url
+    assert_nil performance.festival
+    assert_equal '2d', performance.dimension
+    assert_nil performance.language
+    refute performance.subtitled?
+    assert_equal [], performance.variant
+  end
+
+  def test_language
+    assert_equal 'Tamil', OdeonUk::Performance.new(options(language: 'Tamil')).language
+  end
+
+  def test_festival
+    festival = Cinebase::Festival.new(name: 'Cine-Excess', year: 2026)
+    performance = OdeonUk::Performance.new(options(festival: festival))
+
+    assert_equal festival, performance.festival
+  end
+
+  def test_film_year
+    assert_equal 1996, OdeonUk::Performance.new(options(film_year: 1996)).film_year
+  end
+
+  def test_subtitled
+    assert OdeonUk::Performance.new(options(variant: %w(subtitled))).subtitled?
+  end
+
+  def test_starting_at_converts_local_time_to_utc
+    starting_at = OdeonUk::Performance.new(
+      options(starting_at: Time.new(2013, 9, 12, 11, 0))
+    ).starting_at
+
+    assert_equal Time.utc(2013, 9, 12, 10, 0), starting_at
+    assert starting_at.utc?
+  end
+
+  def test_variant_is_sorted_and_lower_case
+    performance = OdeonUk::Performance.new(options(variant: %w(Subtitled Senior)))
+
+    assert_equal %w(senior subtitled), performance.variant
+  end
+
+  private
+
+  def options(overrides = {})
+    {
+      film_name:   'The Big Short',
+      cinema_id:   '824',
+      cinema_name: 'Brighton',
+      starting_at: Time.utc(2013, 9, 12, 11, 0)
+    }.merge(overrides)
+  end
+
+  def performances_at(cinema_id)
+    with_fake_api { OdeonUk::Performance.at(cinema_id, from: DATE, to: DATE) }
+  end
+
+  def with_fake_api(&block)
+    OdeonUk::Internal::ApiResponse.stub(:new, @api_response, &block)
+  end
 end

data/test/lib/odeon_uk/version_test.rb

@@ -1,7 +1,7 @@
 require_relative '../../test_helper'
 
-describe OdeonUk do
-  it 'must be defined' do
-    OdeonUk::VERSION.wont_be_nil
+class OdeonUk::VersionTest < Minitest::Test
+  def test_version_is_defined
+    refute_nil OdeonUk::VERSION
   end
 end

data/test/live/api_shape_test.rb

@@ -0,0 +1,19 @@
+require_relative 'live_helper'
+
+class LiveApiShapeTest < Minitest::Test
+  include ApiShapes
+
+  def test_sites_are_readable
+    assert_sites_shape('sites' => api.sites)
+  end
+
+  def test_showtimes_are_readable
+    assert_showtimes_shape(api.showtimes('824', Date.today + 1))
+  end
+
+  private
+
+  def api
+    OdeonUk::Internal::ApiResponse.new
+  end
+end

data/test/live/integration_test.rb

@@ -0,0 +1,21 @@
+require_relative 'live_helper'
+
+class LiveCinemaTest < Minitest::Test
+  def test_all_returns_cinemas
+    cinemas = OdeonUk::Cinema.all
+
+    assert(cinemas.all? { |cinema| cinema.instance_of?(OdeonUk::Cinema) })
+    assert_operator cinemas.size, :>, 90
+    assert_equal 'Brighton', cinemas.detect { |cinema| cinema.id == '824' }.name
+  end
+end
+
+class LivePerformanceTest < Minitest::Test
+  def test_at_returns_performances
+    tomorrow = Date.today + 1
+    performances = OdeonUk::Performance.at(824, from: tomorrow, to: tomorrow)
+
+    assert(performances.all? { |p| p.instance_of?(OdeonUk::Performance) })
+    assert_operator performances.count, :>, 10
+  end
+end

data/test/live/live_helper.rb

@@ -0,0 +1,11 @@
+require 'minitest/autorun'
+require 'minitest/reporters'
+
+Minitest::Reporters.use! [
+  Minitest::Reporters::DefaultReporter.new(color: true)
+]
+
+require File.expand_path('../../../lib/odeon_uk.rb', __FILE__)
+require_relative '../support/api_shapes'
+
+OdeonUk.token = ENV.fetch('ODEON_TOKEN') { OdeonUk.acquire_token! }

data/test/package_test.rb

@@ -0,0 +1,56 @@
+# Run separately after building: ODEON_PACKAGE=pkg/...gem bundle exec ruby test/package_test.rb
+require "minitest/autorun"
+require "rubygems/installer"
+require "tmpdir"
+require "fileutils"
+require "open3"
+require "base64"
+require "json"
+
+class PlatformPackageTest < Minitest::Test
+  def setup
+    @directory = Dir.mktmpdir("odeon-package-test")
+    @spec = Gem::Installer.at(
+      ENV.fetch("ODEON_PACKAGE"), install_dir: @directory, ignore_dependencies: true
+    ).install
+    @vendor = File.join(@spec.full_gem_path, "vendor", "curl-impersonate")
+    @binary = File.join(@vendor, "bin", "curl-impersonate")
+  end
+
+  def teardown
+    FileUtils.remove_entry(@directory)
+  end
+
+  def test_installs_runnable_executable_with_notices_and_provenance
+    assert File.executable?(@binary)
+    output, status = Open3.capture2(@binary, "--version")
+    assert status.success?
+    assert_includes output, "curl"
+    assert File.file?(File.join(@vendor, "licenses", "LICENSE_CURL"))
+    assert_includes File.read(File.join(@vendor, "SOURCE")), "2.2.3"
+  end
+
+  def test_acquires_token_with_gem_local_executable_and_no_curl_on_path
+    payload = Base64.urlsafe_encode64(JSON.generate("exp" => Time.now.to_i + 3600), padding: false)
+    token = "header.#{payload}.signature"
+    data = { "api" => { "authToken" => token } }
+    page = "<script>\nwindow.initialData = #{JSON.generate(data)};\n</script>\n200"
+    # Substitute only the network response, not the installed package's path resolution.
+    File.write(@binary, "#!#{RbConfig.ruby}\nSTDOUT.write(#{page.dump})\n")
+    output, error, status = Open3.capture3(
+      { "PATH" => @directory, "RUBYOPT" => nil, "BUNDLER_SETUP" => nil,
+        "GEM_PATH" => Gem.path.join(File::PATH_SEPARATOR) }, RbConfig.ruby,
+      "-I#{File.join(@spec.full_gem_path, 'lib')}", *cinebase_load_path, "-rodeon_uk",
+      "-e", "print OdeonUk.acquire_token!"
+    )
+    assert status.success?, error
+    assert_equal token, output
+  end
+
+  private
+
+  # The package is installed without dependencies, so load the bundle's cinebase.
+  def cinebase_load_path
+    Gem.loaded_specs.fetch("cinebase").full_require_paths.map { |path| "-I#{path}" }
+  end
+end