bulldog 0.0.1

data/spec/unit/attachment/image_spec.rb

@@ -0,0 +1,128 @@
+require 'spec_helper'
+
+describe Attachment::Image do
+  use_model_class(:Thing,
+                  :photo_file_name => :string,
+                  :photo_width => :integer,
+                  :photo_height => :integer,
+                  :photo_aspect_ratio => :float,
+                  :photo_dimensions => :string)
+
+  before do
+    Thing.has_attachment :photo do
+      style :double, :size => '80x60'
+      style :filled, :size => '60x60', :filled => true
+      style :unfilled, :size => '120x120'
+      default_style :double
+    end
+    @thing = Thing.new(:photo => test_image_file)
+  end
+
+  def run(command)
+    `#{command}`
+    $?.success? or
+      raise "command failed: #{command}"
+  end
+
+  describe "#dimensions" do
+    it "should return 1x1 if the style is missing" do
+      Thing.attachment_reflections[:photo].configure do
+        detect_type_by{:image}
+      end
+      @thing.save.should be_true
+      File.unlink(@thing.photo.path(:original))
+      @thing = Thing.find(@thing.id)
+      @thing.photo.is_a?(Attachment::Image)  # sanity check
+      @thing.photo.stream.missing?           # sanity check
+      @thing.photo.dimensions(:original).should == [1, 1]
+    end
+
+    it "should return the width and height of the default style if no style name is given" do
+      @thing.photo.dimensions.should == [80, 60]
+    end
+
+    it "should return the width and height of the given style" do
+      @thing.photo.dimensions(:original).should == [40, 30]
+      @thing.photo.dimensions(:double).should == [80, 60]
+    end
+
+    it "should return the calculated width according to style filledness" do
+      @thing.photo.dimensions(:filled).should == [60, 60]
+      @thing.photo.dimensions(:unfilled).should == [120, 90]
+    end
+
+    it "should honor the exif:Orientation header" do
+      path = create_image("#{temporary_directory}/test.jpg", :size => '40x30')
+      rotated_path = "#{temporary_directory}/rotated-test.jpg"
+      run "exif --create-exif --ifd=EXIF --tag=Orientation --set-value=4 --output=#{rotated_path} #{path}"
+      open(rotated_path) do |file|
+        @thing.photo = file
+        @thing.photo.dimensions(:original).should == [30, 40]
+      end
+    end
+
+    it "should only invoke identify once"
+    it "should log the result"
+  end
+
+  describe "#width" do
+    it "should return the width of the default style if no style name is given" do
+      @thing.photo.width.should == 80
+    end
+
+    it "should return the width of the given style" do
+      @thing.photo.width(:original).should == 40
+      @thing.photo.width(:double).should == 80
+    end
+  end
+
+  describe "#height" do
+    it "should return the height of the default style if no style name is given" do
+      @thing.photo.height.should == 60
+    end
+
+    it "should return the height of the given style" do
+      @thing.photo.height(:original).should == 30
+      @thing.photo.height(:double).should == 60
+    end
+  end
+
+  describe "#aspect_ratio" do
+    it "should return the aspect ratio of the default style if no style name is given" do
+      @thing.photo.aspect_ratio.should be_close(4.0/3, 1e-5)
+    end
+
+    it "should return the aspect ratio of the given style" do
+      @thing.photo.aspect_ratio(:original).should be_close(4.0/3, 1e-5)
+      @thing.photo.aspect_ratio(:filled).should be_close(1, 1e-5)
+    end
+  end
+
+  describe "storable attributes" do
+    it "should set the stored attributes on assignment" do
+      @thing.photo_width.should == 40
+      @thing.photo_height.should == 30
+      @thing.photo_aspect_ratio.should be_close(4.0/3, 1e-5)
+      @thing.photo_dimensions.should == '40x30'
+    end
+
+    describe "after roundtripping through the database" do
+      before do
+        @thing.save
+        @thing = Thing.find(@thing.id)
+      end
+
+      it "should restore the stored attributes" do
+        @thing.photo_width.should == 40
+        @thing.photo_height.should == 30
+        @thing.photo_aspect_ratio.should be_close(4.0/3, 1e-5)
+        @thing.photo_dimensions.should == '40x30'
+      end
+
+      it "should recalculate the dimensions correctly" do
+        @thing.photo.dimensions(:filled).should == [60, 60]
+        @thing.photo.dimensions(:unfilled).should == [120, 90]
+      end
+    end
+  end
+end

data/spec/unit/attachment/maybe_spec.rb

@@ -0,0 +1,126 @@
+require 'spec_helper'
+
+describe Attachment::Maybe do
+  use_model_class(:Thing, :photo_file_name => :string)
+
+  before do
+    spec = self
+    Thing.has_attachment :photo do
+      path "#{spec.temporary_directory}/photo.:style.:extension"
+    end
+  end
+
+  def configure_attachment(&block)
+    spec = self
+    Thing.has_attachment :photo do
+      instance_exec(spec, &block)
+    end
+    @thing = Thing.new
+  end
+
+  describe "#interpolate_path" do
+    it "should return the path that the given style name would be stored at" do
+      spec = self
+      configure_attachment do
+        path "#{spec.temporary_directory}/:attachment.:style.jpg"
+      end
+      @thing.photo.interpolate_path(:original).should == "#{temporary_directory}/photo.original.jpg"
+    end
+
+    it "should use the given interpolation parameters" do
+      spec = self
+      configure_attachment do
+        path "#{spec.temporary_directory}/:attachment.:style.jpg"
+      end
+      @thing.photo.interpolate_path(:original, :style => 'STYLE').should == "#{temporary_directory}/photo.STYLE.jpg"
+    end
+
+    it "should use the style's format attribute for the extension by default" do
+      spec = self
+      configure_attachment do
+        style :processed, :format => 'png'
+        path "#{spec.temporary_directory}/:attachment.:style.:extension"
+      end
+      @thing.photo.interpolate_path(:processed, :format => 'png').should == "#{temporary_directory}/photo.processed.png"
+    end
+
+    describe "for the original style" do
+      it "should support the :basename interpolation key if the basename is given" do
+        spec = self
+        configure_attachment do
+          path "#{spec.temporary_directory}/:attachment.:style/:basename"
+        end
+        @thing.photo.interpolate_path(:original, :basename => 'file.xyz').should == "#{temporary_directory}/photo.original/file.xyz"
+      end
+
+      it "should support the :extension interpolation key if the basename is given" do
+        spec = self
+        configure_attachment do
+          path "#{spec.temporary_directory}/:attachment.:style.:extension"
+        end
+        @thing.photo.interpolate_path(:original, :basename => 'file.xyz').should == "#{temporary_directory}/photo.original.xyz"
+      end
+
+      it "should support the :extension interpolation key if the extension is given" do
+        spec = self
+        configure_attachment do
+          path "#{spec.temporary_directory}/:attachment.:style.:extension"
+        end
+        @thing.photo.interpolate_path(:original, :extension => 'xyz').should == "#{temporary_directory}/photo.original.xyz"
+      end
+    end
+  end
+
+  describe "#interpolate_url" do
+    it "should return the url that the given style name would be found at" do
+      spec = self
+      configure_attachment do
+        url "/:attachment.:style.jpg"
+      end
+      @thing.photo.interpolate_url(:original).should == "/photo.original.jpg"
+    end
+
+    it "should use the given interpolation parameters" do
+      spec = self
+      configure_attachment do
+        url "/:attachment.:style.jpg"
+      end
+      @thing.photo.interpolate_url(:original, :style => 'STYLE').should == "/photo.STYLE.jpg"
+    end
+
+    it "should use the style's format attribute for the extension by default" do
+      spec = self
+      configure_attachment do
+        style :processed, :format => 'png'
+        url "/:attachment.:style.:extension"
+      end
+      @thing.photo.interpolate_url(:processed).should == "/photo.processed.png"
+    end
+
+    describe "for the original style" do
+      it "should support the :basename interpolation key if the basename is given" do
+        spec = self
+        configure_attachment do
+          url "/:attachment.:style/:basename"
+        end
+        @thing.photo.interpolate_url(:original, :basename => 'file.xyz').should == "/photo.original/file.xyz"
+      end
+
+      it "should support the :extension interpolation key if the basename is given" do
+        spec = self
+        configure_attachment do
+          url "/:attachment.:style.:extension"
+        end
+        @thing.photo.interpolate_url(:original, :basename => 'file.xyz').should == "/photo.original.xyz"
+      end
+
+      it "should support the :extension interpolation key if the extension is given" do
+        spec = self
+        configure_attachment do
+          url "/:attachment.:style.:extension"
+        end
+        @thing.photo.interpolate_url(:original, :extension => 'xyz').should == "/photo.original.xyz"
+      end
+    end
+  end
+end

data/spec/unit/attachment/pdf_spec.rb

@@ -0,0 +1,137 @@
+require 'spec_helper'
+
+describe Attachment::Pdf do
+  use_model_class(:Thing,
+                  :attachment_file_name => :string,
+                  :attachment_width => :integer,
+                  :attachment_height => :integer,
+                  :attachment_aspect_ratio => :float,
+                  :attachment_dimensions => :string)
+
+  before do
+    Thing.has_attachment :attachment do
+      style :double, :size => '1224x1584'
+      style :filled, :size => '500x500', :filled => true
+      style :unfilled, :size => '1000x1000'
+      default_style :double
+    end
+    @thing = Thing.new(:attachment => test_file)
+  end
+
+  def test_file
+    path = "#{temporary_directory}/test.pdf"
+    FileUtils.cp("#{ROOT}/spec/data/test.pdf", path)
+    autoclose open(path)
+  end
+
+  def configure(&block)
+    Thing.attachment_reflections[:attachment].configure(&block)
+  end
+
+  describe "#process" do
+    it "should be processed with ImageMagick by default" do
+      context = nil
+      configure do
+        style :output
+        process :on => :event do
+          context = self
+        end
+      end
+
+      @thing.attachment.process(:event)
+      context.should be_a(Processor::ImageMagick)
+    end
+  end
+
+  describe "#dimensions" do
+    it "should return 1x1 if the style is missing" do
+      Thing.attachment_reflections[:attachment].configure do
+        detect_type_by{:pdf}
+      end
+      @thing.save.should be_true
+      File.unlink(@thing.attachment.path(:original))
+      @thing = Thing.find(@thing.id)
+      @thing.attachment.is_a?(Attachment::Pdf)  # sanity check
+      @thing.attachment.stream.missing?         # sanity check
+      @thing.attachment.dimensions(:original).should == [1, 1]
+    end
+
+    it "should return the width and height of the default style if no style name is given" do
+      @thing.attachment.dimensions.should == [1224, 1584]
+    end
+
+    it "should return the width and height of the given style" do
+      @thing.attachment.dimensions(:original).should == [612, 792]
+      @thing.attachment.dimensions(:double).should == [1224, 1584]
+    end
+
+    it "should return the calculated width according to style filledness" do
+      @thing.attachment.dimensions(:filled).should == [500, 500]
+      @thing.attachment.dimensions(:unfilled).should == [773, 1000]
+    end
+
+    it "should only invoke identify once"
+    it "should log the result"
+  end
+
+  describe "#width" do
+    it "should return the width of the default style if no style name is given" do
+      @thing.attachment.width.should == 1224
+    end
+
+    it "should return the width of the given style" do
+      @thing.attachment.width(:original).should == 612
+      @thing.attachment.width(:double).should == 1224
+    end
+  end
+
+  describe "#height" do
+    it "should return the height of the default style if no style name is given" do
+      @thing.attachment.height.should == 1584
+    end
+
+    it "should return the height of the given style" do
+      @thing.attachment.height(:original).should == 792
+      @thing.attachment.height(:double).should == 1584
+    end
+  end
+
+  describe "#aspect_ratio" do
+    it "should return the aspect ratio of the default style if no style name is given" do
+      @thing.attachment.aspect_ratio.should be_close(612.0/792, 1e-5)
+    end
+
+    it "should return the aspect ratio of the given style" do
+      @thing.attachment.aspect_ratio(:original).should be_close(612.0/792, 1e-5)
+      @thing.attachment.aspect_ratio(:filled).should be_close(1, 1e-5)
+    end
+  end
+
+  describe "storable attributes" do
+    it "should set the stored attributes on assignment" do
+      @thing.attachment_width.should == 612
+      @thing.attachment_height.should == 792
+      @thing.attachment_aspect_ratio.should be_close(612.0/792, 1e-5)
+      @thing.attachment_dimensions.should == '612x792'
+    end
+
+    describe "after roundtripping through the database" do
+      before do
+        @thing.save
+        @thing = Thing.find(@thing.id)
+      end
+
+      it "should restore the stored attributes" do
+        @thing.attachment_width.should == 612
+        @thing.attachment_height.should == 792
+        @thing.attachment_aspect_ratio.should be_close(612.0/792, 1e-5)
+        @thing.attachment_dimensions.should == '612x792'
+      end
+
+      it "should recalculate the dimensions correctly" do
+        @thing.attachment.dimensions(:filled).should == [500, 500]
+        @thing.attachment.dimensions(:unfilled).should == [773, 1000]
+      end
+    end
+  end
+end

data/spec/unit/attachment/video_spec.rb

@@ -0,0 +1,176 @@
+require 'spec_helper'
+
+describe Attachment::Video do
+  use_model_class(:Thing,
+                  :video_file_name => :string,
+                  :video_width => :integer,
+                  :video_height => :integer,
+                  :video_aspect_ratio => :float,
+                  :video_dimensions => :string,
+                  :video_duration => :string)
+
+  before do
+    Thing.has_attachment :video do
+      # original video is 640x480.
+      style :half, :size => '320x240'
+      style :filled, :size => '60x60', :filled => true
+      style :unfilled, :size => '120x120'
+      default_style :half
+    end
+    @thing = Thing.new(:video => test_file)
+  end
+
+  def test_file
+    path = "#{temporary_directory}/test.mov"
+    FileUtils.cp("#{ROOT}/spec/data/test.mov", path)
+    autoclose open(path)
+  end
+
+  def run(command)
+    `#{command}`
+    $?.success? or
+      raise "command failed: #{command}"
+  end
+
+  describe "#dimensions" do
+    it "should return 2x2 if the style is missing" do
+      Thing.attachment_reflections[:video].configure do
+        detect_type_by{:video}
+      end
+      @thing.save.should be_true
+      File.unlink(@thing.video.path(:original))
+      @thing = Thing.find(@thing.id)
+      @thing.video.is_a?(Attachment::Video)  # sanity check
+      @thing.video.stream.missing?           # sanity check
+      @thing.video.dimensions(:original).should == [2, 2]
+    end
+
+    it "should return the width and height of the default style if no style name is given" do
+      @thing.video.dimensions.should == [320, 240]
+    end
+
+    it "should return the width and height of the given style" do
+      @thing.video.dimensions(:original).should == [640, 480]
+      @thing.video.dimensions(:half).should == [320, 240]
+    end
+
+    it "should return the calculated width according to style filledness" do
+      @thing.video.dimensions(:filled).should == [60, 60]
+      @thing.video.dimensions(:unfilled).should == [120, 90]
+    end
+
+    it "should round calculated dimensions down to the nearest multiple of 2" do
+      # TODO: ick!
+      Thing.attachment_reflections[:video].styles[:filled][:size] = '59x59'
+      @thing.video.dimensions(:filled).should == [58, 58]
+    end
+
+    it "should only invoke ffmpeg once"
+    it "should log the result"
+  end
+
+  describe "#width" do
+    it "should return the width of the default style if no style name is given" do
+      @thing.video.width.should == 320
+    end
+
+    it "should return the width of the given style" do
+      @thing.video.width(:original).should == 640
+      @thing.video.width(:half).should == 320
+    end
+  end
+
+  describe "#height" do
+    it "should return the height of the default style if no style name is given" do
+      @thing.video.height.should == 240
+    end
+
+    it "should return the height of the given style" do
+      @thing.video.height(:original).should == 480
+      @thing.video.height(:half).should == 240
+    end
+  end
+
+  describe "#aspect_ratio" do
+    it "should return the aspect ratio of the default style if no style name is given" do
+      @thing.video.aspect_ratio.should be_close(4.0/3, 1e-5)
+    end
+
+    it "should return the aspect ratio of the given style" do
+      @thing.video.aspect_ratio(:original).should be_close(4.0/3, 1e-5)
+      @thing.video.aspect_ratio(:filled).should be_close(1, 1e-5)
+    end
+  end
+
+  describe "#duration" do
+    it "should return the duration of the original style if no style name is given" do
+      @thing.video.duration.should == 1.second
+    end
+
+    it "should return the duration of the original style if a style name is given" do
+      @thing.video.duration(:filled).should == 1.second
+    end
+
+    # TODO: make these work instead of the above
+    it "should return the duration of the default style if no style name is given"
+    it "should return the duration of the given style"
+  end
+
+  describe "#video_tracks" do
+    it "should return the video tracks of the original style if no style name is given" do
+      @thing.video.video_tracks.should have(1).video_track
+      @thing.video.video_tracks.first.dimensions.should == [320, 240]
+    end
+
+    it "should return the video tracks of the target style if a style name is given" do
+      @thing.video.video_tracks(:original).should have(1).video_track
+      @thing.video.video_tracks(:original).first.dimensions.should == [640, 480]
+
+      @thing.video.video_tracks(:filled).should have(1).video_track
+      @thing.video.video_tracks(:filled).first.dimensions.should == [60, 60]
+    end
+  end
+
+  describe "#audio_tracks" do
+    it "should return the audio tracks of the original style if no style name is given" do
+      @thing.video.video_tracks.should have(1).video_track
+      @thing.video.video_tracks.first.dimensions.should == [320, 240]
+    end
+
+    it "should return the audio tracks of the target style if a style name is given" do
+      @thing.video.video_tracks(:original).should have(1).video_track
+      @thing.video.video_tracks(:original).first.dimensions.should == [640, 480]
+
+      @thing.video.video_tracks(:filled).should have(1).video_track
+      @thing.video.video_tracks(:filled).first.dimensions.should == [60, 60]
+    end
+  end
+
+  describe "storable attributes" do
+    it "should set the stored attributes on assignment" do
+      @thing.video_width.should == 640
+      @thing.video_height.should == 480
+      @thing.video_aspect_ratio.should be_close(4.0/3, 1e-5)
+      @thing.video_dimensions.should == '640x480'
+    end
+
+    describe "after roundtripping through the database" do
+      before do
+        @thing.save
+        @thing = Thing.find(@thing.id)
+      end
+
+      it "should restore the stored attributes" do
+        @thing.video_width.should == 640
+        @thing.video_height.should == 480
+        @thing.video_aspect_ratio.should be_close(4.0/3, 1e-5)
+        @thing.video_dimensions.should == '640x480'
+      end
+
+      it "should recalculate the dimensions correctly" do
+        @thing.video.dimensions(:filled).should == [60, 60]
+        @thing.video.dimensions(:unfilled).should == [120, 90]
+      end
+    end
+  end
+end