spontaneous 0.1.0.alpha1

data/test/unit/test_images.rb

@@ -0,0 +1,265 @@
+# encoding: UTF-8
+
+
+require 'test_helper'
+# require 'openssl'
+
+class ImagesTest < MiniTest::Spec
+  include Spontaneous
+
+  context "Image fields set using absolute values" do
+    setup do
+      @image = FieldTypes::ImageField.new(:name => "image")
+    end
+    should "accept and not alter URL values" do
+      url =  "http://example.com/image.png"
+      @image.value = url
+      @image.processed_value.should == url
+      @image.src.should == url
+      @image.original.src.should == url
+    end
+
+    should "accept and not alter absolute paths" do
+      path = "/images/house.jpg"
+      @image.value = path
+      @image.processed_value.should == path
+      @image.src.should == path
+      @image.original.src.should == path
+    end
+  end
+
+  context "Image fields" do
+    setup do
+      tmp = File.join(File.dirname(__FILE__), "../../tmp/media")
+      Spontaneous.media_dir = tmp
+      @tmp_dir = Pathname.new(tmp)
+      @upload_dir = @tmp_dir + "tmp/1234"
+      @upload_dir.mkpath
+
+      @revision = 10
+      Site.stubs(:working_revision).returns(@revision)
+
+      @src_image =  Pathname.new(File.join(File.dirname(__FILE__), "../fixtures/images/rose.jpg")).realpath
+      @origin_image = @upload_dir + "rose.jpg"
+      # @origin_image.make_link(@src_image.to_s) unless @origin_image.exist?
+      FileUtils.cp(@src_image.to_s, @origin_image.to_s)
+      @origin_image = @origin_image.realpath.to_s
+      # @digest = OpenSSL::Digest::MD5.new.file(@origin_image).hexdigest
+      # p @digest
+
+      class ::ResizingImageField < FieldTypes::ImageField
+        size :preview do
+          width 200
+        end
+        size :tall do
+          height 200
+        end
+        size :thumbnail do
+          fit 50, 50
+        end
+        size :icon do
+          crop 50, 50
+        end
+        size :greyscale do
+          fit 50, 50
+          greyscale
+          gaussian_blur 10
+        end
+      end
+
+      ResizingImageField.register
+
+      class ::ContentWithImage < Content
+        field :photo, :resizing_image
+      end
+
+      @instance = ContentWithImage.new
+
+      @content_id = 234
+      @instance.stubs(:id).returns(@content_id)
+      @image = @instance.photo
+      @image.owner.should == @instance
+      # silence_logger {
+        @image.value = @origin_image.to_s
+      # }
+    end
+
+    teardown do
+      Object.send(:remove_const, :ContentWithImage)
+      Object.send(:remove_const, :ResizingImageField)
+      (@tmp_dir + "..").rmtree
+    end
+
+    context "with defined sizes" do
+      setup do
+      end
+
+      should "create resized versions of the input image" do
+        ImageSize.read(@image.preview.filepath).should == [200, 267]
+        ImageSize.read(@image.tall.filepath).should == [150, 200]
+        ImageSize.read(@image.thumbnail.filepath).should == [38, 50]
+        ImageSize.read(@image.icon.filepath).should == [50, 50]
+        ImageSize.read(@image.greyscale.filepath).should == [38, 50]
+      end
+    end
+    context "in templates" do
+
+      should "render an <img/> tag in HTML format" do
+        assert_same_elements @image.to_html.split(' '), %(<img src="#{@image.src}" width="400" height="533" alt="" />).split(" ")
+      end
+
+      should "use passed hash to overwrite tag attributes" do
+        attr = {
+          :alt => "Magic",
+          :class => "magic",
+          :rel => "lightbox",
+          :random => "present"
+        }
+        assert_same_elements @image.to_html(attr).split(" "), %(<img src="#{@image.src}" width="400" height="533" alt="Magic" class="magic" rel="lightbox" random="present" />).split(" ")
+      end
+
+      should "be intelligent about setting width & height" do
+        assert_same_elements @image.to_html({ :width => 100 }).split(" "), %(<img src="#{@image.src}" width="100" alt="" />).split(" ")
+        assert_same_elements @image.to_html({ :height => 100 }).split(" "), %(<img src="#{@image.src}" height="100" alt="" />).split(" ")
+        assert_same_elements @image.to_html({ :width => 100, :height => 100 }).split(" "), %(<img src="#{@image.src}" width="100" height="100" alt="" />).split(" ")
+      end
+
+      should "turn off setting with & height if either is passed as false" do
+        assert_same_elements @image.to_html({ :width => false }).split(" "), %(<img src="#{@image.src}" alt="" />).split(" ")
+      end
+
+      should "escape values in params" do
+        assert_same_elements @image.to_html({ :alt => "<danger\">" }).split(" "), %(<img src="#{@image.src}" width="400" height="533" alt="&lt;danger&quot;&gt;" />).split(" ")
+      end
+
+      should "not include size parameters unless known" do
+        @image.value = "/somethingunknown.gif"
+        @image.src.should ==  "/somethingunknown.gif"
+        assert_same_elements @image.to_html.split(" "), %(<img src="#{@image.src}" alt="" />).split(" ")
+      end
+
+      should "output image tags for its sizes too" do
+        assert_same_elements @image.thumbnail.to_html(:alt => "Thumb").split(' '), %(<img src="#{@image.thumbnail.src}" width="38" height="50" alt="Thumb" />).split(" ")
+      end
+    end
+    context "defined by classes" do
+      setup do
+      end
+
+      teardown do
+      end
+
+      should "have image dimension and filesize information" do
+        @image.src.should == "/media/00234/0010/rose.jpg"
+        @image.filesize.should == 54746
+        @image.width.should == 400
+        @image.height.should == 533
+      end
+
+      should "have access to the original uploaded file through field.original" do
+        @image.src.should == "/media/00234/0010/rose.jpg"
+        @image.original.width.should == @image.width
+        @image.original.height.should == @image.height
+        @image.original.filesize.should == @image.filesize
+        @image.filepath.should == File.expand_path(File.join(Spontaneous.media_dir, "00234/0010/rose.jpg"))
+      end
+
+
+      should "have a 'sizes' config option that generates resized versions" do
+        assert_same_elements ResizingImageField.size_definitions.keys, [:preview, :thumbnail, :icon, :tall, :greyscale]
+      end
+
+      should "serialise attributes" do
+        serialised = S::FieldTypes.deserialize_field(@image.serialize_db)[:processed_values]
+        [:preview, :thumbnail, :icon, :tall].each do |size|
+          serialised.key?(size).should be_true
+          serialised[size][:src].should == "/media/00234/0010/rose.#{size}.jpg"
+        end
+        serialised[:preview][:width].should == 200
+        serialised[:tall][:height].should == 200
+        serialised[:thumbnail][:width].should == 38
+        serialised[:thumbnail][:height].should == 50
+        serialised[:icon][:width].should == 50
+        serialised[:icon][:height].should == 50
+        # pp serialised
+      end
+
+      should "persist attributes" do
+        @instance.save
+        @instance = ContentWithImage[@instance[:id]]
+        @instance.photo.thumbnail.src.should == "/media/00234/0010/rose.thumbnail.jpg"
+        @instance.photo.original.src.should == "/media/00234/0010/rose.jpg"
+      end
+
+      should "not throw errors when accessing size before value has been assigned" do
+        instance = ContentWithImage.new
+        instance.photo.thumbnail.should_not be_nil
+        instance.photo.thumbnail.src.should == ""
+      end
+    end
+
+    context "defined anonymously" do
+      setup do
+        # class ::ContentWithImage < Content
+        #   field :photo, :image do
+        #     sizes :preview => { :width => 200 },
+        #       :tall => { :height => 200 },
+        #       :thumbnail => { :fit => [50, 50] },
+        #       :icon => { :crop => [50, 50] }
+        #   end
+        # end
+        # @instance = ContentWithImage.new
+
+        # @content_id = 234
+        # @instance.stubs(:id).returns(@content_id)
+        # @image = @instance.photo
+        # @image.owner.should == @instance
+        # @image.value = @origin_image.to_s
+      end
+
+      teardown do
+        # Object.send(:remove_const, :ContentWithImage)
+        # (@tmp_dir + "..").rmtree
+      end
+
+      should "have image dimension and filesize information" do
+        @image.src.should == "/media/00234/0010/rose.jpg"
+        @image.filesize.should == 54746
+        @image.width.should == 400
+        @image.height.should == 533
+      end
+
+      should "have access to the original uploaded file through field.original" do
+        @image.original.width.should == @image.width
+        @image.original.height.should == @image.height
+        @image.original.filesize.should == @image.filesize
+      end
+      should "have a 'sizes' config option that generates resized versions" do
+        assert_same_elements @image.class.size_definitions.keys, [:preview, :thumbnail, :icon, :tall, :greyscale]
+        assert_same_elements @image.class.sizes.keys, [:preview, :thumbnail, :icon, :tall, :greyscale]
+      end
+
+      should "serialise attributes" do
+        serialised = S::FieldTypes.deserialize_field(@image.serialize_db)[:processed_values]
+        [:preview, :thumbnail, :icon, :tall].each do |size|
+          serialised.key?(size).should be_true
+          serialised[size][:src].should == "/media/00234/0010/rose.#{size}.jpg"
+        end
+        serialised[:preview][:width].should == 200
+        serialised[:tall][:height].should == 200
+        serialised[:thumbnail][:width].should == 38
+        serialised[:thumbnail][:height].should == 50
+        serialised[:icon][:width].should == 50
+        serialised[:icon][:height].should == 50
+        # pp serialised
+      end
+      should "persist attributes" do
+        @instance.save
+        @instance = ContentWithImage[@instance[:id]]
+        @instance.photo.thumbnail.src.should == "/media/00234/0010/rose.thumbnail.jpg"
+        @instance.photo.original.src.should == "/media/00234/0010/rose.jpg"
+      end
+
+    end
+  end
+end

data/test/unit/test_layouts.rb

@@ -0,0 +1,111 @@
+# encoding: UTF-8
+
+require 'test_helper'
+
+class LayoutsTest < MiniTest::Spec
+
+  def setup
+    Spontaneous::Render.use_development_renderer
+    self.template_root = ::File.expand_path('../../fixtures/layouts', __FILE__)
+  end
+
+  context "layouts" do
+
+    setup do
+      Spontaneous::Schema.reset!
+      class ::LayoutPage < Spontaneous::Page; end
+      class ::ABoxClass < Spontaneous::Box; end
+      class ::SubPage < LayoutPage; end
+      ABoxClass.style :monkey
+      ABoxClass.style :crazy
+
+      class ::SomeContent < Spontaneous::Piece; end
+    end
+
+    teardown do
+      [:LayoutPage, :SubPage, :ABoxClass, :SomeContent].each do |c|
+        Object.send(:remove_const, c) rescue nil
+      end
+    end
+
+    context "Page layouts" do
+      setup do
+      end
+      should "default to layouts/standard.html... if nothing defined" do
+        page = LayoutPage.new
+        assert_correct_template(page, 'layouts/standard')
+        page.render.should == "layouts/standard.html.cut\n"
+      end
+
+      should "return the first layout if some are declared but none declared default" do
+        LayoutPage.layout :custom1
+        LayoutPage.layout :custom2
+        page = LayoutPage.new
+        assert_correct_template(page, 'layouts/custom1')
+        page.render.should == "layouts/custom1.html.cut\n"
+      end
+      should "return the layout declared default" do
+        LayoutPage.layout :custom1
+        LayoutPage.layout :custom2, :default => true
+        page = LayoutPage.new
+        assert_correct_template(page, 'layouts/custom2')
+        page.render.should == "layouts/custom2.html.cut\n"
+      end
+
+      should "inherit templates from superclass" do
+        LayoutPage.layout :custom1
+        LayoutPage.layout :custom2, :default => true
+        page = SubPage.new
+        assert_correct_template(page, 'layouts/custom2')
+      end
+
+      should "be able to overwrite inherited templates from superclass" do
+        LayoutPage.layout :custom1
+        LayoutPage.layout :custom2, :default => true
+        SubPage.layout :custom3
+        page = SubPage.new
+        # page.layout.template.should == 'layouts/custom2'
+        assert_correct_template(page, 'layouts/custom2')
+      end
+
+      should "allow setting of style used" do
+        LayoutPage.layout :custom1
+        LayoutPage.layout :custom2
+        page = LayoutPage.new
+        assert_correct_template(page, 'layouts/custom1')
+        # page.layout.template.should == 'layouts/custom1'
+        page.layout = :custom2
+        assert_correct_template(page, 'layouts/custom2')
+        # page.layout.template.should == 'layouts/custom2'
+      end
+
+      should "allow setting of layout in sub-classes" do
+        LayoutPage.layout :custom1
+        LayoutPage.layout :custom2
+        SubPage.layout :custom3, :default => true
+        page = SubPage.new
+        assert_correct_template(page, 'layouts/custom3')
+        page.layout = :custom2
+        assert_correct_template(page, 'layouts/custom2')
+      end
+      # should "raise error when setting unknown layout" do
+      #   LayoutPage.layout :custom1
+      #   LayoutPage.layout :custom2
+      #   SubPage.layout :custom3
+      #   page = SubPage.new
+      #   lambda { page.layout = :wrong }.must_raise(Spontaneous::Errors::UnknownLayoutError)
+      # end
+      # should "have a list of formats" do
+      #   LayoutPage.layout :custom1
+      #   skip("Need to implement formats correctly")
+      # end
+
+      # should "be able to test that an instance supports a format" do
+      #   skip("Need to implement formats correctly")
+      #   LayoutPage.layout :custom1
+      #   page = LayoutPage.new
+      #   # page.provides_format?(:html).should be_true
+      # end
+    end
+  end
+end

data/test/unit/test_logger.rb

@@ -0,0 +1,80 @@
+# encoding: UTF-8
+
+
+require 'test_helper'
+
+# borrowed from Padrino
+class LoggerTest < MiniTest::Spec
+  include Spontaneous
+
+  def setup
+    Spontaneous.env = :test
+    Spontaneous::Logger::Config[:test][:stream] = :null # The default
+    Spontaneous::Logger.setup!
+  end
+
+  def setup_logger(options={})
+    @log    = StringIO.new
+    @logger = Spontaneous::Logger.new(options.merge(:stream => @log))
+  end
+
+  context 'for logger functionality' do
+
+    context 'check stream config' do
+
+      should 'use stdout if stream is nil' do
+        Spontaneous::Logger::Config[:test][:stream] = nil
+        Spontaneous::Logger.setup!
+        assert_equal $stdout, Spontaneous.logger.log
+      end
+
+      should 'use StringIO as default for test' do
+        assert_instance_of StringIO, Spontaneous.logger.log
+      end
+
+      should 'use a custom stream' do
+        my_stream = StringIO.new
+        Spontaneous::Logger::Config[:test][:stream] = my_stream
+        Spontaneous::Logger.setup!
+        assert_equal my_stream, Spontaneous.logger.log
+      end
+
+      context "with custom log file path" do
+        setup do
+          @relative_path = "tmp/log/mylogfile.log"
+        end
+
+        teardown do
+          FileUtils.rm_r(@relative_path) if File.exists?(@relative_path)
+        end
+
+        should "log to given file" do
+          logger = Spontaneous::Logger.setup(:logfile => @relative_path)
+          logger.log.path.should == @relative_path
+        end
+      end
+    end
+
+    should 'log something' do
+      setup_logger(:log_level => :error)
+      @logger.error "You log this error?"
+      assert_match(/You log this error?/, @log.string)
+      @logger.debug "You don't log this error!"
+      refute_match(/You don't log this error!/, @log.string)
+      @logger << "Yep this can be logged"
+      assert_match(/Yep this can be logged/, @log.string)
+    end
+
+    should "be silenceable" do
+      setup_logger(:log_level => :debug)
+      @logger.silent!
+      @logger.error "You log this error?"
+      assert_empty(@log.string)
+      @logger.resume!
+      @logger.error "You log this error?"
+      assert_match(/You log this error?/, @log.string)
+    end
+  end
+
+
+end

data/test/unit/test_media.rb

@@ -0,0 +1,70 @@
+# encoding: UTF-8
+
+require 'test_helper'
+
+class MediaTest < MiniTest::Spec
+  context "Utilitity methods" do
+    should "be able to sanitise filenames" do
+      filename = "Something with-dodgy 'characters'.many.jpg"
+      Media.to_filename(filename).should == "Something-with-dodgy-characters.many.jpg"
+    end
+  end
+  context "Content items" do
+    setup do
+      @media_dir = File.expand_path(File.join(File.dirname(__FILE__), "../../tmp/media"))
+      Spontaneous.media_dir = @media_dir
+      Site.stubs(:working_revision).returns(74)
+      @instance = Content.new
+      @instance.stubs(:id).returns(101)
+    end
+
+    should "be able to generate a revision and id based media path" do
+      @instance.media_filepath("something.jpg").should == File.join(@media_dir, "00101/0074/something.jpg")
+      @instance.media_urlpath("something.jpg").should == "/media/00101/0074/something.jpg"
+    end
+
+    context "file manipulation" do
+      setup do
+        @tmp_dir = Pathname.new(@media_dir)
+        @src_image =  Pathname.new(File.join(File.dirname(__FILE__), "../fixtures/images/rose.jpg")).realpath
+        @upload_dir = @tmp_dir + "tmp/1234"
+        @upload_dir.mkpath
+        @origin_image = @upload_dir + "rose.jpg"
+        FileUtils.cp(@src_image.to_s, @origin_image.to_s)
+      end
+
+      teardown do
+        (@tmp_dir + "..").rmtree
+      end
+
+      should "be able to move a file into the media path" do
+        @instance.make_media_file(@origin_image)
+        @origin_image.exist?.should be_true
+        dest_image = @tmp_dir + "00101/0074/rose.jpg"
+        dest_image.exist?.should be_true
+      end
+
+      should "honour the filename parameter when creating media files" do
+        @instance.make_media_file(@origin_image, 'crysanthemum.jpg')
+        @origin_image.exist?.should be_true
+        dest_image = @tmp_dir + "00101/0074/crysanthemum.jpg"
+        dest_image.exist?.should be_true
+      end
+
+      should "sanitise filenames" do
+        origin_image = @upload_dir + "illegal filename!.jpg"
+        FileUtils.cp(@src_image.to_s, origin_image.to_s)
+        @instance.make_media_file(origin_image)
+        dest_image = @tmp_dir + "00101/0074/illegal-filename.jpg"
+        dest_image.exist?.should be_true
+      end
+      should "sanitise custom filenames" do
+        origin_image = @upload_dir + "rose.jpg"
+        FileUtils.cp(@src_image.to_s, origin_image.to_s)
+        @instance.make_media_file(origin_image,  "other, 'illegal' filename!.jpg")
+        dest_image = @tmp_dir + "00101/0074/other-illegal-filename.jpg"
+        dest_image.exist?.should be_true
+      end
+    end
+  end
+end