tilt 1.2 → 1.3.1

data/test/tilt_blueclothtemplate_test.rb

@@ -5,35 +5,16 @@ begin
   require 'bluecloth'
 
   class BlueClothTemplateTest < Test::Unit::TestCase
-    setup do
-      Tilt.register('markdown', Tilt::BlueClothTemplate)
-      Tilt.register('md', Tilt::BlueClothTemplate)
-      Tilt.register('mkd', Tilt::BlueClothTemplate)
-    end
-    
-    teardown do
-      # Need to revert to RDiscount, otherwise the RDiscount test will fail
-      Tilt.register('markdown', Tilt::RDiscountTemplate)
-      Tilt.register('md', Tilt::RDiscountTemplate)
-      Tilt.register('mkd', Tilt::RDiscountTemplate)
-    end
-    
-    test "registered for '.markdown' files unless RDiscount is loaded" do
-      unless defined?(RDiscount)
-        assert_equal Tilt::BlueClothTemplate, Tilt['test.markdown']
-      end
+    test "registered for '.md' files" do
+      assert Tilt.mappings['md'].include?(Tilt::BlueClothTemplate)
     end
 
-    test "registered for '.md' files unless RDiscount is loaded" do
-      unless defined?(RDiscount)
-        assert_equal Tilt::BlueClothTemplate, Tilt['test.md']
-      end
+    test "registered for '.mkd' files" do
+      assert Tilt.mappings['mkd'].include?(Tilt::BlueClothTemplate)
     end
 
-    test "registered for '.mkd' files unless RDiscount is loaded" do
-      unless defined?(RDiscount)
-        assert_equal Tilt::BlueClothTemplate, Tilt['test.mkd']
-      end
+    test "registered for '.markdown' files" do
+      assert Tilt.mappings['markdown'].include?(Tilt::BlueClothTemplate)
     end
 
     test "preparing and evaluating templates on #render" do
@@ -41,6 +22,11 @@ begin
       assert_equal "<h1>Hello World!</h1>", template.render
     end
 
+    test "can be rendered more than once" do
+      template = Tilt::BlueClothTemplate.new { |t| "# Hello World!" }
+      3.times { assert_equal "<h1>Hello World!</h1>", template.render }
+    end
+
     test "smartypants when :smart is set" do
       template = Tilt::BlueClothTemplate.new(:smartypants => true) { |t|
         "OKAY -- 'Smarty Pants'" }

data/test/tilt_buildertemplate_test.rb

@@ -14,6 +14,11 @@ begin
       assert_equal "<em>Hello World!</em>\n", template.render
     end
 
+    test "can be rendered more than once" do
+      template = Tilt::BuilderTemplate.new { |t| "xml.em 'Hello World!'" }
+      3.times { assert_equal "<em>Hello World!</em>\n", template.render }
+    end
+
     test "passing locals" do
       template = Tilt::BuilderTemplate.new { "xml.em('Hey ' + name + '!')" }
       assert_equal "<em>Hey Joe!</em>\n", template.render(Object.new, :name => 'Joe')
@@ -28,7 +33,7 @@ begin
 
     test "passing a block for yield" do
       template = Tilt::BuilderTemplate.new { "xml.em('Hey ' + yield + '!')" }
-      assert_equal "<em>Hey Joe!</em>\n", template.render { 'Joe' }
+      3.times { assert_equal "<em>Hey Joe!</em>\n", template.render { 'Joe' }}
     end
 
     test "block style templates" do
@@ -38,6 +43,16 @@ begin
         end
       assert_equal "<em>Hey Joe!</em>\n", template.render
     end
+
+    test "allows nesting raw XML" do
+      subtemplate = Tilt::BuilderTemplate.new { "xml.em 'Hello World!'" }
+      template = Tilt::BuilderTemplate.new { "xml.strong { xml << yield }" }
+      3.times do
+        options = { :xml => Builder::XmlMarkup.new }
+        assert_equal "<strong>\n<em>Hello World!</em>\n</strong>\n",
+          template.render(options) { subtemplate.render(options) }
+      end
+    end
   end
 rescue LoadError
   warn "Tilt::BuilderTemplate (disabled)"

data/test/tilt_coffeescripttemplate_test.rb

@@ -14,10 +14,46 @@ begin
       assert_match "puts('Hello, World!');", template.render
     end
 
+    test "can be rendered more than once" do
+      template = Tilt::CoffeeScriptTemplate.new { |t| "puts 'Hello, World!'\n" }
+      3.times { assert_match "puts('Hello, World!');", template.render }
+    end
+
     test "disabling coffee-script wrapper" do
-      template = Tilt::CoffeeScriptTemplate.new(:no_wrap => true) { |t| "puts 'Hello, World!'\n" }
+      str = "puts 'Hello, World!'\n"
+
+      template = Tilt::CoffeeScriptTemplate.new(:bare => true) { str }
+      assert_equal "puts('Hello, World!');", template.render
+
+      template2 = Tilt::CoffeeScriptTemplate.new(:no_wrap => true) { str}
       assert_equal "puts('Hello, World!');", template.render
     end
+
+    context "disabling coffee-script wrapper globally" do
+      setup do
+        @bare = Tilt::CoffeeScriptTemplate.default_bare
+      end
+
+      teardown do
+        Tilt::CoffeeScriptTemplate.default_bare = @bare
+      end
+
+      test "no options" do
+        template = Tilt::CoffeeScriptTemplate.new { |t| "puts 'Hello, World!'\n" }
+        assert_match "puts('Hello, World!');", template.render
+        assert_match "(function() {", template.render
+      end
+
+      test "overridden by :bare" do
+        template = Tilt::CoffeeScriptTemplate.new(:bare => false) { "puts 'Hello, World!'\n" }
+        assert_not_equal "puts('Hello, World!');", template.render
+      end
+
+      test "overridden by :no_wrap" do
+        template = Tilt::CoffeeScriptTemplate.new(:no_wrap => false) { "puts 'Hello, World!'\n" }
+        assert_not_equal "puts('Hello, World!');", template.render
+      end
+    end
   end
 
 rescue LoadError => boom

data/test/tilt_creoletemplate_test.rb

@@ -0,0 +1,24 @@
+require 'contest'
+require 'tilt'
+
+begin
+  require 'creole'
+
+  class CreoleTemplateTest < Test::Unit::TestCase
+    test "is registered for '.creole' files" do
+      assert_equal Tilt::CreoleTemplate, Tilt['test.creole']
+    end
+
+    test "compiles and evaluates the template on #render" do
+      template = Tilt::CreoleTemplate.new { |t| "= Hello World!" }
+      assert_equal "<h1>Hello World!</h1>", template.render
+    end
+
+    test "can be rendered more than once" do
+      template = Tilt::CreoleTemplate.new { |t| "= Hello World!" }
+      3.times { assert_equal "<h1>Hello World!</h1>", template.render }
+    end
+  end
+rescue LoadError => boom
+  warn "Tilt::CreoleTemplate (disabled)\n"
+end

data/test/tilt_erbtemplate_test.rb

@@ -6,12 +6,11 @@ require 'tempfile'
 
 class ERBTemplateTest < Test::Unit::TestCase
   test "registered for '.erb' files" do
-    assert_equal Tilt::ERBTemplate, Tilt['test.erb']
-    assert_equal Tilt::ERBTemplate, Tilt['test.html.erb']
+    assert Tilt.mappings['erb'].include?(Tilt::ERBTemplate)
   end
 
   test "registered for '.rhtml' files" do
-    assert_equal Tilt::ERBTemplate, Tilt['test.rhtml']
+    assert Tilt.mappings['rhtml'].include?(Tilt::ERBTemplate)
   end
 
   test "loading and evaluating templates on #render" do
@@ -19,6 +18,11 @@ class ERBTemplateTest < Test::Unit::TestCase
     assert_equal "Hello World!", template.render
   end
 
+  test "can be rendered more than once" do
+    template = Tilt::ERBTemplate.new { |t| "Hello World!" }
+    3.times { assert_equal "Hello World!", template.render }
+  end
+
   test "passing locals" do
     template = Tilt::ERBTemplate.new { 'Hey <%= name %>!' }
     assert_equal "Hey Joe!", template.render(Object.new, :name => 'Joe')
@@ -62,9 +66,9 @@ class ERBTemplateTest < Test::Unit::TestCase
       fail 'should have raised an exception'
     rescue => boom
       assert_kind_of NameError, boom
-      line = boom.backtrace.first
+      line = boom.backtrace.grep(/^test\.erb:/).first
+      assert line, "Backtrace didn't contain test.erb"
       file, line, meth = line.split(":")
-      assert_equal 'test.erb', file
       assert_equal '13', line
     end
   end
@@ -85,18 +89,18 @@ class ERBTemplateTest < Test::Unit::TestCase
     end
   end
 
-  test "default non-stripping trim mode" do
-    template = Tilt.new('test.erb', 1) { "\n<%= 1 + 1 %>\n" }
-    assert_equal "\n2\n", template.render
+  test "default stripping trim mode" do
+    template = Tilt::ERBTemplate.new('test.erb', 1) { "\n<%= 1 + 1 %>\n" }
+    assert_equal "\n2", template.render
   end
 
   test "stripping trim mode" do
-    template = Tilt.new('test.erb', 1, :trim => '-') { "\n<%= 1 + 1 -%>\n" }
+    template = Tilt::ERBTemplate.new('test.erb', 1, :trim => '-') { "\n<%= 1 + 1 -%>\n" }
     assert_equal "\n2", template.render
   end
 
   test "shorthand whole line syntax trim mode" do
-    template = Tilt.new('test.erb', :trim => '%') { "\n% if true\nhello\n%end\n" }
+    template = Tilt::ERBTemplate.new('test.erb', :trim => '%') { "\n% if true\nhello\n%end\n" }
     assert_equal "\nhello\n", template.render
   end
 
@@ -159,9 +163,9 @@ class CompiledERBTemplateTest < Test::Unit::TestCase
       fail 'should have raised an exception'
     rescue => boom
       assert_kind_of NameError, boom
-      line = boom.backtrace.first
+      line = boom.backtrace.grep(/^test\.erb:/).first
+      assert line, "Backtrace didn't contain test.erb"
       file, line, meth = line.split(":")
-      assert_equal 'test.erb', file
       assert_equal '13', line
     end
   end
@@ -182,18 +186,18 @@ class CompiledERBTemplateTest < Test::Unit::TestCase
     end
   end
 
-  test "default non-stripping trim mode" do
-    template = Tilt.new('test.erb') { "\n<%= 1 + 1 %>\n" }
-    assert_equal "\n2\n", template.render(Scope.new)
+  test "default stripping trim mode" do
+    template = Tilt::ERBTemplate.new('test.erb') { "\n<%= 1 + 1 %>\n" }
+    assert_equal "\n2", template.render(Scope.new)
   end
 
   test "stripping trim mode" do
-    template = Tilt.new('test.erb', :trim => '-') { "\n<%= 1 + 1 -%>\n" }
+    template = Tilt::ERBTemplate.new('test.erb', :trim => '-') { "\n<%= 1 + 1 -%>\n" }
     assert_equal "\n2", template.render(Scope.new)
   end
 
   test "shorthand whole line syntax trim mode" do
-    template = Tilt.new('test.erb', :trim => '%') { "\n% if true\nhello\n%end\n" }
+    template = Tilt::ERBTemplate.new('test.erb', :trim => '%') { "\n% if true\nhello\n%end\n" }
     assert_equal "\nhello\n", template.render(Scope.new)
   end
 
@@ -203,7 +207,7 @@ class CompiledERBTemplateTest < Test::Unit::TestCase
     f.puts('ふが <%= @hoge %>')
     f.close()
     @hoge = "ほげ"
-    erb = Tilt['erb'].new(f.path)
+    erb = Tilt::ERBTemplate.new(f.path)
     3.times { erb.render(self) }
     f.delete
   end
@@ -213,7 +217,7 @@ class CompiledERBTemplateTest < Test::Unit::TestCase
     f.puts('ふが <%= @hoge %>')
     f.close()
     @hoge = "ほげ"
-    erb = Tilt['erb'].new(f.path, :default_encoding => 'UTF-8')
+    erb = Tilt::ERBTemplate.new(f.path, :default_encoding => 'UTF-8')
     3.times { erb.render(self) }
     f.delete
   end

data/test/tilt_erubistemplate_test.rb

@@ -9,11 +9,26 @@ begin
       assert_equal Tilt::ErubisTemplate, Tilt['test.html.erubis']
     end
 
+    test "registered above ERB" do
+      %w[erb rhtml].each do |ext|
+        mappings = Tilt.mappings[ext]
+        erubis_idx = mappings.index(Tilt::ErubisTemplate)
+        erb_idx = mappings.index(Tilt::ERBTemplate)
+        assert erubis_idx < erb_idx,
+          "#{erubis_idx} should be lower than #{erb_idx}"
+      end
+    end
+
     test "preparing and evaluating templates on #render" do
       template = Tilt::ErubisTemplate.new { |t| "Hello World!" }
       assert_equal "Hello World!", template.render
     end
 
+    test "can be rendered more than once" do
+      template = Tilt::ErubisTemplate.new { |t| "Hello World!" }
+      3.times { assert_equal "Hello World!", template.render }
+    end
+
     test "passing locals" do
       template = Tilt::ErubisTemplate.new { 'Hey <%= name %>!' }
       assert_equal "Hey Joe!", template.render(Object.new, :name => 'Joe')
@@ -57,9 +72,9 @@ begin
         fail 'should have raised an exception'
       rescue => boom
         assert_kind_of NameError, boom
-        line = boom.backtrace.first
+        line = boom.backtrace.grep(/^test\.erubis:/).first
+        assert line, "Backtrace didn't contain test.erubis"
         file, line, meth = line.split(":")
-        assert_equal 'test.erubis', file
         assert_equal '13', line
       end
     end

data/test/tilt_fallback_test.rb

@@ -0,0 +1,122 @@
+require 'contest'
+require 'tilt'
+
+class TiltFallbackTest < Test::Unit::TestCase
+  class FailTemplate  < Tilt::Template
+    def self.engine_initialized?; false end
+    def prepare;                        end
+    
+    def initialize_engine
+      raise LoadError, "can't load #{self.class}"
+    end
+  end
+
+  class WinTemplate < Tilt::Template
+    def self.engine_initialized?; true end
+    def prepare;                       end
+  end
+
+  FailTemplate2 = Class.new(FailTemplate)
+  WinTemplate2  = Class.new(WinTemplate)
+
+  def set_ivar(obj, name, value)
+    obj.instance_variable_set("@#{name}", value)
+  end
+  
+  def clear_ivar(obj, name)
+    ivar = "@#{name}"
+    value = obj.instance_variable_get(ivar)
+  ensure
+    obj.instance_variable_set(ivar, value.dup.clear)
+  end
+
+  setup do
+    # Make sure every test have no mappings.
+    @p = clear_ivar(Tilt, :preferred_mappings)
+    @t = clear_ivar(Tilt, :template_mappings)
+  end
+
+  teardown do
+    set_ivar(Tilt, :preferred_mappings, @p)
+    set_ivar(Tilt, :template_mappings, @t)
+  end
+
+  test "returns nil on unregistered extensions" do
+    template = Tilt["md"]
+    assert_equal nil, template
+  end
+
+  test "returns the last registered template" do
+    Tilt.register("md", WinTemplate)
+    Tilt.register("md", WinTemplate2)
+    
+    template = Tilt["md"]
+    assert_equal WinTemplate2, template
+  end
+
+  test "returns the last registered working template" do
+    Tilt.register("md", WinTemplate)
+    Tilt.register("md", FailTemplate)
+
+    template = Tilt["md"]
+    assert_equal WinTemplate, template
+  end
+
+  test "if every template fails, raise the exception from the first template" do
+    Tilt.register("md", FailTemplate)
+    Tilt.register("md", FailTemplate2)
+
+    exc = assert_raise(LoadError) { Tilt["md"] }
+    assert_match /FailTemplate2/, exc.message
+  end
+
+  test ".prefer should also register the template" do
+    Tilt.prefer(WinTemplate, "md")
+    assert Tilt.registered?("md")
+  end
+
+  test ".prefer always win" do
+    Tilt.register("md", FailTemplate)
+    Tilt.register("md", WinTemplate)
+    Tilt.prefer(FailTemplate, "md")
+
+    template = Tilt["md"]
+    assert_equal FailTemplate, template
+  end
+
+  test ".prefer accepts multiple extensions" do
+    extensions = %w[md mkd markdown]
+    Tilt.prefer(FailTemplate, *extensions)
+    
+    extensions.each do |ext|
+      template = Tilt[ext]
+      assert_equal FailTemplate, template
+    end
+  end
+
+  test ".prefer with no extension should use already registered extensions" do
+    extensions = %w[md mkd markdown]
+    
+    extensions.each do |ext|
+      Tilt.register(ext, FailTemplate)
+      Tilt.register(ext, WinTemplate)
+    end
+
+    Tilt.prefer(FailTemplate)
+
+    extensions.each do |ext|
+      template = Tilt[ext]
+      assert_equal FailTemplate, template
+    end
+  end
+  
+  test ".prefer should only override extensions the preferred library is registered for"  do
+    Tilt.register("md", WinTemplate)
+    Tilt.register("mkd", FailTemplate)
+    Tilt.register("mkd", WinTemplate)
+    Tilt.prefer(FailTemplate)
+    assert_equal FailTemplate, Tilt["mkd"]
+    assert_equal WinTemplate, Tilt["md"]
+  end
+end
+

data/test/tilt_hamltemplate_test.rb

@@ -17,6 +17,11 @@ begin
       assert_equal "<p>Hello World!</p>\n", template.render
     end
 
+    test "can be rendered more than once" do
+      template = Tilt::HamlTemplate.new { |t| "%p Hello World!" }
+      3.times { assert_equal "<p>Hello World!</p>\n", template.render }
+    end
+
     test "passing locals" do
       template = Tilt::HamlTemplate.new { "%p= 'Hey ' + name + '!'" }
       assert_equal "<p>Hey Joe!</p>\n", template.render(Object.new, :name => 'Joe')
@@ -43,9 +48,9 @@ begin
         fail 'should have raised an exception'
       rescue => boom
         assert_kind_of NameError, boom
-        line = boom.backtrace.first
+        line = boom.backtrace.grep(/^test\.haml:/).first
+        assert line, "Backtrace didn't contain test.haml"
         file, line, meth = line.split(":")
-        assert_equal 'test.haml', file
         assert_equal '12', line
       end
     end
@@ -103,9 +108,9 @@ begin
         fail 'should have raised an exception'
       rescue => boom
         assert_kind_of NameError, boom
-        line = boom.backtrace.first
+        line = boom.backtrace.grep(/^test\.haml:/).first
+        assert line, "Backtrace didn't contain test.haml"
         file, line, meth = line.split(":")
-        assert_equal 'test.haml', file
         assert_equal '12', line
       end
     end

data/test/tilt_kramdown_test.rb

@@ -0,0 +1,42 @@
+require 'contest'
+require 'tilt'
+
+begin
+  require 'kramdown'
+
+  class MarukuTemplateTest < Test::Unit::TestCase
+    test "registered for '.md' files" do
+      assert Tilt.mappings['md'].include?(Tilt::KramdownTemplate)
+    end
+
+    test "registered for '.mkd' files" do
+      assert Tilt.mappings['mkd'].include?(Tilt::KramdownTemplate)
+    end
+
+    test "registered for '.markdown' files" do
+      assert Tilt.mappings['markdown'].include?(Tilt::KramdownTemplate)
+    end
+
+    test "registered above MarukuTemplate" do
+      %w[md mkd markdown].each do |ext|
+        mappings = Tilt.mappings[ext]
+        kram_idx = mappings.index(Tilt::KramdownTemplate)
+        maru_idx = mappings.index(Tilt::MarukuTemplate)
+        assert kram_idx < maru_idx,
+          "#{kram_idx} should be lower than #{maru_idx}"
+      end
+    end
+
+    test "preparing and evaluating templates on #render" do
+      template = Tilt::KramdownTemplate.new { |t| "# Hello World!" }
+      assert_equal "<h1 id='hello_world'>Hello World!</h1>", template.render
+    end
+
+    test "can be rendered more than once" do
+      template = Tilt::KramdownTemplate.new { |t| "# Hello World!" }
+      3.times { assert_equal "<h1 id='hello_world'>Hello World!</h1>", template.render }
+    end
+  end
+rescue LoadError => boom
+  warn "Tilt::KramdownTemplate (disabled)\n"
+end

data/test/tilt_lesstemplate_test.rb

@@ -13,6 +13,11 @@ begin
       template = Tilt::LessTemplate.new { |t| ".bg { background-color: #0000ff; } \n#main\n { .bg; }\n" }
       assert_equal ".bg, #main { background-color: #0000ff; }\n", template.render
     end
+
+    test "can be rendered more than once" do
+      template = Tilt::LessTemplate.new { |t| ".bg { background-color: #0000ff; } \n#main\n { .bg; }\n" }
+      3.times { assert_equal ".bg, #main { background-color: #0000ff; }\n", template.render }
+    end
   end
 
 rescue LoadError => boom

data/test/tilt_liquidtemplate_test.rb

@@ -14,6 +14,11 @@ begin
       assert_equal "Hello World!", template.render
     end
 
+    test "can be rendered more than once" do
+      template = Tilt::LiquidTemplate.new { |t| "Hello World!" }
+      3.times { assert_equal "Hello World!", template.render }
+    end
+
     test "passing locals" do
       template = Tilt::LiquidTemplate.new { "Hey {{ name }}!" }
       assert_equal "Hey Joe!", template.render(nil, :name => 'Joe')

data/test/tilt_markaby_test.rb

@@ -26,6 +26,11 @@ begin
       assert_equal "<html></html>", tilt.render
     end
 
+    test "can be rendered more than once" do
+      tilt = ::Tilt::MarkabyTemplate.new { "html do; end" }
+      3.times { assert_equal "<html></html>", tilt.render }
+    end
+
     test "should evaluate a template file in the scope given" do
       scope = Object.new
       def scope.foo