Merge pull request #890 from pulibrary/887-keywords-case

Downcase keywords at save time

Gemfile.lock

@@ -307,7 +307,9 @@ GEM
     marcel (0.3.3)
       mimemagic (~> 0.3.2)
     method_source (0.8.2)
-    mimemagic (0.3.5)
+    mimemagic (0.3.10)
+      nokogiri (~> 1)
+      rake
     mini_magick (4.11.0)
     mini_mime (1.0.2)
     mini_portile2 (2.4.0)
@@ -666,4 +668,4 @@ DEPENDENCIES
   webpacker (>= 4.0.x)
 
 BUNDLED WITH
-   2.1.4
+   2.2.6

app/values/manifest_metadata.rb

@@ -75,6 +75,7 @@ def transform_value(value)
       return value["@id"] if value["@id"]
       return link_to_catalog(value) if key == "Link to catalog" || key == "Link to finding aid"
       return actors(value) if key == "Actor"
+      return value.downcase if key.casecmp("keywords").zero?
 
       value
     end

spec/values/manifest_metadata_spec.rb

@@ -23,6 +23,15 @@
       }
     ] }
   end
+  let(:keywords_value) do
+    {
+      "keywords" => [
+        "Christianity",
+        "people's movement",
+        "liberation theology"
+      ]
+    }
+  end
 
   describe "#process_values" do
     it "doesn't modify a plain value" do
@@ -46,5 +55,8 @@
     it "uses @id as a last resort" do
       expect(metadata.process_values(id_value)).to eq("Title" => ["1234"])
     end
+    it "downcases keywords for consistent storage" do
+      expect(metadata.process_values(keywords_value)).to eq("keywords" => ["christianity", "people's movement", "liberation theology"])
+    end
   end
 end