temple 0.3.5 → 0.4.0

data/.travis.yml

@@ -1,7 +1,6 @@
 rvm:
   - 1.8.7
-  - 1.9.2
   - 1.9.3
   - ruby-head
-  - rbx-2.0
   - jruby
+  - rbx-18mode

data/CHANGES

@@ -1,4 +1,8 @@
-master
+0.4.0
+
+  * Split Temple::HTML::AttributeMerger in AttributeSorter,
+    AttributeMerger and AttributeRemover
+  * Fix issue #58
 
 0.3.5
 

data/lib/temple.rb

@@ -1,48 +1,51 @@
 require 'temple/version'
 
 module Temple
-  autoload :InvalidExpression, 'temple/generators'
-  autoload :Generator,         'temple/generators'
-  autoload :Generators,        'temple/generators'
-  autoload :Engine,            'temple/engine'
-  autoload :Utils,             'temple/utils'
-  autoload :Filter,            'temple/filter'
-  autoload :Templates,         'temple/templates'
-  autoload :Grammar,           'temple/grammar'
-  autoload :ImmutableHash,     'temple/hash'
-  autoload :MutableHash,       'temple/hash'
+  autoload :InvalidExpression,    'temple/generators'
+  autoload :Generator,            'temple/generators'
+  autoload :Generators,           'temple/generators'
+  autoload :Engine,               'temple/engine'
+  autoload :Utils,                'temple/utils'
+  autoload :Filter,               'temple/filter'
+  autoload :Templates,            'temple/templates'
+  autoload :Grammar,              'temple/grammar'
+  autoload :ImmutableHash,        'temple/hash'
+  autoload :MutableHash,          'temple/hash'
 
   module Mixins
-    autoload :Dispatcher,      'temple/mixins/dispatcher'
-    autoload :EngineDSL,       'temple/mixins/engine_dsl'
-    autoload :GrammarDSL,      'temple/mixins/grammar_dsl'
-    autoload :Options,         'temple/mixins/options'
-    autoload :DefaultOptions,  'temple/mixins/options'
-    autoload :Template,        'temple/mixins/template'
+    autoload :Dispatcher,         'temple/mixins/dispatcher'
+    autoload :CompiledDispatcher, 'temple/mixins/dispatcher'
+    autoload :EngineDSL,          'temple/mixins/engine_dsl'
+    autoload :GrammarDSL,         'temple/mixins/grammar_dsl'
+    autoload :Options,            'temple/mixins/options'
+    autoload :DefaultOptions,     'temple/mixins/options'
+    autoload :Template,           'temple/mixins/template'
   end
 
   module ERB
-    autoload :Engine,          'temple/erb/engine'
-    autoload :Parser,          'temple/erb/parser'
-    autoload :Trimming,        'temple/erb/trimming'
-    autoload :Template,        'temple/erb/template'
+    autoload :Engine,             'temple/erb/engine'
+    autoload :Parser,             'temple/erb/parser'
+    autoload :Trimming,           'temple/erb/trimming'
+    autoload :Template,           'temple/erb/template'
   end
 
   module Filters
-    autoload :ControlFlow,     'temple/filters/control_flow'
-    autoload :MultiFlattener,  'temple/filters/multi_flattener'
-    autoload :StaticMerger,    'temple/filters/static_merger'
-    autoload :DynamicInliner,  'temple/filters/dynamic_inliner'
-    autoload :Escapable,       'temple/filters/escapable'
-    autoload :Eraser,          'temple/filters/eraser'
-    autoload :Validator,       'temple/filters/validator'
+    autoload :ControlFlow,        'temple/filters/control_flow'
+    autoload :MultiFlattener,     'temple/filters/multi_flattener'
+    autoload :StaticMerger,       'temple/filters/static_merger'
+    autoload :DynamicInliner,     'temple/filters/dynamic_inliner'
+    autoload :Escapable,          'temple/filters/escapable'
+    autoload :Eraser,             'temple/filters/eraser'
+    autoload :Validator,          'temple/filters/validator'
   end
 
   module HTML
-    autoload :Dispatcher,      'temple/html/dispatcher'
-    autoload :Filter,          'temple/html/filter'
-    autoload :Fast,            'temple/html/fast'
-    autoload :Pretty,          'temple/html/pretty'
-    autoload :AttributeMerger, 'temple/html/attribute_merger'
+    autoload :Dispatcher,         'temple/html/dispatcher'
+    autoload :Filter,             'temple/html/filter'
+    autoload :Fast,               'temple/html/fast'
+    autoload :Pretty,             'temple/html/pretty'
+    autoload :AttributeMerger,    'temple/html/attribute_merger'
+    autoload :AttributeSorter,    'temple/html/attribute_sorter'
+    autoload :AttributeRemover,   'temple/html/attribute_remover'
   end
 end

data/lib/temple/html/attribute_merger.rb

@@ -1,9 +1,9 @@
 module Temple
   module HTML
+    # This filter merges html attributes (e.g. used for id and class)
     # @api public
     class AttributeMerger < Filter
       default_options[:attr_delimiter] = {'id' => '_', 'class' => ' '}
-      default_options[:sort_attrs] = true
 
       def on_html_attrs(*attrs)
         names = []
@@ -41,37 +41,7 @@ module Temple
             names << name
           end
         end
-        result = options[:sort_attrs] ? result.sort : names.map {|k| [k, result[k]] }
-        [:multi, *result.map {|name,attr| compile(attr) }]
-      end
-
-      def on_html_attr(name, value)
-        if empty_exp?(value)
-          value
-        elsif contains_static?(value)
-          [:html, :attr, name, value]
-        else
-          tmp = unique_name
-          [:multi,
-           [:capture, tmp, compile(value)],
-           [:if, "!#{tmp}.empty?",
-            [:html, :attr, name, [:dynamic, tmp]]]]
-        end
-      end
-
-      protected
-
-      def contains_static?(exp)
-        case exp[0]
-        when :multi
-          exp[1..-1].any? {|e| contains_static?(e) }
-        when :escape
-          contains_static?(exp[2])
-        when :static
-          true
-        else
-          false
-        end
+        [:html, :attrs, *names.map {|name| result[name] }]
       end
     end
   end

data/lib/temple/html/attribute_remover.rb

@@ -0,0 +1,28 @@
+module Temple
+  module HTML
+    # This filter removes empty attributes
+    # @api public
+    class AttributeRemover < Filter
+      default_options[:remove_empty_attrs] = true
+
+      def on_html_attrs(*attrs)
+        [:multi, *(options[:remove_empty_attrs] ?
+                   attrs.map {|attr| compile(attr) } : attrs)]
+      end
+
+      def on_html_attr(name, value)
+        if empty_exp?(value)
+          value
+        elsif contains_static?(value)
+          [:html, :attr, name, value]
+        else
+          tmp = unique_name
+          [:multi,
+           [:capture, tmp, compile(value)],
+           [:if, "!#{tmp}.empty?",
+            [:html, :attr, name, [:dynamic, tmp]]]]
+        end
+      end
+    end
+  end
+end

data/lib/temple/html/attribute_sorter.rb

@@ -0,0 +1,17 @@
+module Temple
+  module HTML
+    # This filter sorts html attributes.
+    # @api public
+    class AttributeSorter < Filter
+      default_options[:sort_attrs] = true
+
+      def on_html_attrs(*attrs)
+        attrs = attrs.sort_by do |attr|
+          raise(InvalidExpression, 'Attribute is not a html attr') if attr[0] != :html || attr[1] != :attr
+          attr[2].to_s
+        end if options[:sort_attrs]
+        [:html, :attrs, *attrs]
+      end
+    end
+  end
+end

data/lib/temple/mixins/engine_dsl.rb

@@ -88,12 +88,13 @@ module Temple
         chain_modified!
       end
 
-      def filter(name, *options)
-        use(name, Temple::Filters.const_get(name), *options)
-      end
-
-      def generator(name, *options)
-        use(name, Temple::Generators.const_get(name), *options)
+      # Shortcuts to access namespaces
+      { :filter    => Temple::Filters,
+        :generator => Temple::Generators,
+        :html      => Temple::HTML }.each do |method, mod|
+        define_method(method) do |name, *options|
+          use(name, mod.const_get(name), *options)
+        end
       end
 
       private
@@ -113,8 +114,9 @@ module Temple
         if Class === name
           filter = name
           name = filter.name.to_sym
+        else
+          raise(ArgumentError, 'First argument must be Class or Symbol') unless Symbol === name
         end
-        raise(ArgumentError, 'First argument must be Class or Symbol') unless Symbol === name
 
         if block
           raise(ArgumentError, 'Class and block argument are not allowed at the same time') if filter

data/lib/temple/utils.rb

@@ -58,6 +58,19 @@ module Temple
       "_#{prefix}#{@unique_name += 1}"
     end
 
+    def contains_static?(exp)
+      case exp[0]
+      when :multi
+        exp[1..-1].any? {|e| contains_static?(e) }
+      when :escape
+        contains_static?(exp[2])
+      when :static
+        true
+      else
+        false
+      end
+    end
+
     # Check if expression is empty
     #
     # @param exp [Array] Temple expression

data/lib/temple/version.rb

@@ -1,3 +1,3 @@
 module Temple
-  VERSION = '0.3.5'
+  VERSION = '0.4.0'
 end

data/test/html/test_attribute_merger.rb

@@ -3,7 +3,6 @@ require 'helper'
 describe Temple::HTML::AttributeMerger do
   before do
     @merger = Temple::HTML::AttributeMerger.new
-    @ordered_merger = Temple::HTML::AttributeMerger.new :sort_attrs => false
   end
 
   it 'should pass static attributes through' do
@@ -12,12 +11,12 @@ describe Temple::HTML::AttributeMerger do
       [:html, :attrs, [:html, :attr, 'class', [:static, 'b']]],
       [:content]
     ]).should.equal [:html, :tag, "div",
-                     [:multi,
+                     [:html, :attrs,
                       [:html, :attr, "class", [:static, "b"]]],
                      [:content]]
   end
 
-  it 'should sort html attributes by name by default, when :sort_attrs is true' do
+  it 'should preserve the order of html attributes' do
     @merger.call([:html, :tag,
       'meta',
       [:html, :attrs, [:html, :attr, 'c', [:static, '1']],
@@ -25,69 +24,39 @@ describe Temple::HTML::AttributeMerger do
                       [:html, :attr, 'a', [:static, '3']],
                       [:html, :attr, 'b', [:static, '4']]]
     ]).should.equal [:html, :tag, 'meta',
-                     [:multi,
-                      [:html, :attr, 'a', [:static, '3']],
-                      [:html, :attr, 'b', [:static, '4']],
-                      [:html, :attr, 'c', [:static, '1']],
-                      [:html, :attr, 'd', [:static, '2']]]]
-  end
-
-  it 'should preserve the order of html attributes when :sort_attrs is false' do
-    @ordered_merger.call([:html, :tag,
-      'meta',
-      [:html, :attrs, [:html, :attr, 'c', [:static, '1']],
-                      [:html, :attr, 'd', [:static, '2']],
-                      [:html, :attr, 'a', [:static, '3']],
-                      [:html, :attr, 'b', [:static, '4']]]
-    ]).should.equal [:html, :tag, 'meta',
-                     [:multi,
+                     [:html, :attrs,
                       [:html, :attr, 'c', [:static, '1']],
                       [:html, :attr, 'd', [:static, '2']],
                       [:html, :attr, 'a', [:static, '3']],
                       [:html, :attr, 'b', [:static, '4']]]]
 
     # Use case:
-    @ordered_merger.call([:html, :tag,
+    @merger.call([:html, :tag,
       'meta',
       [:html, :attrs, [:html, :attr, 'http-equiv', [:static, 'Content-Type']],
                       [:html, :attr, 'content', [:static, '']]]
     ]).should.equal [:html, :tag, 'meta',
-                     [:multi,
+                     [:html, :attrs,
                       [:html, :attr, 'http-equiv', [:static, 'Content-Type']],
                       [:html, :attr, 'content', [:static, '']]]]
   end
 
-  it 'should check for empty dynamic attribute' do
-    @merger.call([:html, :tag,
-      'div',
-      [:html, :attrs, [:html, :attr, 'class', [:dynamic, 'b']]],
-      [:content]
-    ]).should.equal [:html, :tag, "div",
-                     [:multi,
-                      [:multi,
-                       [:capture, "_temple_html_attributemerger1", [:dynamic, "b"]],
-                       [:if, "!_temple_html_attributemerger1.empty?",
-                        [:html, :attr, "class", [:dynamic, "_temple_html_attributemerger1"]]]]],
-                     [:content]]
-  end
-
   it 'should merge ids' do
     @merger.call([:html, :tag,
       'div',
       [:html, :attrs, [:html, :attr, 'id', [:dynamic, 'a']], [:html, :attr, 'id', [:dynamic, 'b']]],
       [:content]
     ]).should.equal [:html, :tag, "div",
-                     [:multi,
-                      [:multi,
-                       [:capture, "_temple_html_attributemerger2",
-                        [:multi, [:dynamic, "a"],
-                         [:capture, "_temple_html_attributemerger1",
-                          [:dynamic, "b"]],
-                         [:if, "!_temple_html_attributemerger1.empty?",
-                          [:multi, [:static, "_"],
-                           [:dynamic, "_temple_html_attributemerger1"]]]]],
-                       [:if, "!_temple_html_attributemerger2.empty?",
-                        [:html, :attr, "id", [:dynamic, "_temple_html_attributemerger2"]]]]],
+                     [:html, :attrs,
+                      [:html, :attr, "id",
+                       [:multi,
+                        [:dynamic, "a"],
+                        [:capture, "_temple_html_attributemerger1",
+                         [:dynamic, "b"]],
+                        [:if, "!_temple_html_attributemerger1.empty?",
+                         [:multi,
+                          [:static, "_"],
+                          [:dynamic, "_temple_html_attributemerger1"]]]]]],
                      [:content]]
   end
 
@@ -97,12 +66,15 @@ describe Temple::HTML::AttributeMerger do
       [:html, :attrs, [:html, :attr, 'class', [:static, 'a']], [:html, :attr, 'class', [:dynamic, 'b']]],
       [:content]
     ]).should.equal [:html, :tag, "div",
-                     [:multi,
+                     [:html, :attrs,
                       [:html, :attr, "class",
-                       [:multi, [:static, "a"],
-                        [:capture, "_temple_html_attributemerger1", [:dynamic, "b"]],
+                       [:multi,
+                        [:static, "a"],
+                        [:capture, "_temple_html_attributemerger1",
+                         [:dynamic, "b"]],
                         [:if, "!_temple_html_attributemerger1.empty?",
-                         [:multi, [:static, " "],
+                         [:multi,
+                          [:static, " "],
                           [:dynamic, "_temple_html_attributemerger1"]]]]]],
                      [:content]]
   end

data/test/html/test_attribute_remover.rb

@@ -0,0 +1,44 @@
+require 'helper'
+
+describe Temple::HTML::AttributeRemover do
+  before do
+    @remover = Temple::HTML::AttributeRemover.new
+    @disabled_remover = Temple::HTML::AttributeRemover.new :remove_empty_attrs => false
+  end
+
+  it 'should pass static attributes through' do
+    @remover.call([:html, :tag,
+      'div',
+      [:html, :attrs, [:html, :attr, 'class', [:static, 'b']]],
+      [:content]
+    ]).should.equal [:html, :tag, "div",
+                     [:multi,
+                      [:html, :attr, "class", [:static, "b"]]],
+                     [:content]]
+  end
+
+  it 'should check for empty dynamic attribute if :remove_empty_attrs is true' do
+    @remover.call([:html, :tag,
+      'div',
+      [:html, :attrs, [:html, :attr, 'class', [:dynamic, 'b']]],
+      [:content]
+    ]).should.equal [:html, :tag, "div",
+                    [:multi,
+                      [:multi,
+                       [:capture, "_temple_html_attributeremover1", [:dynamic, "b"]],
+                       [:if, "!_temple_html_attributeremover1.empty?",
+                        [:html, :attr, "class", [:dynamic, "_temple_html_attributeremover1"]]]]],
+                     [:content]]
+  end
+
+  it 'should not check for empty dynamic attribute if :remove_empty_attrs is false' do
+    @disabled_remover.call([:html, :tag,
+      'div',
+      [:html, :attrs, [:html, :attr, 'class', [:dynamic, 'b']]],
+      [:content]
+    ]).should.equal [:html, :tag, "div",
+                     [:multi,
+                      [:html, :attr, "class", [:dynamic, "b"]]],
+                     [:content]]
+  end
+end

data/test/html/test_attribute_sorter.rb

@@ -0,0 +1,48 @@
+require 'helper'
+
+describe Temple::HTML::AttributeSorter do
+  before do
+    @ordered   = Temple::HTML::AttributeSorter.new
+    @unordered = Temple::HTML::AttributeSorter.new :sort_attrs => false
+  end
+
+  it 'should sort html attributes by name by default, when :sort_attrs is true' do
+    @ordered.call([:html, :tag,
+      'meta',
+      [:html, :attrs, [:html, :attr, 'c', [:static, '1']],
+                      [:html, :attr, 'd', [:static, '2']],
+                      [:html, :attr, 'a', [:static, '3']],
+                      [:html, :attr, 'b', [:static, '4']]]
+    ]).should.equal [:html, :tag, 'meta',
+                     [:html, :attrs,
+                      [:html, :attr, 'a', [:static, '3']],
+                      [:html, :attr, 'b', [:static, '4']],
+                      [:html, :attr, 'c', [:static, '1']],
+                      [:html, :attr, 'd', [:static, '2']]]]
+  end
+
+  it 'should preserve the order of html attributes when :sort_attrs is false' do
+    @unordered.call([:html, :tag,
+      'meta',
+      [:html, :attrs, [:html, :attr, 'c', [:static, '1']],
+                      [:html, :attr, 'd', [:static, '2']],
+                      [:html, :attr, 'a', [:static, '3']],
+                      [:html, :attr, 'b', [:static, '4']]]
+    ]).should.equal [:html, :tag, 'meta',
+                     [:html, :attrs,
+                      [:html, :attr, 'c', [:static, '1']],
+                      [:html, :attr, 'd', [:static, '2']],
+                      [:html, :attr, 'a', [:static, '3']],
+                      [:html, :attr, 'b', [:static, '4']]]]
+
+    # Use case:
+    @unordered.call([:html, :tag,
+      'meta',
+      [:html, :attrs, [:html, :attr, 'http-equiv', [:static, 'Content-Type']],
+                      [:html, :attr, 'content', [:static, '']]]
+    ]).should.equal [:html, :tag, 'meta',
+                     [:html, :attrs,
+                      [:html, :attr, 'http-equiv', [:static, 'Content-Type']],
+                      [:html, :attr, 'content', [:static, '']]]]
+  end
+end

data/test/test_engine.rb

@@ -118,6 +118,7 @@ describe Temple::Engine do
     engine.after :Parser, :MyFilter0 do |exp|
       exp
     end
+    TestEngine.chain.size.should.equal 8
     engine.chain.size.should.equal 9
     engine.chain[0].first.should.equal :Parser
     engine.chain[1].first.should.equal :MyFilter0
@@ -129,6 +130,7 @@ describe Temple::Engine do
     engine.before :MyFilter1, :MyFilter0 do |exp|
       exp
     end
+    TestEngine.chain.size.should.equal 8
     engine.chain.size.should.equal 9
     engine.chain[0].first.should.equal :Parser
     engine.chain[1].first.should.equal :MyFilter0
@@ -138,6 +140,7 @@ describe Temple::Engine do
   it 'should have #remove' do
     engine = TestEngine.new
     engine.remove :MyFilter1
+    TestEngine.chain.size.should.equal 8
     engine.chain.size.should.equal 7
     engine.chain[0].first.should.equal :Parser
     engine.chain[1].first.should.equal :MyFilter2
@@ -145,10 +148,10 @@ describe Temple::Engine do
 
   it 'should have #replace' do
     engine = TestEngine.new
-    engine.before :Parser, :MyParser do |exp|
+    engine.replace :Parser, :MyParser do |exp|
       exp
     end
-    engine.chain.size.should.equal 9
+    engine.chain.size.should.equal 8
     engine.chain[0].first.should.equal :MyParser
   end
 

metadata

@@ -1,63 +1,58 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: temple
-version: !ruby/object:Gem::Version 
+version: !ruby/object:Gem::Version
+  version: 0.4.0
   prerelease: 
-  version: 0.3.5
 platform: ruby
-authors: 
+authors:
 - Magnus Holm
 - Daniel Mendler
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2012-01-06 00:00:00 +01:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2012-02-26 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: tilt
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
+  requirement: &18003800 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: bacon
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  version_requirements: *18003800
+- !ruby/object:Gem::Dependency
+  name: bacon
+  requirement: &18002180 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
-  name: rake
   prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  version_requirements: *18002180
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: &18000780 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id003
+  prerelease: false
+  version_requirements: *18000780
 description: 
-email: 
+email:
 - judofyr@gmail.com
 - mail@daniel-mendler.de
 executables: []
-
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
+files:
 - .gitignore
 - .travis.yml
 - .yardopts
@@ -85,6 +80,8 @@ files:
 - lib/temple/grammar.rb
 - lib/temple/hash.rb
 - lib/temple/html/attribute_merger.rb
+- lib/temple/html/attribute_remover.rb
+- lib/temple/html/attribute_sorter.rb
 - lib/temple/html/dispatcher.rb
 - lib/temple/html/fast.rb
 - lib/temple/html/filter.rb
@@ -108,6 +105,8 @@ files:
 - test/filters/test_static_merger.rb
 - test/helper.rb
 - test/html/test_attribute_merger.rb
+- test/html/test_attribute_remover.rb
+- test/html/test_attribute_sorter.rb
 - test/html/test_fast.rb
 - test/html/test_pretty.rb
 - test/mixins/test_dispatcher.rb
@@ -119,35 +118,31 @@ files:
 - test/test_grammar.rb
 - test/test_hash.rb
 - test/test_utils.rb
-has_rdoc: true
 homepage: http://dojo.rubyforge.org/
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.6.2
+rubygems_version: 1.8.15
 signing_key: 
 specification_version: 3
 summary: Template compilation framework in Ruby
-test_files: 
+test_files:
 - test/filters/test_control_flow.rb
 - test/filters/test_dynamic_inliner.rb
 - test/filters/test_eraser.rb
@@ -156,6 +151,8 @@ test_files:
 - test/filters/test_static_merger.rb
 - test/helper.rb
 - test/html/test_attribute_merger.rb
+- test/html/test_attribute_remover.rb
+- test/html/test_attribute_sorter.rb
 - test/html/test_fast.rb
 - test/html/test_pretty.rb
 - test/mixins/test_dispatcher.rb