feat: respond to PR comments

app/models/validations/sales/financial_validations.rb

@@ -90,7 +90,8 @@ module Validations::Sales::FinancialValidations
 
     ranges = EQUITY_RANGES_BY_YEAR.fetch(record.collection_start_year, DEFAULT_EQUITY_RANGES)
 
-    if (range = ranges[record.type])
+    return unless (range = ranges[record.type])
+
     if record.equity < range.min
       record.errors.add :type, I18n.t("validations.financial.equity.under_min", min_equity: range.min)
       record.errors.add :equity, I18n.t("validations.financial.equity.under_min", min_equity: range.min)
@@ -99,7 +100,6 @@ module Validations::Sales::FinancialValidations
       record.errors.add :equity, I18n.t("validations.financial.equity.over_max", max_equity: range.max)
     end
   end
-  end
 
 private
 

spec/models/validations/sales/financial_validations_spec.rb

@@ -270,11 +270,10 @@ RSpec.describe Validations::Sales::FinancialValidations do
   describe "#validate_equity_in_range_for_year_and_type" do
     let(:record) { FactoryBot.create(:sales_log, saledate: now) }
 
-    before do
+    around do |example|
-      Timecop.freeze(now)
+      Timecop.freeze(now) do
+        example.run
       end
-
-    after do
       Timecop.unfreeze
     end