gillbus 0.17.6 → 0.18.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 39981156aa20418b3109add8e3d94c63077bd44c
-  data.tar.gz: 50e5710aa809acc6203119011b706034fbdd100f
+  metadata.gz: 061717304c911054d7958701d0453ae6243f1efc
+  data.tar.gz: 481cf904061a7696bdfabc23359710e0a0d83707
 SHA512:
-  metadata.gz: 512923d49ef42592d7f3d1d2fbd4a6e97b888d15980cd1b1ebeee82ded1908cda2fa820e07d8a2769ebf797f2557e7d8beb1395e34dee02f54298705d2c2f4ab
-  data.tar.gz: 15d286724cead0805000c7d4d29c9abea2f7b59ca11561bc8cc6515a2e037cf616df9cbfa6284d9f90c418723de581c00ef6b3ad7d95add3171a6cb0a11c261d
+  metadata.gz: cfc65005717460f4f77283acf4b572b8b45c94d2d67445d89b39fc2a4845fd53adf446e1b3783182615afabb346c487e8d48c28f331fb628d2dc40016a86f955
+  data.tar.gz: 4af31b24aa23bf174da9d40d9abd7f0b9219d8016a6592a9fa62d733b9b033f378181ff815d0204707fc4d9a966e80ced0cbb544bfde3cfbff05e6e98277a424

data/Gemfile

@@ -2,7 +2,3 @@ source 'https://rubygems.org'
 
 # Specify your gem's dependencies in gillbus.gemspec
 gemspec
-
-group :development do
-  gem 'ox'
-end

data/gillbus.gemspec

@@ -22,7 +22,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'faraday'
   spec.add_dependency 'monetize'
   spec.add_dependency 'money'
-  spec.add_dependency 'multi_xml'
+  spec.add_dependency 'ox', '~> 2.9'
 
   spec.add_development_dependency 'bundler', '~> 1.6'
   spec.add_development_dependency 'minitest'

data/lib/gillbus/base_response.rb

@@ -1,4 +1,4 @@
-require 'multi_xml'
+require 'ox'
 
 class Gillbus
   class BaseResponse
@@ -32,12 +32,12 @@ class Gillbus
     end
 
     def self.parse_string(xml_string, **options)
-      xml = MultiXml.parse(xml_string)
+      xml = Ox.load(xml_string, mode: :hash, symbolize_keys: false)
       # <DATA/> is a valid response
       return ParseError.new(xml_string, 'DATA attribute missing') unless xml.key?('DATA')
       data = xml['DATA'] || {}
       parse(data, instance: new, options: options)
-    rescue MultiXml::ParseError, ArgumentError => e
+    rescue Ox::ParseError, ArgumentError => e
       ParseError.new(xml_string, e.message)
     end
   end

data/lib/gillbus/helpers/parser.rb

@@ -3,6 +3,13 @@ require 'active_support/time'
 
 class Gillbus
   class Parser
+    NULL_CONST = 'null'.freeze
+    TRUE_CONST = 'true'.freeze
+    YES_CONST = 'Y'.freeze
+    DATE_FORMAT_CONST = '%d.%m.%Y'.freeze
+    TIME_FORMAT_REGEXP = /^ ( \d\d:\d\d ) (?: :\d\d )? $/x.freeze
+    DEFAULT_TIMEZONE = 'Europe/Kiev'.freeze
+
     attr_accessor :doc
     attr_accessor :instance
     attr_accessor :fields
@@ -45,6 +52,9 @@ class Gillbus
     def make_one_or_many(type, val)
       # [:type]
       if type.is_a? Array
+        if val.is_a?(Array) && val[0].is_a?(Hash) && !val[0].has_key?('__content__')
+          val = [val] if val[1].is_a?(String) # hack to handle attribute parsing by Ox
+        end
         array(val).map { |v| make_one type.first, v }
       # :type
       else
@@ -70,16 +80,16 @@ class Gillbus
     end
 
     def string(val)
-      return if val == 'null'
+      return if val == NULL_CONST
       val
     end
 
     def bool(val)
-      val == 'true'
+      val == TRUE_CONST
     end
 
     def yesno_bool(val)
-      val == 'Y'
+      val == YES_CONST
     end
 
     def int(val)
@@ -87,12 +97,12 @@ class Gillbus
     end
 
     def date(val)
-      Date.strptime(val, '%d.%m.%Y')
+      Date.strptime(val, DATE_FORMAT_CONST)
     end
 
     # rubocop:disable Style/GuardClause, Style/IfUnlessModifier
     def time(val)
-      if val =~ /^ ( \d\d:\d\d ) (?: :\d\d )? $/x
+      if val =~ TIME_FORMAT_REGEXP
         $1
       end
     end
@@ -109,7 +119,7 @@ class Gillbus
     end
 
     def default_timezone
-      @options[:timezone] || 'Europe/Kiev'
+      @options[:timezone] || DEFAULT_TIMEZONE
     end
 
     def decimal(val)

data/lib/gillbus/structs/return_cause.rb

@@ -14,7 +14,7 @@ class Gillbus
 
     parser do
       def money(val)
-        Monetize.parse(val, doc[:_currency])
+        Monetize.parse(val, parent.sale_cur_code)
       end
     end
   end

data/lib/gillbus/structs/tariff/return_cause.rb

@@ -1,6 +1,8 @@
 class Gillbus
   class Tariff
     class ReturnCause
+      LOSSLESS = 'lossless'.freeze
+
       extend Fields
       include UpdateAttrs
 
@@ -10,10 +12,13 @@ class Gillbus
 
       # rubocop:disable Lint/UnusedMethodArgument
       def self.parse(doc, instance: nil, parent: nil, options: {})
-        instance = super
-        if doc.is_a? Hash
+        instance = new
+        if doc.is_a?(Array) && doc.size == 2 && doc.first.is_a?(Hash) && doc.last.is_a?(String)
+          instance.lossless = doc.first[LOSSLESS] == Parser::TRUE_CONST
+          instance.cause = doc.last
+        elsif doc.is_a?(Hash) && doc.has_key?('__content__')
+          instance.lossless = (doc['lossless'] == 'true')
           instance.cause = doc['__content__']
-          instance.lossless = doc['lossless'] == 'true'
         else
           instance.cause = doc
           instance.lossless = false

data/lib/gillbus/structs/ticket.rb

@@ -256,7 +256,7 @@ class Gillbus
     # Array of commissions only for us (with additional fields)
     field :all_commissions, [Commission], key: 'COMMISSION', root: 'ALL_COMMISSIONS'
 
-    field :return_causes, [:return_cause], key: 'RETURN_CAUSE'
+    field :return_causes, [ReturnCause], key: 'RETURN_CAUSE'
 
     # Date to pay reserved ticket
     field :date_to_pay, :datetime

data/lib/gillbus/structs/trip.rb

@@ -181,6 +181,7 @@ class Gillbus
     def self.parse(doc, instance: nil, parent: nil, options: {})
       instance = super
       instance.data = doc
+      instance.options = TripOptions.build_blank if instance.options.nil?
       instance
     end
   end

data/lib/gillbus/structs/trip_options.rb

@@ -39,5 +39,13 @@ class Gillbus
         vals.include?('BUSFOR_RECOMMEND')
       end
     end
+
+    def self.build_blank
+      options = new
+      field_definitions.each do |name:, type:, key:, root:|
+        options.send(:"#{name}=", []) if type.is_a?(Array)
+      end
+      options
+    end
   end
 end

data/lib/gillbus/structs/trip_service.rb

@@ -7,6 +7,21 @@ class Gillbus
     field :id, :int, key: 'ID'
 
     # название услуги в автобусе
-    field :name, :string, key: '__content__'
+    field :name, :string, key: :__content__
+
+    def self.parse(doc, instance: nil, parent: nil, options: {})
+      instance = new
+      if doc.is_a?(Array)
+        raise ArgumentError, "Unable to parse TripService: #{doc.inspect}" unless doc.size == 2
+        instance.id = doc.first.fetch('ID').to_i
+        instance.name = doc.last
+      elsif doc.is_a?(Hash) #legacy data made with MultiXML
+        instance.id = doc.fetch('ID')
+        instance.name = doc.fetch('__content__')
+      else
+        raise ArgumentError, "Unable to parse TripService: #{doc.inspect}"
+      end
+      instance
+    end
   end
 end

data/lib/gillbus/version.rb

@@ -1,3 +1,3 @@
 class Gillbus
-  VERSION = '0.17.6'.freeze
+  VERSION = '0.18.1'.freeze
 end

data/test/error_test.rb

@@ -22,7 +22,7 @@ class ErrorTest < Minitest::Test
 
   def test_malformed_errorness
     assert malformed_response.error?
-    assert_equal "Malformed response: Not Terminated: text not terminated at 0:10", malformed_response.error_message
+    assert_equal "Malformed response: invalid format, expected < at line 1, column 1 [parse.c:146]\n", malformed_response.error_message
   end
 
   def test_malformed_2_errorness

data/test/parse_trip_test.rb

@@ -0,0 +1,25 @@
+require 'test_helper'
+
+class ParseTripYAML < Minitest::Test
+  def test_content_1
+    response = Gillbus::Trip.parse(YAML.load(File.read('test/responses/legacy_trip.yml')))
+    expected_causes = [
+      'Без потерь,  более чем за 2 ч., до отправления рейса. Возвращается 100% стоимости тарифа. Ориентировочная сумма возврата 550',
+      'Добровольно,  менее чем за 2 ч., до отправления рейса. Удерживается при возврате: 15% тарифа. Ориентировочная сумма возврата 425',
+     'Возврат маршрутной квитанции в случае опоздания к отправлению (стоимость не возвращается). Удерживается при возврате: 100% тарифа. Ориентировочная сумма возврата 0',
+     'Возврат маршрутной квитанции по отмене отправления т/с, задержки отправления т/с более чем на час и непредставления пассажиру указанного в билете места. СРОК ОБРАЩЕНИЯ для возврата билета 24 часа с времени отправления. . Возвращается 100% стоимости билета. Ориентировочная сумма возврата 550'
+    ]
+
+    assert_equal true, response.tariffs.first.return_cause.first.lossless
+    assert_equal expected_causes, response.tariffs.first.return_cause.map(&:cause)
+  end
+
+  def test_content_2
+    response = Gillbus::Trip.parse(YAML.load(File.read('test/responses/legacy_trip_2.yml')))
+    assert_equal ["Принудительно, при отмене рейса. Ориентировочная сумма возврата 1044.95RUB",
+ "Добровільно ДБ 4, менш ніж за 10 хвилин до відправлення рейсу. Ориентировочная сумма возврата 574.72RUB",
+ "ДБ 3, не менш ніж за 10 хвилин до відправлення рейсу. Ориентировочная сумма возврата 679.21RUB",
+ "Добровольно,  более чем за 2 ч., до отправления рейса. Ориентировочная сумма возврата 783.71RUB",
+ "не менше ніж за 24 години до відправлення рейсу. Ориентировочная сумма возврата 888.2RUB"], response.tariffs.first.return_cause.map(&:cause)
+  end
+end

data/test/responses/legacy_trip_2.yml

@@ -0,0 +1,217 @@
+---
+ID: '3436521222122325532'
+RES_ID: '0'
+NUMBER: STAGE - ATM 002 Москва - Киев
+START_CITY_NAME: Москва
+END_CITY_NAME: Киев Автовокзал "Центральный"
+START_TIMEZONE: 
+END_TIMEZONE: 
+START_DATE: 10.06.2017
+END_DATE: 11.06.2017
+START_TIME: '21:20'
+END_TIME: '15:30'
+BUS_MODEL: 'Неоплан '
+BUS_CLASS: 
+BUS_ID: 
+PLATFORM: 
+TRIP_SEATS_MAP: 'true'
+SHOW_PICKUP_DROPOFF: 'false'
+BRANDED: 'false'
+HAS_TRANSFER: 'false'
+PLACES_COUNT: '49'
+FREE_PLACES_COUNT: '10'
+FREE_BUSY_PLACES: 10/49
+CARRIER_ID: 4A5DC2609C320E39E0530300F00A7BFB
+CARRIER_NAME: '"АТМ Транс" ООО'
+TIME_IN_ROAD: '18:10'
+FAKE_TIME_IN_ROAD: 'false'
+STATION_BEGIN_ID: 3206819326B763A8E0530300F00AA4FB
+STATION_BEGIN_NAME: Международный автовокзал "Южные Ворота"
+STATION_BEGIN_ADDRESS: '19-й км МКАД, владение 20, стр. 1, Москва, Россия
+
+'
+STATION_END_ID: 1C7DB38A37BCCE21E0508E5B35570395
+STATION_END_NAME: Центральный автовокзал
+STATION_END_ADDRESS: проспект Науки; дом 1/2
+RESERVATION_ENABLED: 'false'
+SALE_ENABLED: 'true'
+RETURN_ENABLED: 'false'
+TIME_FOR_CANCEL: '40'
+TRIP_FULL_SALE: 'false'
+CAN_DISCOUNT: 'true'
+RECOMMENDED: 'false'
+INTERNATIONAL: 'true'
+JOIN_BOOK_BUY: 'true'
+TRIP_MODE: '1'
+SPECIAL_CONDITIONS: 
+IMAGE_URL: 'null'
+START_POINT_TYPE: 'null'
+VEHICLE_DESCRIPTION: 'null'
+VEHICLE_FEATURES: 'null'
+TOTAL_COST: '1044.95'
+CURRENCY: RUB
+CURRENCY_CODE: '643'
+TARIFF:
+  SHORT_NAME: Y
+  FULL_NAME: Базовый тариф
+  START_DATE: 
+  END_DATE: 
+  DESCRIPTION: |+
+    Базовый тариф стоимости проезда одного пассажиров в одном направлении
+    В стоимость  билета включена стоимость перевозки одного места багажа, весом до 30 кг., на 1 лицо и ручной поклажи весом до 5 кг. размером до 60х40х20 см. При превышении данных лимитов относительно веса и размеров, берется доплата как за дополнительный багаж. Дополнительный багаж перевозится при наличии свободного места в багажном отделении автобуса, за который берется доплата в размере 10 % от стоимости билета за одно дополнительное багажное место.
+
+  TIME_TO_BUY_RESERVED: '24'
+  TIME_TO_STOP_BOOKING: '48'
+  TIME_TO_PAY_RESERVATION: 
+  COST: '1044.95'
+  INSURANCE_COST: '0'
+  INSURANCE_ID: 
+  INSURANCE_SUM: '0'
+  INSURANCE_CUR: 
+  AFTER_DISPATCH_NO_RETURN: 'true'
+  PASSENGER_BIRTHDAY: 
+  PASSENGER_ISIC_NUMBER: 
+  PASSENGER_STUDENT_TICKET: 
+  NOTE: 
+  RETURN_CAUSE:
+  - max_time: '10'
+    min_time: "-1440000"
+    percent: '0'
+    __content__: Принудительно, при отмене рейса. Ориентировочная сумма возврата 1044.95RUB
+  - max_time: '10'
+    min_time: '10'
+    percent: '45.00'
+    __content__: Добровільно ДБ 4, менш ніж за 10 хвилин до відправлення рейсу. Ориентировочная
+      сумма возврата 574.72RUB
+  - max_time: '120'
+    min_time: '10'
+    percent: '35.00'
+    __content__: ДБ 3, не менш ніж за 10 хвилин до відправлення рейсу. Ориентировочная
+      сумма возврата 679.21RUB
+  - max_time: '1440'
+    min_time: '120'
+    percent: '25.00'
+    __content__: Добровольно,  более чем за 2 ч., до отправления рейса. Ориентировочная
+      сумма возврата 783.71RUB
+  - min_time: '1440'
+    percent: '15.00'
+    __content__: не менше ніж за 24 години до відправлення рейсу. Ориентировочная
+      сумма возврата 888.2RUB
+  PROMOTE: 'false'
+POINT:
+- NUMBER: '0'
+  GEOGRAPHY_NAME: Москва
+  NAME: Международный автовокзал "Южные Ворота"
+  ADDRESS: '19-й км МКАД, владение 20, стр. 1, Москва, Россия
+
+'
+  TIMEZONE: 
+  ARRIVAL_DATE: 10.06.2017
+  ARRIVAL_TIME: 
+  DISPATCH_DATE: 10.06.2017
+  DISPATCH_TIME: '21:20'
+  CHECK_POINT: 'false'
+  DISTANCE: '0'
+  TRANSFER_POINT: 'false'
+  PLATFORM: 
+  BUSMODEL: 'Неоплан '
+- NUMBER: '1'
+  GEOGRAPHY_NAME: Москва
+  NAME: Автостанция "Красногвардейская"
+  ADDRESS: метро Красногвардейская; бульвар Ореховый; дом 24к1Г
+  TIMEZONE: 
+  ARRIVAL_DATE: 10.06.2017
+  ARRIVAL_TIME: '21:40'
+  DISPATCH_DATE: 10.06.2017
+  DISPATCH_TIME: '22:00'
+  CHECK_POINT: 'false'
+  DISTANCE: '5'
+  TRANSFER_POINT: 'false'
+  PLATFORM: 
+  BUSMODEL: 'Неоплан '
+- NUMBER: '2'
+  GEOGRAPHY_NAME: Орел
+  NAME: Автовокзал "Орел"
+  ADDRESS: улица  Автовокзальная; дом 1
+  TIMEZONE: 
+  ARRIVAL_DATE: 11.06.2017
+  ARRIVAL_TIME: '05:05'
+  DISPATCH_DATE: 11.06.2017
+  DISPATCH_TIME: '05:30'
+  CHECK_POINT: 'false'
+  DISTANCE: '371'
+  TRANSFER_POINT: 'false'
+  PLATFORM: 
+  BUSMODEL: 'Неоплан '
+- NUMBER: '3'
+  GEOGRAPHY_NAME: Железногорск
+  NAME: Автовокзал   "Железногорск"
+  ADDRESS: улица  Советской Армии; дом 8
+  TIMEZONE: 
+  ARRIVAL_DATE: 11.06.2017
+  ARRIVAL_TIME: '07:20'
+  DISPATCH_DATE: 11.06.2017
+  DISPATCH_TIME: '07:30'
+  CHECK_POINT: 'false'
+  DISTANCE: '466'
+  TRANSFER_POINT: 'false'
+  PLATFORM: 
+  BUSMODEL: 'Неоплан '
+- NUMBER: '4'
+  GEOGRAPHY_NAME: Хомутовка
+  NAME: АС
+  ADDRESS: ул. Ленина, 5
+  TIMEZONE: 
+  ARRIVAL_DATE: 11.06.2017
+  ARRIVAL_TIME: '08:45'
+  DISPATCH_DATE: 11.06.2017
+  DISPATCH_TIME: '08:50'
+  CHECK_POINT: 'false'
+  DISTANCE: '534'
+  TRANSFER_POINT: 'false'
+  PLATFORM: 
+  BUSMODEL: 'Неоплан '
+- NUMBER: '5'
+  GEOGRAPHY_NAME: Бачевск КПП(Украина-Россия)
+  NAME: 
+  ADDRESS: 
+  TIMEZONE: 
+  ARRIVAL_DATE: 11.06.2017
+  ARRIVAL_TIME: '09:05'
+  DISPATCH_DATE: 11.06.2017
+  DISPATCH_TIME: '09:45'
+  CHECK_POINT: 'true'
+  DISTANCE: '557'
+  TRANSFER_POINT: 'false'
+  PLATFORM: 
+  BUSMODEL: 'Неоплан '
+- NUMBER: '6'
+  GEOGRAPHY_NAME: Глухов
+  NAME: Остановка Глухов
+  ADDRESS: улица Путивльская; дом 39А
+  TIMEZONE: 
+  ARRIVAL_DATE: 11.06.2017
+  ARRIVAL_TIME: '10:15'
+  DISPATCH_DATE: 11.06.2017
+  DISPATCH_TIME: '10:35'
+  CHECK_POINT: 'false'
+  DISTANCE: '594'
+  TRANSFER_POINT: 'false'
+  PLATFORM: 
+  BUSMODEL: 'Неоплан '
+- NUMBER: '7'
+  GEOGRAPHY_NAME: Киев
+  NAME: Центральный автовокзал
+  ADDRESS: проспект Науки; дом 1/2
+  TIMEZONE: 
+  ARRIVAL_DATE: 11.06.2017
+  ARRIVAL_TIME: '15:30'
+  DISPATCH_DATE: 11.06.2017
+  DISPATCH_TIME: '16:00'
+  CHECK_POINT: 'false'
+  DISTANCE: '914'
+  TRANSFER_POINT: 'false'
+  PLATFORM: 
+  BUSMODEL: 'Неоплан '
+EXTRA_CHARGE: '0.0'
+

data/test/search_trips_test.rb

@@ -84,6 +84,7 @@ class SearchTripsResponseTest < Minitest::Test
 
   def test_fields_parsing
     response = get_successful_search_trips
+    assert_equal '3451212695530327295', response.trips.first.id
     assert_equal Date.new(2014, 8, 23), response.trips.first.start_date
     assert_equal Money.new(1410_00, 'RUB'), response.trips.first.total_cost
     assert_equal Money.new(1410_00, 'RUB'), response.trips.first.tariffs.first.cost

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gillbus
 version: !ruby/object:Gem::Version
-  version: 0.17.6
+  version: 0.18.1
 platform: ruby
 authors:
 - Alexey "codesnik" Trofimenko
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-04-27 00:00:00.000000000 Z
+date: 2018-06-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -68,19 +68,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: multi_xml
+  name: ox
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.9'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -237,6 +237,7 @@ files:
 - test/get_trip_seats_test.rb
 - test/get_trip_segments_test.rb
 - test/lock_seats_test.rb
+- test/parse_trip_test.rb
 - test/reserve_tickets_test.rb
 - test/responses/bookingDifferentCurrencies.xml
 - test/responses/buyBooking.xml
@@ -253,6 +254,7 @@ files:
 - test/responses/getTripSeats-segments.xml
 - test/responses/getTripSeats.xml
 - test/responses/getTripSegments.xml
+- test/responses/legacy_trip_2.yml
 - test/responses/lockSeats.xml
 - test/responses/reserveTickets.xml
 - test/responses/reserveTickets.yml
@@ -313,6 +315,7 @@ test_files:
 - test/get_trip_seats_test.rb
 - test/get_trip_segments_test.rb
 - test/lock_seats_test.rb
+- test/parse_trip_test.rb
 - test/reserve_tickets_test.rb
 - test/responses/bookingDifferentCurrencies.xml
 - test/responses/buyBooking.xml
@@ -329,6 +332,7 @@ test_files:
 - test/responses/getTripSeats-segments.xml
 - test/responses/getTripSeats.xml
 - test/responses/getTripSegments.xml
+- test/responses/legacy_trip_2.yml
 - test/responses/lockSeats.xml
 - test/responses/reserveTickets.xml
 - test/responses/reserveTickets.yml