Add viewingDirection as optional data value on manifest, fixes #12

iiif_manifest.gemspec

@@ -21,9 +21,9 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'iiif-presentation', '~> 0.2.0'
 
   spec.add_development_dependency 'bundler', '~> 1.11'
+  spec.add_development_dependency 'pry-byebug'
   spec.add_development_dependency 'rake', '~> 10.0'
   spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'rubocop'
   spec.add_development_dependency 'rubocop-rspec'
-  spec.add_development_dependency 'pry-byebug'
 end

lib/iiif_manifest/manifest_builder/record_property_builder.rb

@@ -6,21 +6,28 @@ def initialize(record)
         @record = record
       end
 
+      # rubocop:disable Metrics/AbcSize
       def apply(manifest)
         manifest['@id'] = record.manifest_url.to_s
         manifest.label = record.to_s
         manifest.description = record.description
         manifest.viewing_hint = viewing_hint if viewing_hint.present?
+        manifest.viewing_direction = viewing_direction if viewing_direction.present?
         manifest.metadata = record.manifest_metadata if valid_metadata?
         manifest
       end
+      # rubocop:enable Metrics/AbcSize
 
       private
 
       def viewing_hint
         (record.respond_to?(:viewing_hint) && record.send(:viewing_hint))
       end
 
+      def viewing_direction
+        (record.respond_to?(:viewing_direction) && record.send(:viewing_direction))
+      end
+
       # Validate manifest_metadata against the IIIF spec format for metadata
       #
       # @return [Boolean]

spec/lib/iiif_manifest/manifest_factory_spec.rb

@@ -110,7 +110,7 @@ def display_image
       end
     end
 
-    context 'where there is a no sequence_rendering method' do
+    context 'when there is a no sequence_rendering method' do
       let(:file_presenter) { DisplayImagePresenter.new }
 
       it 'does not have a rendering on the sequence' do
@@ -120,7 +120,7 @@ def display_image
       end
     end
 
-    context 'where there is a sequence_rendering method' do
+    context 'when there is a sequence_rendering method' do
       let(:file_presenter) { DisplayImagePresenter.new }
 
       before do
@@ -163,7 +163,7 @@ def sequence_rendering
       end
     end
 
-    context 'where there is a no manifest_metadata method' do
+    context 'when there is no manifest_metadata method' do
       let(:file_presenter) { DisplayImagePresenter.new }
 
       it 'does not have a metadata element' do
@@ -173,7 +173,7 @@ def sequence_rendering
       end
     end
 
-    context 'where there is a manifest_metadata method' do
+    context 'when there is a manifest_metadata method' do
       let(:metadata) { [{ 'label' => 'Title', 'value' => 'Title of the Item' }] }
 
       it 'has metadata' do
@@ -183,7 +183,7 @@ def sequence_rendering
       end
     end
 
-    context 'where there is a manifest_metadata method with invalid data' do
+    context 'when there is a manifest_metadata method with invalid data' do
       let(:metadata) { 'invalid data' }
 
       it 'has no metadata' do
@@ -236,5 +236,18 @@ def sequence_rendering
         expect(result['sequences'].first['canvases'].length).to eq 2
       end
     end
+
+    context 'when there is no viewing_direction method' do
+      it 'does not have a viewingDirection element' do
+        expect(result['viewingDirection']).to eq nil
+      end
+    end
+
+    context 'when there is a viewing_direction method' do
+      it 'has a viewingDirection' do
+        allow(book_presenter).to receive(:viewing_direction).and_return('right-to-left')
+        expect(result.viewingDirection).to eq 'right-to-left'
+      end
+    end
   end
 end