artwork 0.2.1 → 0.3.0

data/.rspec

@@ -0,0 +1 @@
+--color

data/README.md

@@ -36,12 +36,17 @@ you have a _2x versions of your thumbs, the helper will choose the _2x one.
 - Ruby 1.8.7 or newer
 - Rails 2.3 or newer
 - Paperclip 2.3 or newer
+- A JavaScript runtime
 
 ## Installation
 
-Add this line to your application's Gemfile:
+Add these lines to your application's Gemfile:
 
     gem 'artwork'
+    gem 'therubyracer'
+
+You can skip `therubyracer` if you have other JavaScript environments available
+on your machine (including on the prodiction one).
 
 And then execute:
 
@@ -100,6 +105,8 @@ the `artwork_tag` view helper. Example:
 
 1. [Fork it](https://github.com/mitio/artwork/fork)
 2. Create your feature branch (`git checkout -b my-new-feature`)
-3. Commit your changes (`git commit -am 'Add some feature'`)
-4. Push to the branch (`git push origin my-new-feature`)
-5. Create a new Pull Request
+3. Make your changes
+4. Make sure the tests pass (`bundle exec rake`)
+5. Commit your changes (`git commit -am 'Add some feature'`)
+6. Push to the branch (`git push origin my-new-feature`)
+7. Create a new Pull Request

data/Rakefile

@@ -1,2 +1,9 @@
 require 'bundler/gem_tasks'
+require 'rspec/core'
+require 'rspec/core/rake_task'
 
+RSpec::Core::RakeTask.new(:spec) do |spec|
+  spec.pattern = FileList['spec/**/*_spec.rb']
+end
+
+task :default => :spec

data/artwork.gemspec

@@ -20,6 +20,7 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency 'bundler', '~> 1.6'
   spec.add_development_dependency 'rake'
+  spec.add_development_dependency 'rspec', '~> 3'
 
   spec.add_dependency 'rails', '>= 2.3'
   spec.add_dependency 'paperclip', '>= 2.3'

data/lib/artwork.rb

@@ -3,7 +3,7 @@ require 'artwork/configuration'
 require 'artwork/model'
 require 'artwork/view'
 require 'artwork/controller'
-require 'artwork/engine' if Rails.const_defined?(:Engine)
+require 'artwork/engine' if Object.const_defined?(:Rails) and Rails.const_defined?(:Engine)
 
 module Artwork
   extend Configuration

data/lib/artwork/model.rb

@@ -1,34 +1,23 @@
+require 'artwork/thumbnail'
+
 module Artwork
   module Model
-    THUMBNAIL_NAME_PATTERN = /^[0-9]+x(\w*?)(_2x)?$/i.freeze
-
     def artwork_thumb_for(attachment_name, size)
-      size = size.to_s
+      desired_thumb = Thumbnail.new(size)
       matching_thumb_name = nil
 
-      if size =~ THUMBNAIL_NAME_PATTERN
-        desired_size = size.to_i / ratio_for_current_resolution
-
-        available_attachments = []
-
-        # Pick attachments which follow our naming conventions, skipping retina images
-        attachment_styles_for(attachment_name).each do |thumb_name|
-          if thumb_name.to_s =~ THUMBNAIL_NAME_PATTERN
-            is_retina   = $2
-            thumb_width = thumb_name.to_s.to_i
+      if desired_thumb.compatible?
+        desired_size = desired_thumb.width / ratio_for_current_resolution
 
-            available_attachments << [thumb_name, thumb_width] unless is_retina
-          end
-        end
-
-        # Sort attachments by width, in ascending order
-        available_attachments = available_attachments.sort_by do |thumb_name, thumb_width|
-          thumb_width
-        end
+        thumbs = attachment_styles_for(attachment_name) \
+          .map { |thumb_name| Thumbnail.new(thumb_name) } \
+          .select(&:compatible?) \
+          .reject(&:retina?) \
+          .sort
 
-        available_attachments.each do |thumb_name, thumb_width|
-          if desired_size <= thumb_width
-            matching_thumb_name = thumb_name
+        thumbs.each do |thumb|
+          if desired_size <= thumb.width and thumb.label == desired_thumb.label
+            matching_thumb_name = thumb.name
             break
           end
         end
@@ -36,7 +25,7 @@ module Artwork
         # If we did not find any matching attachment definitions,
         # the desired size is probably larger than all of our thumb widths,
         # So pick the last (largest) one we have.
-        matching_thumb_name ||= available_attachments.last.first
+        matching_thumb_name ||= thumbs.last.name
       end
 
       matching_thumb_name ||= size.to_sym
@@ -48,7 +37,7 @@ module Artwork
       matching_thumb_name.to_sym
     end
 
-    def artwork_url(attachment_name, size, options = {})
+    def artwork_url(attachment_name, size, options = nil)
       thumb_name = artwork_thumb_for(attachment_name, size)
       send(attachment_name).url(thumb_name, options)
     end

data/lib/artwork/thumbnail.rb

@@ -0,0 +1,46 @@
+module Artwork
+  class Thumbnail
+    include Comparable
+
+    NAME_PATTERN = /^(\d+)x(\w*?)(_2x)?$/i.freeze
+
+    attr :name
+    attr :width
+    attr :label
+
+    def initialize(name)
+      @name = name.to_s
+
+      if match = @name.match(NAME_PATTERN)
+        @width       = match[1].to_i
+        @label       = match[2].to_s.gsub(/^_|_$/, '')
+        @retina_flag = match[3]
+      end
+    end
+
+    def compatible?
+      not width.nil?
+    end
+
+    def retina?
+      @retina_flag == '_2x'
+    end
+
+    def <=>(other_thumb)
+      width <=> other_thumb.width
+    end
+
+    def eq(other)
+      name    == other.name and \
+      width   == other.width and \
+      label   == other.label and \
+      retina? == other.retina?
+    end
+
+    alias == eq
+
+    def self.compatible?(name)
+      name.to_s =~ NAME_PATTERN ? true : false
+    end
+  end
+end

data/lib/artwork/version.rb

@@ -1,3 +1,3 @@
 module Artwork
-  VERSION = '0.2.1'
+  VERSION = '0.3.0'
 end

data/lib/artwork/view.rb

@@ -1,9 +1,7 @@
 module Artwork
   module View
     def activate_resolution_independence
-      content_tag :script do
-        Thread.current[:artwork_script] ||= compile_artwork_script
-      end
+      Thread.current[:artwork_script] ||= content_tag :script, compile_artwork_script
     end
 
     def artwork_tag(record, attachment_name, size, options = {})

data/spec/artwork/configuration_spec.rb

@@ -0,0 +1,114 @@
+require 'spec_helper'
+
+module Artwork
+  describe Configuration do
+    let(:config) { Class.new { extend Configuration } }
+
+    before :each do
+      thread_variables = {}
+      allow(Thread).to receive(:current) { thread_variables }
+    end
+
+    [:default_resolution, :supported_resolutions_list].each do |option_name|
+      describe "##{option_name}" do
+        it 'raises an error if not set' do
+          expect { config.send(option_name) }.to raise_error
+        end
+
+        it 'uses Thread.current to store values' do
+          expect(Thread).to receive(:current).and_return(option_name => 'value')
+          expect(config.send(option_name)).to eq 'value'
+        end
+      end
+    end
+
+    describe '#supported_resolutions_list=' do
+      it 'converts the resolutions to integers' do
+        config.supported_resolutions_list = ['123', '456']
+        expect(config.supported_resolutions_list).to eq [123, 456]
+      end
+    end
+
+    describe '#current_resolution' do
+      it 'falls back to default_resolution' do
+        expect(config).to receive(:default_resolution).and_return('default')
+        expect(config.current_resolution).to eq 'default'
+      end
+    end
+
+    describe '#current_resolution=' do
+      it 'allows arbitrary resolutions if called directly' do
+        config.current_resolution = 12345
+        expect(config.current_resolution).to eq 12345
+      end
+    end
+
+    describe '#reset_configuration' do
+      it 'resets the current_resolution' do
+        config.current_resolution = 'current'
+        expect(config.current_resolution).to eq 'current'
+
+        config.reset_configuration
+
+        expect(config).to receive(:default_resolution).and_return('default')
+        expect(config.current_resolution).to eq 'default'
+      end
+
+      it 'resets the retina flag' do
+        config.load_2x_images = 'retina_flag'
+        expect(config.load_2x_images?).to eq 'retina_flag'
+
+        config.reset_configuration
+
+        expect(config.load_2x_images?).to eq false
+      end
+    end
+
+    describe '#configure_for' do
+      def make_request(cookies_hash = {})
+        double :cookies => cookies_hash
+      end
+
+      it 'sets current_resolution and load_2x_images from request cookies' do
+        config.supported_resolutions_list = [1000, 2000, 3000]
+        config.configure_for make_request('_retina' => '1', '_width' => '2000')
+
+        expect(config.current_resolution).to eq 2000
+        expect(config.load_2x_images?).to eq true
+      end
+
+      it 'sets the load_2x_images flag from the _retina cookie' do
+        config.supported_resolutions_list = [1000, 2000, 3000]
+        config.default_resolution = 1000
+
+        config.configure_for make_request('_retina' => '0')
+        expect(config.load_2x_images?).to eq false
+
+        config.configure_for make_request('_retina' => '1')
+        expect(config.load_2x_images?).to eq true
+      end
+
+      it 'picks only from the supported resolutions list' do
+        config.supported_resolutions_list = [1000, 2000, 3000]
+
+        config.configure_for make_request('_width' => '1234')
+        expect(config.current_resolution).to eq 2000
+
+        config.configure_for make_request('_width' => '234')
+        expect(config.current_resolution).to eq 1000
+
+        config.configure_for make_request('_width' => '10234')
+        expect(config.current_resolution).to eq 3000
+      end
+
+      it 'falls back to default_resolution if no _width cookie' do
+        config.supported_resolutions_list = []
+        config.default_resolution = 5000
+
+        config.configure_for make_request
+
+        expect(config.current_resolution).to eq 5000
+      end
+    end
+  end
+end

data/spec/artwork/controller_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+module Artwork
+  describe Controller do
+    let(:application_controller) { Class.new }
+
+    it 'adds an around_filter and declares view helpers when included' do
+      expect(application_controller).to receive(:around_filter).with(:initialize_artwork_env)
+      expect(application_controller).to receive(:helper).with(View)
+
+      application_controller.send :include, Controller
+    end
+  end
+end

data/spec/artwork/model_spec.rb

@@ -0,0 +1,170 @@
+require 'spec_helper'
+
+module Artwork
+  describe Model do
+    let(:model) { Class.new { include Artwork::Model } }
+    let(:instance) { model.new }
+
+    before :each do
+      allow(model).to receive(:attachment_definitions).and_return({
+        :image => {
+          :styles => {
+            :'320x'               => '320x>',
+            :'320x_2x'            => '640x>',
+            :'640x_2x'            => '1280x>',
+            :'640x'               => '640x>',
+            :'1280x'              => '1280x>',
+            :'1280x_2x'           => '2560x>',
+            :'2000x'              => '2000x>',
+            :'1500x_2x'           => '3000x>',
+            :'320x_some_label'    => '320x>',
+            :'320x_some_label_2x' => '640x>',
+            :'320x500'            => '320x500>',
+            :'320x500_2x'         => '640x1000>',
+            :'320x500_crop'       => '320x500#',
+            :'320x500_crop_2x'    => '640x1000#',
+            :unsupported          => '100x100>'
+          },
+        },
+      })
+    end
+
+    describe '#attachment_styles_for' do
+      it 'returns the list of available thumbnails' do
+        expect(instance.attachment_styles_for(:image)).to match_array [
+          :'320x',
+          :'320x_2x',
+          :'640x_2x',
+          :'640x',
+          :'1280x',
+          :'1280x_2x',
+          :'2000x',
+          :'1500x_2x',
+          :'320x_some_label',
+          :'320x_some_label_2x',
+          :'320x500',
+          :'320x500_2x',
+          :'320x500_crop',
+          :'320x500_crop_2x',
+          :unsupported,
+        ]
+      end
+    end
+
+    describe '#artwork_url' do
+      it 'returns the computed url of an attachment by delegating to artwork_thumb_for' do
+        expect(instance).to receive(:artwork_thumb_for).with(:photo, :size).and_return(:computed_size)
+
+        attachment = double
+        expect(attachment).to receive(:url).with(:computed_size, 'options').and_return 'some/url'
+        expect(instance).to receive(:photo).and_return(attachment)
+
+        expect(instance.artwork_url(:photo, :size, 'options')).to eq 'some/url'
+      end
+
+      it 'works with two arguments and a hash options' do
+        expect(instance).to receive(:artwork_thumb_for).with(:photo, :size).and_return(:computed_size)
+
+        attachment = double
+        expect(attachment).to receive(:url).with(:computed_size, :some => 'options').and_return 'some/url'
+        expect(instance).to receive(:photo).and_return(attachment)
+
+        expect(instance.artwork_url(:photo, :size, :some => 'options')).to eq 'some/url'
+      end
+
+      it 'works with two arguments only without any options hash' do
+        expect(instance).to receive(:artwork_thumb_for).with(:photo, :size).and_return(:computed_size)
+
+        attachment = double
+        expect(attachment).to receive(:url).with(:computed_size, nil).and_return 'some/url'
+        expect(instance).to receive(:photo).and_return(attachment)
+
+        expect(instance.artwork_url(:photo, :size)).to eq 'some/url'
+      end
+    end
+
+    describe '#artwork_thumb_for' do
+      before :each do
+        Artwork.default_resolution = 1000
+        Artwork.current_resolution = 1000
+        Artwork.load_2x_images     = false
+      end
+
+      def expect_thumb(size, expected)
+        expect(instance.artwork_thumb_for(:image, size)).to eq expected
+      end
+
+      it 'picks the exact requested size if it exists' do
+        expect_thumb '2000x', :'2000x'
+      end
+
+      it 'accepts sizes passed as both a symbol or a string' do
+        expect_thumb '2000x', :'2000x'
+        expect_thumb :'2000x', :'2000x'
+      end
+
+      it 'scales the required size according to current_resolution' do
+        Artwork.default_resolution = 1000
+        Artwork.current_resolution = 2000
+
+        expect_thumb '1000x', :'2000x'
+        expect_thumb '640x',  :'1280x'
+      end
+
+      it 'ignores the retina thumbs when looking for a given size' do
+        expect_thumb '1500x', :'2000x'
+      end
+
+      it 'uses the _2x thumb for retina screens' do
+        Artwork.load_2x_images = true
+        expect_thumb '640x', :'640x_2x'
+        expect_thumb '640x', :'640x_2x'
+      end
+
+      it 'uses the non-retina thumb for retina screens if no _2x thumb is available' do
+        Artwork.load_2x_images = true
+        expect_thumb '2000x', :'2000x'
+      end
+
+      it 'passes through unsupported thumb names' do
+        expect_thumb 'unsupported', :unsupported
+
+        Artwork.load_2x_images     = true
+        Artwork.default_resolution = 1000
+        Artwork.current_resolution = 5000
+
+        expect_thumb 'unsupported', :unsupported
+      end
+
+      it 'picks the nearest non-retina size to our desizred size' do
+        expect_thumb '400x', :'640x'
+      end
+
+      it 'picks the largest available size if requesting a too large thumb' do
+        expect_thumb '5000x', :'2000x'
+      end
+
+      it 'picks the smallest available size if requesting a too small thumb' do
+        expect_thumb '100x', :'320x'
+      end
+
+      it 'distinguishes thumbs by the supplied text label' do
+        expect_thumb '320x', :'320x'
+        expect_thumb '320x_some_label', :'320x_some_label'
+        expect_thumb '200x_some_label', :'320x_some_label'
+      end
+
+      it 'distinguishes thumbs by the supplied numerical label, as a vertical height' do
+        expect_thumb :'200x500', :'320x500'
+        Artwork.load_2x_images = true
+        expect_thumb :'200x500', :'320x500_2x'
+      end
+
+      it 'distinguishes thumbs by the supplied numerical label ignoring retina flags' do
+        expect_thumb :'200x500_2x', :'320x500'
+        Artwork.load_2x_images = true
+        expect_thumb :'200x500_2x', :'320x500_2x'
+      end
+    end
+  end
+end

data/spec/artwork/thumbnail_spec.rb

@@ -0,0 +1,70 @@
+require 'spec_helper'
+
+module Artwork
+  describe Thumbnail do
+    expected_defaults = {:compatible? => true}
+
+    examples = {
+      :'320x'               => {:width => 320,  :retina? => false, :label => ''},
+      :'320x_2x'            => {:width => 320,  :retina? => true,  :label => ''},
+      :'640x_2x'            => {:width => 640,  :retina? => true,  :label => ''},
+      :'640x'               => {:width => 640,  :retina? => false, :label => ''},
+      :'1280x'              => {:width => 1280, :retina? => false, :label => ''},
+      :'1280x_2x'           => {:width => 1280, :retina? => true,  :label => ''},
+      :'2000x'              => {:width => 2000, :retina? => false, :label => ''},
+      :'1500x_2x'           => {:width => 1500, :retina? => true,  :label => ''},
+      :'320x_some_label'    => {:width => 320,  :retina? => false, :label => 'some_label'},
+      :'320x_some_label_2x' => {:width => 320,  :retina? => true,  :label => 'some_label'},
+      :'320x500'            => {:width => 320,  :retina? => false, :label => '500'},
+      :'320x500_2x'         => {:width => 320,  :retina? => true,  :label => '500'},
+      :'320x500_crop'       => {:width => 320,  :retina? => false, :label => '500_crop'},
+      :'320x500_crop_2x'    => {:width => 320,  :retina? => true,  :label => '500_crop'},
+      :unsupported          => {:compatible? => false},
+      'unsupported_thumb'   => {:compatible? => false},
+    }
+
+    examples.each do |thumb_name, expected_properties|
+      expected_properties = expected_defaults.merge(expected_properties)
+      expected_properties.each do |field_name, expected_value|
+        it "##{field_name} is #{expected_value.inspect} for #{thumb_name}" do
+          thumbnail = Thumbnail.new(thumb_name)
+          expect(thumbnail.send(field_name)).to eq expected_value
+        end
+      end
+
+      compatible = expected_properties[:compatible?]
+      it "compatible? is true for #{thumb_name.inspect}" do
+        expect(Thumbnail.compatible?(thumb_name)).to eq compatible
+      end
+    end
+
+    describe '#eq' do
+      it 'is true for different objects with the same width, label and retina flag' do
+        expect(Thumbnail.new(:'1200x500_crop_2x')).to eq Thumbnail.new('1200x500_crop_2x')
+        expect(Thumbnail.new(:'1200x_2x')).to eq Thumbnail.new('1200x_2x')
+        expect(Thumbnail.new(:'1200x')).to eq Thumbnail.new('1200x')
+        expect(Thumbnail.new(:'1200x_black_and_white')).to eq Thumbnail.new('1200x_black_and_white')
+      end
+
+      it 'is false for different objects if any of the width, label or retina flag differ' do
+        expect(Thumbnail.new(:'1200x500_crop_2x')).not_to eq Thumbnail.new('1200x500_crop')
+        expect(Thumbnail.new(:'1200x500_crop_2x')).not_to eq Thumbnail.new('1200x500crop_2x')
+        expect(Thumbnail.new(:'1200x500_crop_2x')).not_to eq Thumbnail.new('1201x500_crop_2x')
+        expect(Thumbnail.new(:'1200x500_crop_2x')).not_to eq Thumbnail.new('1500x')
+      end
+    end
+
+    describe 'comparison' do
+      it 'is based on the thumb width' do
+        small  = Thumbnail.new('90x_crop')
+        medium = Thumbnail.new('500x_crop')
+        large  = Thumbnail.new('1090x_2x')
+        huge   = Thumbnail.new('3200x')
+
+        unsorted = [huge, large, small, medium]
+
+        expect(unsorted.sort).to eq [small, medium, large, huge]
+      end
+    end
+  end
+end

data/spec/artwork/view_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+module Artwork
+  describe View do
+    let(:view_context) { Class.new { include View }.new }
+
+    describe '#activate_resolution_independence' do
+      it 'returns an HTML script tag' do
+        stub_const('Uglifier', double(:compile => 'compiled_script'))
+        expect(Thread).to receive(:current).and_return({})
+        expect(view_context).to receive(:content_tag).with(:script, 'compiled_script').and_return('compiled_script_html')
+
+        expect(view_context.activate_resolution_independence).to eq 'compiled_script_html'
+      end
+
+      it 'caches the compiled script in Thread.current' do
+        expect(Thread).to receive(:current).and_return(:artwork_script => 'cached_compiled_script')
+        expect(view_context.activate_resolution_independence).to eq 'cached_compiled_script'
+      end
+    end
+
+    it 'defines the artwork_tag helper method' do
+      expect(View.instance_methods.map(&:to_sym)).to include(:artwork_tag)
+    end
+  end
+end

data/spec/artwork_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe Artwork do
+  it 'responds to root_path' do
+    expect(Artwork).to respond_to(:root_path)
+  end
+
+  it 'responds to configuration methods' do
+    expect(Artwork).to respond_to(:configure_for)
+    expect(Artwork).to respond_to(:reset_configuration)
+    expect(Artwork).to respond_to(:default_resolution)
+    expect(Artwork).to respond_to(:default_resolution=)
+    expect(Artwork).to respond_to(:supported_resolutions_list)
+    expect(Artwork).to respond_to(:supported_resolutions_list=)
+    expect(Artwork).to respond_to(:load_2x_images?)
+    expect(Artwork).to respond_to(:current_resolution)
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,2 @@
+require 'rspec'
+require 'artwork'

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: artwork
 version: !ruby/object:Gem::Version 
-  hash: 21
+  hash: 19
   prerelease: 
   segments: 
   - 0
-  - 2
-  - 1
-  version: 0.2.1
+  - 3
+  - 0
+  version: 0.3.0
 platform: ruby
 authors: 
 - Dimitar Dimitrov
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2014-06-11 00:00:00 +03:00
+date: 2014-06-17 00:00:00 +03:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -48,9 +48,23 @@ dependencies:
   type: :development
   version_requirements: *id002
 - !ruby/object:Gem::Dependency 
-  name: rails
+  name: rspec
   prerelease: false
   requirement: &id003 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        hash: 5
+        segments: 
+        - 3
+        version: "3"
+  type: :development
+  version_requirements: *id003
+- !ruby/object:Gem::Dependency 
+  name: rails
+  prerelease: false
+  requirement: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -61,11 +75,11 @@ dependencies:
         - 3
         version: "2.3"
   type: :runtime
-  version_requirements: *id003
+  version_requirements: *id004
 - !ruby/object:Gem::Dependency 
   name: paperclip
   prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  requirement: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -76,11 +90,11 @@ dependencies:
         - 3
         version: "2.3"
   type: :runtime
-  version_requirements: *id004
+  version_requirements: *id005
 - !ruby/object:Gem::Dependency 
   name: uglifier
   prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  requirement: &id006 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -90,7 +104,7 @@ dependencies:
         - 0
         version: "0"
   type: :runtime
-  version_requirements: *id005
+  version_requirements: *id006
 description: Automated user resolution based image size choosing for your Rails views, but done at the backend.
 email: 
 - dimitar@live.bg
@@ -102,6 +116,7 @@ extra_rdoc_files: []
 
 files: 
 - .gitignore
+- .rspec
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -112,9 +127,17 @@ files:
 - lib/artwork/controller.rb
 - lib/artwork/engine.rb
 - lib/artwork/model.rb
+- lib/artwork/thumbnail.rb
 - lib/artwork/version.rb
 - lib/artwork/view.rb
 - lib/assets/javascripts/artwork.js
+- spec/artwork/configuration_spec.rb
+- spec/artwork/controller_spec.rb
+- spec/artwork/model_spec.rb
+- spec/artwork/thumbnail_spec.rb
+- spec/artwork/view_spec.rb
+- spec/artwork_spec.rb
+- spec/spec_helper.rb
 has_rdoc: true
 homepage: https://github.com/mitio/artwork
 licenses: 
@@ -149,5 +172,11 @@ rubygems_version: 1.6.2
 signing_key: 
 specification_version: 3
 summary: Automated user resolution based image size choosing for Rails.
-test_files: []
-
+test_files: 
+- spec/artwork/configuration_spec.rb
+- spec/artwork/controller_spec.rb
+- spec/artwork/model_spec.rb
+- spec/artwork/thumbnail_spec.rb
+- spec/artwork/view_spec.rb
+- spec/artwork_spec.rb
+- spec/spec_helper.rb