spec/lib/onebox/engine/pdf_onebox_spec.rb in onebox-1.8.17 vs spec/lib/onebox/engine/pdf_onebox_spec.rb in onebox-1.8.18

@@ -2,19 +2,27 @@
 
 describe Onebox::Engine::PdfOnebox do
   let(:link) { "https://acrobatusers.com/assets/uploads/public_downloads/2217/adobe-acrobat-xi-merge-pdf-files-tutorial-ue.pdf" }
   let(:html) { described_class.new(link).to_html }
 
+  let(:no_content_length_link) { "https://dspace.lboro.ac.uk/dspace-jspui/bitstream/2134/14294/3/greiffenhagen-ca_and_consumption.pdf" }
+  let(:no_filesize_html) { described_class.new(no_content_length_link).to_html }
+
   before do
     FakeWeb.register_uri(:head, link, :content_length => "335562")
+    FakeWeb.register_uri(:head, no_content_length_link, :content_length => nil)
   end
 
   describe "#to_html" do
     it "includes filename" do
       expect(html).to include("adobe-acrobat-xi-merge-pdf-files-tutorial-ue.pdf")
     end
 
     it "includes filesize" do
       expect(html).to include("327.70 KB")
+    end
+
+    it "doesn’t include filesize when unknown" do
+      expect(no_filesize_html).to_not include("<p class='filesize'>")
     end
   end
 end