Boy Scout: Create document parser spec file

Regroup DocumentParser spec. Now they're duplicates in
local_census_specs and census_api_spec.

Move duplicate private method "dni?(document_type) from
LocalCensus and CensusAPI to DocumentParser
This commit is contained in:
taitus
2019-04-16 16:43:35 +02:00
committed by Javi Martín
parent a382f7c301
commit 090390c09f
6 changed files with 35 additions and 56 deletions

View File

@@ -119,8 +119,4 @@ class CensusApi
{get_habita_datos_response: {get_habita_datos_return: {datos_habitante: {}, datos_vivienda: {}}}} {get_habita_datos_response: {get_habita_datos_return: {datos_habitante: {}, datos_vivienda: {}}}}
end end
def dni?(document_type)
document_type.to_s == "1"
end
end end

View File

@@ -57,4 +57,9 @@ module DocumentParser
end end
variants variants
end end
def dni?(document_type)
document_type.to_s == "1"
end
end end

View File

@@ -63,8 +63,4 @@ class LocalCensus
LocalCensusRecord.find_by(document_type: document_type, document_number: document_number) LocalCensusRecord.find_by(document_type: document_type, document_number: document_number)
end end
def dni?(document_type)
document_type.to_s == "1"
end
end end

View File

@@ -3,30 +3,6 @@ require "rails_helper"
describe CensusApi do describe CensusApi do
let(:api) { described_class.new } let(:api) { described_class.new }
describe "#get_document_number_variants" do
it "trims and cleans up entry" do
expect(api.get_document_number_variants(2, " 1 2@ 34")).to eq(["1234"])
end
it "returns only one try for passports & residence cards" do
expect(api.get_document_number_variants(2, "1234")).to eq(["1234"])
expect(api.get_document_number_variants(3, "1234")).to eq(["1234"])
end
it "takes only the last 8 digits for dnis and resicence cards" do
expect(api.get_document_number_variants(1, "543212345678")).to eq(["12345678"])
end
it "tries all the dni variants padding with zeroes" do
expect(api.get_document_number_variants(1, "0123456")).to eq(["123456", "0123456", "00123456"])
expect(api.get_document_number_variants(1, "00123456")).to eq(["123456", "0123456", "00123456"])
end
it "adds upper and lowercase letter when the letter is present" do
expect(api.get_document_number_variants(1, "1234567A")).to eq(%w(1234567 01234567 1234567a 1234567A 01234567a 01234567A))
end
end
describe "#call" do describe "#call" do
let(:invalid_body) { {get_habita_datos_response: {get_habita_datos_return: {datos_habitante: {}}}} } let(:invalid_body) { {get_habita_datos_response: {get_habita_datos_return: {datos_habitante: {}}}} }
let(:valid_body) do let(:valid_body) do

View File

@@ -0,0 +1,30 @@
require "rails_helper"
include DocumentParser
describe DocumentParser do
describe "#get_document_number_variants" do
it "trims and cleans up entry" do
expect(DocumentParser.get_document_number_variants(2, " 1 2@ 34")).to eq(["1234"])
end
it "returns only one try for passports & residence cards" do
expect(DocumentParser.get_document_number_variants(2, "1234")).to eq(["1234"])
expect(DocumentParser.get_document_number_variants(3, "1234")).to eq(["1234"])
end
it "takes only the last 8 digits for dnis and resicence cards" do
expect(DocumentParser.get_document_number_variants(1, "543212345678")).to eq(["12345678"])
end
it "tries all the dni variants padding with zeroes" do
expect(DocumentParser.get_document_number_variants(1, "0123456")).to eq(["123456", "0123456", "00123456"])
expect(DocumentParser.get_document_number_variants(1, "00123456")).to eq(["123456", "0123456", "00123456"])
end
it "adds upper and lowercase letter when the letter is present" do
expect(DocumentParser.get_document_number_variants(1, "1234567A")).to eq(%w(1234567 01234567 1234567a 1234567A 01234567a 01234567A))
end
end
end

View File

@@ -3,30 +3,6 @@ require "rails_helper"
describe LocalCensus do describe LocalCensus do
let(:api) { described_class.new } let(:api) { described_class.new }
describe "#get_document_number_variants" do
it "trims and cleans up entry" do
expect(api.get_document_number_variants(2, " 1 2@ 34")).to eq(["1234"])
end
it "returns only one try for passports & residence cards" do
expect(api.get_document_number_variants(2, "1234")).to eq(["1234"])
expect(api.get_document_number_variants(3, "1234")).to eq(["1234"])
end
it "takes only the last 8 digits for dnis and resicence cards" do
expect(api.get_document_number_variants(1, "543212345678")).to eq(["12345678"])
end
it "tries all the dni variants padding with zeroes" do
expect(api.get_document_number_variants(1, "0123456")).to eq(["123456", "0123456", "00123456"])
expect(api.get_document_number_variants(1, "00123456")).to eq(["123456", "0123456", "00123456"])
end
it "adds upper and lowercase letter when the letter is present" do
expect(api.get_document_number_variants(1, "1234567A")).to eq(["1234567", "01234567", "1234567a", "1234567A", "01234567a", "01234567A"])
end
end
describe "#call" do describe "#call" do
let(:invalid_body) { nil } let(:invalid_body) { nil }
let(:valid_body) { create(:local_census_record) } let(:valid_body) { create(:local_census_record) }