Merge branch 'feature-tags' into 'master'

Feature tags




See merge request !241

app/services/institution_importer.rb

@@ -16,8 +16,11 @@ class InstitutionImporter
         institution = Institution.new(name: publisher)
         institution.save
         @names << publisher
+      else
+        institution = Institution.find_by_name(publisher)
       end
 
+
       learning_object.publisher = institution
       learning_object.save
     end

test/fixtures/learning_objects.yml

@@ -44,4 +44,30 @@ institution_lo:
   name: 'Institution Object 1'
   description: 'Testing'
   author: 'Mauricio'
-  publisher: ufpr (Institution)
\ No newline at end of file
+  publisher: ufpr (Institution)
+
+lo_complete:
+  name: 'Institution Object 1'
+  description: 'Testing'
+  author: 'Mauricio'
+  id_dspace: '1'
+  object_type: image
+  language: portuguese
+
+lo_metadata:
+  name: 'Institution Object 1'
+  description: 'Testing'
+  author: 'Mauricio'
+  id_dspace: '2'
+  object_type: image
+  language: portuguese
+  metadata:  '[{"key": "dc.creator", "value": "ufpr", "language": "nil"}]'
+
+lo_metadata2:
+  name: 'Institution Object 2'
+  description: 'Testing'
+  author: 'Mauricio'
+  id_dspace: '3'
+  object_type: image
+  language: portuguese
+  metadata:  '[{"key": "dc.creator", "value": "ufpr", "language": "nil"}]'

test/services/institution_importer_test.rb

+require 'test_helper'
+
+class InstitutionImporterTest < ActiveSupport::TestCase
+
+  test 'importing learning object with metadata publisher = nil' do
+    importer = InstitutionImporter.new()
+    lo = learning_objects(:lo_complete)
+    importer.items = [lo]
+    importer.import
+    assert_not_nil Institution.find_by_name(lo.publisher.name)
+  end
+
+  test 'importing learning object with metadata publisher != nil' do
+    importer = InstitutionImporter.new()
+    lo = learning_objects(:lo_metadata)
+    importer.items = [lo]
+    importer.import
+    assert_not_nil Institution.find_by_name(lo.publisher.name)
+  end
+
+  test 'importing learning object with institution that already exists' do
+    importer = InstitutionImporter.new()
+    lo1 = learning_objects(:lo_metadata)
+    lo2 = learning_objects(:lo_metadata2)
+    importer.items = [lo1, lo2]
+    importer.import
+    assert_equal Institution.find_by_name(lo1.publisher.name), Institution.find_by_name(lo2.publisher.name)
+  end
+
+end