Revert CSV ingestor changes and fix test

Author: fbacall

lib/ingestors/material_csv_ingestor.rb

@@ -42,7 +42,7 @@ def read(url)
           material.date_published = get_column row, 'Published'
           material.date_modified = get_column row, 'Modified'
           material.difficulty_level = process_competency row, 'Competency'
-          material.person_links_attributes = process_authors(process_array(row, 'Authors'))
+          material.authors = process_array row, 'Authors'
           material.contributors = process_array row, 'Contributors'
           material.fields = process_array row, 'Fields'
           material.target_audience = process_array row, 'Audiences'
@@ -65,21 +65,6 @@ def read(url)
 
     private
 
-    def process_authors(authors_array)
-      return [] if authors_array.blank?
-
-      authors_array.map do |author_name|
-        # Store the full name directly without trying to parse it
-        {
-          role: 'author',
-          person_attributes: {
-            full_name: author_name.to_s.strip,
-            orcid: nil
-          }
-        }
-      end
-    end
-
     def process_competency(row, header)
       row[header].nil? ? 'notspecified' : row[header]
     end

test/unit/ingestors/material_csv_ingestor_test.rb

@@ -94,8 +94,9 @@ def run
     # contributors
     refute material.contributors.nil?, 'material contributors is nil'
     assert_equal 1, material.contributors.size, 'material contributors count not matched.'
-    assert material.contributors.include?('Sam Smiths'), 'material contributor[Sam Smiths] missing.'
-    refute material.contributors.include?('Wily Coyote'), 'material contributor[Wily Coyote] exists.'
+    contributor_names = material.contributors.map(&:display_name)
+    assert contributor_names.include?('Sam Smiths'), 'material contributor[Sam Smiths] missing.'
+    refute contributor_names.include?('Wily Coyote'), 'material contributor[Wily Coyote] exists.'
     # fields
     refute material.fields.nil?, 'material fields is nil'
     refute_empty material.fields, 'material fields is empty'
@@ -195,8 +196,9 @@ def run
     assert_equal 2, updated.keywords.size, 'Updated keywords count not matched!'
     assert updated.keywords.include?('book'), 'Updated keywords missing value!'
     assert_equal 2, updated.authors.size, "Updated authors count not matched! ... #{updated.authors.inspect}"
-    assert updated.authors.include?('Steven Smith'), 'Updated authors[Steven Smith] missing!'
-    assert updated.authors.include?('Sam Harpic'), 'Updated authors[Sam Harpic] missing!'
+    author_names = updated.authors.map(&:display_name)
+    assert author_names.include?('Steven Smith'), 'Updated authors[Steven Smith] missing!'
+    assert author_names.include?('Sam Harpic'), 'Updated authors[Sam Harpic] missing!'
     assert_equal 0, updated.contributors.size, 'Updated contributors count not matched!'
 
     # check locked fields not updated