Makes more sense to have image iterator simply provide functionality …

…as a module..

actions/image.rb

@@ -7,17 +7,6 @@ module Actions
   class Image
     AnnotationException = Class.new(Exception)
 
-    module Didntfindshit
-
-      def file
-        File.new './public/didntfindshit.jpg'
-      end
-
-      def max_age
-        30
-      end
-    end
-
     attr_accessor :file, :max_age
 
     def initialize(noun)

sources/bing.rb

@@ -4,6 +4,7 @@
 
 class Bing
   attr_reader :noun, :images, :result, :json
+  include ImageIterator
 
   def initialize(noun)
     @noun = noun

sources/image_iterator.rb

@@ -1,14 +1,13 @@
-class ImageIterator
-  def initialize(urls=[])
-    @urls = urls
-    @enum = @urls.to_enum
+module ImageIterator
+  def enum
+    @enum ||= @images.to_enum
   end
 
   def next
-    Kernel.open(@enum.next)
+    Kernel.open(enum.next)
   end
 
   def rewind
-    @enum.rewind
+    enum.rewind
   end
 end

spec/sources/bing_spec.rb

@@ -28,7 +28,7 @@
 
     let(:subject)     { Bing.fetch('xbox') }
     its(:images)      { should respond_to(:each) }
-    its(:images)      { should ==(['http://example.com/example.jpg'])}
+    its(:images)      { should == ['http://example.com/example.jpg'] }
 
     its('best_image.path')  { should == './spec/support/test_image.jpg' }
   end

spec/sources/image_iterator_spec.rb

@@ -1,12 +1,21 @@
 require './sources/image_iterator'
 
+class Dummy
+  include ImageIterator
+  attr_accessor :images
+
+  def initialize(images=[])
+    @images = images
+  end
+end
+
 describe 'ImageIterator' do
   context 'basic' do
     before do
       Kernel.stub(:open) { |arg| arg }
     end
 
-    subject {  ImageIterator.new([:a,:b,:c]) }
+    subject {  Dummy.new([:a,:b,:c]) }
 
     it 'iterates as expected' do
       subject.next.should == :a