dragonfly 0.6.2 → 0.7.0

data/spec/dragonfly/{active_record_extensions → active_model_extensions}/model_spec.rb

@@ -1,17 +1,18 @@
 require File.dirname(__FILE__) + '/spec_helper'
+require 'active_support/core_ext/hash/except' # Needed for Rails 3.0.0.rc but subsequently fixed - TODO: remove when gem is updated
 
 describe Item do
 
   # See extra setup in models / initializer files
 
-  describe "registering dragonfly apps" do
+  describe "defining accessors" do
+
+    let(:app1){ Dragonfly[:images] }
+    let(:app2){ Dragonfly[:videos] }
 
-    let(:app1){ Dragonfly::App[:images] }
-    let(:app2){ Dragonfly::App[:videos] }
-    
     it "should return the mapping of apps to attributes" do
-      Dragonfly.active_record_macro(:image, app1)
-      Dragonfly.active_record_macro(:video, app2)
+      app1.define_macro(model_class, :image_accessor)
+      app2.define_macro(model_class, :video_accessor)
       Item.class_eval do
         image_accessor :preview_image
         video_accessor :trailer_video
@@ -19,58 +20,99 @@ describe Item do
       Item.dragonfly_apps_for_attributes.should == {:preview_image => app1, :trailer_video => app2}
     end
 
+    it "should work for included modules (e.g. Mongoid::Document)" do
+      mongoid_document = Module.new
+      app1.define_macro_on_include(mongoid_document, :dog_accessor)
+      model_class = Class.new do
+        def self.before_save(*args); end
+        def self.before_destroy(*args); end
+        include mongoid_document
+        dog_accessor :doogie
+      end
+      model_class.dragonfly_apps_for_attributes.should == {:doogie => app1}
+    end
+
   end
-  
-  describe "defining accessors" do
 
-    it "should raise an error if the wrong method prefix is used" do
-      lambda{
-        Item.class_eval do
-          dog_accessor :preview_image
-        end
-      }.should raise_error(NameError)
+  describe "correctly defined" do
+
+    before(:each) do
+      @app = Dragonfly[:images]
+      @app.define_macro(model_class, :image_accessor)
+      Item.class_eval do
+        image_accessor :preview_image
+      end
+      @item = Item.new
     end
 
-    describe "correctly defined" do
-    
-      before(:each) do
-        @app = Dragonfly::App[:images]
-        Dragonfly.active_record_macro(:image, @app)
-        Item.class_eval do
-          image_accessor :preview_image
-        end
-        @item = Item.new
-      end    
+    it "should provide a reader" do
+      @item.should respond_to(:preview_image)
+    end
+
+    it "should provide a writer" do
+      @item.should respond_to(:preview_image=)
+    end
 
-      it "should provide a reader" do
-        @item.should respond_to(:preview_image)
+    describe "when there has been nothing assigned" do
+      it "the reader should return nil" do
+        @item.preview_image.should be_nil
+      end
+      it "the uid should be nil" do
+        @item.preview_image_uid.should be_nil
+      end
+      it "should not try to store anything on save" do
+        @app.datastore.should_not_receive(:store)
+        @item.save!
       end
+      it "should not try to destroy anything on save" do
+        @app.datastore.should_not_receive(:destroy)
+        @item.save!
+      end
+      it "should not try to destroy anything on destroy" do
+        @app.datastore.should_not_receive(:destroy)
+        @item.destroy
+      end
+    end
 
-      it "should provide a writer" do
-        @item.should respond_to(:preview_image=)
+    describe "when the uid is set manually" do
+      before(:each) do
+        @item.preview_image_uid = 'some_known_uid'
+      end
+      it "should not try to retrieve any data" do
+        @app.datastore.should_not_receive(:retrieve)
+        @item.save!
+      end
+      it "should not try to destroy any data" do
+        @app.datastore.should_not_receive(:destroy)
+        @item.save!
       end
+      it "should not try to store any data" do
+        @app.datastore.should_not_receive(:store)
+        @item.save!
+      end
+    end
 
-      describe "when there has been nothing assigned" do
-        it "the reader should return nil" do
-          @item.preview_image.should be_nil
-        end
-        it "the uid should be nil" do
-          @item.preview_image_uid.should be_nil
-        end
-        it "should not try to store anything on save" do
-          @app.datastore.should_not_receive(:store)
-          @item.save!
-        end
-        it "should not try to destroy anything on save" do
-          @app.datastore.should_not_receive(:destroy)
-          @item.save!
-        end
-        it "should not try to destroy anything on destroy" do
-          @app.datastore.should_not_receive(:destroy)
-          @item.destroy
-        end
+    describe "when there has been some thing assigned but not saved" do
+      before(:each) do
+        @item.preview_image = "DATASTRING"
+      end
+      it "the reader should return an attachment" do
+        @item.preview_image.should be_a(Dragonfly::ActiveModelExtensions::Attachment)
+      end
+      it "the uid should be nil" do
+        @item.preview_image_uid.should be_nil
+      end
+      it "should store the image when saved" do
+        @app.datastore.should_receive(:store).with(a_temp_object_with_data("DATASTRING"), {})
+        @item.save!
+      end
+      it "should not try to destroy anything on destroy" do
+        @app.datastore.should_not_receive(:destroy)
+        @item.destroy
+      end
+      it "should return nil for the url" do
+        @item.preview_image.url.should be_nil
       end
-      
       describe "when the uid is set manually" do
         before(:each) do
           @item.preview_image_uid = 'some_known_uid'
@@ -88,179 +130,186 @@ describe Item do
           @item.save!
         end
       end
-      
-      describe "when there has been some thing assigned but not saved" do
+
+    end
+
+    describe "when something has been assigned and saved" do
+
+      before(:each) do
+        @item.preview_image = "DATASTRING"
+        @app.datastore.should_receive(:store).with(a_temp_object_with_data("DATASTRING"), {}).once.and_return('some_uid')
+        @item.save!
+      end
+
+      it "should have the correct uid" do
+        @item.preview_image_uid.should == 'some_uid'
+      end
+      it "should not try to store anything if saved again" do
+        @app.datastore.should_not_receive(:store)
+        @item.save!
+      end
+
+      it "should not try to destroy anything if saved again" do
+        @app.datastore.should_not_receive(:destroy)
+        @item.save!
+      end
+
+      it "should destroy the data on destroy" do
+        @app.datastore.should_receive(:destroy).with('some_uid')
+        @item.destroy
+      end
+
+      it "should return the url for the data" do
+        @app.should_receive(:url_for).with(an_instance_of(Dragonfly::Job)).and_return('some.url')
+        @item.preview_image.url.should == 'some.url'
+      end
+
+      it "should destroy the old data when the uid is set manually" do
+        @app.datastore.should_receive(:destroy).with('some_uid')
+        @item.preview_image_uid = 'some_known_uid'
+        @item.save!
+      end
+
+      describe "when accessed by a new model object" do
+        before(:each) do
+          @item = Item.find(@item.id)
+        end
+        it "should destroy the data on destroy" do
+          @app.datastore.should_receive(:destroy).with(@item.preview_image_uid)
+          @item.destroy
+        end
+      end
+
+      describe "when something new is assigned" do
         before(:each) do
-          @item.preview_image = "DATASTRING"
+          @item.preview_image = "ANEWDATASTRING"
+          @app.datastore.stub!(:store).and_return('some_uid')
         end
-        it "the reader should return an attachment" do
-          @item.preview_image.should be_a(Dragonfly::ActiveRecordExtensions::Attachment)
+        it "should set the uid to nil" do
+          @item.preview_image_uid.should be_nil
         end
-        it "the uid should be a 'pending' object" do
-          @item.preview_image_uid.should be_a(Dragonfly::ActiveRecordExtensions::PendingUID)
+        it "should destroy the old data when saved" do
+          @app.datastore.should_receive(:destroy).with('some_uid')
+          @item.save!
         end
-        it "should store the image when saved" do
-          @app.datastore.should_receive(:store).with(a_temp_object_with_data("DATASTRING"))
+        it "should not try to destroy the old data if saved again" do
+          @app.datastore.should_receive(:destroy).with('some_uid')
+          @item.save!
+          @app.datastore.should_not_receive(:destroy).with('some_uid')
           @item.save!
         end
-        it "should not try to destroy anything on destroy" do
-          @app.datastore.should_not_receive(:destroy)
+        it "should destroy the old data when saved, even if yet another thing is assigned" do
+          @item.preview_image = "YET ANOTHER DATA STRING"
+          @app.datastore.should_receive(:destroy).with('some_uid')
+          @item.save!
+        end
+        it "should store the new data when saved" do
+          @app.datastore.should_receive(:store).with(a_temp_object_with_data("ANEWDATASTRING"), {})
+          @item.save!
+        end
+        it "should destroy the old data on destroy" do
+          @app.datastore.should_receive(:destroy).with('some_uid')
           @item.destroy
         end
-        it "should return nil for the url" do
-          @item.preview_image.url.should be_nil
+        it "should destroy the old data on destroy, even if yet another thing is assigned" do
+          @item.preview_image = "YET ANOTHER DATA STRING"
+          @app.datastore.should_receive(:destroy).with('some_uid')
+          @item.destroy
         end
-        it "should return the temp_object" do
-          temp_object = @item.preview_image.temp_object
-          temp_object.should be_a(Dragonfly::ExtendedTempObject)
-          temp_object.data.should == 'DATASTRING'
+        it "should destroy the old data when the uid has been set manually" do
+          @app.datastore.should_receive(:destroy).with('some_uid')
+          @item.preview_image_uid = 'some_known_uid'
+          @item.save!
         end
-        describe "when the uid is set manually" do
-          before(:each) do
-            @item.preview_image_uid = 'some_known_uid'
-          end
-          it "should not try to retrieve any data" do
-            @app.datastore.should_not_receive(:retrieve)
-            @item.save!
-          end
-          it "should not try to destroy any data" do
-            @app.datastore.should_not_receive(:destroy)
-            @item.save!
-          end
-          it "should not try to store any data" do
-            @app.datastore.should_not_receive(:store)
-            @item.save!
-          end
+        it "should return the new size" do
+          @item.preview_image.size.should == 14
+        end
+        it "should return the new data" do
+          @item.preview_image.data.should == 'ANEWDATASTRING'
         end
-        
       end
-      
-      describe "when something has been assigned and saved" do
 
+      describe "when it is set to nil" do
         before(:each) do
-          @item.preview_image = "DATASTRING"
-          @app.datastore.should_receive(:store).with(a_temp_object_with_data("DATASTRING")).once.and_return('some_uid')
-          @app.datastore.stub!(:store).and_return('some_uid')
-          @app.datastore.stub!(:destroy)
-          @item.save!
+          @item.preview_image = nil
         end
-        it "should have the correct uid" do
-          @item.preview_image_uid.should == 'some_uid'
+        it "should set the uid to nil" do
+          @item.preview_image_uid.should be_nil
         end
-        it "should not try to store anything if saved again" do
-          @app.datastore.should_not_receive(:store)
-          @item.save!
+        it "should return the attribute as nil" do
+          @item.preview_image.should be_nil
         end
-
-        it "should not try to destroy anything if saved again" do
-          @app.datastore.should_not_receive(:destroy)
+        it "should destroy the data on save" do
+          @app.datastore.should_receive(:destroy).with('some_uid')
           @item.save!
+          @item.preview_image.should be_nil
         end
-        
-        it "should destroy the data on destroy" do
+        it "should destroy the old data on destroy" do
           @app.datastore.should_receive(:destroy).with('some_uid')
           @item.destroy
         end
+      end
 
-        it "should return the url for the data" do
-          @app.should_receive(:url_for).with(@item.preview_image_uid, :arg).and_return('some.url')
-          @item.preview_image.url(:arg).should == 'some.url'
+      describe "when the data can't be found" do
+        it "should log a warning if the data wasn't found on destroy" do
+          @app.datastore.should_receive(:destroy).with('some_uid').and_raise(Dragonfly::DataStorage::DataNotFound)
+          @app.log.should_receive(:warn)
+          @item.destroy
         end
-        
-        it "should destroy the old data when the uid is set manually" do
-          @app.datastore.should_receive(:destroy).with('some_uid')
-          @item.preview_image_uid = 'some_known_uid'
-          @item.save!
+      end
+
+    end
+
+    describe "other types of assignment" do
+      before(:each) do
+        @app.generator.add :egg do
+          "Gungedin"
         end
-        
-        describe "when accessed by a new model object" do
-          before(:each) do
-            @item = Item.find(@item.id)
-          end
-          it "should destroy the data on destroy" do
-            @app.datastore.should_receive(:destroy).with(@item.preview_image_uid)
-            @item.destroy
-          end
-          it "should return the temp_object" do
-            @app.should_receive(:fetch).with('some_uid').and_return(temp_object = mock('extended temp_object'))
-            @item.preview_image.temp_object.should == temp_object
-          end
+        @app.processor.add :doogie do |temp_object|
+          temp_object.data.upcase
         end
+      end
 
-        describe "when something new is assigned" do
-          before(:each) do
-            @item.preview_image = "ANEWDATASTRING"
-          end
-          it "should set the uid to pending" do
-            @item.preview_image_uid.should be_a(Dragonfly::ActiveRecordExtensions::PendingUID)
-          end
-          it "should destroy the old data when saved" do
-            @app.datastore.should_receive(:store).with(a_temp_object_with_data("ANEWDATASTRING")).once.and_return('some_uid')
-            
-            @app.datastore.should_receive(:destroy).with('some_uid')
-            @item.save!
-          end
-          it "should store the new data when saved" do
-            @app.datastore.should_receive(:store).with(a_temp_object_with_data("ANEWDATASTRING"))
-            @item.save!
-          end
-          it "should destroy the old data on destroy" do
-            @app.datastore.should_receive(:destroy).with('some_uid')
-            @item.destroy
-          end
-          it "should return the new size" do
-            @item.preview_image.size.should == 14
-          end
-          it "should return the new temp_object" do
-            temp_object = @item.preview_image.temp_object
-            temp_object.should be_a(Dragonfly::ExtendedTempObject)
-            temp_object.data.should == 'ANEWDATASTRING'
-          end
+      describe "assigning with a job" do
+        before(:each) do
+          @job = @app.generate(:egg)
+          @item.preview_image = @job
         end
-        
-        describe "when it is set to nil" do
-          before(:each) do
-            @item.preview_image = nil
-          end
-          it "should set the uid to nil" do
-            @item.preview_image_uid.should be_nil
-          end
-          it "should return the attribute as nil" do
-            @item.preview_image.should be_nil
-          end
-          it "should destroy the data on save" do
-            @app.datastore.should_receive(:destroy).with('some_uid')
-            @item.save!
-            @item.preview_image.should be_nil
-          end
-          it "should destroy the old data on destroy" do
-            @app.datastore.should_receive(:destroy).with('some_uid')
-            @item.destroy
-          end
+
+        it "should work" do
+          @item.preview_image.data.should == 'Gungedin'
         end
 
-        describe "when the data can't be found" do
-          before(:each) do
-            @app.datastore.stub!(:destroy).with('some_uid').and_raise(Dragonfly::DataStorage::DataNotFound)
-            @app.datastore.stub!(:retrieve).with('some_uid').and_raise(Dragonfly::DataStorage::DataNotFound)
-          end
-          it "should log a warning if the data wasn't found on destroy" do
-            @app.log.should_receive(:warn)
-            @item.destroy
-          end 
+        it "should not be affected by subsequent changes to the job" do
+          @job.process!(:doogie)
+          @item.preview_image.data.should == 'Gungedin'
         end
+      end
 
+      describe "assigning with another attachment" do
+        before(:each) do
+          Item.class_eval do
+            image_accessor :other_image
+          end
+        end
+        it "should work like assigning the job" do
+          @item.preview_image = 'eggheads'
+          @item.other_image = @item.preview_image
+          @item.preview_image = 'dogchin'
+          @item.other_image.data.should == 'eggheads'
+        end
       end
     end
+
   end
 
   describe "validations" do
 
     before(:all) do
-      @app = Dragonfly::App[:images]
-      Dragonfly.active_record_macro(:image, @app)
+      @app = Dragonfly[:images]
+      @app.define_macro(model_class, :image_accessor)
     end
-    
+
     describe "validates_presence_of" do
 
       before(:all) do
@@ -279,9 +328,9 @@ describe Item do
       end
 
     end
-    
+
     describe "validates_size_of" do
-      
+
       before(:all) do
         Item.class_eval do
           image_accessor :preview_image
@@ -296,9 +345,9 @@ describe Item do
       it "should be invalid if too small" do
         Item.new(:preview_image => "12345").should_not be_valid
       end
-      
+
     end
-    
+
     describe "validates_property" do
 
       before(:each) do
@@ -306,8 +355,7 @@ describe Item do
       end
 
       before(:all) do
-        custom_analyser = Class.new(Dragonfly::Analysis::Base)
-        custom_analyser.class_eval do
+        custom_analyser = Class.new do
           def mime_type(temp_object)
             case temp_object.data
             when "WRONG TYPE" then 'wrong/type'
@@ -320,8 +368,8 @@ describe Item do
             temp_object.data.count('G')
           end
         end
-        @app.register_analyser(custom_analyser)
-        
+        @app.analyser.register(custom_analyser)
+
         Item.class_eval do
           validates_property :mime_type, :of => :preview_image, :in => ['how/special', 'how/crazy'], :if => :its_friday
           validates_property :mime_type, :of => [:other_image, :yet_another_image], :as => 'how/special'
@@ -339,12 +387,12 @@ describe Item do
 
         end
       end
-    
+
       it "should be valid if nil, if not validated on presence (even with validates_property)" do
         @item.other_image = nil
         @item.should be_valid
       end
-    
+
       it "should be invalid if the property is nil" do
         @item.preview_image = "OTHER TYPE"
         @item.should_not be_valid
@@ -356,7 +404,7 @@ describe Item do
         @item.should_not be_valid
         @item.errors[:preview_image].should match_ar_error("mime type is incorrect. It needs to be one of 'how/special', 'how/crazy', but was 'wrong/type'")
       end
-      
+
       it "should work for a range" do
         @item.preview_image = "GOOGLE GUM"
         @item.should_not be_valid
@@ -376,7 +424,7 @@ describe Item do
         @item.preview_image = "WRONG TYPE"
         @item.should be_valid
       end
-    
+
       it "should require either :as or :in as an argument" do
         lambda{
           Item.class_eval do
@@ -401,77 +449,67 @@ describe Item do
 
     end
 
-    describe "validates_mime_type_of" do
-      it "should provide validates_mime_type as a convenience wrapper for validates_property" do
-        Item.should_receive(:validates_property).with(:mime_type, :of => :preview_image, :in => ['how/special', 'how/crazy'], :if => :its_friday)
-        Item.class_eval do
-          validates_mime_type_of :preview_image, :in => ['how/special', 'how/crazy'], :if => :its_friday
-        end
-      end
-    end
-
   end
 
   describe "extra properties" do
 
     before(:each) do
-      @app = Dragonfly::App[:images]
-      custom_analyser = Class.new(Dragonfly::Analysis::Base)
-      custom_analyser.class_eval do
+      @app = Dragonfly[:images]
+      custom_analyser = Class.new do
         def some_analyser_method(temp_object)
           "abc" + temp_object.data[0..0]
         end
         def number_of_As(temp_object); temp_object.data.count('A'); end
       end
-      @app.register_analyser(custom_analyser)
-      Dragonfly.active_record_macro(:image, @app)
+      @app.analyser.register(custom_analyser)
+      @app.define_macro(model_class, :image_accessor)
       Item.class_eval do
         image_accessor :preview_image
       end
       @item = Item.new
     end
-    
+
     describe "magic attributes" do
-    
+
       it "should default the magic attribute as nil" do
         @item.preview_image_some_analyser_method.should be_nil
       end
-    
+
       it "should set the magic attribute when assigned" do
         @item.preview_image = '123'
         @item.preview_image_some_analyser_method.should == 'abc1'
       end
-    
+
       it "should not set non-magic attributes with the same prefix when assigned" do
         @item.preview_image_blah_blah = 'wassup'
         @item.preview_image = '123'
         @item.preview_image_blah_blah.should == 'wassup'
       end
-    
+
       it "should update the magic attribute when something else is assigned" do
         @item.preview_image = '123'
         @item.preview_image = '456'
         @item.preview_image_some_analyser_method.should == 'abc4'
       end
-    
+
       it "should reset the magic attribute when set to nil" do
         @item.preview_image = '123'
         @item.preview_image = nil
         @item.preview_image_some_analyser_method.should be_nil
       end
-    
+
       it "should not reset non-magic attributes with the same prefix when set to nil" do
         @item.preview_image_blah_blah = 'wassup'
         @item.preview_image = '123'
         @item.preview_image = nil
         @item.preview_image_blah_blah.should == 'wassup'
       end
-    
+
       it "should work for size too" do
         @item.preview_image = '123'
         @item.preview_image_size.should == 3
       end
-    
+
       it "should store the original file extension if it exists" do
         data = 'jasdlkf sadjl'
         data.stub!(:original_filename).and_return('hello.png')
@@ -486,9 +524,9 @@ describe Item do
         @item.preview_image_name.should == 'hello.png'
       end
     end
-  
-  
-    describe "delegating methods to the temp_object" do
+
+
+    describe "delegating methods to the job" do
       before(:each) do
         @item.preview_image = "DATASTRING"
       end
@@ -504,7 +542,7 @@ describe Item do
       it "should include analyser methods in public_methods" do
         @item.preview_image.public_methods.include?('number_of_As'.to_method_name).should be_true
       end
-      
+
       it "should update when something new is assigned" do
         @item.preview_image = 'ANEWDATASTRING'
         @item.preview_image.number_of_As.should == 3
@@ -515,35 +553,33 @@ describe Item do
           @app.datastore.stub!(:store).and_return('my_uid')
           item = Item.create!(:preview_image => 'DATASTRING')
           @item = Item.find(item.id)
-          @temp_object = @app.create_object('DATASTRING')
-          @temp_object.name = 'jonny.briggs'
         end
-        it "should load the temp_object then delegate the method" do
-          @app.should_receive(:fetch).with('my_uid').and_return(@temp_object)
+        it "should load the content then delegate the method" do
+          @app.datastore.should_receive(:retrieve).with('my_uid').and_return(['DATASTRING', {}])
           @item.preview_image.number_of_As.should == 2
         end
-        it "should use the magic attribute if there is one, and not load the temp_object" do
-          @app.should_not_receive(:fetch)
+        it "should use the magic attribute if there is one, and not load the content" do
+          @app.datastore.should_not_receive(:retrieve)
           @item.should_receive(:preview_image_some_analyser_method).and_return('result yo')
           @item.preview_image.some_analyser_method.should == 'result yo'
         end
-        
+
         %w(size name ext).each do |attr|
-          it "should use the magic attribute for #{attr} if there is one, and not load the temp_object" do
-            @app.should_not_receive(:fetch)
+          it "should use the magic attribute for #{attr} if there is one, and not load the content" do
+            @app.datastore.should_not_receive(:retrieve)
             @item.should_receive("preview_image_#{attr}".to_sym).and_return('result yo')
             @item.preview_image.send(attr).should == 'result yo'
           end
-          it "should load the temp_object then delegate '#{attr}' if there is no magic attribute for it" do
-            Item.should_receive(:column_names).and_return(['preview_image_uid']) # no magic attributes
-            
-            @app.should_receive(:fetch).with('my_uid').and_return(@temp_object)
-            @item.preview_image.send(attr).should == @temp_object.send(attr)
+
+          it "should load the content then delegate '#{attr}' if there is no magic attribute for it" do
+            @item.should_receive(:public_methods).and_return(['preview_image_uid']) # no magic attributes
+            @app.datastore.should_receive(:retrieve).with('my_uid').and_return(['DATASTRING', {}])
+            @item.preview_image.send(attr).should == @item.preview_image.send(:job).send(attr)
           end
         end
-        
+
       end
-    
+
       it "should not raise an error if a non-existent method is called" do
         # Just checking method missing works ok
         lambda{
@@ -552,14 +588,14 @@ describe Item do
       end
     end
   end
-  
+
   describe "inheritance" do
-    
+
     before(:all) do
-      @app = Dragonfly::App[:images]
-      @app2 = Dragonfly::App[:egg]
-      Dragonfly.active_record_macro(:image, @app)
-      Dragonfly.active_record_macro(:egg, @app2)
+      @app = Dragonfly[:images]
+      @app2 = Dragonfly[:egg]
+      @app.define_macro(model_class, :image_accessor)
+      @app2.define_macro(model_class, :egg_accessor)
       Car.class_eval do
         image_accessor :image
       end
@@ -568,11 +604,13 @@ describe Item do
       end
 
       @base_class = Car
-      @subclass = Class.new(Car){ image_accessor :reliant_image }
-      @subclass_with_module = Class.new(Car) do
+      class ReliantRobin < Car; image_accessor :reliant_image; end
+      @subclass = ReliantRobin
+      class ReliantRobinWithModule < Car
         include Module.new
         image_accessor :reliant_image
       end
+      @subclass_with_module = ReliantRobinWithModule
       @unrelated_class = Photo
     end