minor changes for testing and linting

app/models/validations/sales/financial_validations.rb

@@ -41,7 +41,7 @@ module Validations::Sales::FinancialValidations
   end
 
   def validate_monthly_leasehold_charges(record)
-    record.errors.add :mscharge, I18n.t("validations.financial.monthly_leasehold_charges.not_zero") if record.mscharge == 0
+    record.errors.add :mscharge, I18n.t("validations.financial.monthly_leasehold_charges.not_zero") if record.mscharge&.zero?
   end
 
   def validate_percentage_bought_not_greater_than_percentage_owned(record)

spec/fixtures/imports/sales_logs/discounted_ownership_sales_log.xml

@@ -280,7 +280,7 @@
     <Q34b>33</Q34b>
     <Q35Borrowing>2 No</Q35Borrowing>
     <Q36CashDeposit>0</Q36CashDeposit>
-    <Q37MonthlyCharges>0.00</Q37MonthlyCharges>
+    <Q37MonthlyCharges>10.00</Q37MonthlyCharges>
   </Group>
   <Group>
     <Q40PurchasePrice override-field=""/>

spec/models/form/sales/questions/leasehold_charges_spec.rb

@@ -36,11 +36,11 @@ RSpec.describe Form::Sales::Questions::LeaseholdCharges, type: :model do
   end
 
   it "has the correct width" do
-    expect(question.width).to eq(5)
+    expect(question.width).to be 5
   end
 
   it "has the correct min" do
-    expect(question.min).to eq(0)
+    expect(question.min).to be 1
   end
 
   it "has the correct prefix" do