General upgrade (#463)

app/services/create_mkv_service.rb

@@ -64,7 +64,7 @@ def video_blob
                         video: disk_title.video,
                         episode: disk_title.episode,
                         extra_type: :feature_films,
-                        edition:
+                        edition: edition.presence
                       )
                     else
                       VideoBlob.create!(

current_version.txt

@@ -1 +1 @@
-v5.1.0
+v5.2.0

spec/factories/video_blobs.rb

@@ -33,7 +33,6 @@
   factory :video_blob do
     video factory: %i[movie]
     filename { 'Back to the Future Part III.mkv' }
-    sequence(:key) { "error-incidunt/#{_1}/perferendis-iure/Back to the Future Part III.mkv" }
     content_type { 'video/x-matroska' }
     byte_size { 123_456_789 }
 

spec/models/video_blob_spec.rb

@@ -136,7 +136,7 @@
   describe '#set_extra_type_number' do
     context 'when using movies types' do
       let!(:video_blob_a) { create(:video_blob, video: movie, extra_type: :feature_films) }
-      let!(:video_blob_b) { create(:video_blob, video: movie, extra_type: :feature_films) }
+      let!(:video_blob_b) { create(:video_blob, key: 'uniq', video: movie, extra_type: :feature_films) }
       let(:movie) { create(:movie) }
 
       it { expect(video_blob_a.extra_type_number).to be_nil }

spec/services/create_mkv_service_spec.rb

@@ -20,5 +20,20 @@
         expect(call).to be_a(described_class::Result)
       end
     end
+
+    context 'when a video blob already exists' do
+      let(:service) { described_class.new(disk_title:, edition: '') }
+      let(:disk_title) { create(:disk_title, video: movie, episode: nil) }
+      let(:movie) { create(:movie) }
+
+      before do
+        allow(service).to receive(:cmd).and_return('ls /not-a-real-folder')
+        create(:video_blob, video: movie)
+      end
+
+      it 'responds with a result object' do
+        expect(call).to be_a(described_class::Result)
+      end
+    end
   end
 end