Merge pull request #297 from iainbeeston/validate-video-attachments

Added a test to validate video attachments

Author: Mth0158

test/active_storage_validations_test.rb

@@ -29,6 +29,7 @@ class ActiveStorageValidations::Test < ActiveSupport::TestCase
     u.proc_image_regex.attach(image_150x150_file)
     u.photos.attach(bad_dummy_file)
     u.proc_photos.attach(bad_dummy_file)
+    u.video.attach(video_file)
     assert !u.valid?
     assert_equal u.errors.full_messages, ['Photos has an invalid content type', 'Proc photos has an invalid content type']
 
@@ -39,6 +40,7 @@ class ActiveStorageValidations::Test < ActiveSupport::TestCase
     u.proc_image_regex.attach(image_150x150_file)
     u.photos.attach(image_150x150_file)
     u.proc_photos.attach(image_150x150_file)
+    u.video.attach(video_file)
     assert !u.valid?
     assert_equal u.errors.full_messages, ['Avatar has an invalid content type', 'Proc avatar has an invalid content type']
     assert_equal u.errors.details, avatar: [
@@ -68,6 +70,7 @@ class ActiveStorageValidations::Test < ActiveSupport::TestCase
     u.proc_image_regex.attach(image_150x150_file)
     u.photos.attach(pdf_file) # Should be handled by regex match.
     u.proc_photos.attach(pdf_file) # Should be handled by regex match.
+    u.video.attach(video_file)
     assert u.valid?
 
     u = User.new(name: 'John Smith')
@@ -77,6 +80,7 @@ class ActiveStorageValidations::Test < ActiveSupport::TestCase
     u.proc_image_regex.attach(bad_dummy_file)
     u.photos.attach(image_150x150_file)
     u.proc_photos.attach(image_150x150_file)
+    u.video.attach(video_file)
     assert !u.valid?
     assert_equal u.errors.full_messages, ['Image regex has an invalid content type', 'Proc image regex has an invalid content type']
 
@@ -87,6 +91,7 @@ class ActiveStorageValidations::Test < ActiveSupport::TestCase
     u.proc_image_regex.attach(bad_dummy_file)
     u.photos.attach(bad_dummy_file)
     u.proc_photos.attach(bad_dummy_file)
+    u.video.attach(video_file)
     assert !u.valid?
     assert_equal u.errors.full_messages, ['Avatar has an invalid content type', 'Photos has an invalid content type', 'Image regex has an invalid content type', 'Proc avatar has an invalid content type', 'Proc photos has an invalid content type', 'Proc image regex has an invalid content type']
 
@@ -96,6 +101,7 @@ class ActiveStorageValidations::Test < ActiveSupport::TestCase
     u.photos.attach(image_150x150_file)
     u.proc_photos.attach(image_150x150_file)
     u.conditional_image_2.attach(image_150x150_file)
+    u.video.attach(video_file)
     assert u.valid?
     assert_equal u.errors.full_messages, []
 
@@ -105,6 +111,7 @@ class ActiveStorageValidations::Test < ActiveSupport::TestCase
     u.photos.attach(bad_dummy_file)
     u.proc_photos.attach(image_150x150_file)
     u.conditional_image_2.attach(bad_dummy_file)
+    u.video.attach(video_file)
     assert !u.valid?
     assert_equal u.errors.full_messages, ["Avatar has an invalid content type", "Photos has an invalid content type", "Conditional image 2 has an invalid content type", "Proc avatar has an invalid content type"]
   end

test/dummy/app/models/user.rb

@@ -20,6 +20,7 @@ class User < ApplicationRecord
   has_many_attached :proc_photos
   has_many_attached :proc_photo_with_messages
   has_one_attached :proc_image_regex
+  has_one_attached :video
 
   validates :name, presence: true
 
@@ -29,6 +30,7 @@ class User < ApplicationRecord
   validates :image_regex, content_type: /\Aimage\/.*\z/
   validates :conditional_image, attached: true, if: -> { name == 'Foo' }
   validates :conditional_image_2, attached: true, content_type: -> (record) {[/\Aimage\/.*\z/]}, size: { less_than: 10.megabytes }, if: -> { name == 'Peter Griffin' }
+  validates :video, content_type: [:mp4, :mpeg, :webm]
 
   validates :moon_picture, content_type: ['image/png'], size: { greater_than: 0 }
 

test/dummy/public/video.mp4

@@ -193,6 +193,14 @@ def not_identifiable_io_file
   }
 end
 
+def video_file
+  {
+    io: File.open(Rails.root.join('public', 'video.mp4')),
+    filename: 'video',
+    content_type: 'video/mp4'
+  }
+end
+
 def create_blob_from_file(file)
   ActiveStorage::Blob.create_and_upload!(
     io: file[:io],