feat(smrtlink): adds custom validator for use_adaptive_loading smrt l…

…ink option

app/validators/smrt_link_options_validator.rb

@@ -22,7 +22,7 @@
 # - Or you need to create your own validator as per
 # https://api.rubyonrails.org/classes/ActiveModel/Validator.html
 class SmrtLinkOptionsValidator < ActiveModel::Validator
-  def validate(record)
+  def validate(record) # rubocop:disable Metrics/MethodLength
     # If the version is not present no point validating
     return if record&.run&.smrt_link_version.blank?
 
@@ -38,7 +38,15 @@ def validate(record)
         # @example
         #  key = required
         #  validator => ActiveModel::Validations::RequiredValidator
-        validator = "ActiveModel::Validations::#{key.capitalize}Validator".constantize
+
+        # Check if there is a custom validator first
+        validator_class_name = "#{key.camelize}Validator"
+        validator = if Object.const_defined?(validator_class_name)
+                      validator_class_name.constantize
+                    else
+                      # If no custom validator then use an active model one
+                      "ActiveModel::Validations::#{key.camelize}Validator".constantize
+                    end
 
         # We then need to create a new instance of the validator
         # and pass the options along with the attribute name which is the key
@@ -52,5 +60,5 @@ def validate(record)
         instance.validate(record)
       end
     end
-  end
+  end # rubocop:enable Metrics/MethodLength
 end

app/validators/use_adaptive_loading_validator.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+# Validator for ensuring use_adaptive_loading is consistent across all wells
+class UseAdaptiveLoadingValidator < ActiveModel::Validator
+  # @param [ActiveRecord::Base] record
+  def validate(record)
+    return if record.use_adaptive_loading == record.run.wells.first.use_adaptive_loading
+
+    record.errors.add(:base, "well #{record.position} has a differing 'Use Adaptive Loading' value")
+  end
+end

config/pacbio_smrt_link_versions.yml

@@ -285,6 +285,7 @@ default: &default
         presence: {}
         inclusion:
           in: *true_false
+        use_adaptive_loading: {}
       data_type: list
       select_options: *select_true_false
       versions:

spec/exchanges/run_csv/pacbio_sample_sheet_spec.rb

@@ -10,7 +10,6 @@
   before do
     # Create a default smrt link version
     create(:pacbio_smrt_link_version, name: 'v13_revio', default: true)
-    create(:pacbio_smrt_link_version, name: 'v25_1_revio')
   end
 
   describe '#payload' do

spec/validators/smrt_link_options_validator_spec.rb

@@ -63,7 +63,7 @@
       end
     end
 
-    it 'onlies mark the record as invalid for the correct version' do
+    it 'only marks the record as invalid for the correct version' do
       run = create(:pacbio_sequel_run, smrt_link_version: versions.first)
       well = build(:pacbio_well, plate: run.plates.first, movie_time: nil)
       described_class.new.validate(well)

spec/validators/use_adaptive_loading_validator_spec.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe UseAdaptiveLoadingValidator do
+  before do
+    create(:pacbio_smrt_link_version, name: 'v13_1_revio', default: true)
+  end
+
+  let(:run) { create(:pacbio_revio_run) }
+  let(:validator) { described_class.new(run:, exclude_marked_for_destruction: true) }
+
+  it 'is valid when all wells have the same use_adaptive_loading value' do
+    # Factory defaults to false for use_adaptive_loading for all wells
+    well = run.wells.first
+    validator.validate(well)
+    expect(well.errors).to be_empty
+  end
+
+  it 'is invalid when all wells do not have the same use_adaptive_loading value' do
+    well = run.wells.last
+    well.use_adaptive_loading = true
+
+    validator.validate(well)
+    expect(well.errors[:base]).to include("well #{run.wells.last.position} has a differing 'Use Adaptive Loading' value")
+  end
+end