Merge 275569a into 94cfa83

.rubocop_todo.yml

@@ -199,6 +199,7 @@ RSpec/ExpectInHook:
 RSpec/FilePath:
   Exclude:
     - 'spec/goobi_spec.rb'
+    - 'spec/goobi_tag_spec.rb'
     - 'spec/registration_response_spec.rb'
     - 'spec/service_item_spec.rb'
     - 'spec/update_marc_spec.rb'

app/models/dor/goobi_tag.rb

@@ -13,7 +13,7 @@ def initialize(p_hash)
     end
 
     def to_xml
-      "<tag name=\"#{name}\" value=\"#{value}\"></tag>"
+      "<tag name=\"#{name.encode(xml: :text)}\" value=\"#{value.encode(xml: :text)}\"></tag>"
     end
   end
 end

spec/goobi_spec.rb

@@ -22,8 +22,8 @@
   end
 
   it 'creates the correct xml request without ocr tag present' do
-    allow(item).to receive(:tags).and_return(['DPG : Workflow : book_workflow', 'Process : Content Type : Book', 'LAB : MAPS'])
-    expect(@goobi.goobi_xml_tags).to eq('<tag name="DPG" value="Workflow : book_workflow"></tag><tag name="Process" value="Content Type : Book"></tag><tag name="LAB" value="MAPS"></tag>')
+    allow(item).to receive(:tags).and_return(['DPG : Workflow : book_workflow & stuff', 'Process : Content Type : Book', 'LAB : MAPS'])
+    expect(@goobi.goobi_xml_tags).to eq('<tag name="DPG" value="Workflow : book_workflow &amp; stuff"></tag><tag name="Process" value="Content Type : Book"></tag><tag name="LAB" value="MAPS"></tag>')
     expect(@goobi.xml_request).to be_equivalent_to <<-END
       <stanfordCreationRequest>
         <objectId>#{pid}</objectId>
@@ -40,7 +40,7 @@
         <goobiWorkflow>goobi_workflow</goobiWorkflow>
         <ocr>false</ocr>
         <tags>
-            <tag name="DPG" value="Workflow : book_workflow"></tag>
+            <tag name="DPG" value="Workflow : book_workflow &amp; stuff"></tag>
             <tag name="Process" value="Content Type : Book"></tag>
             <tag name="LAB" value="MAPS"></tag>
         </tags>

spec/goobi_tag_spec.rb

@@ -0,0 +1,13 @@
+require 'rails_helper'
+
+RSpec.describe Dor::GoobiTag do
+  it 'encodes a tag as XML' do
+    goobi_tag = described_class.new(name: 'tag_name', value: 'tag value')
+    expect(goobi_tag.to_xml).to eq('<tag name="tag_name" value="tag value"></tag>')
+  end
+
+  it 'encodes a tag as XML with special characters' do
+    goobi_tag = described_class.new(name: 'tag_name > and odd characters', value: 'tag value & other things')
+    expect(goobi_tag.to_xml).to eq('<tag name="tag_name &gt; and odd characters" value="tag value &amp; other things"></tag>')
+  end
+end