oahu-dragonfly 0.8.2

data/spec/functional/deprecations_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+describe 'deprecations' do
+  
+  before(:each) do
+    @app = test_app
+  end
+  
+  describe "url_suffix" do
+    it "should give an appropriate error" do
+      @app.configure do |c|
+        expect{
+          c.url_suffix = 'darble'
+        }.to raise_error(NoMethodError, /deprecated.*please use url_format/)
+      end
+    end
+  end
+  
+  describe "url_path_prefix" do
+    it "should give an appropriate error" do
+      @app.configure do |c|
+        expect{
+          c.url_path_prefix = '/darble'
+        }.to raise_error(NoMethodError, /deprecated.*please use url_format/)
+      end
+    end
+  end
+  
+  describe "middleware" do
+    it "should give an appropriate error" do
+      app = Rack::Builder.new do
+        use Dragonfly::Middleware, :images, '/media'
+        run proc{[200, {}, []]}
+      end
+      expect{
+        app.call({})
+      }.to raise_error(ArgumentError, /deprecated/)
+    end
+  end
+  
+  describe "infer_mime_type_from_file_ext" do
+    it "should give an appropriate error" do
+      @app.configure do |c|
+        expect{
+          c.infer_mime_type_from_file_ext = false
+        }.to raise_error(NoMethodError, /deprecated.*please use trust_file_extensions = false/)
+      end
+    end
+  end
+  
+end

data/spec/functional/image_magick_app_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe "a configured imagemagick app" do
+  
+  before(:each) do
+    @app = Dragonfly[:images].configure_with(:imagemagick)
+  end
+  
+  describe "convert command path" do
+    before(:each) do
+      @processor = @app.processor.get_registered(Dragonfly::ImageMagick::Processor)
+    end
+    
+    it "should default to 'convert'" do
+      @processor.convert_command.should == 'convert'
+    end
+    
+    it "should change when configured through the app" do
+      @app.configure do |c|
+        c.convert_command = '/usr/eggs'
+      end
+      @processor.convert_command.should == '/usr/eggs'
+    end
+    
+  end
+  
+end

data/spec/functional/model_urls_spec.rb

@@ -0,0 +1,85 @@
+require 'dragonfly/active_model_extensions/spec_helper'
+require 'tempfile'
+
+describe "model urls" do
+  
+  def new_tempfile(data='HELLO', filename='hello.txt')
+    tempfile = Tempfile.new('test')
+    tempfile.write(data)
+    tempfile.rewind
+    tempfile.stub!(:original_filename).and_return(filename)
+    tempfile
+  end
+  
+  before(:each) do
+    @app = test_app.configure do |c|
+      c.url_format = '/media/:job/:basename.:format'
+    end
+    @app.define_macro(MyModel, :image_accessor)
+    Item.class_eval do
+      image_accessor :preview_image # has name etc.
+      image_accessor :other_image # doesn't have magic stuff
+    end
+    @item = Item.new
+  end
+  
+  it "should include the name in the url if it has the magic attribute" do
+    @item.preview_image = new_tempfile
+    @item.save!
+    item = Item.find(@item.id)
+    item.preview_image.url.should =~ %r{^/media/\w+/hello\.txt$}
+  end
+  
+  it "should work without the name if the name magic attr doesn't exist" do
+    @item.other_image = new_tempfile
+    @item.save!
+    item = Item.find(@item.id)
+    item.other_image.url.should =~ %r{^/media/\w+$}
+  end
+  
+  it "should include the name in the url even if it has no ext" do
+    @item.preview_image = new_tempfile("hello", 'hello')
+    @item.save!
+    item = Item.find(@item.id)
+    item.preview_image.url.should =~ %r{^/media/\w+/hello$}
+  end
+  
+  it "should change the ext when there's an encoding step" do
+    @item.preview_image = new_tempfile
+    @item.save!
+    item = Item.find(@item.id)
+    item.preview_image.encode(:bum).url.should =~ %r{^/media/\w+/hello\.bum$}
+  end
+  
+  it "should not include the name if it has none" do
+    @item.preview_image = "HELLO"
+    @item.save!
+    item = Item.find(@item.id)
+    item.preview_image.url.should =~ %r{^/media/\w+$}
+  end
+  
+  it "should have an ext when there's an encoding step but no name" do
+    @item.preview_image = "HELLO"
+    @item.save!
+    item = Item.find(@item.id)
+    item.preview_image.encode(:bum).url.should =~ %r{^/media/\w+\.bum$}
+  end
+  
+  it "should work as normal with dos protection" do
+    @app.server.protect_from_dos_attacks = true
+    @item.preview_image = new_tempfile
+    @item.save!
+    item = Item.find(@item.id)
+    item.preview_image.url.should =~ %r{^/media/\w+/hello\.txt\?sha=\w+$}
+  end
+  
+  it "should allow configuring the url" do
+    @app.configure do |c|
+      c.url_format = '/img/:job'
+    end
+    @item.preview_image = new_tempfile
+    @item.save!
+    item = Item.find(@item.id)
+    item.preview_image.url.should =~ %r{^/img/\w+$}
+  end
+end

data/spec/functional/remote_on_the_fly_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+describe "remote on-the-fly urls" do
+  
+  THUMBS = {}
+  
+  before(:each) do
+    @app = test_app.configure do |c|
+      c.generator.add :test do
+        "TEST"
+      end
+      c.server.before_serve do |job, env|
+        uid = job.store(:path => 'yay.txt')
+        THUMBS[job.serialize] = uid
+      end
+      c.define_url do |app, job, opts|
+        uid = THUMBS[job.serialize]
+        if uid
+          app.datastore.url_for(uid)
+        else
+          app.server.url_for(job)
+        end
+      end
+      c.datastore = Dragonfly::DataStorage::FileDataStore.new
+      c.datastore.root_path = '/var/tmp/dragonfly'
+      c.datastore.server_root = '/var/tmp'
+    end
+    @job = @app.generate(:test)
+  end
+
+  after(:each) do
+    THUMBS.delete_if{true}
+    FileUtils.rm_f('/var/tmp/dragonfly/yay.txt')
+  end
+  
+  it "should give the url for the server" do
+    @job.url.should == "/#{@job.serialize}"
+  end
+  
+  it "should store the content when first called" do
+    File.exist?('/var/tmp/dragonfly/yay.txt').should be_false
+    @app.server.call('PATH_INFO' => @job.url, 'REQUEST_METHOD' => 'GET')
+    File.read('/var/tmp/dragonfly/yay.txt').should == 'TEST'
+  end
+
+  it "should point to the external url the second time" do
+    @app.server.call('PATH_INFO' => @job.url, 'REQUEST_METHOD' => 'GET')
+    @job.url.should == '/dragonfly/yay.txt'
+  end
+
+end

data/spec/functional/to_response_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe "getting rack response directly" do
+  
+  before(:each) do
+    @app = Dragonfly[:to_response].configure do |c|
+      c.generator.add :test do
+        "bunheads"
+      end
+    end
+  end
+  
+  it "should give a rack response" do
+    response = @app.generate(:test, 1, 1).to_response
+    response.should be_a(Array)
+    response.length.should == 3
+    response[0].should == 200
+    response[1]['Content-Type'].should == 'application/octet-stream'
+    response[2].data.should == 'bunheads'
+  end
+  
+  it "should allow passing in the env" do
+    response = @app.generate(:test, 1, 1).to_response('REQUEST_METHOD' => 'POST')
+    response.should be_a(Array)
+    response.length.should == 3
+    response[0].should == 405
+    response[1]['Content-Type'].should == 'text/plain'
+    response[2].should == ["POST method not allowed"]
+  end
+  
+end

data/spec/spec_helper.rb

@@ -0,0 +1,51 @@
+require "rubygems"
+require "bundler"
+Bundler.setup(:default, :test)
+
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+$LOAD_PATH.unshift(File.dirname(__FILE__))
+require 'rspec'
+require 'dragonfly'
+require 'fileutils'
+
+require 'webmock/rspec'
+
+# Requires supporting files with custom matchers and macros, etc,
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f}
+
+SAMPLES_DIR = File.expand_path(File.dirname(__FILE__) + '/../samples') unless defined?(SAMPLES_DIR)
+
+def todo
+  raise "TODO"
+end
+
+require 'logger'
+LOG_FILE = File.dirname(__FILE__) + '/spec.log' unless defined?(LOG_FILE)
+FileUtils.rm_rf(LOG_FILE)
+def mock_app(extra_stubs={})
+  mock('app', {
+    :datastore => mock('datastore', :store => 'some_uid', :retrieve => ["SOME_DATA", {}], :destroy => nil),
+    :processor => mock('processor', :process => "SOME_PROCESSED_DATA"),
+    :encoder => mock('encoder', :encode => "SOME_ENCODED_DATA"),
+    :analyser => mock('analyser', :analyse => "some_result", :analysis_methods => Module.new),
+    :generator => mock('generator', :generate => "SOME_GENERATED_DATA"),
+    :log => Logger.new(LOG_FILE),
+    :cache_duration => 10000,
+    :job_definitions => Module.new
+  }.merge(extra_stubs)
+  )
+end
+
+def test_app
+  Dragonfly::App.send(:new)
+end
+
+def suppressing_stderr
+  original_stderr = $stderr.dup
+  tempfile = Tempfile.new('stderr')
+  $stderr.reopen(tempfile)
+  yield
+ensure
+  tempfile.close!
+  $stderr.reopen(original_stderr)
+end

data/spec/support/argument_matchers.rb

@@ -0,0 +1,19 @@
+def string_matching(regexp)
+  Spec::Mocks::ArgumentMatchers::RegexpMatcher.new(regexp)
+end
+
+class TempObjectArgumentMatcher
+  def initialize(data, opts)
+    @data = data
+    @opts = opts
+  end
+  def ==(actual)
+    actual.is_a?(Dragonfly::TempObject) &&
+      actual.data == @data &&
+      @opts.all?{|k,v| actual.send(k) == v }
+  end
+end
+
+def a_temp_object_with_data(data, opts={})
+  TempObjectArgumentMatcher.new(data, opts)
+end

data/spec/support/image_matchers.rb

@@ -0,0 +1,47 @@
+def image_properties(image)
+  if image.is_a?(Tempfile)
+    tempfile = image
+  else
+    tempfile = Tempfile.new('image')
+    tempfile.write(image.is_a?(Dragonfly::TempObject) ? image.data : image)
+    tempfile.close
+  end
+  details = `identify #{tempfile.path}`
+  # example of details string:
+  # myimage.png PNG 200x100 200x100+0+0 8-bit DirectClass 31.2kb
+  filename, format, geometry, geometry_2, depth, image_class, size = details.split(' ')
+  width, height = geometry.split('x')
+  {
+    :filename => filename,
+    :format => format.downcase,
+    :width => width,
+    :height => height,
+    :depth => depth,
+    :image_class => image_class,
+    :size => size
+  }
+end
+
+RSpec::Matchers.define :have_width do |width|
+  match do |given|
+    width.should === image_properties(given)[:width].to_i
+  end
+end
+
+RSpec::Matchers.define :have_height do |height|
+  match do |given|
+    height.should === image_properties(given)[:height].to_i
+  end
+end
+
+RSpec::Matchers.define :have_format do |format|
+  match do |given|
+    image_properties(given)[:format].should == format
+  end
+end
+
+RSpec::Matchers.define :have_size do |size|
+  match do |given|
+    image_properties(given)[:size].should == size
+  end
+end

data/spec/support/simple_matchers.rb

@@ -0,0 +1,53 @@
+RSpec::Matchers.define :match_url do |url|
+  match do |given|
+    given_path, given_query_string = given.split('?')
+    path, query_string = url.split('?')
+
+    path == given_path && given_query_string.split('&').sort == query_string.split('&').sort
+  end
+end
+
+RSpec::Matchers.define :be_an_empty_directory do
+  match do |given|
+    Dir.entries(given) == ['.','..']
+  end
+end
+
+RSpec::Matchers.define :include_hash do |hash|
+  match do |given|
+    given.merge(hash) == given
+  end
+end
+
+def memory_usage
+  GC.start # Garbage collect
+  `ps -o rss= -p #{$$}`.strip.to_i
+end
+
+RSpec::Matchers.define :leak_memory do
+  match do |given|
+    memory_before = memory_usage
+    given.call
+    memory_after = memory_usage
+    result = memory_after > memory_before
+    puts "#{memory_after} > #{memory_before}" if result
+    result
+  end
+end
+
+RSpec::Matchers.define :match_attachment_classes do |classes|
+  match do |given_classes|
+    given_classes.length == classes.length &&
+      classes.zip(given_classes).all? do |(klass, given)|
+        given.model_class == klass[0] && given.attribute == klass[1] && given.app == klass[2]
+      end
+  end
+end
+
+RSpec::Matchers.define :be_a_text_response do
+  match do |given_response|
+    given_response.status.should == 200
+    given_response.body.length.should > 0
+    given_response.content_type.should == 'text/plain'
+  end
+end

data/yard/handlers/configurable_attr_handler.rb

@@ -0,0 +1,38 @@
+class ConfigurableAttrHandler < YARD::Handlers::Ruby::Legacy::Base
+  handles(/^\s*configurable_attr/)
+
+  def process
+    namespace[:configurable_attributes] ||= []
+    
+    attribute = token_to_object(statement.tokens[2])
+    comments = statement.comments.join(' ') if statement.comments
+    
+    if statement.block
+    # e.g. configurable_attr :datastore do FileDataStore.new end
+      lazy_default_value = statement.block.to_s
+    else
+    # e.g. configurable_attr :fallback_mime_type, 'application/octet-stream'    
+      default_value = token_to_object(statement.tokens[5..-1])
+    end
+    namespace[:configurable_attributes] << {
+      :attribute => attribute,
+      :default_value => default_value,
+      :lazy_default_value => lazy_default_value,
+      :comments => comments
+    }
+  end
+
+  private
+
+  def token_to_object(token)
+    if token
+      if token.is_a?(YARD::Parser::Ruby::Legacy::TokenList)
+        eval(token.to_s)
+      else # is a single token
+        eval(token.text)
+      end
+    end
+  end
+
+end
+