Merge pull request #461 from nla/feat/blac-673_bibref-structure

fix: handle case where bibref is a Hash

app/presenters/rendering/bibref.rb

@@ -20,7 +20,11 @@ def render_bibref
           # Sometimes the value is a single string instead of an array of strings
           Array.wrap(values).each do |value|
             doc.li do |list_item|
-              list_item << value
+              list_item << if value.is_a?(Hash)
+                value["content"]
+              else
+                value
+              end
             end
           end
         }

spec/files/notes/bibref_content.json

@@ -0,0 +1,6 @@
+{
+  "bibref": {
+    "persname": "Sexton, Christopher",
+    "content": "<span class='persname'>Sexton, Christopher</span>. <a href=\"http://nla.gov.au/nla.cat-vn1619352\"><em>Peggy van Praagh, a life of dance</em>.</a> South Melbourne : Macmillan, 1985."
+  }
+}

spec/presenters/rendering/bibref_spec.rb

@@ -43,5 +43,20 @@
         expect(rendered).to have_css("em", text: "Who's who in Australia")
       end
     end
+
+    context "when the bibref contains a hash" do
+      let(:note) { JSON.parse(IO.read("spec/files/notes/bibref_content.json")) }
+      let(:values) { note["bibref"] }
+
+      it "renders the bibref" do
+        expect(rendered).to have_css("h4", text: "Bibliographic Reference(s)")
+        expect(rendered).to have_css("ul.list-unstyled")
+        expect(rendered).to have_css("li", count: 1)
+      end
+
+      it "inserts the existing HTML" do
+        expect(rendered).to have_css("span.persname", text: "Sexton, Christopher")
+      end
+    end
   end
 end