VTT detection not dependent on filename

The logic for determining if a file is a VTT file is now based on the media type, not the filename or its extension.
sul-dlss · May 16, 2023 · 7568bb1 · 7568bb1
1 parent 033d2ba
commit 7568bb1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/app/models/embed/purl/resource_file.rb b/app/models/embed/purl/resource_file.rb
@@ -48,7 +48,7 @@ def vtt
       end
 
       def vtt?
-        mimetype == 'text/vtt' && title.end_with?('.vtt')
+        mimetype == 'text/vtt'
       end
 
       def pdf?

diff --git a/spec/fixtures/purl_fixtures.rb b/spec/fixtures/purl_fixtures.rb
@@ -972,7 +972,7 @@ def video_purl_with_vtt
         <contentMetadata type="media">
           <resource sequence="1" id="abc123_1" type="video">
             <file id="abc_123.mp4" mimetype="video/mp4" size="152000000"></file>
-            <file id="abc_123_cap.vtt" mimetype="text/vtt" size="176218"></file>
+            <file id="abc_123_cap.webvtt" mimetype="text/vtt" size="176218"></file>
           </resource>
         </contentMetadata>
         <rightsMetadata>

diff --git a/spec/models/embed/purl/resource_spec.rb b/spec/models/embed/purl/resource_spec.rb
@@ -58,7 +58,7 @@
 
       before { stub_purl_response_with_fixture(video_purl_with_vtt) }
 
-      it { is_expected.to eq 'abc_123_cap.vtt' }
+      it { is_expected.to eq 'abc_123_cap.webvtt' }
     end
 
     context 'when it does not have a vtt transcript' do