Issue #15 working forc csv files with empty lines

app/importers/csv_importer.rb

@@ -6,8 +6,21 @@ def initialize(file)
   end
   def import
     CSV.foreach(@file) do |row|
+      next if row.empty?
       image = Image.new
       image.title = [row[1]]
+      image.source = [row[2]]
+      image.visibility = "open"
+      image.depositor = @user.email
+      # Attach the image file and run it through the actor stack
+      # Try entering Hyrax::CurationConcern.actor on a console to see all of the
+      # actors this object will run through.
+      image_binary = File.open("#{::Rails.root}/spec/fixtures/images/#{row[0]}")
+      uploaded_file = Hyrax::UploadedFile.create(user: @user, file: image_binary)
+      attributes_for_actor = { uploaded_files: [uploaded_file.id] }
+      env = Hyrax::Actors::Environment.new(image, ::Ability.new(@user), attributes_for_actor)
+      Hyrax::CurationConcern.actor.create(env)
+      image_binary.close
     end
   end
 end

config/environments/development.rb

@@ -5,7 +5,7 @@
   # every request. This slows down response time but is perfect for development
   # since you don't have to restart the web server when you make code changes.
   config.cache_classes = false
-
+  config.active_job.queue_adapter = :inline
   # Do not eager load code on boot.
   config.eager_load = false
 

config/environments/test.rb

@@ -6,7 +6,7 @@
   # your test database is "scratch space" for the test suite and is wiped
   # and recreated between test runs. Don't rely on the data there!
   config.cache_classes = true
-
+  config.active_job.queue_adapter     = :inline
   # Do not eager load code on boot. This avoids loading your whole application
   # just for the purpose of running a single test. If you are using a tool that
   # preloads Rails for running tests, you may have to set it to true.

lib/tasks/csv_import_task.rake

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+ CSV_FILE = "#{::Rails.root}/spec/fixtures/three_line_example.csv"
+ namespace :sample do
+  desc 'Import the three line sample CSV'
+  task :import_three_line_csv => [:environment] do |_task|
+    CsvImporter.new(CSV_FILE).import
+  end
+   desc 'Import a different CSV'
+  task :import, [:filename] => [:environment] do |_task, args|
+    csv_file = args[:filename]
+    CsvImporter.new(csv_file).import
+  end
+end

spec/importers/csv_importer_spec.rb

@@ -9,12 +9,28 @@
     DatabaseCleaner.clean
     ActiveFedora::Cleaner.clean!
   end
-
+  it "imports a csv" do
+    expect { CsvImporter.new(three_line_example).import }.to change { Image.count }.by 3
+  end
   it "puts the title into the title field" do
     CsvImporter.new(one_line_example).import
     imported_image = Image.first
     expect(imported_image.title.first).to eq 'A Cute Dog'
   end
-
-
-end
+  it "puts the url into the source field" do
+    CsvImporter.new(one_line_example).import
+    imported_image = Image.first
+    expect(imported_image.source.first).to eq 'https://www.pexels.com/photo/animal-blur-canine-close-up-551628/'
+  end
+  it "creates publicly visible objects" do
+    CsvImporter.new(one_line_example).import
+    imported_image = Image.first
+    expect(imported_image.visibility).to eq 'open'
+  end
+  it "attaches a file" do
+    CsvImporter.new(one_line_example).import
+    imported_image = Image.first
+    expect(imported_image.members.first).to be_instance_of(FileSet)
+    expect(imported_image.members.first.title.first).to eq 'dog.jpg'
+  end
+end