iiif-presentation 1.4.2 → 1.5.0

data/spec/unit/iiif/v3/presentation/annotation_collection_spec.rb

@@ -1,36 +1,33 @@
 describe IIIF::V3::Presentation::AnnotationCollection do
-
   let(:fixed_values) do
     {
-      '@context' => [
-        'http://iiif.io/api/presentation/3/context.json',
-        'http://www.w3.org/ns/anno.jsonld'
+      "@context" => [
+        "http://iiif.io/api/presentation/3/context.json",
+        "http://www.w3.org/ns/anno.jsonld"
       ],
-      'id' => 'http://www.example.org/iiif/book1/annoColl/transcription',
-      'type' => 'AnnotationCollection',
-      'label' => 'Diplomatic Transcription',
-      'first' => 'http://www.example.org/iiif/book1/list/l1',
-      'last' => 'http://www.example.org/iiif/book1/list/l4',
+      "id" => "http://www.example.org/iiif/book1/annoColl/transcription",
+      "type" => "AnnotationCollection",
+      "label" => "Diplomatic Transcription",
+      "first" => "http://www.example.org/iiif/book1/list/l1",
+      "last" => "http://www.example.org/iiif/book1/list/l4"
     }
   end
 
-
-  describe '#initialize' do
-    it 'sets type' do
-      expect(subject['type']).to eq 'AnnotationCollection'
+  describe "#initialize" do
+    it "sets type" do
+      expect(subject["type"]).to eq "AnnotationCollection"
     end
   end
 
   describe "#{described_class}.define_methods_for_string_only_keys" do
-    it_behaves_like 'it has the appropriate methods for string-only keys v3'
+    it_behaves_like "it has the appropriate methods for string-only keys v3"
   end
 
   describe "#{described_class}.define_methods_for_array_only_keys" do
-    it_behaves_like 'it has the appropriate methods for array-only keys v3'
+    it_behaves_like "it has the appropriate methods for array-only keys v3"
   end
 
   describe "#{described_class}.define_methods_for_any_type_keys" do
-    it_behaves_like 'it has the appropriate methods for any-type keys v3'
+    it_behaves_like "it has the appropriate methods for any-type keys v3"
   end
-
 end

data/spec/unit/iiif/v3/presentation/annotation_page_spec.rb

@@ -1,129 +1,131 @@
 describe IIIF::V3::Presentation::AnnotationPage do
-
-  describe '#required_keys' do
-    it 'id' do
-      expect(subject.required_keys).to include('id')
+  describe "#required_keys" do
+    it "id" do
+      expect(subject.required_keys).to include("id")
     end
   end
 
-  describe '#prohibited_keys' do
-    it 'contains the expected key names' do
+  describe "#prohibited_keys" do
+    it "contains the expected key names" do
       keys = described_class::CONTENT_RESOURCE_PROPERTIES +
-       %w{
-         first
-         last
-         total
-         nav_date
-         viewing_direction
-         start_canvas
-         content_annotations
-       }
-     expect(subject.prohibited_keys).to include(*keys)
-   end
+        %w[
+          first
+          last
+          total
+          nav_date
+          viewing_direction
+          start_canvas
+          content_annotations
+        ]
+      expect(subject.prohibited_keys).to include(*keys)
+    end
   end
 
-  describe '#uri_only_keys' do
-    it 'id' do
-      expect(subject.uri_only_keys).to include('id')
+  describe "#uri_only_keys" do
+    it "id" do
+      expect(subject.uri_only_keys).to include("id")
     end
   end
 
-  describe '#array_only_keys' do
-    it 'items' do
-      expect(subject.array_only_keys).to include('items')
+  describe "#array_only_keys" do
+    it "items" do
+      expect(subject.array_only_keys).to include("items")
     end
   end
 
-  describe '#legal_viewing_hint_values' do
-    it 'contains none' do
-      expect(subject.legal_viewing_hint_values).to contain_exactly('none')
+  describe "#legal_viewing_hint_values" do
+    it "contains none" do
+      expect(subject.legal_viewing_hint_values).to contain_exactly("none")
     end
   end
 
-  describe '#initialize' do
-    it 'sets type to AnnotationPage by default' do
-      expect(subject['type']).to eq 'AnnotationPage'
+  describe "#initialize" do
+    it "sets type to AnnotationPage by default" do
+      expect(subject["type"]).to eq "AnnotationPage"
     end
-    it 'allows subclasses to override type' do
+    it "allows subclasses to override type" do
       subclass = Class.new(described_class) do
-        def initialize(hsh={})
-          hsh = { 'type' => 'a:SubClass' }
-          super(hsh)
+        def initialize(_hsh = {})
+          super({"type" => "a:SubClass"})
         end
       end
       sub = subclass.new
-      expect(sub['type']).to eq 'a:SubClass'
+      expect(sub["type"]).to eq "a:SubClass"
     end
-    it 'allows type to be passed in' do
-      ap = described_class.new('type' => 'bar')
-      expect(ap.type).to eq 'bar'
+    it "allows type to be passed in" do
+      ap = described_class.new("type" => "bar")
+      expect(ap.type).to eq "bar"
     end
   end
 
-  describe '#validate' do
-    it 'raises IllegalValueError if id is not URI' do
+  describe "#validate" do
+    it "raises IllegalValueError if id is not URI" do
       exp_err_msg = "id value must be a String containing a URI for #{described_class}"
-      subject['id'] = 'foo'
+      subject["id"] = "foo"
       expect { subject.validate }.to raise_error(IIIF::V3::Presentation::IllegalValueError, exp_err_msg)
     end
-    it 'raises IllegalValueError if id is not http(s)' do
-      subject['id'] = 'ftp://www.example.org'
+    it "raises IllegalValueError if id is not http(s)" do
+      subject["id"] = "ftp://www.example.org"
       exp_err_msg = "id must be an http(s) URI for #{described_class}"
       expect { subject.validate }.to raise_error(IIIF::V3::Presentation::IllegalValueError, exp_err_msg)
     end
-    it 'raises IllegalValueError for items entry that is not an Annotation' do
-      subject['id'] = 'http://example.com/iiif3/annotation_page/666'
-      subject['items'] = [IIIF::V3::Presentation::ImageResource.new, IIIF::V3::Presentation::Annotation.new]
+    it "raises IllegalValueError for items entry that is not an Annotation" do
+      subject["id"] = "http://example.com/iiif3/annotation_page/666"
+      subject["items"] = [IIIF::V3::Presentation::ImageResource.new, IIIF::V3::Presentation::Annotation.new]
       exp_err_msg = "All entries in the items list must be a IIIF::V3::Presentation::Annotation"
       expect { subject.validate }.to raise_error(IIIF::V3::Presentation::IllegalValueError, exp_err_msg)
     end
   end
 
-  describe 'realistic examples' do
-    let(:ap_id) { 'http://example.com/iiif3/annotation_page/666' }
-    let(:anno) { IIIF::V3::Presentation::Annotation.new(
-      'id' => 'http://example.com/anno/666',
-      'target' => 'http://example.com/canvas/abc'
-      )}
+  describe "realistic examples" do
+    let(:ap_id) { "http://example.com/iiif3/annotation_page/666" }
+    let(:anno) {
+      IIIF::V3::Presentation::Annotation.new(
+        "id" => "http://example.com/anno/666",
+        "target" => "http://example.com/canvas/abc"
+      )
+    }
 
-    describe 'stanford (purl code)' do
+    describe "stanford (purl code)" do
       let(:anno_page) {
         anno_page = described_class.new
-        anno_page['id'] = ap_id
+        anno_page["id"] = ap_id
         anno_page.items << anno
         anno_page
       }
-      it 'validates' do
-        expect{anno_page.validate}.not_to raise_error
+      it "validates" do
+        expect { anno_page.validate }.not_to raise_error
       end
-      it 'has expected required values' do
+      it "has expected required values" do
         expect(anno_page.id).to eq ap_id
-        expect(anno_page['type']).to eq 'AnnotationPage'
+        expect(anno_page["type"]).to eq "AnnotationPage"
       end
-      it 'has expected additional values' do
+      it "has expected additional values" do
         expect(anno_page.items).to eq [anno]
       end
     end
 
-    describe 'two items' do
-      let(:anno2) { IIIF::V3::Presentation::Annotation.new(
-        'id' => 'http://example.com/anno/333',
-        'target' => 'http://example.com/canvas/abc'
-        )}
+    describe "two items" do
+      let(:anno2) {
+        IIIF::V3::Presentation::Annotation.new(
+          "id" => "http://example.com/anno/333",
+          "target" => "http://example.com/canvas/abc"
+        )
+      }
       let(:anno_page) {
         anno_page = described_class.new
-        anno_page['id'] = ap_id
+        anno_page["id"] = ap_id
         anno_page.items = [anno, anno2]
         anno_page
       }
-      it 'validates' do
-        expect{anno_page.validate}.not_to raise_error
+      it "validates" do
+        expect { anno_page.validate }.not_to raise_error
       end
-      it 'has expected required values' do
+      it "has expected required values" do
         expect(anno_page.id).to eq ap_id
-        expect(anno_page['type']).to eq 'AnnotationPage'
+        expect(anno_page["type"]).to eq "AnnotationPage"
       end
-      it 'has expected additional values' do
+      it "has expected additional values" do
         expect(anno_page.items).to eq [anno, anno2]
       end
     end