Update field name and required fields

app/services/imports/local_authorities_service.rb

@@ -13,7 +13,7 @@ module Imports
       CSV.foreach(path, headers: true) do |row|
         LocalAuthority.upsert(
           { code: row["code"],
-            la_name: row["la_name"],
+            name: row["name"],
             start_date: Time.zone.local(row["start_year"], 4, 1),
             end_date: (Time.zone.local(row["end_year"], 3, 31) if row["end_year"]),
             previous_location_only: row["previous_location_only"] || false },

config/local_authorities_data/initial_local_authorities.csv

@@ -1,4 +1,4 @@
-code,la_name,start_year,end_year,previous_location_only
+code,name,start_year,end_year,previous_location_only
 S12000033,Aberdeen City,2021,,true
 S12000034,Aberdeenshire,2021,,true
 E07000223,Adur,2021,,

db/migrate/20230308101826_create_local_authorities.rb

@@ -1,9 +1,9 @@
 class CreateLocalAuthorities < ActiveRecord::Migration[7.0]
   def change
     create_table :local_authorities do |t|
-      t.string :code
+      t.string :code, null: false
-      t.string :la_name
+      t.string :name, null: false
-      t.datetime :start_date
+      t.datetime :start_date, null: false
       t.datetime :end_date
       t.boolean :previous_location_only, default: false
       t.index %w[code], name: "index_local_authority_code", unique: true

db/schema.rb

@@ -290,9 +290,9 @@ ActiveRecord::Schema[7.0].define(version: 2023_03_08_101826) do
   end
 
   create_table "local_authorities", force: :cascade do |t|
-    t.string "code"
+    t.string "code", null: false
-    t.string "la_name"
+    t.string "name", null: false
-    t.datetime "start_date"
+    t.datetime "start_date", null: false
     t.datetime "end_date"
     t.boolean "previous_location_only", default: false
     t.datetime "created_at", null: false

spec/fixtures/files/local_authorities.csv

@@ -1,4 +1,4 @@
-code,la_name,start_year,end_year,previous_location_only
+code,name,start_year,end_year,previous_location_only
 S12000033,Aberdeen City,2021,,true
 S12000034,Aberdeenshire,2021,,true
 E07000223,Adur,2021,,

spec/lib/tasks/local_authorities_import_spec.rb

@@ -29,7 +29,7 @@ RSpec.describe "data_import" do
       end
 
       context "when a record already exists with a matching code index" do
-        let!(:local_authority) { LocalAuthority.create(code: "S12000041", la_name: "Angus", start_date: Time.zone.local(2021, 4, 1), previous_location_only: false) }
+        let!(:local_authority) { LocalAuthority.create(code: "S12000041", name: "Angus", start_date: Time.zone.local(2021, 4, 1), previous_location_only: false) }
 
         it "updates local authority if the record is matched on code" do
           task.invoke(local_authorities_file_path)