fog-dragonfly 0.8.1

data/spec/dragonfly/generation/shared_generator_spec.rb

@@ -0,0 +1,91 @@
+require 'spec_helper'
+
+# Needs: @generator
+describe "image generator", :shared => true do
+  
+  describe "generating an image with the given dimensions" do
+    before(:each) do
+      @image, @extra = @generator.plasma(23,12)
+    end
+    it {@image.should have_width(23)}
+    it {@image.should have_height(12)}
+    it {@image.should have_format('png')}
+    it {@extra.should == {:format => :png, :name => 'plasma.png'}}
+  end
+
+  describe "specifying the format" do
+    before(:each) do
+      @image, @extra = @generator.plasma(23, 12, :gif)
+    end
+    it {@image.should have_format('gif')}
+    it {@extra.should == {:format => :gif, :name => 'plasma.gif'}}
+  end
+
+  describe "text" do
+    before(:each) do
+      @text = "mmm"
+    end
+
+    describe "creating a text image" do
+      before(:each) do
+        @image, @extra = @generator.text(@text, :font_size => 12)
+      end
+      it {@image.should have_width(20..40)} # approximate
+      it {@image.should have_height(10..20)}
+      it {@image.should have_format('png')}
+      it {@extra.should == {:format => :png, :name => 'text.png'}}
+    end
+
+    describe "specifying the format" do
+      before(:each) do
+        @image, @extra = @generator.text(@text, :format => :gif)
+      end
+      it {@image.should have_format('gif')}
+      it {@extra.should == {:format => :gif, :name => 'text.gif'}}
+    end
+
+    describe "padding" do
+      before(:each) do
+        no_padding_text, extra = @generator.text(@text, :font_size => 12)
+        @width = image_properties(no_padding_text)[:width].to_i
+        @height = image_properties(no_padding_text)[:height].to_i
+      end
+      it "1 number shortcut" do
+        image, extra = @generator.text(@text, :padding => '10')
+        image.should have_width(@width + 20)
+        image.should have_height(@height + 20)
+      end
+      it "2 numbers shortcut" do
+        image, extra = @generator.text(@text, :padding => '10 5')
+        image.should have_width(@width + 10)
+        image.should have_height(@height + 20)
+      end
+      it "3 numbers shortcut" do
+        image, extra = @generator.text(@text, :padding => '10 5 8')
+        image.should have_width(@width + 10)
+        image.should have_height(@height + 18)
+      end
+      it "4 numbers shortcut" do
+        image, extra = @generator.text(@text, :padding => '1 2 3 4')
+        image.should have_width(@width + 6)
+        image.should have_height(@height + 4)
+      end
+      it "should override the general padding declaration with the specific one (e.g. 'padding-left')" do
+        image, extra = @generator.text(@text, :padding => '10', 'padding-left' => 9)
+        image.should have_width(@width + 19)
+        image.should have_height(@height + 20)
+      end
+      it "should ignore 'px' suffixes" do
+        image, extra = @generator.text(@text, :padding => '1px 2px 3px 4px')
+        image.should have_width(@width + 6)
+        image.should have_height(@height + 4)
+      end
+      it "bad padding string" do
+        lambda{
+          @generator.text(@text, :padding => '1 2 3 4 5')
+        }.should raise_error(ArgumentError)
+      end
+    end
+  end
+
+end

data/spec/dragonfly/image_magick_utils_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+
+describe Dragonfly::ImageMagickUtils do
+
+  before(:each) do
+    @obj = Object.new
+    @obj.extend(Dragonfly::ImageMagickUtils)
+  end
+  
+  it "should raise an error if the identify command isn't found" do
+    lambda{
+      @obj.send(:run, "non-existent-command")
+    }.should raise_error(Dragonfly::ImageMagickUtils::ShellCommandFailed)
+  end
+
+end

data/spec/dragonfly/job_builder_spec.rb

@@ -0,0 +1,37 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Dragonfly::JobBuilder do
+
+  describe "a multi-step job" do
+
+    before(:each) do
+      @job_builder = Dragonfly::JobBuilder.new do |size, format|
+        process :thumb, size
+        encode format unless format.nil?
+      end
+    end
+
+    it "should correctly call job steps" do
+      job = mock
+      job.should_receive(:process).with(:thumb, '30x30#').and_return(job2=mock)
+      job2.should_receive(:encode).with(:jpg).and_return(job3=mock)
+      @job_builder.build(job, '30x30#', :jpg).should == job3
+    end
+
+    it "should work consistently with bang methods" do
+      job = mock
+      job.should_receive(:process!).with(:thumb, '30x30#').and_return(job)
+      job.should_receive(:encode!).with(:jpg).and_return(job)
+      @job_builder.build!(job, '30x30#', :jpg).should == job
+    end
+
+    it "should yield nil if the arg isn't passed in" do
+      job = mock
+      job.should_receive(:process).with(:thumb, '30x30#').and_return(job2=mock)
+      job2.should_not_receive(:encode)
+      @job_builder.build(job, '30x30#').should == job2
+    end
+
+  end
+  
+end

data/spec/dragonfly/job_definitions_spec.rb

@@ -0,0 +1,35 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Dragonfly::JobDefinitions do
+
+  describe "defining jobs" do
+
+    before(:each) do
+      @job_definitions = Dragonfly::JobDefinitions.new
+      @object = Object.new
+      @object.extend @job_definitions
+    end
+
+    describe "a simple job" do
+
+      before(:each) do
+        @job_definitions.add :thumb do |size|
+          process :thumb, size
+        end
+      end
+
+      it "correctly call job steps" do
+        @object.should_receive(:process).with(:thumb, '30x30#').and_return(job=mock)
+        @object.thumb('30x30#').should == job
+      end
+
+      it "should correctly call job steps when bang is given" do
+        @object.should_receive(:process!).with(:thumb, '30x30#').and_return(@object)
+        @object.thumb!('30x30#').should == @object
+      end
+
+    end
+
+  end
+  
+end

data/spec/dragonfly/job_endpoint_spec.rb

@@ -0,0 +1,120 @@
+# encoding: utf-8
+require File.dirname(__FILE__) + '/../spec_helper'
+
+## General tests for Response go here as it's a pretty simple wrapper around that
+
+describe Dragonfly::JobEndpoint do
+
+  def make_request(job, opts={})
+    endpoint = Dragonfly::JobEndpoint.new(job)
+    Rack::MockRequest.new(endpoint).get('', opts)
+  end
+
+  before(:each) do
+    @app = test_app
+    @app.datastore.stub!(:retrieve).with('egg').and_return(["GUNGLE", {:name => 'gung.txt'}])
+    @job = Dragonfly::Job.new(@app).fetch('egg')
+  end
+
+  it "should return a correct response if successful" do
+    response = make_request(@job)
+    response.status.should == 200
+    response['ETag'].should =~ /^"\w+"$/
+    response['Cache-Control'].should == "public, max-age=31536000"
+    response['Content-Type'].should == 'text/plain'
+    response['Content-Length'].should == '6'
+    response['Content-Disposition'].should == 'filename="gung.txt"'
+    response.body.should == 'GUNGLE'
+  end
+
+  it "should return 404 if the datastore raises data not found" do
+    @job.should_receive(:apply).and_raise(Dragonfly::DataStorage::DataNotFound)
+    response = make_request(@job)
+    response.status.should == 404
+  end
+
+  describe "ETag" do
+    it "should return an ETag" do
+      response = make_request(@job)
+      response.headers['ETag'].should =~ /^"\w+"$/
+    end
+
+    [
+      "dingle",
+      "dingle, eggheads",
+      '"dingle", "eggheads"',
+      '*'
+    ].each do |header|
+      it "should return a 304 if the correct ETag is specified in HTTP_IF_NONE_MATCH header e.g. #{header}" do
+        @job.should_receive(:unique_signature).at_least(:once).and_return('dingle')
+        response = make_request(@job, 'HTTP_IF_NONE_MATCH' => header)
+        response.status.should == 304
+        response['ETag'].should == '"dingle"'
+        response['Cache-Control'].should == "public, max-age=31536000"
+        response.body.should be_empty
+      end
+    end
+
+    it "should not have applied any steps if the correct ETag is specified in HTTP_IF_NONE_MATCH header" do
+      response = make_request(@job, 'HTTP_IF_NONE_MATCH' => @job.unique_signature)
+      @job.applied_steps.should be_empty
+    end
+  end
+
+  describe "Content Disposition" do
+    before(:each) do
+      @app.encoder.add{|temp_object, format| temp_object }
+    end
+
+    describe "filename" do
+      it "should return the original name" do
+        response = make_request(@job)
+        response['Content-Disposition'].should == 'filename="gung.txt"'
+      end
+      it "should return a filename with a different extension if it's been encoded" do
+        response = make_request(@job.encode(:doogs))
+        response['Content-Disposition'].should == 'filename="gung.doogs"'
+      end
+      it "should not have the filename if name doesn't exist" do
+        response = make_request(@app.new_job("ADFSDF"))
+        response['Content-Disposition'].should be_nil
+      end
+      it "should cope with filenames with no ext" do
+        response = make_request(@app.new_job("ASDF", :name => 'asdf'))
+        response['Content-Disposition'].should == 'filename="asdf"'
+      end
+      it "should uri encode funny characters" do
+        response = make_request(@app.new_job("ASDF", :name => '£@$£ `'))
+        response['Content-Disposition'].should == 'filename="%C2%A3@$%C2%A3%20%60"'
+      end
+      it "should allow for setting the filename using a block" do
+        @app.content_filename = proc{|job, request|
+          job.basename.reverse.upcase + request['a']
+        }
+        response = make_request(@job, 'QUERY_STRING' => 'a=egg')
+        response['Content-Disposition'].should == 'filename="GNUGegg"'
+      end
+      it "should not include the filename if configured to be nil" do
+        @app.content_filename = nil
+        response = make_request(@job)
+        response['Content-Disposition'].should be_nil
+      end
+    end
+    
+    describe "content disposition" do
+      it "should use the app's configured content-disposition" do
+        @app.content_disposition = :attachment
+        response = make_request(@job)
+        response['Content-Disposition'].should == 'attachment; filename="gung.txt"'
+      end
+      it "should allow using a block to set the content disposition" do
+        @app.content_disposition = proc{|job, request|
+          job.basename + request['blah']
+        }
+        response = make_request(@job, 'QUERY_STRING' => 'blah=yo')
+        response['Content-Disposition'].should == 'gungyo; filename="gung.txt"'
+      end
+    end
+  end
+
+end

data/spec/dragonfly/job_spec.rb

@@ -0,0 +1,773 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+# Matchers
+Spec::Matchers.define :match_steps do |steps|
+  match do |given|
+    given.map{|step| step.class } == steps
+  end
+end
+
+describe Dragonfly::Job do
+
+  describe "Step types" do
+
+    {
+      Dragonfly::Job::Fetch => :fetch,
+      Dragonfly::Job::Process => :process,
+      Dragonfly::Job::Encode => :encode,
+      Dragonfly::Job::Generate => :generate,
+      Dragonfly::Job::FetchFile => :fetch_file
+    }.each do |klass, step_name|
+      it "should return the correct step name for #{klass}" do
+        klass.step_name.should == step_name
+      end
+    end
+
+    {
+      Dragonfly::Job::Fetch => :f,
+      Dragonfly::Job::Process => :p,
+      Dragonfly::Job::Encode => :e,
+      Dragonfly::Job::Generate => :g,
+      Dragonfly::Job::FetchFile => :ff
+    }.each do |klass, abbreviation|
+      it "should return the correct abbreviation for #{klass}" do
+        klass.abbreviation.should == abbreviation
+      end
+    end
+
+    describe "step_names" do
+      it "should return the available step names" do
+        Dragonfly::Job.step_names.should == [:fetch, :process, :encode, :generate, :fetch_file]
+      end
+    end
+
+  end
+
+  describe "without temp_object" do
+
+    before(:each) do
+      @app = mock_app
+      @job = Dragonfly::Job.new(@app)
+    end
+
+    it "should allow initializing with content" do
+      job = Dragonfly::Job.new(@app, Dragonfly::TempObject.new('eggheads'))
+      job.temp_object.data.should == 'eggheads'
+    end
+
+    describe "fetch" do
+      before(:each) do
+        @job.fetch!('some_uid')
+      end
+
+      it { @job.steps.should match_steps([Dragonfly::Job::Fetch]) }
+
+      it "should retrieve from the app's datastore when applied" do
+        @app.datastore.should_receive(:retrieve).with('some_uid').and_return('HELLO')
+        @job.apply
+        @job.temp_object.data.should == 'HELLO'
+      end
+
+      it "should set extra data if returned from the datastore" do
+        @app.datastore.should_receive(:retrieve).with('some_uid').and_return(['HELLO', {:name => 'test.txt', :meta => {1=>2}}])
+        @job.apply
+        @job.temp_object.data.should == 'HELLO'
+        @job.temp_object.name.should == 'test.txt'
+        @job.temp_object.meta.should == {1 => 2}
+      end
+    end
+
+    describe "process" do
+      it "should raise an error when applying" do
+        @job.process!(:resize, '20x30')
+        lambda{
+          @job.apply
+        }.should raise_error(Dragonfly::Job::NothingToProcess)
+      end
+    end
+
+    describe "encode" do
+      it "should raise an error when applying" do
+        @job.encode!(:gif)
+        lambda{
+          @job.apply
+        }.should raise_error(Dragonfly::Job::NothingToEncode)
+      end
+    end
+
+    describe "analyse" do
+      it "should raise an error" do
+        lambda{
+          @job.analyse(:width)
+        }.should raise_error(Dragonfly::Job::NothingToAnalyse)
+      end
+    end
+
+    describe "generate" do
+      before(:each) do
+        @job.generate!(:plasma, 20, 30)
+      end
+
+      it { @job.steps.should match_steps([Dragonfly::Job::Generate]) }
+
+      it "should use the generator when applied" do
+        @app.generator.should_receive(:generate).with(:plasma, 20, 30).and_return('hi')
+        @job.apply.data.should == 'hi'
+      end
+
+      it "should save extra data if the generator returns it" do
+        @app.generator.should_receive(:generate).with(:plasma, 20, 30).and_return(['hi', {:name => 'plasma.png', :format => :png, :meta => {:a => :b}}])
+        @job.apply
+        @job.temp_object.data.should == 'hi'
+        @job.temp_object.name.should == 'plasma.png'
+        @job.temp_object.format.should == :png
+        @job.temp_object.meta.should == {:a => :b}
+      end
+    end
+
+    describe "fetch_file" do
+      before(:each) do
+        @job.fetch_file!(File.dirname(__FILE__) + '/../../samples/egg.png')
+      end
+
+      it { @job.steps.should match_steps([Dragonfly::Job::FetchFile]) }
+
+      it "should fetch the specified file when applied" do
+        @job.apply
+        @job.temp_object.size.should == 62664
+      end
+
+    end
+
+  end
+
+  describe "with temp_object already there" do
+
+    before(:each) do
+      @app = mock_app
+      @temp_object = Dragonfly::TempObject.new('HELLO', :name => 'hello.txt', :meta => {:a => :b}, :format => :txt)
+      @job = Dragonfly::Job.new(@app)
+      @job.temp_object = @temp_object
+    end
+
+    describe "apply" do
+      it "should return itself" do
+        @job.apply.should == @job
+      end
+    end
+
+    describe "process" do
+      before(:each) do
+        @job.process!(:resize, '20x30')
+      end
+
+      it { @job.steps.should match_steps([Dragonfly::Job::Process]) }
+
+      it "should use the processor when applied" do
+        @app.processor.should_receive(:process).with(@temp_object, :resize, '20x30').and_return('hi')
+        @job.apply.data.should == 'hi'
+      end
+
+      it "should maintain the temp object attributes" do
+        @app.processor.should_receive(:process).with(@temp_object, :resize, '20x30').and_return('hi')
+        temp_object = @job.apply.temp_object
+        temp_object.data.should == 'hi'
+        temp_object.name.should == 'hello.txt'
+        temp_object.meta.should == {:a => :b}
+        temp_object.format.should == :txt
+      end
+
+      it "should allow returning an array with extra attributes from the processor" do
+        @app.processor.should_receive(:process).with(@temp_object, :resize, '20x30').and_return(['hi', {:name => 'hello_20x30.txt', :meta => {:eggs => 'asdf'}}])
+        temp_object = @job.apply.temp_object
+        temp_object.data.should == 'hi'
+        temp_object.name.should == 'hello_20x30.txt'
+        temp_object.meta.should == {:a => :b, :eggs => 'asdf'}
+        temp_object.format.should == :txt
+      end
+    end
+
+    describe "encode" do
+      before(:each) do
+        @job.encode!(:gif, :bitrate => 'mumma')
+      end
+
+      it { @job.steps.should match_steps([Dragonfly::Job::Encode]) }
+
+      it "should use the encoder when applied" do
+        @app.encoder.should_receive(:encode).with(@temp_object, :gif, :bitrate => 'mumma').and_return('alo')
+        @job.apply.data.should == 'alo'
+      end
+
+      it "should maintain the temp object attributes (except format)" do
+        @app.encoder.should_receive(:encode).with(@temp_object, :gif, :bitrate => 'mumma').and_return('alo')
+        temp_object = @job.apply.temp_object
+        temp_object.data.should == 'alo'
+        temp_object.name.should == 'hello.txt'
+        temp_object.meta.should == {:a => :b}
+      end
+
+      it "should update the format" do
+        @app.encoder.should_receive(:encode).with(@temp_object, :gif, :bitrate => 'mumma').and_return('alo')
+        @job.apply.temp_object.format.should == :gif
+      end
+
+      it "should allow returning an array with extra attributes form the encoder" do
+        @app.encoder.should_receive(:encode).with(@temp_object, :gif, :bitrate => 'mumma').and_return(['alo', {:name => 'doobie', :meta => {:eggs => 'fish'}}])
+        temp_object = @job.apply.temp_object
+        temp_object.data.should == 'alo'
+        temp_object.name.should == 'doobie'
+        temp_object.meta.should == {:a => :b, :eggs => 'fish'}
+      end
+
+      it "not allow overriding the format" do
+        @app.encoder.should_receive(:encode).with(@temp_object, :gif, :bitrate => 'mumma').and_return(['alo', {:format => :png}])
+        temp_object = @job.apply.temp_object
+        temp_object.format.should == :gif
+      end
+    end
+  end
+
+  describe "analysis" do
+    before(:each) do
+      @app = test_app
+      @job = Dragonfly::Job.new(@app, Dragonfly::TempObject.new('HELLO'))
+      @app.analyser.add(:num_letters){|temp_object, letter| temp_object.data.count(letter) }
+    end
+    it "should return correctly when calling analyse" do
+      @job.analyse(:num_letters, 'L').should == 2
+    end
+    it "should have mixed in the analyser method" do
+      @job.num_letters('L').should == 2
+    end
+    it "should return nil from analyse if calling any old method" do
+      @job.analyse(:robin_van_persie).should be_nil
+    end
+    it "should not allow calling any old method" do
+      lambda{
+        @job.robin_van_persie
+      }.should raise_error(NoMethodError)
+    end
+    it "should work correctly with chained jobs, applying before analysing" do
+      @app.processor.add(:double){|temp_object| temp_object.data * 2 }
+      @job.process(:double).num_letters('L').should == 4
+    end
+  end
+
+  describe "adding jobs" do
+
+    before(:each) do
+      @app = mock_app
+    end
+
+    it "should raise an error if the app is different" do
+      job1 = Dragonfly::Job.new(@app)
+      job2 = Dragonfly::Job.new(mock_app)
+      lambda {
+        job1 + job2
+      }.should raise_error(Dragonfly::Job::AppDoesNotMatch)
+    end
+
+    describe "both belonging to the same app" do
+      before(:each) do
+        @job1 = Dragonfly::Job.new(@app, Dragonfly::TempObject.new('hello'))
+        @job1.process! :resize
+        @job2 = Dragonfly::Job.new(@app, Dragonfly::TempObject.new('hola'))
+        @job2.encode! :png
+      end
+
+      it "should concatenate jobs" do
+        job3 = @job1 + @job2
+        job3.steps.should match_steps([
+          Dragonfly::Job::Process,
+          Dragonfly::Job::Encode
+        ])
+      end
+
+      it "should raise an error if the second job has applied steps" do
+        @job2.apply
+        lambda {
+          @job1 + @job2
+        }.should raise_error(Dragonfly::Job::JobAlreadyApplied)
+      end
+
+      it "should not raise an error if the first job has applied steps" do
+        @job1.apply
+        lambda {
+          @job1 + @job2
+        }.should_not raise_error
+      end
+
+      it "should have the first job's temp_object" do
+        (@job1 + @job2).temp_object.data.should == 'hello'
+      end
+
+      it "should have the correct applied steps" do
+        @job1.apply
+        (@job1 + @job2).applied_steps.should match_steps([
+          Dragonfly::Job::Process
+        ])
+      end
+
+      it "should have the correct pending steps" do
+        @job1.apply
+        (@job1 + @job2).pending_steps.should match_steps([
+          Dragonfly::Job::Encode
+        ])
+      end
+    end
+
+  end
+
+  describe "defining extra steps after applying" do
+    before(:each) do
+      @app = mock_app
+      @job = Dragonfly::Job.new(@app)
+      @job.temp_object = Dragonfly::TempObject.new("hello")
+      @job.process! :resize
+      @job.apply
+      @job.encode! :micky
+    end
+    it "should not call apply on already applied steps" do
+      @job.steps[0].should_not_receive(:apply)
+      @job.apply
+    end
+    it "should call apply on not yet applied steps" do
+      @job.steps[1].should_receive(:apply)
+      @job.apply
+    end
+    it "should return all steps" do
+      @job.steps.should match_steps([
+        Dragonfly::Job::Process,
+        Dragonfly::Job::Encode
+      ])
+    end
+    it "should return applied steps" do
+      @job.applied_steps.should match_steps([
+        Dragonfly::Job::Process
+      ])
+    end
+    it "should return the pending steps" do
+      @job.pending_steps.should match_steps([
+        Dragonfly::Job::Encode
+      ])
+    end
+    it "should not call apply on any steps when already applied" do
+      @job.apply
+      @job.steps[0].should_not_receive(:apply)
+      @job.steps[1].should_not_receive(:apply)
+      @job.apply
+    end
+  end
+
+  describe "chaining" do
+
+    before(:each) do
+      @app = mock_app
+      @job = Dragonfly::Job.new(@app)
+    end
+
+    it "should return itself if bang is used" do
+      @job.fetch!('some_uid').should == @job
+    end
+
+    it "should return a new job if bang is not used" do
+      @job.fetch('some_uid').should_not == @job
+    end
+
+    describe "when a chained job is defined" do
+      before(:each) do
+        @job.fetch!('some_uid')
+        @job2 = @job.process(:resize, '30x30')
+      end
+
+      it "should return the correct steps for the original job" do
+        @job.applied_steps.should match_steps([
+        ])
+        @job.pending_steps.should match_steps([
+          Dragonfly::Job::Fetch
+        ])
+      end
+
+      it "should return the correct data for the original job" do
+        @job.data.should == 'SOME_DATA'
+      end
+
+      it "should return the correct steps for the new job" do
+        @job2.applied_steps.should match_steps([
+        ])
+        @job2.pending_steps.should match_steps([
+          Dragonfly::Job::Fetch,
+          Dragonfly::Job::Process
+        ])
+      end
+
+      it "should return the correct data for the new job" do
+        @job2.data.should == 'SOME_PROCESSED_DATA'
+      end
+
+      it "should not affect the other one when one is applied" do
+        @job.apply
+        @job.applied_steps.should match_steps([
+          Dragonfly::Job::Fetch
+        ])
+        @job.pending_steps.should match_steps([
+        ])
+        @job.temp_object.data.should == 'SOME_DATA'
+        @job2.applied_steps.should match_steps([
+        ])
+        @job2.pending_steps.should match_steps([
+          Dragonfly::Job::Fetch,
+          Dragonfly::Job::Process
+        ])
+        @job2.temp_object.should be_nil
+      end
+    end
+
+  end
+
+  describe "to_a" do
+    before(:each) do
+      @app = mock_app
+    end
+    it "should represent all the steps in array form" do
+      job = Dragonfly::Job.new(@app)
+      job.fetch! 'some_uid'
+      job.generate! :plasma # you wouldn't really call this after fetch but still works
+      job.process! :resize, '30x40'
+      job.encode! :gif, :bitrate => 20
+      job.to_a.should == [
+        [:f, 'some_uid'],
+        [:g, :plasma],
+        [:p, :resize, '30x40'],
+        [:e, :gif, {:bitrate => 20}]
+      ]
+    end
+  end
+
+  describe "from_a" do
+
+    before(:each) do
+      @app = test_app
+    end
+
+    describe "a well-defined array" do
+      before(:each) do
+        @job = Dragonfly::Job.from_a([
+          [:f, 'some_uid'],
+          [:g, :plasma],
+          [:p, :resize, '30x40'],
+          [:e, :gif, {:bitrate => 20}]
+        ], @app)
+      end
+      it "should have the correct step types" do
+        @job.steps.should match_steps([
+          Dragonfly::Job::Fetch,
+          Dragonfly::Job::Generate,
+          Dragonfly::Job::Process,
+          Dragonfly::Job::Encode
+        ])
+      end
+      it "should have the correct args" do
+        @job.steps[0].args.should == ['some_uid']
+        @job.steps[1].args.should == [:plasma]
+        @job.steps[2].args.should == [:resize, '30x40']
+        @job.steps[3].args.should == [:gif, {:bitrate => 20}]
+      end
+      it "should have no applied steps" do
+        @job.applied_steps.should be_empty
+      end
+      it "should have all steps pending" do
+        @job.steps.should == @job.pending_steps
+      end
+    end
+
+    [
+      :f,
+      [:f],
+      [[]],
+      [[:egg]]
+    ].each do |object|
+      it "should raise an error if the object passed in is #{object.inspect}" do
+        lambda {
+          Dragonfly::Job.from_a(object, @app)
+        }.should raise_error(Dragonfly::Job::InvalidArray)
+      end
+    end
+
+    it "should initialize an empty job if the array is empty" do
+      job = Dragonfly::Job.from_a([], @app)
+      job.steps.should be_empty
+    end
+  end
+
+  describe "from_path" do
+    before(:each) do
+      @app = test_app
+      @serialized = @app.fetch('eggs').serialize
+    end
+    it "should work with a simple path" do
+      Dragonfly::Job.from_path("/#{@serialized}", @app).should be_a(Dragonfly::Job)
+    end
+    it "should work with no slash" do
+      Dragonfly::Job.from_path(@serialized, @app).should be_a(Dragonfly::Job)
+    end
+    it "should ignore the app's url_path_prefix" do
+      @app.url_path_prefix = '/images/yo'
+      Dragonfly::Job.from_path("/images/yo/#{@serialized}", @app).should be_a(Dragonfly::Job)
+    end
+    it "should not work with an incorrect url_path_prefix" do
+      @app.url_path_prefix = '/images/yo'
+      lambda{
+        Dragonfly::Job.from_path("/images/#{@serialized}", @app).should be_a(Dragonfly::Job)
+      }.should raise_error(Dragonfly::Serializer::BadString)
+    end
+    it "should ignore any suffix" do
+      Dragonfly::Job.from_path("/#{@serialized}asdfJASKLDF*()!@$%{}|/GGG/.png.your.mum", @app).should be_a(Dragonfly::Job)
+    end
+  end
+
+  describe "serialization" do
+    before(:each) do
+      @app = test_app
+      @job = Dragonfly::Job.new(@app).fetch('mumma').process(:resize, '1x50')
+    end
+    it "should serialize itself" do
+      @job.serialize.should =~ /^\w{1,}$/
+    end
+    it "should deserialize to the same as the original" do
+      new_job = Dragonfly::Job.deserialize(@job.serialize, @app)
+      new_job.to_a.should == @job.to_a
+    end
+  end
+
+  describe "to_app" do
+    before(:each) do
+      @app = test_app
+      @job = Dragonfly::Job.new(@app)
+    end
+    it "should return an endpoint" do
+      endpoint = @job.to_app
+      endpoint.should be_a(Dragonfly::JobEndpoint)
+      endpoint.job.should == @job
+    end
+  end
+
+  describe "url" do
+    before(:each) do
+      @app = mock_app(:url_for => 'hello')
+      @job = Dragonfly::Job.new(@app)
+    end
+    it "should return a url" do
+      @job.generate!(:plasma)
+      @job.url.should == 'hello'
+    end
+    it "should return nil if there are no steps" do
+      @job.url.should be_nil
+    end
+  end
+
+  describe "to_fetched_job" do
+    it "should maintain the same temp_object and be already applied" do
+      app = mock_app
+      job = Dragonfly::Job.new(app, Dragonfly::TempObject.new("HELLO"))
+      new_job = job.to_fetched_job('some_uid')
+      new_job.temp_object.data.should == 'HELLO'
+      new_job.to_a.should == [
+        [:f, 'some_uid']
+      ]
+      new_job.pending_steps.should be_empty
+    end
+  end
+
+  describe "format" do
+    before(:each) do
+      @app = test_app
+    end
+    it "should default to nil" do
+      job = @app.new_job("HELLO")
+      job.format.should be_nil
+    end
+    it "should use the temp_object format if it exists" do
+      job = @app.new_job("HELLO", :format => :txt)
+      job.format.should == :txt
+    end
+    it "should use the analyser format if it exists" do
+      @app.analyser.add :format do |temp_object|
+        :egg
+      end
+      job = @app.new_job("HELLO")
+      job.format.should == :egg
+    end
+    it "should prefer the temp_object format if both exist" do
+      @app.analyser.add :format do |temp_object|
+        :egg
+      end
+      job = @app.new_job("HELLO", :format => :txt)
+      job.format.should == :txt
+    end
+  end
+
+  describe "validate_sha!" do
+    before(:each) do
+      @app = test_app
+      @job = @app.fetch('eggs')
+    end
+    it "should raise an error if nothing is given" do
+      lambda{
+        @job.validate_sha!(nil)
+      }.should raise_error(Dragonfly::Job::NoSHAGiven)
+    end
+    it "should raise an error if the wrong SHA is given" do
+      lambda{
+        @job.validate_sha!('asdf')
+      }.should raise_error(Dragonfly::Job::IncorrectSHA)
+    end
+    it "should return self if ok" do
+      @job.validate_sha!(@job.sha).should == @job
+    end
+  end
+
+  describe "setting the name" do
+    before(:each) do
+      @app = test_app
+      @job = @app.new_job("HELLO", :name => 'not.me')
+    end
+    it "should allow setting the name" do
+      @job.name = 'wassup.doc'
+      @job.name.should == 'wassup.doc'
+    end
+  end
+
+  describe "setting the meta" do
+    before(:each) do
+      @app = test_app
+      @job = @app.new_job("HiThere", :meta => {:five => 'beans'})
+    end
+    it "should allow setting the meta" do
+      @job.meta = {:doogie => 'ladders'}
+      @job.meta.should == {:doogie => 'ladders'}
+    end
+    it "should allow updating the meta" do
+      @job.meta[:doogie] = 'ladders'
+      @job.meta.should == {:five => 'beans', :doogie => 'ladders'}
+    end
+  end
+
+  describe "b64_data" do
+    before(:each) do
+      @app = test_app
+    end
+    it "should return a string using the data:URI schema" do
+      job = @app.new_job("HELLO", :name => 'text.txt')
+      job.b64_data.should == "data:text/plain;base64,SEVMTE8=\n"
+    end
+  end
+
+  describe "querying stuff without applying steps" do
+    before(:each) do
+      @app = test_app
+    end
+    
+    describe "fetch_step" do
+      it "should return nil if it doesn't exist" do
+        @app.generate(:ponies).process(:jam).fetch_step.should be_nil
+      end
+      it "should return the fetch step otherwise" do
+        step = @app.fetch('hello').process(:cheese).fetch_step
+        step.should be_a(Dragonfly::Job::Fetch)
+        step.uid.should == 'hello'
+      end
+    end
+    describe "fetched uid" do
+      describe "when there's no fetch step" do
+        before(:each) do
+          @job = @app.new_job("AGG")
+        end
+        it "should return nil for uid" do
+          @job.uid.should be_nil
+        end
+        it "should return nil for uid_basename" do
+          @job.uid_basename.should be_nil
+        end
+        it "should return nil for uid_extname" do
+          @job.uid_extname.should be_nil
+        end
+      end
+      describe "when there is a fetch step" do
+        before(:each) do
+          @job = @app.fetch('gungedin/innit.blud')
+        end
+        it "should return the uid" do
+          @job.uid.should == 'gungedin/innit.blud'
+        end
+        it "should return the uid_basename" do
+          @job.uid_basename.should == 'innit'
+        end
+        it "should return the uid_extname" do
+          @job.uid_extname.should == '.blud'
+        end
+      end
+    end
+
+    describe "fetch_file_step" do
+      it "should return nil if it doesn't exist" do
+        @app.generate(:ponies).process(:jam).fetch_file_step.should be_nil
+      end
+      it "should return the fetch_file step otherwise" do
+        step = @app.fetch_file('/my/file.png').process(:cheese).fetch_file_step
+        step.should be_a(Dragonfly::Job::FetchFile)
+        step.path.should == '/my/file.png'
+      end
+    end
+
+    describe "generate_step" do
+      it "should return nil if it doesn't exist" do
+        @app.fetch('many/ponies').process(:jam).generate_step.should be_nil
+      end
+      it "should return the generate step otherwise" do
+        step = @app.generate(:ponies).process(:cheese).generate_step
+        step.should be_a(Dragonfly::Job::Generate)
+        step.args.should == [:ponies]
+      end
+    end
+    
+    describe "process_steps" do
+      it "should return the processing steps" do
+        job = @app.fetch('many/ponies').process(:jam).process(:eggs).encode(:gif)
+        job.process_steps.should match_steps([
+          Dragonfly::Job::Process,
+          Dragonfly::Job::Process
+        ])
+      end
+    end
+
+    describe "encode_step" do
+      it "should return nil if it doesn't exist" do
+        @app.generate(:ponies).encode_step.should be_nil
+      end
+      it "should return the last encode step otherwise" do
+        step = @app.fetch('hello').encode(:smells).encode(:cheese).encode_step
+        step.should be_a(Dragonfly::Job::Encode)
+        step.format.should == :cheese
+      end
+    end
+    describe "encoded_format" do
+      it "should return nil if there's no encode step" do
+        @app.new_job('asdf').encoded_format.should be_nil
+      end
+      it "should return the last encoded format if it exists" do
+        @app.fetch('gungedin').encode(:a).encode(:b).encoded_format.should == :b
+      end
+    end
+    describe "encoded_extname" do
+      it "should return nil if there's no encode step" do
+        @app.new_job('asdf').encoded_extname.should be_nil
+      end
+      it "should return the last encoded format as an extname if it exists" do
+        @app.fetch('gungedin').encode(:a).encode(:b).encoded_extname.should == '.b'
+      end
+    end
+  end
+
+end