sample_file 0.0.3 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7f164867285558f9f7193a2397a3987dcd2344c2
-  data.tar.gz: 1442c5272e887e94065e0990c3c525cbde092829
+  metadata.gz: fdaf48797664f50a5c11170b2c258faf7d38a629
+  data.tar.gz: 7f7ba026781765ec2ad341b8552f8cbb755960e6
 SHA512:
-  metadata.gz: ed734e265f79d85d4f982502d12e10550730a78ebfbe54e349ed688c24feab2751e608a1f5f8f7ba3d6f1d102dae89a323530dc77fe80c7a3d3fcdcf1cf38dc4
-  data.tar.gz: 114d53567e8d7925a9750243b48d459d0504b4da6f415cfa6ed7a01f41666b07d1b4c9ec18e3228f6e288dc0e52ace85eaeabe89ead2d7f83957c7d7bc364ebf
+  metadata.gz: 6391e288725356c4444e41f7e639497a184aae0b3391b917deb03a771acc1748750e559d5a27af692f277d9524a43630556bdbc40b069fded6cd5b664433d5c1
+  data.tar.gz: a130a31921aa4547e2b9f2084ae8c5eece52817899f583d29003762b52cf334beedfe5286d06e3330dd81c0bdb8b80ef5a282e2ab321c6f6829a8c5ef4fd93bf

data/README.md

@@ -24,12 +24,14 @@ SampleFile.image
 SampleFile.image :jpg
 SampleFile.image :gif
 SampleFile.image :png
+SampleFile.image :jpg, width: 300, height: 50
 
 # absolute path to image
 SampleFile.image_path 
 SampleFile.image_path :jpg
 SampleFile.image_path :gif
 SampleFile.image_path :png
+SampleFile.image_path :jpg, width: 300, height: 50
 
 # returns a video file
 SampleFile.video

data/lib/sample_file.rb

@@ -1,3 +1,7 @@
+require "rubygems"
+require "bundler/setup"
+require "mini_magick"
+
 require "sample_file/version"
 require "sample_file/base"
 require "sample_file/image"
@@ -5,20 +9,20 @@ require "sample_file/video"
 
 module SampleFile
   class << self
-    def image(type='png')
-      Image.file(type)
+    def image(type='png', opts={})
+      Image.new(type, opts).file
     end
 
-    def image_path(type='png')
-      Image.file_path(type)
+    def image_path(type='png', opts={})
+      Image.new(type, opts).file_path
     end
 
-    def video(type='h264')
-      Video.file(type)
+    def video(type='h264', opts={})
+      Video.new(type, opts).file
     end
 
-    def video_path(type='h264')
-      Video.file_path(type)
+    def video_path(type='h264', opts={})
+      Video.new(type, opts).file_path
     end
   end
 end

data/lib/sample_file/base.rb

@@ -1,10 +1,10 @@
 module SampleFile
   class Base
-    def file(type=nil)
-      File.open file_path(type)
+    def file
+      File.open file_path
     end
 
-    def file_path(type=nil)
+    def file_path
       raise NotImplementedError, "#{self.class} does not implement the file_path method"
     end
 
@@ -13,20 +13,5 @@ module SampleFile
     def base_path
       @base_path ||= File.expand_path File.join(File.dirname(__FILE__), '..', 'files')
     end
-
-    class << self
-      def method_missing method, *args
-        instance.send method, *args
-      end
-
-      def respond_to? method
-        return true if instance.respond_to? method
-        super
-      end
-
-      def instance
-        @@instance ||= self.new
-      end
-    end
   end
 end

data/lib/sample_file/image.rb

@@ -1,22 +1,74 @@
+require 'fileutils'
+
 module SampleFile
   class Image < Base
-    def file_path(type='png')
-      filename = filename_for_type(type)
-      File.join(image_dir_path, filename)
+    attr_reader :type, :width, :height
+
+    def initialize(type='png', opts={})
+      @type = type
+      @width  = opts[:width]  || opts[:height]
+      @height = opts[:height] || opts[:width]
     end
 
-    def file(type='png')
-      super
+    def file_path
+      if width && height
+        path_to_resized_image
+      else
+        path_to_base_image
+      end
+    end
+
+    # Temporary directory where images will be stored
+    #
+    def tmpdir
+      @tmpdir ||= File.join(Dir.tmpdir, 'sample_file', 'image')
     end
 
     protected
 
-    def image_dir_path
-      File.join(base_path, 'image')
+    def path_to_base_image
+      @path_to_base_image ||= File.join(base_path, 'image', base_filename)
     end
 
-    def filename_for_type(type)
+    def base_filename
       "sample.#{type}"
     end
+
+    def ensure_tmpdir_exists
+      FileUtils.mkdir_p(tmpdir) unless File.exists?(tmpdir)
+    end
+
+    def path_to_resized_image
+      @path_to_resized_image ||= determine_resized_image_path
+    end
+
+    def determine_resized_image_path
+      destination = File.join(tmpdir, filename)
+      ensure_tmpdir_exists
+      FileUtils.cp(create_resized_image.path, destination)
+      destination
+    end
+
+    def filename
+      @filename ||= determine_filename
+    end
+
+    def determine_filename
+      if width && height
+        filename_with_dimensions
+      else
+        base_filename
+      end
+    end
+
+    def filename_with_dimensions
+      "sample_#{width}x#{height}.#{type}"
+    end
+
+    def create_resized_image
+      image = ::MiniMagick::Image.open(path_to_base_image)
+      image.resize "#{width.to_i}x#{height.to_i}!"
+      File.open image.path
+    end
   end
 end

data/lib/sample_file/version.rb

@@ -1,3 +1,3 @@
 module SampleFile
-  VERSION = "0.0.3"
+  VERSION = "0.1.0"
 end

data/lib/sample_file/video.rb

@@ -1,6 +1,10 @@
 module SampleFile
   class Video < Base
-    def file_path(type='h264')
+    def initialize(type='h264', opts={})
+      # TODO Implement provide samples of multiple video codecs
+    end
+
+    def file_path
       File.join(video_dir_path, 'h264.mp4')
     end
 

data/sample_file.gemspec

@@ -22,4 +22,6 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rspec"
   spec.add_development_dependency "ruby-imagespec", "~> 0.3.1"
+
+  spec.add_dependency "mini_magick", ">= 3.6.0"
 end

data/spec/sample_file/image_spec.rb

@@ -1,4 +1,5 @@
 require 'spec_helper'
+require 'fileutils'
 
 describe SampleFile::Image do
   it_behaves_like "a sample file", %w(gif png jpg)
@@ -11,13 +12,57 @@ describe SampleFile::Image do
 
   image_types.each_pair do |image_type, content_type|
     context "when requesting a '#{image_type}' image" do
+      subject { described_class.new(image_type) }
+
       describe :file do
         it "is of the correct content type" do
-          data = ImageSpec.new(subject.file(image_type))
-          data.content_type.should == content_type
+          image = MiniMagick::Image.open subject.file.path
+          image.mime_type.should == content_type
+        end
+
+        context "when passing :width and :height options" do
+          subject { described_class.new(image_type, width: 75, height: 200) }
+          it "returns an image of those dimensions" do
+            image = MiniMagick::Image.open subject.file.path
+            image['width'].should == 75
+            image['height'].should == 200
+          end
+        end
+
+        context "when only :width is set" do
+          subject { described_class.new(image_type, width: 35) }
+          it "returns a square image" do
+            image = MiniMagick::Image.open subject.file.path
+            image['width'].should == 35
+            image['height'].should eq image['width']
+          end
+        end
+
+        context "when only :height is set" do
+          subject { described_class.new(image_type, height: 45) }
+          it "returns a square image" do
+            image = MiniMagick::Image.open subject.file.path
+            image['width'].should == 45
+            image['height'].should eq image['width']
+          end
         end
       end
-    end
-  end
+
+      describe :file_path do
+        context "when passing :width and :height options" do
+          subject { described_class.new(image_type, width: 75, height: 200) }
+          before(:each) do
+            FileUtils.rm_r(subject.tmpdir)
+          end
+          its(:file_path) { should match(/sample_75x200\.#{image_type}$/) }
+          it "returns a path to an image that exists" do
+            path = subject.file_path
+            File.exists?(path).should be_true
+          end
+        end
+      end
+
+    end # context
+  end # each image_type
 
 end

data/spec/sample_file_spec.rb

@@ -1,32 +1,24 @@
 require 'spec_helper'
 
 describe SampleFile do
+  %w(Image Video).each do |class_name|
+    klass = const_get("SampleFile::#{class_name}")
 
-  describe :image do
-    it "should call #file on a new instance of Image" do
-      described_class::Image.should_receive(:file)
-      described_class.image
+    file_method = class_name.downcase
+    describe file_method do
+      it "should call #file on an instance of #{class_name}" do
+        klass.any_instance.should_receive(:file)
+        described_class.send file_method
+      end
     end
-  end
-
-  describe :image_path do
-    it "should call #file_path on Image" do
-      described_class::Image.should_receive(:file_path)
-      described_class.image_path
-    end
-  end
 
-  describe :video do
-    it "should call #file on Video" do
-      described_class::Video.should_receive(:file)
-      described_class.video
+    file_path_method = "#{file_method}_path"
+    describe file_path_method do
+      it "should call #file_path on an instance of #{class_name}" do
+        klass.any_instance.should_receive(:file_path)
+        described_class.send file_path_method
+      end
     end
-  end
 
-  describe :video_path do
-    it "should call #file_path on Video" do
-      described_class::Video.should_receive(:file_path)
-      described_class.video_path
-    end
   end
 end

data/spec/support/sample_file.rb

@@ -1,39 +1,31 @@
 shared_examples_for 'a sample file' do |file_types=[]|
-  subject { described_class }
+  subject { described_class.new }
 
   describe :file_path do
     it { should respond_to(:file_path) }
-
     context "when no file type is provided" do
       it "should point to an existing file" do
         File.exists?(subject.file_path).should be_true
       end
     end
-
-    file_types.each do |type|
-      context "when file_type is #{type}" do
-        it "should point to an existing file" do
-          File.exists?(subject.file_path(type)).should be_true
-        end
-      end
-    end
   end
 
   describe :file do
     it { should respond_to(:file) }
-
     context "when no file type is provided" do
       it "opens a file using file_path" do
         File.should_receive(:open).with(subject.file_path)
         subject.file
       end
     end
+  end
 
-    file_types.each do |type|
-      context "when file_type is #{type}" do
-        it "opens a file using file_path" do
-          File.should_receive(:open).with(subject.file_path(type))
-          subject.file(type)
+  file_types.each do |type|
+    context "when file_type is #{type}" do
+      subject { described_class.new(type) }
+      describe :file_path do
+        it "should point to an existing file" do
+          File.exists?(subject.file_path).should be_true
         end
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sample_file
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.1.0
 platform: ruby
 authors:
 - Maarten Claes
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-25 00:00:00.000000000 Z
+date: 2013-05-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -66,6 +66,20 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 0.3.1
+- !ruby/object:Gem::Dependency
+  name: mini_magick
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 3.6.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 3.6.0
 description: Sample files for testing purposes
 email:
 - maartencls@gmail.com