iiif-presentation 1.4.1 → 1.5.0

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

@@ -1,88 +1,87 @@
 describe IIIF::V3::Presentation::Service do
-
-  describe '#required_keys' do
+  describe "#required_keys" do
     it '"type" is not required' do
-      expect(subject.required_keys).not_to include('type')
+      expect(subject.required_keys).not_to include("type")
     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 +
         described_class::PAGING_PROPERTIES +
-        %w{
+        %w[
           nav_date
           viewing_direction
           start_canvas
           content_annotations
-        }
+        ]
       expect(subject.prohibited_keys).to include(*keys)
     end
   end
 
-  describe '#uri_only_keys' do
-    it '@context' do
-      expect(subject.uri_only_keys).to include('@context')
+  describe "#uri_only_keys" do
+    it "@context" do
+      expect(subject.uri_only_keys).to include("@context")
     end
-    it '@id' do
-      expect(subject.uri_only_keys).to include('@id')
+    it "@id" do
+      expect(subject.uri_only_keys).to include("@id")
     end
-    it 'id' do
-      expect(subject.uri_only_keys).to include('id')
+    it "id" do
+      expect(subject.uri_only_keys).to include("id")
     end
   end
 
-  describe '#any_type_keys' do
-    it 'profile' do
-      expect(subject.any_type_keys).to include('profile')
+  describe "#any_type_keys" do
+    it "profile" do
+      expect(subject.any_type_keys).to include("profile")
     end
   end
 
   let(:id_uri) { "https://example.org/image1" }
 
-  describe '#initialize' do
-    it 'assigns hash values passed in' do
-      label_val = 'foo'
-      inner_service_id = 'http://example.org/whatever'
-      inner_service_profile = 'http://iiif.io/api/auth/1/token'
+  describe "#initialize" do
+    it "assigns hash values passed in" do
+      label_val = "foo"
+      inner_service_id = "http://example.org/whatever"
+      inner_service_profile = "http://iiif.io/api/auth/1/token"
       inner_service_val = described_class.new({
-        'id' => inner_service_id,
-        'profile' => inner_service_profile
-        })
+        "id" => inner_service_id,
+        "profile" => inner_service_profile
+      })
       service_obj = described_class.new({
-        'type' => described_class::IIIF_IMAGE_V2_TYPE,
-        'id' => id_uri,
-        'profile' => described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE,
-        'label' => label_val,
-        'service' => [inner_service_val]
+        "type" => described_class::IIIF_IMAGE_V2_TYPE,
+        "id" => id_uri,
+        "profile" => described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE,
+        "label" => label_val,
+        "service" => [inner_service_val]
       })
       expect(service_obj.keys.size).to eq 5
-      expect(service_obj['id']).to eq id_uri
-      expect(service_obj['type']).to eq described_class::IIIF_IMAGE_V2_TYPE
-      expect(service_obj['profile']).to eq described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
-      expect(service_obj['label']).to eq label_val
-      expect(service_obj['service'][0]['id']).to eq inner_service_id
-      expect(service_obj['service'][0]['profile']).to eq inner_service_profile
+      expect(service_obj["id"]).to eq id_uri
+      expect(service_obj["type"]).to eq described_class::IIIF_IMAGE_V2_TYPE
+      expect(service_obj["profile"]).to eq described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
+      expect(service_obj["label"]).to eq label_val
+      expect(service_obj["service"][0]["id"]).to eq inner_service_id
+      expect(service_obj["service"][0]["profile"]).to eq inner_service_profile
     end
     it 'allows both "id" and "@id" as keys' do
       id_uri = "https://stacks.stanford.edu/image/iiif/wy534zh7137%2FSULAIR_rosette"
       service_obj = described_class.new({
-        'id' => id_uri,
-        '@id' => id_uri
+        "id" => id_uri,
+        "@id" => id_uri
       })
       expect(service_obj.keys.size).to eq 2
-      expect(service_obj['id']).to eq id_uri
-      expect(service_obj['@id']).to eq id_uri
+      expect(service_obj["id"]).to eq id_uri
+      expect(service_obj["@id"]).to eq id_uri
     end
-    it 'allows non-URI profile value' do
-      expect{
+    it "allows non-URI profile value" do
+      expect {
         described_class.new({
           "profile" => [
             "http://iiif.io/api/image/2/level2.json",
             {
-              "formats" => [ "gif", "pdf" ],
-              "qualities" => [ "color", "gray" ],
-              "supports" => [ "canonicalLinkHeader", "rotationArbitrary", "http://example.com/feature" ]
+              "formats" => ["gif", "pdf"],
+              "qualities" => ["color", "gray"],
+              "supports" => ["canonicalLinkHeader", "rotationArbitrary", "http://example.com/feature"]
             }
           ]
         })
@@ -90,111 +89,111 @@ describe IIIF::V3::Presentation::Service do
     end
   end
 
-  describe '#validate' do
-    describe 'type = IIIF_IMAGE_API_V2_TYPE' do
+  describe "#validate" do
+    describe "type = IIIF_IMAGE_API_V2_TYPE" do
       it 'must have a "@id" or "id"' do
         service_obj = described_class.new({
-          'type' => described_class::IIIF_IMAGE_V2_TYPE,
-          'nope' => id_uri,
-          'profile' => described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
-          })
-        exp_err_msg = 'id or @id values are required for IIIF::V3::Presentation::Service with type or @type ImageService2'
-        expect{service_obj.validate}.to raise_error(IIIF::V3::Presentation::MissingRequiredKeyError, exp_err_msg)
+          "type" => described_class::IIIF_IMAGE_V2_TYPE,
+          "nope" => id_uri,
+          "profile" => described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
+        })
+        exp_err_msg = "id or @id values are required for IIIF::V3::Presentation::Service with type or @type ImageService2"
+        expect { service_obj.validate }.to raise_error(IIIF::V3::Presentation::MissingRequiredKeyError, exp_err_msg)
       end
-      it 'should have a profile' do
+      it "should have a profile" do
         service_obj = described_class.new({
-          '@type' => described_class::IIIF_IMAGE_V2_TYPE,
-          '@id' => id_uri
-          })
-        exp_err_msg = 'profile should be present for IIIF::V3::Presentation::Service with type or @type ImageService2'
-        expect{service_obj.validate}.to raise_error(IIIF::V3::Presentation::MissingRequiredKeyError, exp_err_msg)
+          "@type" => described_class::IIIF_IMAGE_V2_TYPE,
+          "@id" => id_uri
+        })
+        exp_err_msg = "profile should be present for IIIF::V3::Presentation::Service with type or @type ImageService2"
+        expect { service_obj.validate }.to raise_error(IIIF::V3::Presentation::MissingRequiredKeyError, exp_err_msg)
       end
       it 'should have matching values for "@id" and "id" if both are specified' do
         service_obj = described_class.new({
-          '@type' => described_class::IIIF_IMAGE_V2_TYPE,
-          '@id' => id_uri,
-          'id' => "#{id_uri}/foo",
-          'profile' => described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
-          })
-        exp_err_msg = 'id and @id values must match for IIIF::V3::Presentation::Service with type or @type ImageService2'
-        expect{service_obj.validate}.to raise_error(IIIF::V3::Presentation::IllegalValueError, exp_err_msg)
+          "@type" => described_class::IIIF_IMAGE_V2_TYPE,
+          "@id" => id_uri,
+          "id" => "#{id_uri}/foo",
+          "profile" => described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
+        })
+        exp_err_msg = "id and @id values must match for IIIF::V3::Presentation::Service with type or @type ImageService2"
+        expect { service_obj.validate }.to raise_error(IIIF::V3::Presentation::IllegalValueError, exp_err_msg)
       end
     end
   end
 
-  describe 'realistic examples' do
-    describe 'from Stanford purl manifests' do
-      it 'iiif image v2 service' do
+  describe "realistic examples" do
+    describe "from Stanford purl manifests" do
+      it "iiif image v2 service" do
         service_obj = described_class.new({
-          'type' => described_class::IIIF_IMAGE_V2_TYPE,
-          'id' => id_uri,
-          '@id' => id_uri,
-          'profile' => described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
+          "type" => described_class::IIIF_IMAGE_V2_TYPE,
+          "id" => id_uri,
+          "@id" => id_uri,
+          "profile" => described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
         })
-        expect{service_obj.validate}.not_to raise_error
+        expect { service_obj.validate }.not_to raise_error
         expect(service_obj.keys.size).to eq 4
-        expect(service_obj.keys).to include('type', '@id', 'id', 'profile')
-        expect(service_obj['type']).to eq described_class::IIIF_IMAGE_V2_TYPE
-        expect(service_obj['id']).to eq id_uri
-        expect(service_obj['@id']).to eq id_uri
-        expect(service_obj['profile']).to eq described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
+        expect(service_obj.keys).to include("type", "@id", "id", "profile")
+        expect(service_obj["type"]).to eq described_class::IIIF_IMAGE_V2_TYPE
+        expect(service_obj["id"]).to eq id_uri
+        expect(service_obj["@id"]).to eq id_uri
+        expect(service_obj["profile"]).to eq described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
       end
-      it 'login service' do
+      it "login service" do
         token_service = described_class.new({
-          'id' => 'https://example.org/image/iiif/token',
-          'profile' => described_class::IIIF_AUTHENTICATION_V1_TOKEN_PROFILE
+          "id" => "https://example.org/image/iiif/token",
+          "profile" => described_class::IIIF_AUTHENTICATION_V1_TOKEN_PROFILE
         })
-        expect{token_service.validate}.not_to raise_error
+        expect { token_service.validate }.not_to raise_error
         expect(token_service.class).to eq described_class
         expect(token_service.keys.size).to eq 2
-        expect(token_service.keys).to include('id', 'profile')
-        expect(token_service['id']).to eq 'https://example.org/image/iiif/token'
-        expect(token_service['profile']).to eq described_class::IIIF_AUTHENTICATION_V1_TOKEN_PROFILE
+        expect(token_service.keys).to include("id", "profile")
+        expect(token_service["id"]).to eq "https://example.org/image/iiif/token"
+        expect(token_service["profile"]).to eq described_class::IIIF_AUTHENTICATION_V1_TOKEN_PROFILE
         service_obj = described_class.new(
-          'id' => 'https://example.org/auth/iiif',
-          'profile' => described_class::IIIF_AUTHENTICATION_V1_LOGIN_PROFILE,
-          'label' => 'label value',
-          'service' => [token_service]
+          "id" => "https://example.org/auth/iiif",
+          "profile" => described_class::IIIF_AUTHENTICATION_V1_LOGIN_PROFILE,
+          "label" => "label value",
+          "service" => [token_service]
         )
-        expect{service_obj.validate}.not_to raise_error
+        expect { service_obj.validate }.not_to raise_error
         expect(service_obj.keys.size).to eq 4
-        expect(service_obj.keys).to include('id', 'profile', 'label', 'service')
-        expect(service_obj['id']).to eq 'https://example.org/auth/iiif'
-        expect(service_obj['profile']).to eq described_class::IIIF_AUTHENTICATION_V1_LOGIN_PROFILE
-        expect(service_obj['label']).to eq 'label value'
-        inner_service = service_obj['service'][0]
+        expect(service_obj.keys).to include("id", "profile", "label", "service")
+        expect(service_obj["id"]).to eq "https://example.org/auth/iiif"
+        expect(service_obj["profile"]).to eq described_class::IIIF_AUTHENTICATION_V1_LOGIN_PROFILE
+        expect(service_obj["label"]).to eq "label value"
+        inner_service = service_obj["service"][0]
         expect(inner_service.class).to eq described_class
-        expect{inner_service.validate}.not_to raise_error
+        expect { inner_service.validate }.not_to raise_error
         expect(inner_service.keys.size).to eq 2
-        expect(inner_service.keys).to include('id', 'profile')
-        expect(inner_service['id']).to eq 'https://example.org/image/iiif/token'
-        expect(inner_service['profile']).to eq described_class::IIIF_AUTHENTICATION_V1_TOKEN_PROFILE
+        expect(inner_service.keys).to include("id", "profile")
+        expect(inner_service["id"]).to eq "https://example.org/image/iiif/token"
+        expect(inner_service["profile"]).to eq described_class::IIIF_AUTHENTICATION_V1_TOKEN_PROFILE
       end
-      it 'triply nested service' do
+      it "triply nested service" do
         inner = described_class.new({
-          'id' => 'https://example.org/image/iiif/token',
-          'profile' => described_class::IIIF_AUTHENTICATION_V1_TOKEN_PROFILE
-          })
-        expect{inner.validate}.not_to raise_error
+          "id" => "https://example.org/image/iiif/token",
+          "profile" => described_class::IIIF_AUTHENTICATION_V1_TOKEN_PROFILE
+        })
+        expect { inner.validate }.not_to raise_error
         middle = described_class.new({
           "id" => "https://example.org/auth/iiif",
           "profile" => described_class::IIIF_AUTHENTICATION_V1_LOGIN_PROFILE,
           "label" => "Stanford-affiliated? Login to view",
           "service" => [inner]
-          })
-        expect{middle.validate}.not_to raise_error
+        })
+        expect { middle.validate }.not_to raise_error
         outer = described_class.new({
-            "@type" => described_class::IIIF_IMAGE_V2_TYPE,
-            "@id" => "https://example.org/iiif/yy816tv6021_img_1",
-            "id" => "https://example.org/iiif/yy816tv6021_img_1",
-            "profile" => described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE,
-            "service" => [middle]
-          })
-        expect{outer.validate}.not_to raise_error
+          "@type" => described_class::IIIF_IMAGE_V2_TYPE,
+          "@id" => "https://example.org/iiif/yy816tv6021_img_1",
+          "id" => "https://example.org/iiif/yy816tv6021_img_1",
+          "profile" => described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE,
+          "service" => [middle]
+        })
+        expect { outer.validate }.not_to raise_error
       end
     end
-    describe 'example from http://prezi3.iiif.io/api/presentation/3.0' do
-      it 'iiif image v2' do
+    describe "example from http://prezi3.iiif.io/api/presentation/3.0" do
+      it "iiif image v2" do
         service_obj = described_class.new({
           "type" => described_class::IIIF_IMAGE_V2_TYPE,
           "id" => "http://example.org/images/book1-page2",
@@ -202,16 +201,16 @@ describe IIIF::V3::Presentation::Service do
           "profile" => "http://iiif.io/api/image/2/level1.json",
           "height" => 8000,
           "width" => 6000,
-          "tiles" => [{"width" => 512, "scaleFactors" => [1,2,4,8,16]}]
+          "tiles" => [{"width" => 512, "scaleFactors" => [1, 2, 4, 8, 16]}]
         })
         expect(service_obj.keys.size).to eq 7
-        expect(service_obj['type']).to eq described_class::IIIF_IMAGE_V2_TYPE
-        expect(service_obj['id']).to eq 'http://example.org/images/book1-page2'
-        expect(service_obj['@id']).to eq 'http://example.org/images/book1-page2'
-        expect(service_obj['profile']).to eq described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
-        expect(service_obj['height']).to eq 8000
-        expect(service_obj['width']).to eq 6000
-        expect(service_obj['tiles']).to eq [{"width" => 512, "scaleFactors" => [1,2,4,8,16]}]
+        expect(service_obj["type"]).to eq described_class::IIIF_IMAGE_V2_TYPE
+        expect(service_obj["id"]).to eq "http://example.org/images/book1-page2"
+        expect(service_obj["@id"]).to eq "http://example.org/images/book1-page2"
+        expect(service_obj["profile"]).to eq described_class::IIIF_IMAGE_V2_LEVEL1_PROFILE
+        expect(service_obj["height"]).to eq 8000
+        expect(service_obj["width"]).to eq 6000
+        expect(service_obj["tiles"]).to eq [{"width" => 512, "scaleFactors" => [1, 2, 4, 8, 16]}]
         # TODO:  note that this won't validate because "height is a prohibited key in IIIF::V3::Presentation::Service"
         # expect{service_obj.validate}.not_to raise_error
       end

data/spec/unit/iiif/v3/presentation/shared_examples/abstract_resource_only_keys.rb

@@ -1,7 +1,5 @@
-shared_examples 'it has the appropriate methods for abstract_resource_only_keys v3' do
-
+shared_examples "it has the appropriate methods for abstract_resource_only_keys v3" do
   described_class.new.abstract_resource_only_keys.each do |entry|
-
     describe "#{entry[:key]}=" do
       it "sets #{entry[:key]}" do
         @ex = entry[:type].new
@@ -17,11 +15,11 @@ shared_examples 'it has the appropriate methods for abstract_resource_only_keys
       end
       it "raises an exception when attempting to set it to something other than an #{entry[:type]}" do
         e = IIIF::V3::Presentation::IllegalValueError
-        expect { subject.send("#{entry[:key]}=", 'Foo') }.to raise_error e
+        expect { subject.send("#{entry[:key]}=", "Foo") }.to raise_error e
       end
     end
 
-    describe "#{entry[:key]}" do
+    describe entry[:key].to_s do
       it "gets #{entry[:key]}" do
         @ex = entry[:type].new
         subject[entry[:key]] = @ex
@@ -31,11 +29,9 @@ shared_examples 'it has the appropriate methods for abstract_resource_only_keys
         it "is aliased as ##{entry[:key].camelize(:lower)}" do
           @ex = entry[:type].new
           subject[entry[:key]] = @ex
-          expect(subject.send("#{entry[:key].camelize(:lower)}")).to eq @ex
+          expect(subject.send(entry[:key].camelize(:lower).to_s)).to eq @ex
         end
       end
     end
-
   end
-
 end

data/spec/unit/iiif/v3/presentation/shared_examples/any_type_keys.rb

@@ -1,5 +1,4 @@
-shared_examples 'it has the appropriate methods for any-type keys v3' do
-
+shared_examples "it has the appropriate methods for any-type keys v3" do
   described_class.new.any_type_keys.each do |prop|
     describe "##{prop}=" do
       it "sets self['#{prop}']" do
@@ -16,16 +15,15 @@ shared_examples 'it has the appropriate methods for any-type keys v3' do
 
     describe "##{prop}" do
       it "gets self[#{prop}]" do
-        subject.send("[]=", prop, fixed_values[prop])
-        expect(subject.send("#{prop}")).to eq fixed_values[prop]
+        subject.send(:[]=, prop, fixed_values[prop])
+        expect(subject.send(prop.to_s)).to eq fixed_values[prop]
       end
       if prop.camelize(:lower) != prop
         it "is aliased as ##{prop.camelize(:lower)}" do
-          subject.send("[]=", prop, fixed_values[prop])
-          expect(subject.send("#{prop.camelize(:lower)}")).to eq fixed_values[prop]
+          subject.send(:[]=, prop, fixed_values[prop])
+          expect(subject.send(prop.camelize(:lower).to_s)).to eq fixed_values[prop]
         end
       end
     end
   end
-
 end

data/spec/unit/iiif/v3/presentation/shared_examples/array_only_keys.rb

@@ -1,40 +1,36 @@
-shared_examples 'it has the appropriate methods for array-only keys v3' do
-
+shared_examples "it has the appropriate methods for array-only keys v3" do
   described_class.new.array_only_keys.each do |prop|
-
     describe "#{prop}=" do
       it "sets #{prop}" do
-        ex = [{'label' => 'XYZ'}]
+        ex = [{"label" => "XYZ"}]
         subject.send("#{prop}=", ex)
         expect(subject[prop]).to eq ex
       end
       if prop.camelize(:lower) != prop
         it "is aliased as ##{prop.camelize(:lower)}=" do
-          ex = [{'label' => 'XYZ'}]
+          ex = [{"label" => "XYZ"}]
           subject.send("#{prop.camelize(:lower)}=", ex)
           expect(subject[prop]).to eq ex
         end
       end
-      it 'raises an exception when attempting to set it to something other than an Array' do
-        expect { subject.send("#{prop}=", 'Foo') }.to raise_error IIIF::V3::Presentation::IllegalValueError
+      it "raises an exception when attempting to set it to something other than an Array" do
+        expect { subject.send("#{prop}=", "Foo") }.to raise_error IIIF::V3::Presentation::IllegalValueError
       end
     end
 
-    describe "#{prop}" do
+    describe prop.to_s do
       it "gets #{prop}" do
-        ex = [{'label' => 'XYZ'}]
+        ex = [{"label" => "XYZ"}]
         subject[prop] = ex
         expect(subject.send(prop)).to eq ex
       end
       if prop.camelize(:lower) != prop
         it "is aliased as ##{prop.camelize(:lower)}" do
-          ex = [{'label' => 'XYZ'}]
+          ex = [{"label" => "XYZ"}]
           subject[prop] = ex
-          expect(subject.send("#{prop.camelize(:lower)}")).to eq ex
+          expect(subject.send(prop.camelize(:lower).to_s)).to eq ex
         end
       end
     end
-
   end
-
 end

data/spec/unit/iiif/v3/presentation/shared_examples/hash_only_keys.rb

@@ -1,40 +1,36 @@
-shared_examples 'it has the appropriate methods for hash-only keys v3' do
-
+shared_examples "it has the appropriate methods for hash-only keys v3" do
   described_class.new.hash_only_keys.each do |prop|
-
     describe "#{prop}=" do
       it "sets #{prop}" do
-        ex = {'label' => 'XYZ', 'fooBar' => 'bar'}
+        ex = {"label" => "XYZ", "fooBar" => "bar"}
         subject.send("#{prop}=", ex)
         expect(subject[prop]).to eq ex
       end
       if prop.camelize(:lower) != prop
         it "is aliased as ##{prop.camelize(:lower)}=" do
-          ex = {'label' => 'XYZ'}
+          ex = {"label" => "XYZ"}
           subject.send("#{prop.camelize(:lower)}=", ex)
           expect(subject[prop]).to eq ex
         end
       end
-      it 'raises an exception when attempting to set it to something other than a Hash' do
-        expect { subject.send("#{prop}=", ['Foo']) }.to raise_error IIIF::V3::Presentation::IllegalValueError
+      it "raises an exception when attempting to set it to something other than a Hash" do
+        expect { subject.send("#{prop}=", ["Foo"]) }.to raise_error IIIF::V3::Presentation::IllegalValueError
       end
     end
 
-    describe "#{prop}" do
+    describe prop.to_s do
       it "gets #{prop}" do
-        ex = {'label' => 'XYZ', 'fooBar' => 'bar'}
+        ex = {"label" => "XYZ", "fooBar" => "bar"}
         subject[prop] = ex
         expect(subject.send(prop)).to eq ex
       end
       if prop.camelize(:lower) != prop
         it "is aliased as ##{prop.camelize(:lower)}" do
-          ex = {'fooBar' => 'bar'}
+          ex = {"fooBar" => "bar"}
           subject[prop] = ex
-          expect(subject.send("#{prop.camelize(:lower)}")).to eq ex
+          expect(subject.send(prop.camelize(:lower).to_s)).to eq ex
         end
       end
     end
-
   end
-
 end

data/spec/unit/iiif/v3/presentation/shared_examples/int_only_keys.rb

@@ -1,7 +1,5 @@
-shared_examples 'it has the appropriate methods for integer-only keys v3' do
-
+shared_examples "it has the appropriate methods for integer-only keys v3" do
   described_class.new.int_only_keys.each do |prop|
-
     describe "#{prop}=" do
       before(:all) do
         @ex = 7200
@@ -16,15 +14,15 @@ shared_examples 'it has the appropriate methods for integer-only keys v3' do
           expect(subject[prop]).to eq @ex
         end
       end
-      it 'raises an exception when attempting to set it to something other than an Integer' do
-        expect { subject.send("#{prop}=", 'Foo') }.to raise_error IIIF::V3::Presentation::IllegalValueError
+      it "raises an exception when attempting to set it to something other than an Integer" do
+        expect { subject.send("#{prop}=", "Foo") }.to raise_error IIIF::V3::Presentation::IllegalValueError
       end
-      it 'raises an exception when attempting to set it to a negative number' do
+      it "raises an exception when attempting to set it to a negative number" do
         expect { subject.send("#{prop}=", -1) }.to raise_error IIIF::V3::Presentation::IllegalValueError
       end
     end
 
-    describe "#{prop}" do
+    describe prop.to_s do
       before(:all) do
         @ex = 7200
       end
@@ -35,11 +33,9 @@ shared_examples 'it has the appropriate methods for integer-only keys v3' do
       if prop.camelize(:lower) != prop
         it "is aliased as ##{prop.camelize(:lower)}" do
           subject[prop] = @ex
-          expect(subject.send("#{prop.camelize(:lower)}")).to eq @ex
+          expect(subject.send(prop.camelize(:lower).to_s)).to eq @ex
         end
       end
     end
-
   end
-
 end

data/spec/unit/iiif/v3/presentation/shared_examples/numeric_only_keys.rb

@@ -1,7 +1,5 @@
-shared_examples 'it has the appropriate methods for numeric-only keys v3' do
-
+shared_examples "it has the appropriate methods for numeric-only keys v3" do
   described_class.new.numeric_only_keys.each do |prop|
-
     describe "#{prop}=" do
       before(:all) do
         @ex = 7200.0
@@ -16,15 +14,15 @@ shared_examples 'it has the appropriate methods for numeric-only keys v3' do
           expect(subject[prop]).to eq @ex
         end
       end
-      it 'raises an exception when attempting to set it to something other than an Integer' do
-        expect { subject.send("#{prop}=", 'Foo') }.to raise_error IIIF::V3::Presentation::IllegalValueError
+      it "raises an exception when attempting to set it to something other than an Integer" do
+        expect { subject.send("#{prop}=", "Foo") }.to raise_error IIIF::V3::Presentation::IllegalValueError
       end
-      it 'raises an exception when attempting to set it to a negative number' do
+      it "raises an exception when attempting to set it to a negative number" do
         expect { subject.send("#{prop}=", -1.0) }.to raise_error IIIF::V3::Presentation::IllegalValueError
       end
     end
 
-    describe "#{prop}" do
+    describe prop.to_s do
       before(:all) do
         @ex = 7200.0
       end
@@ -35,11 +33,9 @@ shared_examples 'it has the appropriate methods for numeric-only keys v3' do
       if prop.camelize(:lower) != prop
         it "is aliased as ##{prop.camelize(:lower)}" do
           subject[prop] = @ex
-          expect(subject.send("#{prop.camelize(:lower)}")).to eq @ex
+          expect(subject.send(prop.camelize(:lower).to_s)).to eq @ex
         end
       end
     end
-
   end
-
 end

data/spec/unit/iiif/v3/presentation/shared_examples/string_only_keys.rb

@@ -1,26 +1,22 @@
-shared_examples 'it has the appropriate methods for string-only keys v3' do
-
+shared_examples "it has the appropriate methods for string-only keys v3" do
   described_class.new.string_only_keys.each do |prop|
-
     describe "#{prop}=" do
       it "sets #{prop}" do
-        ex = 'foo'
+        ex = "foo"
         subject.send("#{prop}=", ex)
         expect(subject[prop]).to eq ex
       end
-      it 'raises an exception when attempting to set it to something other than a String' do
-        expect { subject.send("#{prop}=", ['Foo']) }.to raise_error IIIF::V3::Presentation::IllegalValueError
+      it "raises an exception when attempting to set it to something other than a String" do
+        expect { subject.send("#{prop}=", ["Foo"]) }.to raise_error IIIF::V3::Presentation::IllegalValueError
       end
     end
 
-    describe "#{prop}" do
+    describe prop.to_s do
       it "gets #{prop}" do
-        ex = 'bar'
+        ex = "bar"
         subject[prop] = ex
         expect(subject.send(prop)).to eq ex
       end
     end
-
   end
-
 end