(feature) Raise exception if given hash is missing required keys

We want to ensure that the data import runs successfully, with no
unexpected side effects, so this checks that the input hash includes the
fields we use during the import process.

Author: leeky

lib/supplier_import_row.rb

@@ -1,6 +1,10 @@
 class SupplierImportRow
+  REQUIRED_KEYS = %i[frameworkreference suppliername].freeze
+
   def initialize(row)
     @row = row
+
+    check_for_required_keys!
   end
 
   def import!
@@ -9,4 +13,12 @@ def import!
 
     supplier.agreements.find_or_create_by!(framework: framework)
   end
+
+  private
+
+  def check_for_required_keys!
+    raise MissingKey unless (REQUIRED_KEYS - @row.keys).empty?
+  end
+
+  class MissingKey < StandardError; end
 end

spec/lib/supplier_import_row_spec.rb

@@ -20,6 +20,16 @@
     }
   end
 
+  describe '#initialize' do
+    it 'throws an error if the hash is missing required fields' do
+      data = {
+        frameworkreference: 'RM3756'
+      }
+
+      expect { SupplierImportRow.new(data) }.to raise_error('SupplierImportRow::MissingKey')
+    end
+  end
+
   describe '#import!' do
     it 'creates a new supplier and adds it to the framework' do
       framework = FactoryBot.create(:framework, short_name: 'RM3756')