timezone 1.3.2 → 1.3.30

data/lib/timezone/lookup/geonames.rb

@@ -55,6 +55,7 @@ module Timezone
         return unless data['gmtOffset'].is_a? Numeric
 
         return 'Etc/UTC' if data['gmtOffset'].zero?
+
         "Etc/GMT#{format('%+d', -data['gmtOffset'])}"
       end
 

data/lib/timezone/lookup/google.rb

@@ -14,7 +14,7 @@ module Timezone
     class Google < ::Timezone::Lookup::Basic
       # Indicates that no time zone data could be found for the specified
       # <lat, lng>. This can occur if the query is incomplete or ambiguous.
-      NO_TIMEZONE_INFORMATION = 'ZERO_RESULTS'.freeze
+      NO_TIMEZONE_INFORMATION = 'ZERO_RESULTS'
 
       def initialize(config)
         if config.api_key.nil?
@@ -34,7 +34,8 @@ module Timezone
           raise(Timezone::Error::Google, '403 Forbidden')
         end
 
-        return unless response.code =~ /^2\d\d$/
+        return unless /^2\d\d$/.match?(response.code)
+
         data = JSON.parse(response.body)
 
         return if data['status'] == NO_TIMEZONE_INFORMATION

data/lib/timezone/lookup.rb

@@ -4,12 +4,13 @@ require 'timezone/lookup/geonames'
 require 'timezone/lookup/google'
 require 'timezone/lookup/test'
 require 'timezone/net_http_client'
+require 'ostruct'
 
 module Timezone
   # Configure timezone lookups.
   module Lookup
     class << self
-      MISSING_LOOKUP = 'No lookup configured'.freeze
+      MISSING_LOOKUP = 'No lookup configured'
       private_constant :MISSING_LOOKUP
 
       # Returns the lookup object
@@ -45,7 +46,7 @@ module Timezone
         test: ::Timezone::Lookup::Test
       }.freeze
 
-      INVALID_LOOKUP = 'Invalid lookup specified'.freeze
+      INVALID_LOOKUP = 'Invalid lookup specified'
 
       attr_reader :config
 

data/lib/timezone/parser.rb

@@ -9,30 +9,45 @@ module Timezone
     MIN_YEAR = -500
     MAX_YEAR = 2039
 
-    LINE = /\s*(.+)\s*=\s*(.+)\s*isdst=(\d+)\s*gmtoff=([\+\-]*\d+)/
+    LINE = /\s*(.+)\s*=\s*(.+)\s*isdst=(\d+)\s*gmtoff=([+\-]*\d+)/.freeze
 
-    ZONEINFO_DIR = '/usr/share/zoneinfo'.freeze
+    # Bookkeeping files that we do not want to parse.
+    IGNORE = ['leapseconds', 'posixrules', 'tzdata.zi'].freeze
 
-    attr_reader :zoneinfo
-
-    def initialize(zoneinfo = ZONEINFO_DIR)
-      @zoneinfo = zoneinfo
+    def initialize(root)
+      @config = Config.new(root)
     end
 
     def perform
       FileUtils.rm_rf('data')
 
-      Dir["#{zoneinfo}/posix/**/*"].each do |file|
+      Dir["#{@config.zoneinfo}/**/*"].each do |file|
         next if File.directory?(file)
+        next if file.end_with?('.tab')
+        next if IGNORE.include?(File.basename(file))
+
         parse(file)
       end
     end
 
+    # Represents a timezone database config.
+    class Config
+      def initialize(root)
+        @root = root
+        @zoneinfo = File.join(@root, 'usr/share/zoneinfo')
+        @zdump = File.join(@root, 'usr/bin/zdump')
+      end
+
+      attr_reader :root, :zoneinfo, :zdump
+    end
+
+    private_constant :Config
+
     # Represents a single timezone data file line for a reference timezone.
     class RefLine
-      def initialize(zone)
+      def initialize(config, file)
         first =
-          `zdump -i posix/#{zone}`
+          `#{config.zdump} -i #{file}`
             .split("\n")
             .reject(&:empty?)
             .reject { |line| line.start_with?('TZ=') }
@@ -51,7 +66,7 @@ module Timezone
       def parse_offset(offset)
         arity = offset.start_with?('-') ? -1 : 1
 
-        match = offset.match(/^[\-\+](\d{2})$/)
+        match = offset.match(/^[\-+](\d{2})$/)
         arity * match[1].to_i * 60 * 60
       end
     end
@@ -62,10 +77,10 @@ module Timezone
     class Line
       attr_accessor :source, :name, :dst, :offset
 
-      SOURCE_FORMAT = '%a %b %e %H:%M:%S %Y %Z'.freeze
+      SOURCE_FORMAT = '%a %b %e %H:%M:%S %Y %Z'
 
       def initialize(match)
-        self.source = Time.strptime(match[1] + 'C', SOURCE_FORMAT).to_i
+        self.source = Time.strptime("#{match[1]}C", SOURCE_FORMAT).to_i
         self.name = match[2].split(' ').last
         self.dst = match[3].to_i
         self.offset = match[4].to_i
@@ -85,15 +100,15 @@ module Timezone
     private
 
     def parse(file)
-      zone = file.gsub("#{zoneinfo}/posix/", '')
+      zone = file.gsub("#{@config.zoneinfo}/", '')
       print "Parsing #{zone}... "
-      data = zdump(zone)
+      data = zdump(file)
 
       last = 0
       result = []
 
       data.split("\n").each do |line|
-        match = line.gsub('posix/' + zone + ' ', '').match(LINE)
+        match = line.gsub(/^#{file}\s+/, '').match(LINE)
         next if match.nil?
 
         line = Line.new(match)
@@ -112,14 +127,14 @@ module Timezone
         result << line
       end
 
-      result << RefLine.new(zone) if result.empty?
+      result << RefLine.new(@config, file) if result.empty?
 
       write(zone, result)
       puts 'DONE'
     end
 
-    def zdump(zone)
-      `zdump -v -c #{MIN_YEAR},#{MAX_YEAR} posix/#{zone}`
+    def zdump(file)
+      `#{@config.zdump} -V -c #{MIN_YEAR},#{MAX_YEAR} #{file}`
     end
 
     def write(zone, data)

data/lib/timezone/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Timezone
-  VERSION = '1.3.2'.freeze
+  VERSION = '1.3.30'
 end

data/lib/timezone/zone.rb

@@ -6,7 +6,6 @@ require 'time'
 
 require 'timezone/loader'
 require 'timezone/error'
-require 'timezone/loader'
 
 module Timezone
   # This object represents a real-world timezone. Each instance provides
@@ -179,10 +178,6 @@ module Timezone
 
     private
 
-    def private_rules
-      @rules ||= Loader.load(name)
-    end
-
     def sanitize(time)
       time.to_time
     end
@@ -200,11 +195,15 @@ module Timezone
 
     def rule_for_local(local)
       local = local.to_i
+      rules = Loader.load(name)
 
       # For each rule, convert the local time into the UTC equivalent for
       # that rule offset, and then check if the UTC time matches the rule.
-      index = binary_search(local) { |t, r| match?(t - r[OFFSET_BIT], r) }
-      match = private_rules[index]
+      index =
+        binary_search(rules, local) do |t, r|
+          match?(t - r[OFFSET_BIT], r)
+        end
+      match = rules[index]
 
       utc = local - match[OFFSET_BIT]
 
@@ -213,7 +212,7 @@ module Timezone
       return RuleSet.new(:missing, [match]) if rule_for_utc(utc) != match
 
       # If the match is the last rule, then return it.
-      return RuleSet.new(:single, [match]) if index == private_rules.length - 1
+      return RuleSet.new(:single, [match]) if index == rules.length - 1
 
       # If the UTC equivalent time falls within the last hour(s) of the time
       # change which were replayed during a fall-back in time, then return
@@ -234,10 +233,10 @@ module Timezone
       last_hour =
         match[SOURCE_BIT] -
         match[OFFSET_BIT] +
-        private_rules[index + 1][OFFSET_BIT]
+        rules[index + 1][OFFSET_BIT]
 
       if utc > last_hour
-        RuleSet.new(:double, private_rules[index..(index + 1)])
+        RuleSet.new(:double, rules[index..(index + 1)])
       else
         RuleSet.new(:single, [match])
       end
@@ -245,27 +244,28 @@ module Timezone
 
     def rule_for_utc(time)
       time = time.to_i
+      rules = Loader.load(name)
 
-      private_rules[binary_search(time) { |t, r| match?(t, r) }]
+      rules[binary_search(rules, time) { |t, r| match?(t, r) }]
     end
 
     # Find the first rule that matches using binary search.
-    def binary_search(time, from = 0, to = nil, &block)
-      to = private_rules.length - 1 if to.nil?
+    def binary_search(rules, time, from = 0, to = nil, &block)
+      to = rules.length - 1 if to.nil?
 
       return from if from == to
 
       mid = (from + to).div(2)
 
-      unless yield(time, private_rules[mid])
-        return binary_search(time, mid + 1, to, &block)
+      unless yield(time, rules[mid])
+        return binary_search(rules, time, mid + 1, to, &block)
       end
 
       return mid if mid.zero?
 
-      return mid unless yield(time, private_rules[mid - 1])
+      return mid unless yield(time, rules[mid - 1])
 
-      binary_search(time, from, mid - 1, &block)
+      binary_search(rules, time, from, mid - 1, &block)
     end
   end
 end

data/test/test_timezone.rb

@@ -32,13 +32,11 @@ class TestTimezone < ::Minitest::Test
   end
 
   def test_fetch_warning
-    warning = false
-
-    Timezone.stub(:warn, ->(_) { warning = true }) do
+    _out, err = capture_io do
       Timezone.fetch('foo/bar', 'a') { 'b' }
     end
 
-    assert warning, 'warning was not issued'
+    assert_equal "warning: block supersedes default value argument\n", err
   end
 
   def test_lookup

data/test/timezone/lookup/test_geonames.rb

@@ -2,6 +2,7 @@
 
 require 'timezone/lookup/geonames'
 require 'minitest/autorun'
+require 'ostruct'
 require_relative '../../http_test_client'
 
 class TestGeonames < ::Minitest::Test
@@ -19,6 +20,10 @@ class TestGeonames < ::Minitest::Test
     }
   end
 
+  def lookup_file(file_name, &block)
+    lookup(File.open(File.join(mock_path, file_name)).read, &block)
+  end
+
   def lookup(body = nil, &_block)
     config = OpenStruct.new
     config.username = 'timezone'
@@ -40,23 +45,25 @@ class TestGeonames < ::Minitest::Test
   end
 
   def test_lookup
-    mine = lookup(File.open(mock_path + '/lat_lon_coords.txt').read)
+    mine = lookup_file('/lat_lon_coords.txt')
 
     assert_equal 'Australia/Adelaide', mine.lookup(*coordinates)
   end
 
   def test_lookup_with_etc
     etc_data.each do |key, data|
-      body = File.open(mock_path + "/lat_lon_coords_#{key}.txt").read
-      mine = lookup(body) { |c| c.offset_etc_zones = true }
+      mine = lookup_file("/lat_lon_coords_#{key}.txt") do |c|
+        c.offset_etc_zones = true
+      end
 
       assert_equal data[:name], mine.lookup(*data[:coordinates])
     end
   end
 
   def test_wrong_offset
-    body = File.open(mock_path + '/lat_lon_coords_wrong_offset.txt').read
-    mine = lookup(body) { |c| c.offset_etc_zones = true }
+    mine = lookup_file('/lat_lon_coords_wrong_offset.txt') do |c|
+      c.offset_etc_zones = true
+    end
 
     assert_nil mine.lookup(*coordinates)
   end
@@ -80,7 +87,7 @@ class TestGeonames < ::Minitest::Test
   end
 
   def test_api_limit
-    mine = lookup(File.open(mock_path + '/api_limit_reached.json').read)
+    mine = lookup_file('/api_limit_reached.json')
 
     assert_exception(
       mine,
@@ -90,19 +97,19 @@ class TestGeonames < ::Minitest::Test
   end
 
   def test_invalid_latlong
-    mine = lookup(File.open(mock_path + '/invalid_latlong.json').read)
+    mine = lookup_file('/invalid_latlong.json')
 
     assert_exception(mine, 'invalid lat/lng')
   end
 
   def test_no_result_found
-    mine = lookup(File.open(mock_path + '/no_result_found.json').read)
+    mine = lookup_file('/no_result_found.json')
 
     assert_nil(mine.lookup(10, 10))
   end
 
   def test_invalid_parameter
-    mine = lookup(File.open(mock_path + '/invalid_parameter.json').read)
+    mine = lookup_file('/invalid_parameter.json')
 
     assert_exception(mine, 'error parsing parameter')
   end

data/test/timezone/lookup/test_google.rb

@@ -3,6 +3,7 @@
 require 'timezone/lookup/google'
 require 'minitest/autorun'
 require 'timecop'
+require 'ostruct'
 require_relative '../../http_test_client'
 
 class TestGoogle < ::Minitest::Test
@@ -12,6 +13,10 @@ class TestGoogle < ::Minitest::Test
     [-34.92771808058, 138.477041423321]
   end
 
+  def lookup_file(file_name, &block)
+    lookup(File.open(File.join(mock_path, file_name)).read, &block)
+  end
+
   def lookup(body = nil, &_block)
     config = OpenStruct.new
     config.api_key = 'MTIzYWJj'
@@ -33,7 +38,7 @@ class TestGoogle < ::Minitest::Test
   end
 
   def test_google_using_lat_long_coordinates
-    mine = lookup(File.open(mock_path + '/google_lat_lon_coords.txt').read)
+    mine = lookup_file('/google_lat_lon_coords.txt')
 
     assert_equal 'Australia/Adelaide', mine.lookup(*coordinates)
   end
@@ -76,7 +81,7 @@ class TestGoogle < ::Minitest::Test
   end
 
   def test_no_result_found
-    mine = lookup(File.open(mock_path + '/google_no_result_found.json').read)
+    mine = lookup_file('/google_no_result_found.json')
 
     assert_nil(mine.lookup(26.188703, -78.987053))
   end

data/test/timezone/lookup/test_test.rb

@@ -3,6 +3,7 @@
 require 'timezone/lookup/test'
 require 'timezone'
 require 'minitest/autorun'
+require 'ostruct'
 
 class TestTest < ::Minitest::Test
   parallelize_me!

data/test/timezone/test_lookup.rb

@@ -8,7 +8,7 @@ class TestLookup < ::Minitest::Test
     Timezone::Lookup.config(:test)
 
     assert_equal Timezone::Lookup::Test,
-      Timezone::Lookup.lookup.class
+                 Timezone::Lookup.lookup.class
   end
 
   def test_geonames_config
@@ -17,10 +17,10 @@ class TestLookup < ::Minitest::Test
     end
 
     assert_equal Timezone::Lookup::Geonames,
-      Timezone::Lookup.lookup.class
+                 Timezone::Lookup.lookup.class
 
     assert_equal Timezone::NetHTTPClient,
-      Timezone::Lookup.lookup.config.request_handler
+                 Timezone::Lookup.lookup.config.request_handler
   end
 
   def test_google_config
@@ -29,10 +29,10 @@ class TestLookup < ::Minitest::Test
     end
 
     assert_equal Timezone::Lookup::Google,
-      Timezone::Lookup.lookup.class
+                 Timezone::Lookup.lookup.class
 
     assert_equal Timezone::NetHTTPClient,
-      Timezone::Lookup.lookup.config.request_handler
+                 Timezone::Lookup.lookup.config.request_handler
   end
 
   def test_custom_config

data/test/timezone/test_zone.rb

@@ -130,12 +130,12 @@ class TestZone < ::Minitest::Test
 
   # http://www.timeanddate.com/worldclock/clockchange.html?n=2364&year=1940
   def test_historical_time_change_in_hebron
-    local = Time.strptime('1940-05-31T23:59:59 UTC', '%Y-%m-%dT%H:%M:%S %Z')
-    utc = Time.strptime('1940-05-31T21:59:59 UTC', '%Y-%m-%dT%H:%M:%S %Z')
+    local = Time.strptime('1940-06-01T01:59:59 UTC', '%Y-%m-%dT%H:%M:%S %Z')
+    utc = Time.strptime('1940-05-31T23:59:59 UTC', '%Y-%m-%dT%H:%M:%S %Z')
     assert_equal local.to_i, zone('Asia/Hebron').time(utc).to_i
 
-    local = Time.strptime('1940-06-01T01:00:00 UTC', '%Y-%m-%dT%H:%M:%S %Z')
-    utc = Time.strptime('1940-05-31T22:00:00 UTC', '%Y-%m-%dT%H:%M:%S %Z')
+    local = Time.strptime('1940-06-01T03:00:00 UTC', '%Y-%m-%dT%H:%M:%S %Z')
+    utc = Time.strptime('1940-06-01T00:00:00 UTC', '%Y-%m-%dT%H:%M:%S %Z')
     assert_equal local.to_i, zone('Asia/Hebron').time(utc).to_i
   end
 

data/timezone.gemspec

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
-# -*- encoding: utf-8 -*-
 
-$:.push File.expand_path('../lib', __FILE__)
+$:.push File.expand_path('lib', __dir__)
 require 'timezone/version'
 
 Gem::Specification.new do |s|
@@ -16,7 +15,9 @@ Gem::Specification.new do |s|
   s.description = 'Accurate current and historical timezones for Ruby with ' \
     'support for Geonames and Google latitude - longitude lookups.'
 
-  s.rubyforge_project = 'timezone'
+  s.metadata =    {
+    'changelog_uri' => 'https://github.com/panthomakos/timezone/blob/master/CHANGES.markdown'
+  }
 
   s.files       = `git ls-files`.split("\n")
   s.test_files  = `git ls-files -- {test,spec,features}/*`.split("\n")
@@ -27,8 +28,12 @@ Gem::Specification.new do |s|
   s.rdoc_options     = ['--charset=UTF-8']
   s.require_paths    = ['lib']
 
+  s.add_runtime_dependency('base64', '~> 0.2')
+  s.add_runtime_dependency('ostruct', '~> 0.6')
+
   s.add_development_dependency('minitest', '~> 5.8')
-  s.add_development_dependency('rake', '~> 12')
-  s.add_development_dependency('rubocop', '= 0.51')
+  s.add_development_dependency('rake', '~> 13')
+  s.add_development_dependency('rubocop', '= 1.5.1')
+  s.add_development_dependency('rubocop-performance', '= 1.5.1')
   s.add_development_dependency('timecop', '~> 0.8')
 end

metadata

@@ -1,15 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: timezone
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.3.30
 platform: ruby
 authors:
 - Pan Thomakos
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-11-02 00:00:00.000000000 Z
+date: 2025-12-11 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: base64
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.2'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.2'
+- !ruby/object:Gem::Dependency
+  name: ostruct
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -30,28 +58,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12'
+        version: '13'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12'
+        version: '13'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: '0.51'
+        version: 1.5.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.5.1
+- !ruby/object:Gem::Dependency
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.5.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: '0.51'
+        version: 1.5.1
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement
@@ -76,10 +118,11 @@ extra_rdoc_files:
 - README.markdown
 - License.txt
 files:
+- ".codeclimate.yml"
 - ".editorconfig"
+- ".github/workflows/ci.yml"
 - ".gitignore"
 - ".rubocop.yml"
-- ".travis.yml"
 - CHANGES.markdown
 - CODE_OF_CONDUCT.md
 - CONTRIBUTING.markdown
@@ -183,9 +226,11 @@ files:
 - data/America/Cayman
 - data/America/Chicago
 - data/America/Chihuahua
+- data/America/Ciudad_Juarez
 - data/America/Coral_Harbour
 - data/America/Cordoba
 - data/America/Costa_Rica
+- data/America/Coyhaique
 - data/America/Creston
 - data/America/Cuiaba
 - data/America/Curacao
@@ -263,6 +308,7 @@ files:
 - data/America/North_Dakota/Beulah
 - data/America/North_Dakota/Center
 - data/America/North_Dakota/New_Salem
+- data/America/Nuuk
 - data/America/Ojinaga
 - data/America/Panama
 - data/America/Pangnirtung
@@ -389,6 +435,7 @@ files:
 - data/Asia/Pontianak
 - data/Asia/Pyongyang
 - data/Asia/Qatar
+- data/Asia/Qostanay
 - data/Asia/Qyzylorda
 - data/Asia/Rangoon
 - data/Asia/Riyadh
@@ -535,6 +582,7 @@ files:
 - data/Europe/Kaliningrad
 - data/Europe/Kiev
 - data/Europe/Kirov
+- data/Europe/Kyiv
 - data/Europe/Lisbon
 - data/Europe/Ljubljana
 - data/Europe/London
@@ -630,6 +678,7 @@ files:
 - data/Pacific/Guam
 - data/Pacific/Honolulu
 - data/Pacific/Johnston
+- data/Pacific/Kanton
 - data/Pacific/Kiritimati
 - data/Pacific/Kosrae
 - data/Pacific/Kwajalein
@@ -674,7 +723,6 @@ files:
 - data/US/Michigan
 - data/US/Mountain
 - data/US/Pacific
-- data/US/Pacific-New
 - data/US/Samoa
 - data/UTC
 - data/Universal
@@ -726,8 +774,9 @@ files:
 homepage: https://github.com/panthomakos/timezone
 licenses:
 - MIT
-metadata: {}
-post_install_message: 
+metadata:
+  changelog_uri: https://github.com/panthomakos/timezone/blob/master/CHANGES.markdown
+post_install_message:
 rdoc_options:
 - "--charset=UTF-8"
 require_paths:
@@ -743,11 +792,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: timezone
-rubygems_version: 2.6.11
-signing_key: 
+rubygems_version: 3.4.19
+signing_key:
 specification_version: 4
-summary: timezone-1.3.2
+summary: timezone-1.3.30
 test_files:
 - test/data/Helsinki_rules_without_timestamps.json
 - test/data/asia
@@ -776,4 +824,3 @@ test_files:
 - test/timezone/test_mathn_compatibility.rb
 - test/timezone/test_nil_zone.rb
 - test/timezone/test_zone.rb
-has_rdoc: