Merge pull request #25 from dxw/289-code-reference-against-suppliers

Add coda_reference to the Supplier model

Author: tekin

app/models/supplier.rb

@@ -5,4 +5,8 @@ class Supplier < ApplicationRecord
   has_many :memberships, dependent: :destroy
 
   validates :name, presence: true
+  validates :coda_reference, allow_nil: true, format: {
+    with: /\AC0\d{5}\z/,
+    message: 'must start with “C0” and have five additional numbers, for example: “C012345”'
+  }
 end

db/data_migrate/20180807084912_set_coda_reference_for_suppliers.rb

@@ -0,0 +1,33 @@
+
+NAME_TO_CODA_REFERENCE_MAPPING = {
+  'Addleshaw Goddard LLP' => 'C000976',
+  'Ashurst LLP' => 'C005187',
+  'DLA Piper LLP' => 'C002107',
+  'Eversheds Sutherland (International) LLP' => 'C000941',
+  'Linklaters LLP' => 'C002112',
+  'Norton Rose Fulbright LLP' => 'C002119',
+  'Osborne Clarke LLP' => 'C009841',
+  'Stephenson Harwood' => 'C005196',
+  'Bond Dickinson LLP' => 'C000940',
+  'Burges Salmon LLP' => 'C000977',
+  'DAC Beachcroft LLP' => 'C009492',
+  'Dentons UK MEA LLP' => 'C000851',
+  'Gowling WLG (UK) LLP' => 'C000855',
+  'Mills & Reeve LLP' => 'C002031',
+  'Pinsent Mason' => 'C000853',
+  'PricewaterhouseCoopers LLP' => 'C000568',
+  'TLT LLP' => 'C009493',
+  'Bevan Brittan LLP' => 'C000850',
+  'Browne Jacobson LLP' => 'C005191',
+  'Field Fisher Waterhouse' => 'C000911',
+  'Hogan Lovells International LLP' => 'C002113',
+  'Simmons & Simmons LLP' => 'C000917',
+  'Slaughter & May' => 'C002123',
+  'BRYAN CAVE LEIGHTON PAISNER LLP' => 'C002101',
+  'Clifford Chance LLP' => 'C009889',
+  'Freshfields Bruckhaus Deringer' => 'C002108',
+}.freeze
+
+NAME_TO_CODA_REFERENCE_MAPPING.each_pair do |name, coda_reference|
+  Supplier.find_by!(name: name).update!(coda_reference: coda_reference)
+end

db/migrate/20180806154052_add_coda_reference_to_suppliers.rb

@@ -0,0 +1,5 @@
+class AddCodaReferenceToSuppliers < ActiveRecord::Migration[5.2]
+  def change
+    add_column :suppliers, :coda_reference, :string, limit: 7
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2018_07_27_170406) do
+ActiveRecord::Schema.define(version: 2018_08_06_154052) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "pgcrypto"
@@ -102,6 +102,7 @@
 
   create_table "suppliers", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|
     t.string "name", null: false
+    t.string "coda_reference", limit: 7
   end
 
   create_table "tasks", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|

spec/models/supplier_spec.rb

@@ -7,4 +7,19 @@
   it { is_expected.to have_many(:agreements) }
   it { is_expected.to have_many(:frameworks).through(:agreements) }
   it { is_expected.to have_many(:memberships) }
+
+  it 'validates coda_reference begins with C0 and ends with 5 more digits' do
+    valid_coda_references = %w[C012345 C002928 C099999]
+    invalid_coda_references = %w[C012 D012345 C0AB123]
+
+    valid_coda_references.each do |coda_reference|
+      expect(FactoryBot.create(:supplier, coda_reference: coda_reference)).to be_valid
+    end
+
+    invalid_coda_references.each do |coda_reference|
+      supplier = FactoryBot.build(:supplier, coda_reference: coda_reference)
+      expect(supplier).not_to be_valid
+      expect(supplier.errors[:coda_reference]).to be_present
+    end
+  end
 end