hydra-works 1.0.0 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 85a73441a989a8a6103b8461445b0bd1cd2a5b89
-  data.tar.gz: 46c6ea127d5b5ea125f6d467276856df7eaaf94b
+SHA256:
+  metadata.gz: f17bd910e50c6baeda9000b13a4f625615be042df52f102206b051dcb32c6039
+  data.tar.gz: ec76af677423c06b7bfe68e297aef518a8cd43ba5b4fbcffef7c9a8e55fe5312
 SHA512:
-  metadata.gz: 6f7afde9dfc6d577b7ccea96aa997d7cdf654a8001bfb280fa3631b346e4ac4e6c0b574e0f396dbd8403fe0975a0060e6a037df2ded391435288d6c9969061a3
-  data.tar.gz: 15992b88bbf1f9a5f3a63b282ed3d37d700d1cdb5ea45e7a24b3e77e54c9c0d2b584b45cadf22c9e66c710914b8898133fdb98570694731353ae89889b3f1cc4
+  metadata.gz: 38064607b5073e2618072580e5f9b242666658fcd0d7415b819d2462b1f18fd1d20d350be64f10b268f88b8ea570b0055428d3e2932ddd8cb8b32c6cbf272944
+  data.tar.gz: 7a725c734c6bca2b25e62387f281013cc9dcaf55a8584cfa98318a5a74e6c644cbb75fefb53af81cc52a64e156aedd9bb9dc65b223c49fa73873706642a7d544

data/.rubocop.yml

@@ -2,6 +2,7 @@ require: rubocop-rspec
 inherit_from: .rubocop_todo.yml
 
 AllCops:
+  TargetRubyVersion: 2.3
   DisplayCopNames: true
   Include:
     - '**/Rakefile'

data/.rubocop_todo.yml

@@ -63,6 +63,7 @@ RSpec/NamedSubject:
 RSpec/NestedGroups:
   Exclude:
     - 'spec/hydra/works/models/collection_spec.rb'
+    - 'spec/hydra/works/models/file_set_spec.rb'
     - 'spec/hydra/works/models/work_spec.rb'
     - 'spec/hydra/works/services/characterization_service_spec.rb'
     - 'spec/hydra/works/services/virus_checker_service_spec.rb'
@@ -103,3 +104,7 @@ Style/NumericPredicate:
   Exclude:
     - 'spec/**/*'
     - 'lib/hydra/works/virus_scanner.rb'
+
+Style/FrozenStringLiteralComment:
+  Exclude:
+    - '**/*'

data/hydra-works.gemspec

@@ -19,18 +19,17 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'activesupport', '>= 4.2.10', '< 5.2'
+  spec.add_dependency 'activesupport', '>= 4.2.10', '< 6.0'
   spec.add_dependency 'hydra-derivatives', '~> 3.0'
   spec.add_dependency 'hydra-file_characterization', '~> 0.3', '>= 0.3.3'
   spec.add_dependency 'hydra-pcdm', '>= 0.9'
-  spec.add_dependency 'om', '~> 3.1'
 
   spec.add_development_dependency 'bundler', '~> 1.7'
   spec.add_development_dependency 'coveralls'
   spec.add_development_dependency 'fcrepo_wrapper', '~> 0.1'
-  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'rake', '~> 12.3'
   spec.add_development_dependency 'rspec'
   spec.add_development_dependency 'rspec-rails', '~> 3.1'
-  spec.add_development_dependency 'solr_wrapper', '~> 0.4'
+  spec.add_development_dependency 'solr_wrapper', '~> 2.0'
   spec.add_development_dependency 'sqlite3'
 end

data/lib/hydra/works/characterization/fits_document.rb

@@ -1,175 +1,351 @@
-require 'om'
-
 module Hydra::Works::Characterization
   class FitsDocument
-    include OM::XML::Document
-
-    set_terminology do |t|
-      t.root(path: 'fits',
-             xmlns: 'http://hul.harvard.edu/ois/xml/ns/fits/fits_output',
-             schema: 'http://hul.harvard.edu/ois/xml/xsd/fits/fits_output.xsd')
-      t.fits_v(path: { attribute: 'version' })
-      t.identification do
-        t.identity do
-          t.format_label(path: { attribute: 'format' })
-          t.mime_type(path: { attribute: 'mimetype' })
-          t.tool(attributes: { toolname: "Exiftool" }) do
-            t.exif_tool_version(path: { attribute: 'toolversion' })
-          end
-        end
-      end
-      t.fileinfo do
-        t.file_size(path: 'size')
-        t.last_modified(path: 'lastmodified', attributes: { toolname: "Exiftool" })
-        t.filename(path: 'filename')
-        t.original_checksum(path: 'md5checksum')
-        t.date_created(path: 'created', attributes: { toolname: "Exiftool" })
-        t.rights_basis(path: 'rightsBasis')
-        t.copyright_basis(path: 'copyrightBasis')
-        t.copyright_note(path: 'copyrightNote')
-      end
-      t.filestatus do
-        t.well_formed(path: 'well-formed')
-        t.valid(path: 'valid')
-        t.status_message(path: 'message')
-      end
-      t.metadata do
-        t.document do
-          t.file_title(path: 'title')
-          t.file_author(path: 'author')
-          t.file_language(path: 'language')
-          t.page_count(path: 'pageCount')
-          t.word_count(path: 'wordCount')
-          t.character_count(path: 'characterCount')
-          t.paragraph_count(path: 'paragraphCount')
-          t.line_count(path: 'lineCount')
-          t.table_count(path: 'tableCount')
-          t.graphics_count(path: 'graphicsCount')
-        end
-        t.image do
-          t.byte_order(path: 'byteOrder')
-          t.compression(path: 'compressionScheme')
-          t.width(path: 'imageWidth')
-          t.height(path: 'imageHeight')
-          t.color_space(path: 'colorSpace')
-          t.profile_name(path: 'iccProfileName')
-          t.profile_version(path: 'iccProfileVersion')
-          t.orientation(path: 'orientation')
-          t.color_map(path: 'colorMap')
-          t.image_producer(path: 'imageProducer')
-          t.capture_device(path: 'captureDevice')
-          t.scanning_software(path: 'scanningSoftwareName')
-          t.exif_version(path: 'exifVersion', attributes: { toolname: "Exiftool" })
-          t.gps_timestamp(path: 'gpsTimeStamp')
-          t.latitude(path: 'gpsDestLatitude')
-          t.longitude(path: 'gpsDestLongitude')
-        end
-        t.text do
-          t.character_set(path: 'charset')
-          t.markup_basis(path: 'markupBasis')
-          t.markup_language(path: 'markupLanguage')
-        end
-        t.audio do
-          t.duration(path: 'duration')
-          t.bit_depth(path: 'bitDepth')
-          t.bit_rate(path: 'bitRate')
-          t.sample_rate(path: 'sampleRate')
-          t.channels(path: 'channels')
-          t.data_format(path: 'dataFormatType')
-          t.offset(path: 'offset')
-        end
-        t.video do
-          t.width(path: 'imageWidth') # for fits_0.8.5
-          t.height(path: 'imageHeight') # for fits_0.8.5
-          t.duration(path: 'duration')
-          t.bit_rate(path: 'bitRate') # for fits_1.2.0
-          t.sample_rate(path: 'sampleRate') # for fits_0.8.5
-          t.audio_sample_rate(path: 'audioSampleRate') # for fits_0.8.5
-          t.frame_rate(path: 'frameRate') # for fits_0.8.5
-          t.track(path: 'track', attributes: { type: 'video' }) do # for fits_1.2.0
-            t.width(path: 'width')
-            t.height(path: 'height')
-            t.aspect_ratio(path: 'aspectRatio')
-            t.frame_rate(path: 'frameRate')
-          end
-        end
-      end
-      # fits_version needs a different name than it's target node since they're at the same level
-      t.fits_version(proxy: [:fits, :fits_v])
-      t.format_label(proxy: [:identification, :identity, :format_label])
-      # Can't use .mime_type because it's already defined for this dcoument so method missing won't work.
-      t.file_mime_type(proxy: [:identification, :identity, :mime_type])
-      t.exif_tool_version(proxy: [:identification, :identity, :tool, :exif_tool_version])
-      t.file_size(proxy: [:fileinfo, :file_size])
-      t.date_modified(proxy: [:fileinfo, :last_modified])
-      t.filename(proxy: [:fileinfo, :filename])
-      t.original_checksum(proxy: [:fileinfo, :original_checksum])
-      t.date_created(proxy: [:fileinfo, :date_created])
-      t.rights_basis(proxy: [:fileinfo, :rights_basis])
-      t.copyright_basis(proxy: [:fileinfo, :copyright_basis])
-      t.copyright_note(proxy: [:fileinfo, :copyright_note])
-      t.well_formed(proxy: [:filestatus, :well_formed])
-      t.valid(proxy: [:filestatus, :valid])
-      t.filestatus_message(proxy: [:filestatus, :status_message])
-      t.file_title(proxy: [:metadata, :document, :file_title])
-      t.file_author(proxy: [:metadata, :document, :file_author])
-      t.page_count(proxy: [:metadata, :document, :page_count])
-      t.file_language(proxy: [:metadata, :document, :file_language])
-      t.word_count(proxy: [:metadata, :document, :word_count])
-      t.character_count(proxy: [:metadata, :document, :character_count])
-      t.paragraph_count(proxy: [:metadata, :document, :paragraph_count])
-      t.line_count(proxy: [:metadata, :document, :line_count])
-      t.table_count(proxy: [:metadata, :document, :table_count])
-      t.graphics_count(proxy: [:metadata, :document, :graphics_count])
-      t.byte_order(proxy: [:metadata, :image, :byte_order])
-      t.compression(proxy: [:metadata, :image, :compression])
-      t.width(proxy: [:metadata, :image, :width])
-      t.video_width(proxy: [:metadata, :video, :width])
-      t.video_track_width(proxy: [:metadata, :video, :track, :width])
-      t.height(proxy: [:metadata, :image, :height])
-      t.video_height(proxy: [:metadata, :video, :height])
-      t.video_track_height(proxy: [:metadata, :video, :track, :height])
-      t.color_space(proxy: [:metadata, :image, :color_space])
-      t.profile_name(proxy: [:metadata, :image, :profile_name])
-      t.profile_version(proxy: [:metadata, :image, :profile_version])
-      t.orientation(proxy: [:metadata, :image, :orientation])
-      t.color_map(proxy: [:metadata, :image, :color_map])
-      t.image_producer(proxy: [:metadata, :image, :image_producer])
-      t.capture_device(proxy: [:metadata, :image, :capture_device])
-      t.scanning_software(proxy: [:metadata, :image, :scanning_software])
-      t.exif_version(proxy: [:metadata, :image, :exif_version])
-      t.gps_timestamp(proxy: [:metadata, :image, :gps_timestamp])
-      t.latitude(proxy: [:metadata, :image, :latitude])
-      t.longitude(proxy: [:metadata, :image, :longitude])
-      t.character_set(proxy: [:metadata, :text, :character_set])
-      t.markup_basis(proxy: [:metadata, :text, :markup_basis])
-      t.markup_language(proxy: [:metadata, :text, :markup_language])
-      t.audio_duration(proxy: [:metadata, :audio, :duration])
-      t.video_duration(proxy: [:metadata, :video, :duration])
-      t.bit_depth(proxy: [:metadata, :audio, :bit_depth])
-      t.audio_sample_rate(proxy: [:metadata, :audio, :sample_rate])
-      t.video_sample_rate(proxy: [:metadata, :video, :sample_rate])
-      t.video_audio_sample_rate(proxy: [:metadata, :video, :audio_sample_rate])
-      t.channels(proxy: [:metadata, :audio, :channels])
-      t.data_format(proxy: [:metadata, :audio, :data_format])
-      t.offset(proxy: [:metadata, :audio, :offset])
-      t.frame_rate(proxy: [:metadata, :video, :frame_rate])
-      t.audio_bit_rate(proxy: [:metadata, :audio, :bit_rate])
-      t.video_bit_rate(proxy: [:metadata, :video, :bit_rate])
-      t.track_frame_rate(proxy: [:metadata, :video, :track, :frame_rate])
-      t.aspect_ratio(proxy: [:metadata, :video, :track, :aspect_ratio])
+    attr_accessor :ng_xml
+
+    PROXIED_TERMS = %i(
+      fits_version format_label file_mime_type exif_tool_version file_size
+      date_modified filename original_checksum date_created rights_basis
+      copyright_basis copyright_note well_formed valid filestatus_message
+      file_title file_author page_count file_language word_count
+      character_count paragraph_count line_count table_count graphics_count
+      byte_order compression width video_width video_track_width height
+      video_height video_track_height color_space profile_name profile_version
+      orientation color_map image_producer capture_device scanning_software
+      exif_version gps_timestamp latitude longitude character_set markup_basis
+      markup_language audio_duration video_duration bit_depth audio_sample_rate
+      video_sample_rate video_audio_sample_rate channels data_format offset
+      frame_rate audio_bit_rate video_bit_rate track_frame_rate aspect_ratio
+    ).freeze
+
+    def proxied_term_hash
+      PROXIED_TERMS.map { |t| [t, send(t)] }.to_h
+    end
+
+    def self.terminology
+      struct = Struct.new(:proxied_term).new
+      terminology = Struct.new(:terms)
+      terminology.new(PROXIED_TERMS.map { |t| [t, struct] }.to_h)
+    end
+
+    #  t.fits_version(proxy: [:fits, :fits_v])
+    def fits_version
+      ng_xml.css("fits").map { |n| n['version'].text }
+    end
+
+    # t.format_label(proxy: [:identification, :identity, :format_label])
+    def format_label
+      ng_xml.css("fits > identification > identity").map { |n| n['format'] }
+    end
+
+    # Can't use .mime_type because it's already defined for this document so method missing won't work.
+    # t.file_mime_type(proxy: [:identification, :identity, :mime_type])
+    def file_mime_type
+      # Sometimes, FITS reports the mimetype attribute as a comma-separated string.
+      # All terms are arrays and, in this case, there is only one element, so scan the first.
+      ng_xml.css("fits > identification > identity").map { |n| n['mimetype'].split(',').first }
+    end
+
+    # t.exif_tool_version(proxy: [:identification, :identity, :tool, :exif_tool_version])
+    def exif_tool_version
+      ng_xml.css("fits > identification > identity > tool[toolname='Exiftool']").map { |n| n['toolversion'] }
+    end
+
+    # t.file_size(proxy: [:fileinfo, :file_size])
+    def file_size
+      ng_xml.css("fits > fileinfo > size").map(&:text)
+    end
+
+    # t.date_modified(proxy: [:fileinfo, :last_modified])
+    def date_modified
+      ng_xml.css("fits > fileinfo > lastmodified[toolname='Exiftool']").map(&:text)
+    end
+
+    #  t.filename(proxy: [:fileinfo, :filename])
+    def filename
+      ng_xml.css("fits > fileinfo > filename").map(&:text)
+    end
+
+    # t.original_checksum(proxy: [:fileinfo, :original_checksum])
+    def original_checksum
+      ng_xml.css("fits > fileinfo > md5checksum").map(&:text)
+    end
+
+    # t.date_created(proxy: [:fileinfo, :date_created])
+    def date_created
+      ng_xml.css("fits > fileinfo > created[toolname='Exiftool']").map(&:text)
+    end
+
+    # t.rights_basis(proxy: [:fileinfo, :rights_basis])
+    def rights_basis
+      ng_xml.css("fits > fileinfo > rightsBasis").map(&:text)
+    end
+
+    # t.copyright_basis(proxy: [:fileinfo, :copyright_basis])
+    def copyright_basis
+      ng_xml.css("fits > fileinfo > copyrightBasis").map(&:text)
+    end
+
+    # t.copyright_basis(proxy: [:fileinfo, :copyright_note])
+    def copyright_note
+      ng_xml.css("fits > fileinfo > copyrightNote").map(&:text)
+    end
+
+    # t.well_formed(proxy: [:filestatus, :well_formed])
+    def well_formed
+      ng_xml.css("fits > filestatus > well-formed").map(&:text)
+    end
+
+    # t.valid(proxy: [:filestatus, :valid])
+    def valid
+      ng_xml.css("fits > filestatus > valid").map(&:text)
+    end
+
+    # t.filestatus_message(proxy: [:filestatus, :status_message])
+    def filestatus_message
+      ng_xml.css("fits > filestatus > message").map(&:text)
+    end
+
+    # t.file_title(proxy: [:metadata, :document, :file_title])
+    def file_title
+      ng_xml.css("fits > metadata > document > title").map(&:text)
+    end
+
+    # t.file_author(proxy: [:metadata, :document, :file_author])
+    def file_author
+      ng_xml.css("fits > metadata > document > author").map(&:text)
+    end
+
+    # t.file_language(proxy: [:metadata, :document, :file_language])
+    def file_language
+      ng_xml.css("fits > metadata > document > language").map(&:text)
+    end
+
+    # t.page_count(proxy: [:metadata, :document, :page_count])
+    def page_count
+      ng_xml.css("fits > metadata > document > pageCount").map(&:text)
+    end
+
+    # t.word_count(proxy: [:metadata, :document, :word_count])
+    def word_count
+      ng_xml.css("fits > metadata > document > wordCount").map(&:text)
+    end
+
+    # t.character_count(proxy: [:metadata, :document, :character_count])
+    def character_count
+      ng_xml.css("fits > metadata > document > characterCount").map(&:text)
+    end
+
+    # t.paragraph_count(proxy: [:metadata, :document, :paragraph_count])
+    def paragraph_count
+      ng_xml.css("fits > metadata > document > paragraphCount").map(&:text)
+    end
+
+    # t.line_count(proxy: [:metadata, :document, :line_count])
+    def line_count
+      ng_xml.css("fits > metadata > document > lineCount").map(&:text)
+    end
+
+    # t.table_count(proxy: [:metadata, :document, :table_count])
+    def table_count
+      ng_xml.css("fits > metadata > document > tableCount").map(&:text)
+    end
+
+    # t.graphics_count(proxy: [:metadata, :document, :graphics_count])
+    def graphics_count
+      ng_xml.css("fits > metadata > document > graphicsCount").map(&:text)
+    end
+
+    # t.byte_order(proxy: [:metadata, :image, :byte_order])
+    def byte_order
+      ng_xml.css("fits > metadata > image > byteOrder").map(&:text)
+    end
+
+    # t.compression(proxy: [:metadata, :image, :compression])
+    def compression
+      ng_xml.css("fits > metadata > image > compressionScheme").map(&:text)
+    end
+
+    # t.width(proxy: [:metadata, :image, :width])
+    def width
+      ng_xml.css("fits > metadata > image > imageWidth").map(&:text)
+    end
+
+    # t.height(proxy: [:metadata, :image, :height])
+    def height
+      ng_xml.css("fits > metadata > image > imageHeight").map(&:text)
+    end
+
+    # t.video_width(proxy: [:metadata, :video, :width])
+    def video_width
+      ng_xml.css("fits > metadata > video > imageWidth").map(&:text)
+    end
+
+    # t.video_height(proxy: [:metadata, :video, :height])
+    def video_height
+      ng_xml.css("fits > metadata > video > imageHeight").map(&:text)
+    end
+
+    # for fits 1
+    # t.video_track_width(proxy: [:metadata, :video, :track, :width])
+    def video_track_width
+      ng_xml.css("fits > metadata > video > track[type='video'] > width").map(&:text)
+    end
+
+    # for fits 1
+    # t.video_track_height(proxy: [:metadata, :video, :track, :height])
+    def video_track_height
+      ng_xml.css("fits > metadata > video > track[type='video'] > height").map(&:text)
+    end
+
+    #  t.color_space(proxy: [:metadata, :image, :color_space])
+    def color_space
+      ng_xml.css("fits > metadata > image > colorSpace").map(&:text)
+    end
+
+    # t.profile_name(proxy: [:metadata, :image, :profile_name])
+    def profile_name
+      ng_xml.css("fits > metadata > image > iccProfileName").map(&:text)
+    end
+
+    # t.profile_version(proxy: [:metadata, :image, :profile_version])
+    def profile_version
+      ng_xml.css("fits > metadata > image > iccProfileVersion").map(&:text)
+    end
+
+    # t.orientation(proxy: [:metadata, :image, :orientation])
+    def orientation
+      ng_xml.css("fits > metadata > image > orientation").map(&:text)
+    end
+
+    # t.color_map(proxy: [:metadata, :image, :color_map])
+    def color_map
+      ng_xml.css("fits > metadata > image > colorMap").map(&:text)
+    end
+
+    # t.image_producer(proxy: [:metadata, :image, :image_producer])
+    def image_producer
+      ng_xml.css("fits > metadata > image > imageProducer").map(&:text)
+    end
+
+    # t.capture_device(proxy: [:metadata, :image, :capture_device])
+    def capture_device
+      ng_xml.css("fits > metadata > image > captureDevice").map(&:text)
+    end
+
+    # t.scanning_software(proxy: [:metadata, :image, :scanning_software])
+    def scanning_software
+      ng_xml.css("fits > metadata > image > scanningSoftwareName").map(&:text)
+    end
+
+    # t.exif_version(proxy: [:metadata, :image, :exif_version])
+    def exif_version
+      ng_xml.css("fits > metadata > image > exifVersion[toolname='Exiftool']").map(&:text)
+    end
+
+    # t.gps_timestamp(proxy: [:metadata, :image, :gps_timestamp])
+    def gps_timestamp
+      ng_xml.css("fits > metadata > image > gpsTimeStamp").map(&:text)
+    end
+
+    # t.latitude(proxy: [:metadata, :image, :latitude])
+    def latitude
+      ng_xml.css("fits > metadata > image > gpsDestLatitude").map(&:text)
+    end
+
+    # t.longitude(proxy: [:metadata, :image, :longitude])
+    def longitude
+      ng_xml.css("fits > metadata > image > gpsDestLongitude").map(&:text)
+    end
+
+    # t.character_set(proxy: [:metadata, :text, :character_set])
+    def character_set
+      ng_xml.css("fits > metadata > text > charset").map(&:text)
+    end
+
+    # t.markup_basis(proxy: [:metadata, :text, :markup_basis])
+    def markup_basis
+      ng_xml.css("fits > metadata > text > markupBasis").map(&:text)
+    end
+
+    # t.markup_language(proxy: [:metadata, :text, :markup_language])
+    def markup_language
+      ng_xml.css("fits > metadata > text > markupLanguage").map(&:text)
+    end
+
+    # t.audio_duration(proxy: [:metadata, :audio, :duration])
+    def audio_duration
+      ng_xml.css("fits > metadata > audio > duration").map(&:text)
+    end
+
+    # t.bit_depth(proxy: [:metadata, :audio, :bit_depth])
+    def bit_depth
+      ng_xml.css("fits > metadata > audio > bitDepth").map(&:text)
+    end
+
+    # t.audio_sample_rate(proxy: [:metadata, :audio, :sample_rate])
+    def audio_sample_rate
+      ng_xml.css("fits > metadata > audio > sampleRate").map(&:text)
+    end
+
+    # t.channels(proxy: [:metadata, :audio, :channels])
+    def channels
+      ng_xml.css("fits > metadata > audio > channels").map(&:text)
+    end
+
+    # t.data_format(proxy: [:metadata, :audio, :data_format])
+    def data_format
+      ng_xml.css("fits > metadata > audio > dataFormatType").map(&:text)
+    end
+
+    # t.offset(proxy: [:metadata, :audio, :offset])
+    def offset
+      ng_xml.css("fits > metadata > audio > offset").map(&:text)
+    end
+
+    # t.audio_bit_rate(proxy: [:metadata, :audio, :bit_rate])
+    def audio_bit_rate
+      ng_xml.css("fits > metadata > audio > bitRate").map(&:text)
+    end
+
+    # t.video_duration(proxy: [:metadata, :video, :duration])
+    def video_duration
+      ng_xml.css("fits > metadata > video > duration").map(&:text)
+    end
+
+    # t.video_sample_rate(proxy: [:metadata, :video, :sample_rate])
+    def video_sample_rate
+      ng_xml.css("fits > metadata > video > sampleRate").map(&:text)
+    end
+
+    # t.video_audio_sample_rate(proxy: [:metadata, :video, :audio_sample_rate])
+    def video_audio_sample_rate
+      ng_xml.css("fits > metadata > video > audioSampleRate").map(&:text)
+    end
+
+    # t.frame_rate(proxy: [:metadata, :video, :frame_rate])
+    def frame_rate
+      ng_xml.css("fits > metadata > video > frameRate").map(&:text)
+    end
+
+    # t.video_bit_rate(proxy: [:metadata, :video, :bit_rate])
+    def video_bit_rate
+      ng_xml.css("fits > metadata > video > bitRate").map(&:text)
+    end
+
+    # t.track_frame_rate(proxy: [:metadata, :video, :track, :frame_rate])
+    def track_frame_rate
+      ng_xml.css("fits > metadata > video > track[type='video'] > frameRate").map(&:text)
+    end
+
+    # t.aspect_ratio(proxy: [:metadata, :video, :track, :aspect_ratio])
+    def aspect_ratio
+      ng_xml.css("fits > metadata > video > track[type='video'] > aspectRatio").map(&:text)
     end
 
     # Cleanup phase; ugly name to avoid collisions.
     # The send construct here is required to fix up values because the setters
     # are not defined, but rather applied with method_missing.
     def __cleanup__
-      # Sometimes, FITS reports the mimetype attribute as a comma-separated string.
-      # All terms are arrays and, in this case, there is only one element, so scan the first.
-      if file_mime_type.present? && file_mime_type.first.include?(',')
-        send("file_mime_type=", [file_mime_type.first.split(',').first])
-      end
-
       # Add any other scrubbers here; don't return any particular value
       nil
     end