fias 0.0.2 → 1.0.0

data/spec/lib/import/copy_spec.rb

@@ -0,0 +1,44 @@
+require 'spec_helper'
+
+describe Fias::Import::Copy do
+  let(:name) { 'actual_statuses' }
+  let(:files) { Fias::Import::Dbf.new('spec/fixtures').only(name) }
+  let(:table_name) { "fias_#{name}".to_sym }
+  let(:db) { double('db') }
+  let(:raw_connection) { double('raw_connection') }
+
+  subject { Fias::Import::Tables.new(db, files).copy.first }
+
+  before do
+    stub_const('Fias::Import::Tables::UUID', name.to_sym => %w(name))
+  end
+
+  context '#encode' do
+    it do
+      expect(PgDataEncoder::EncodeForCopy).to receive(:new).with(
+        column_types: { 1 => :uuid }
+      ).and_call_original
+
+      subject.encode
+    end
+  end
+
+  context '#copy' do
+    let(:result) { double('result') }
+
+    before do
+      table_obj = double(table_name)
+      expect(db).to receive(:[]).with(table_name).and_return(table_obj)
+      expect(table_obj).to receive(:truncate)
+      expect(db).to receive(:run).with(/SET client/)
+      expect(db).to receive(:copy_into).with(
+        :fias_actual_statuses, columns: [:actstatid, :name], format: :binary
+      ).and_yield.and_yield
+    end
+
+    it do
+      subject.encode
+      subject.copy
+    end
+  end
+end

data/spec/lib/import/dbf_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe Fias::Import::Dbf do
+  subject { described_class.new('spec/fixtures') }
+
+  context '#initialize' do
+    it 'fails without files' do
+      expect { described_class.new('foo') }.to raise_error
+    end
+
+    it 'returns correct file list' do
+      expect(subject.files.keys).to eq(
+        [:actual_statuses, :structure_statuses, :nordoc99]
+      )
+      expect(subject.files.values).to all(be_present)
+      expect(subject.files.values).to all(be_kind_of(DBF::Table))
+    end
+  end
+
+  context '#only' do
+    it 'returns only houses' do
+      expect(subject.only(:nordocs).keys).to eq([:nordoc99])
+      expect(subject.only(:address_objects, :structure_statuses).keys).to eq(
+        [:structure_statuses]
+      )
+    end
+  end
+end

data/spec/lib/import/download_service_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe Fias::Import::DownloadService do
+  it 'parses url' do
+    stub_request(
+      :post,
+      'http://fias.nalog.ru/WebServices/Public/DownloadService.asmx'
+    ).with(described_class::OPTIONS).to_return(
+      status: 200,
+      body: '<FiasCompleteDbfUrl>http://www.ya.ru</FiasCompleteDbfUrl>'
+    )
+
+    expect(described_class.url).to eq('http://www.ya.ru')
+  end
+end

data/spec/lib/import/restore_parent_id_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe Fias::Import::RestoreParentId do
+  let(:db) { Sequel.sqlite }
+  let(:table) { db[:address_objects] }
+  let(:records) { table.select_map([:id, :parent_id]).index_by(&:first) }
+
+  subject { described_class.new(table) }
+
+  before do
+    db.create_table! :address_objects do
+      primary_key :id
+      column :aoguid, :string
+      column :parentguid, :string
+      column :parent_id, :integer
+    end
+
+    table.insert(id: 1, aoguid: 'g1')
+    table.insert(id: 2, aoguid: 'g2')
+    table.insert(id: 3, aoguid: 'g3', parentguid: 'g1')
+    table.insert(id: 4, aoguid: 'g4', parentguid: 'g1')
+    table.insert(id: 5, aoguid: 'g5', parentguid: 'g2')
+  end
+
+  it do
+    subject.restore
+
+    expect(records[1].last).to be_nil
+    expect(records[2].last).to be_nil
+    expect(records[3].last).to eq(1)
+    expect(records[4].last).to eq(1)
+    expect(records[5].last).to eq(2)
+  end
+end

data/spec/lib/import/tables_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe Fias::Import::Tables do
+  let(:files) { Fias::Import::Dbf.new('spec/fixtures').files }
+  let(:db) { double('db') }
+
+  subject { described_class.new(db, files, '_fias') }
+
+  it '#create' do
+    stub_const('Fias::Import::Tables::UUID', actual_statuses: %w(name))
+
+    expect(db).to receive(:create_table).with(:_fias_actual_statuses).and_yield
+    expect(db).to receive(:create_table).with(:_fias_nordoc99)
+    expect(db).to receive(:create_table).with(:_fias_structure_statuses)
+
+    expect(subject).to receive(:primary_key).with(:id)
+    expect(subject).to receive(:column).with(:name, :uuid)
+    expect(subject).to receive(:column).with(:actstatid, :float)
+
+    expect { subject.create }.to_not raise_error
+  end
+
+  it '#copy' do
+    expect(subject.copy.size).to eq(3)
+  end
+end

data/spec/lib/name/append_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+describe Fias::Name::Append do
+  context '#append' do
+    YAML.load_file('spec/fixtures/status_append.yml').each do |item|
+      name, toponym = *item.first
+      short, long = *item.slice(1..-1)
+
+      it "#{toponym} #{name} must generate #{short} and #{long}" do
+        expect(described_class.append(name, toponym)).to eq([short, long])
+      end
+    end
+  end
+end

data/spec/lib/name/canonical_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+
+describe Fias::Name::Canonical do
+  context '#canonical' do
+    {
+      'ул' => %w(улица ул ул.),
+      'Улица' => %w(улица ул ул.),
+      'микр' => %w(микрорайон мкр мкр. мкрн микр),
+      'микрорайон' => %w(микрорайон мкр мкр. мкрн микр),
+      'АО' => ['автономный округ', 'АО', 'АО'],
+      'Аобл' => ['автономная область', 'Аобл', 'Аобл'],
+      'республика' => %w(Республика Респ Респ.),
+      'Чувашия' => ['Чувашская Республика - Чувашия', 'Чувашия']
+    }.each do |to, normalized|
+      it "#{to} must become #{normalized.inspect}" do
+        expect(described_class.canonical(to)).to eq(normalized)
+      end
+    end
+  end
+end

data/spec/lib/name/extract_spec.rb

@@ -0,0 +1,67 @@
+require 'spec_helper'
+
+describe Fias::Name::Extract do
+  {
+    'г Краснодар' => %w(Краснодар город г г.),
+    'г. Краснодар' => %w(Краснодар город г г.),
+    'Краснодар город' => %w(Краснодар город г г.),
+    'Раменский район' => ['Раменский', 'район', 'р-н', 'р-н'],
+    'Ямало-Ненецкий АО' => ['Ямало-Ненецкий', 'автономный округ', 'АО', 'АО'],
+    'Еврейский автономный округ' => [
+      'Еврейский', 'автономный округ', 'АО', 'АО'
+    ],
+    'Корягинский район' => ['Корягинский', 'район', 'р-н', 'р-н'],
+    'гопотека Южное Бутово' => ['гопотека Южное Бутово'],
+    'ул. Длинная' => %w(Длинная улица ул ул.),
+    'ул. им. Злых Марсиан-3' => ['им. Злых Марсиан-3', 'улица', 'ул', 'ул.'],
+    'Тверь г' => ['Тверь', 'город', 'г', 'г.'],
+    'Тверь г.' => ['Тверь', 'город', 'г', 'г.'],
+    'Гаврилова' => ['Гаврилова'],
+    'пр Космонавтов' => ['Космонавтов', 'проспект', 'пр-кт', 'пр-кт'],
+    'Искровский проспект' => ['Искровский', 'проспект', 'пр-кт', 'пр-кт'],
+    'коттеджный поселок Морозов' => [
+      'Морозов', 'коттеджный поселок', 'кп', 'кп'
+    ],
+    'поселок городского типа Свердловский' => [
+      'Свердловский', 'поселок городского типа', 'пгт', 'пгт'
+    ],
+    'поселок Павловская Слобода' => [
+      'Павловская Слобода', 'поселок', 'п', 'п.'
+    ],
+    'Павловская Слобода поселок' => [
+      'Павловская Слобода', 'поселок', 'п', 'п.'
+    ],
+    'Иваново рабочий поселок' => ['Иваново', 'рабочий поселок', 'рп', 'рп'],
+    'ул Славянский бульвар' => ['Славянский бульвар', 'улица', 'ул', 'ул.'],
+    'ул Набережная' => ['Набережная', 'улица', 'ул', 'ул.'],
+    'Сокольнический Вал ул.' => ['Сокольнический Вал', 'улица', 'ул', 'ул.'],
+    'Сокольнический Вал улица' => ['Сокольнический Вал', 'улица', 'ул', 'ул.'],
+    'ул Сокольнический Вал' => ['Сокольнический Вал', 'улица', 'ул', 'ул.'],
+    'улица Сокольнический Вал' => ['Сокольнический Вал', 'улица', 'ул', 'ул.'],
+    'М.Бронная ул.' => ['М.Бронная', 'улица', 'ул', 'ул.'],
+    '1-я улица Машиностроения' => ['1-я Машиностроения', 'улица', 'ул', 'ул.'],
+    'Аллея Ильича ул' => ['Аллея Ильича', 'улица', 'ул', 'ул.'],
+    'ул.Сычева' => %w(Сычева улица ул ул.),
+    'Мира улица.' => %w(Мира улица ул ул.),
+    'Малаховка городского типа поселок' => [
+      'Малаховка', 'поселок городского типа', 'пгт', 'пгт'
+    ],
+    'Калининград город' => ['Калининград', 'город', 'г', 'г.'],
+    'коттеджный поселок Морозов' => [
+      'Морозов', 'коттеджный поселок', 'кп', 'кп'
+    ],
+    'коттеджный Морозов' => ['Морозов', 'коттеджный поселок', 'кп', 'кп'],
+    'Морозов коттеджный' => ['Морозов', 'коттеджный поселок', 'кп', 'кп'],
+    'Морозов коттеджный поселок' => [
+      'Морозов', 'коттеджный поселок', 'кп', 'кп'
+    ],
+    '' => nil,
+    nil => nil
+  }.each do |name, expected|
+    it "must extract status from #{name} correctly" do
+      given = described_class.extract(name)
+      given = given.first(4) if given.is_a?(Array)
+      expect(given).to eq(expected)
+    end
+  end
+end

data/spec/lib/name/house_number_spec.rb

@@ -0,0 +1,45 @@
+require 'spec_helper'
+
+describe Fias::Name::HouseNumber do
+  {
+    '14'                            => ['14', nil],
+    'мкр. 14'                       => ['мкр. 14', nil],
+    'лин. 5'                        => ['лин. 5', nil],
+    'линия 14'                      => ['линия 14', nil],
+    '2 мкр'                         => ['2 мкр', nil],
+    '2-я советская'                 => ['2-я советская', nil],
+    '50 лет октября'                => ['50 лет октября', nil],
+    'советская 2-я'                 => ['советская 2-я', nil],
+    'Советской Армии ул,238а'       => ['Советской Армии ул', '238а'],
+    'Советской Армии ул,181,корп.в' => ['Советской Армии ул', '181,корп.в'],
+    'пр.Энергетиков, 72/2'          => ['пр.Энергетиков', '72/2'],
+    'Засечное с, 34'                => ['Засечное с', '34'],
+    'Ново-Садовая ул,303а'          => ['Ново-Садовая ул', '303а'],
+    'ЖК Бутово Парк-2, корп. 11'    => ['ЖК Бутово Парк-2', 'корп. 11'],
+    'советская 2-я'                 => ['советская 2-я', nil],
+    'Вербицкого 25'                 => %w(Вербицкого 25),
+    'Новоколомяжский 11к1'          => %w(Новоколомяжский 11к1),
+    'пр.Энергетиков 72/2'           => ['пр.Энергетиков', '72/2'],
+    '2-я линия В.О. 12'             => ['2-я линия В.О.', '12'],
+    'Шуваловский пр.88к1'           => ['Шуваловский пр.', '88к1'],
+    'Мкр. Северное Чертан, вл.1А'   => ['Мкр. Северное Чертан', 'вл.1А'],
+    'Московское ш,43  а,корп.стр.'  => ['Московское ш', '43  а,корп.стр.'],
+    'Заводская ул 35-А'             => ['Заводская ул 35-А', nil],
+    '50 лет октября'                => ['50 лет октября', nil],
+    'ул 1905 года 28'               => ['ул 1905 года', '28'],
+    'Новослободская 29 корп 3'      => ['Новослободская', '29 корп 3'],
+    'Александрова д 29'             => %w(Александрова 29),
+    'Николаева дом 29'              => %w(Николаева 29),
+    'Невский д. 29'                 => %w(Невский 29),
+    'ул 1905 года'                  => ['ул 1905 года', nil],
+    'ул 1905 года 28'               => ['ул 1905 года', '28'],
+    'Малая Морская ом. 29'          => ['Малая Морская', '29'],
+    '26 линия В.О. д.15 к.A'        => ['26 линия В.О.', '15 к.A'],
+    'Дунайский пр.'                 => ['Дунайский пр.', nil],
+    'Пионерстроя ул. 14 к 1'        => ['Пионерстроя ул.', '14 к 1']
+  }.each do |street, extracted|
+    it "Must extract house_number from #{street}" do
+      expect(described_class.extract(street)).to eq(extracted)
+    end
+  end
+end

data/spec/lib/name/query_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe Fias::Query::Params do
+  YAML.load_file('spec/fixtures/query_sanitization.yml').each do |pair|
+    from, to = pair
+    from.symbolize_keys!
+    to.symbolize_keys!
+
+    it from.values.to_s do
+      expect(described_class.new(from).sanitized).to eq(to)
+    end
+  end
+
+  it 'must not resanitize already sanitized parts' do
+    sanitized = described_class.new(
+      city: 'г Краснодар', street: 'Ушинского'
+    ).sanitized
+
+    expect(described_class.new(sanitized).sanitized).to eq(sanitized)
+  end
+end

data/spec/lib/name/split_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe Fias::Name::Split do
+  {
+    '"Новый" Оккервиль' => %w(новый оккервиль),
+    'Старый Дом (ограниченное снт)' => %w(старый дом ограниченное снт),
+    'ю.р.г.Эрвье' => %w(ю.р.г. эрвье),
+    'А.Невского' => %w(а. невского),
+    'им.Максима Горького' => %w(им. максима горького),
+    '50-летия Октября' => ['50-летия', 'октября'],
+    'героя советского союза Жукова' => ['героя советского союза', 'жукова']
+  }.each do |name, tokens|
+    it(name) { expect(described_class.split(name)).to eq(tokens) }
+  end
+end

data/spec/lib/name/synonyms_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+describe Fias::Name::Synonyms do
+  context '#expand' do
+    {
+      '2-я Советская улица' => [
+        %w(
+          2й 2-й 2я 2-я 2е 2-е 2ая 2-ая 2ий 2-ий 2ый 2-ый
+          2ой 2-ой 2ые 2-ые 2ое 2-ое 2го 2-го 2
+        ),
+        %w(советская),
+        %w(улица)],
+      '2-ой проспект им. 50 лет Николая Лавочкина героя соцтруда улица' => [
+        %w(
+          2й 2-й 2я 2-я 2е 2-е 2ая 2-ая 2ий 2-ий 2ый 2-ый
+          2ой 2-ой 2ые 2-ые 2ое 2-ое 2го 2-го 2
+        ),
+        ['проспект'],
+        ['им', 'имени', 'им.', ''],
+        ['50-летия', '50-лет', '50 летия', '50 лет', '50-летие', '50 летие'],
+        ['николая', ''],
+        ['лавочкина'],
+        ['героя', ''],
+        ['соцтруда', 'соц.труда', ''],
+        ['улица']
+      ],
+      'ул. И.А.Покрышкина' => [
+        ['ул.'], ['и.а.', ''], ['покрышкина']
+      ]
+    }.each do |name, synonyms|
+      it(name) do
+        expect(described_class.expand(name)).to eq(synonyms)
+      end
+    end
+  end
+
+  it '#forms' do
+    expect(described_class.forms('им. И.П.Павлова')).to eq(
+      [
+        'и.п. им павлова',
+        'им павлова',
+        'и.п. имени павлова',
+        'имени павлова',
+        'и.п. им. павлова',
+        'им. павлова',
+        'и.п. павлова',
+        'павлова'
+      ]
+    )
+  end
+end

data/spec/lib/query/params_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe Fias::Query::Params do
+  context '#initialize' do
+    YAML.load_file('spec/fixtures/query_sanitization.yml').each do |pair|
+      from, to = pair
+      from.symbolize_keys!
+      to.symbolize_keys!
+
+      it from.values.to_s do
+        expect(described_class.new(from).sanitized).to eq(to)
+      end
+    end
+  end
+end

data/spec/lib/query_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe Fias::Query do
+  context '#perform' do
+    context 'by default' do
+      subject { UndefinedQuery.new(city: 'Санкт-Петербург') }
+
+      it 'raises error' do
+        expect { subject.perform }.to raise_error(NotImplementedError)
+      end
+    end
+
+    context 'with provided #find method' do
+      YAML.load_file('spec/fixtures/query.yml').each do |query, indexes|
+        query = query.symbolize_keys
+        it "looks up #{query.inspect}" do
+          result = TestQuery.new(query).perform
+          result = result.map(&:last).sort_by { |r| r[:id] }
+
+          expected = SHORTCUTS.values_at(*indexes).sort_by { |r| r[:id] }
+
+          expect(result.first(expected.size)).to eq(expected)
+        end
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,30 @@
+$LOAD_PATH << '.' unless $LOAD_PATH.include?('.')
+
+require 'rubygems'
+require 'bundler/setup'
+require 'simplecov'
+require 'webmock/rspec'
+require 'sequel'
+
+SimpleCov.start do
+  add_filter 'spec'
+end
+
+if ENV['CODECLIMATE_REPO_TOKEN']
+  require 'codeclimate-test-reporter'
+  CodeClimate::TestReporter.start
+end
+
+require 'fias'
+require 'spec/support/query'
+require 'spec/support/db'
+
+WebMock.disable_net_connect!(allow: %w(codeclimate.com))
+
+RSpec.configure do |config|
+  config.order = :random
+  config.run_all_when_everything_filtered = true
+  config.filter_run :focus
+end
+
+$LOAD_PATH << File.join(File.dirname(__FILE__), '..', 'lib')