|
|
|
@ -871,6 +871,32 @@ RSpec.describe Imports::SalesLogsImportService do |
|
|
|
end |
|
|
|
end |
|
|
|
end |
|
|
|
end |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
context "and it has an invalid mscharge" do |
|
|
|
|
|
|
|
let(:sales_log_id) { "shared_ownership_sales_log" } |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
before do |
|
|
|
|
|
|
|
sales_log_xml.at_xpath("//xmlns:Q29MonthlyCharges").content = "0.1" |
|
|
|
|
|
|
|
end |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
it "intercepts the relevant validation error" do |
|
|
|
|
|
|
|
expect(logger).to receive(:warn).with(/Removing mscharge with error: Monthly leasehold charges must be at least £1/) |
|
|
|
|
|
|
|
expect(logger).to receive(:warn).with(/Removing has_mscharge with error: Monthly leasehold charges must be at least £1/) |
|
|
|
|
|
|
|
expect { sales_log_service.send(:create_log, sales_log_xml) } |
|
|
|
|
|
|
|
.not_to raise_error |
|
|
|
|
|
|
|
end |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
it "clears out the invalid answers" do |
|
|
|
|
|
|
|
allow(logger).to receive(:warn) |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
sales_log_service.send(:create_log, sales_log_xml) |
|
|
|
|
|
|
|
sales_log = SalesLog.find_by(old_id: sales_log_id) |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
expect(sales_log).not_to be_nil |
|
|
|
|
|
|
|
expect(sales_log.mscharge).to be_nil |
|
|
|
|
|
|
|
expect(sales_log.has_mscharge).to be_nil |
|
|
|
|
|
|
|
end |
|
|
|
|
|
|
|
end |
|
|
|
|
|
|
|
|
|
|
|
context "when inferring default answers for completed sales logs" do |
|
|
|
context "when inferring default answers for completed sales logs" do |
|
|
|
context "when the armedforcesspouse is not answered" do |
|
|
|
context "when the armedforcesspouse is not answered" do |
|
|
|
let(:sales_log_id) { "discounted_ownership_sales_log" } |
|
|
|
let(:sales_log_id) { "discounted_ownership_sales_log" } |
|
|
|
|