deface 0.7.2 → 0.8.0

data/spec/deface/override_spec.rb

@@ -37,8 +37,9 @@ module Deface
         @original = Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace => "h1", :text => "<h1>Argh!</h1>")
       end
 
-      it "should not validate" do
-        @override.validate_original("<p>this gets ignored</p>").should be_true
+      it "should warn but not validate" do
+        Rails.logger.should_receive(:info).once
+        @override.validate_original("<p>this gets ignored</p>").should be_nil
       end
 
     end
@@ -61,6 +62,9 @@ module Deface
     describe "#find" do
       it "should find by virtual_path" do
         Deface::Override.find({:virtual_path => "posts/index"}).size.should == 1
+        Deface::Override.find({:virtual_path => "/posts/index"}).size.should == 1
+        Deface::Override.find({:virtual_path => "/posts/index.html"}).size.should == 1
+        Deface::Override.find({:virtual_path => "posts/index.html"}).size.should == 1
       end
 
       it "should return empty array when no details hash passed" do
@@ -88,7 +92,31 @@ module Deface
       end
     end
 
-    describe "with :partial" do
+    describe "with :erb" do
+
+      before(:each) do
+        @override = Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace => "h1", :text => "<h1 id=\"<%= dom_id @pirate %>\">Argh!</h1>")
+      end
+
+      it "should return un-convert text as source" do
+        @override.source.should == "<h1 id=\"<%= dom_id @pirate %>\">Argh!</h1>"
+      end
+    end
+
+    describe "with :haml" do
+
+      before(:each) do
+        @override = Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace => "h1",
+                                         :haml => %q{%strong{:class => "code", :id => "message"}= 'Hello, World!'})
+      end
+
+      it "should return erb converted from haml as source" do
+        @override.source.should == "<strong class='code' id='message'><%= 'Hello, World!' %>\n</strong>\n"
+      end
+    end
+
+
+    describe "with :partial containing erb" do
 
       before(:each) do
         #stub view paths to be local spec/assets directory
@@ -139,7 +167,7 @@ module Deface
       end
 
       it "should return escaped source" do
-        @override.source_element.should be_an_instance_of Nokogiri::HTML::DocumentFragment 
+        @override.source_element.should be_an_instance_of Nokogiri::HTML::DocumentFragment
         @override.source_element.to_s.should == "<code erb-loud> method :opt =&gt; 'x' &amp; 'y' </code>"
         #do it twice to ensure it doesn't change as it's destructive
         @override.source_element.to_s.should == "<code erb-loud> method :opt =&gt; 'x' &amp; 'y' </code>"
@@ -229,7 +257,6 @@ module Deface
 
     end
 
-
     describe "#end_selector" do
       it "should return nil when closing_selector is not defined" do
         @override.end_selector.should be_nil
@@ -251,6 +278,100 @@ module Deface
       end
     end
 
+    describe "#digest" do
+      before { @digest = @override.digest }
+
+      it "should return hex digest based on override's args" do
+        @override.digest.should =~ /[a-f0-9]{32}/
+      end
+
+      it "should change the digest when any args change" do
+        @override = Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace => "h2", :text => "<h1>Argh!</h1>")
+        @override.digest.should_not == @digest
+
+        @override = Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace => "h1", :text => "<h1>Argh!</h1>")
+        @override.digest.should == @digest
+
+        @override = Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace => "h2", :text => "<h1>I'm a pirate!</h1>")
+        @override.digest.should_not == @digest
+      end
+    end
+
+    describe "self#digest" do
+      before do
+        @second = Deface::Override.new(:virtual_path => "posts/index", :name => "second", :insert_after => "p", :text => "<pre>this is code?</pre>")
+
+        @digest = Deface::Override.digest(:virtual_path =>  "posts/index")
+      end
+
+      it "should return hex digest based on all applicable overrides" do
+        Deface::Override.digest(:virtual_path =>  "posts/index").should =~ /[a-f0-9]{32}/
+      end
+
+      it "should change the digest when any args change for any override" do
+        @override = Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace => "h2", :text => "<h1>Argh!</h1>")
+        Deface::Override.digest(:virtual_path =>  "posts/index").should_not == @digest
+
+        @override = Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace => "h1", :text => "<h1>Argh!</h1>")
+        Deface::Override.digest(:virtual_path =>  "posts/index").should == @digest
+
+        @second = Deface::Override.new(:virtual_path => "posts/index", :name => "2nd", :insert_after => "p", :text => "<pre>this is code?</pre>")
+        Deface::Override.digest(:virtual_path =>  "posts/index").should_not == @digest
+      end
+
+      it "should change the digest when overrides are removed / added" do
+        Deface::Override.all.clear
+
+        @new_digest = Deface::Override.digest(:virtual_path =>  "posts/index")
+        @new_digest.should_not == @digest
+
+        @override = Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace => "h1", :text => "<h1>Argh!</h1>")
+        Deface::Override.digest(:virtual_path =>  "posts/index").should_not == @new_digest
+      end
+    end
+
+    describe "#expire_compiled_template" do
+      before do
+        @compiled_templates = ActionView::CompiledTemplates
+
+        ActionView::CompiledTemplates.instance_methods.each do |method_name|
+          ActionView::CompiledTemplates.send :remove_method, method_name
+        end
+      end
+
+      it "should remove compiled method when method name matches virtual path but not digest" do
+        module ActionView::CompiledTemplates
+          def _e235fa404c3c2281d4f6791162b1c638_posts_index_123123123
+            true #not a real method
+          end
+
+          def _f34556de606cec51d4f6791163fab456_posts_edit_123123123
+            true #not a real method
+          end
+
+        end
+
+        ActionView::CompiledTemplates.instance_methods.size.should == 2
+        @override.send(:expire_compiled_template)
+        ActionView::CompiledTemplates.instance_methods.size.should == 1
+      end
+
+      it "should not remove compiled method when virtual path and digest matach" do
+
+        module ActionView::CompiledTemplates
+          def _e235fa404c3c2281d4f6791162b1c638_posts_index_123123123
+            true #not a real method
+          end
+        end
+
+        Deface::Override.should_receive(:digest).and_return('e235fa404c3c2281d4f6791162b1c638')
+
+        ActionView::CompiledTemplates.instance_methods.size.should == 1
+        @override.send(:expire_compiled_template)
+        ActionView::CompiledTemplates.instance_methods.size.should == 1
+      end
+    end
+
   end
 
 end

data/spec/deface/template_helper_spec.rb

@@ -16,10 +16,18 @@ module Deface
           load_template_source("shared/post", true).should == "<p>I'm from shared/post partial</p>\n<%= \"And I've got ERB\" %>\n"
         end
 
+        it "should return converted source for partial containing haml" do
+          load_template_source("shared/hello", true).should =="<div class='<%= @some %>' id='message'><%= 'Hello, World!' %>\n</div>\n"
+        end
+
         it "should return source for template" do
           load_template_source("shared/person", false).should == "<p>I'm from shared/person template</p>\n<%= \"I've got ERB too\" %>\n"
         end
 
+        it "should return converted source for template containing haml" do
+          load_template_source("shared/pirate", false).gsub(/\s/, '').should == "<divid='content'><divclass='left'><p><%=print_information%></p></div><divclass='right'id='<%=@right%>'><%=render:partial=>\"sidebar\"%></div></div>"
+        end
+
         it "should return source for namespaced template" do
           load_template_source("admin/posts/index", false).should == "<h1>Manage Posts</h1>\n"
         end
@@ -36,21 +44,31 @@ module Deface
         before(:each) do
           Deface::Override.new(:virtual_path => "shared/_post", :name => "shared#post", :remove => "p")
           Deface::Override.new(:virtual_path => "shared/person", :name => "shared#person", :replace => "p", :text => "<h1>Argh!</h1>")
+          Deface::Override.new(:virtual_path => "shared/_hello", :name => "shared#hello", :replace_contents => "div#message", :text => "<%= 'Goodbye World!' %>")
+          Deface::Override.new(:virtual_path => "shared/pirate", :name => "shared#pirate", :replace => "p", :text => "<h1>Argh!</h1>")
           Deface::Override.new(:virtual_path => "admin/posts/index", :name => "admin#posts#index", :replace => "h1", :text => "<h1>Argh!</h1>")
         end
 
-        it "should return source for partial including overrides" do
+        it "should return overridden source for partial including overrides" do
           load_template_source("shared/post", true).should == "\n<%= \"And I've got ERB\" %>"
         end
 
-        it "should return source for partial excluding overrides" do
+        it "should return converted and overridden source for partial containing haml" do
+          load_template_source("shared/hello", true).should =="<div class=\"<%= @some %>\" id=\"message\"><%= 'Goodbye World!' %></div>"
+        end
+
+        it "should return overridden source for partial excluding overrides" do
           load_template_source("shared/post", true, false).should == "<p>I'm from shared/post partial</p>\n<%= \"And I've got ERB\" %>\n"
         end
 
-        it "should return source for template including overrides" do
+        it "should return overridden source for template including overrides" do
           load_template_source("shared/person", false).should == "<h1>Argh!</h1>\n<%= \"I've got ERB too\" %>"
         end
 
+        it "should return converted and overridden source for template containing haml" do
+          load_template_source("shared/pirate", false).gsub(/\s/, '').should == "<divid=\"content\"><divclass=\"left\"><h1>Argh!</h1></div><divclass=\"right\"id=\"<%=@right%>\"><%=render:partial=>\"sidebar\"%></div></div>"
+        end
+
         it "should return source for namespaced template including overrides" do
           load_template_source("admin/posts/index", false).should == "<h1>Argh!</h1>"
         end

data/spec/spec_helper.rb

@@ -3,21 +3,37 @@ require 'rspec'
 require 'action_view'
 require 'action_controller'
 require 'deface'
+#have to manually require following three for testing purposes
+require 'deface/action_view_extensions'
+require 'haml'
+require 'deface/haml_converter'
+
+Haml.init_rails(nil)
 
 RSpec.configure do |config|
   config.mock_framework = :rspec
 end
 
+module ActionView::CompiledTemplates
+  #empty module for testing purposes
+end
+
 shared_context "mock Rails" do
   before(:each) do
     unless defined? Rails
       Rails = mock 'Rails'
     end
+
     Rails.stub :application => mock('application')
     Rails.application.stub :config => mock('config')
     Rails.application.config.stub :cache_classes => true
     Rails.application.config.stub :deface => ActiveSupport::OrderedOptions.new
     Rails.application.config.deface.enabled = true
+
+    Rails.stub :logger => mock('logger')
+    Rails.logger.stub(:error)
+    Rails.logger.stub(:warning)
+    Rails.logger.stub(:info)
   end
 end
 
@@ -26,5 +42,6 @@ shared_context "mock Rails.application" do
 
   before(:each) do
     Rails.application.config.stub :deface => Deface::Environment.new
+    Rails.application.config.deface.haml_support = true
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: deface
 version: !ruby/object:Gem::Version 
-  hash: 7
+  hash: 63
   prerelease: 
   segments: 
   - 0
-  - 7
-  - 2
-  version: 0.7.2
+  - 8
+  - 0
+  version: 0.8.0
 platform: ruby
 authors: 
 - Brian D Quinn
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-11-28 00:00:00 Z
+date: 2012-03-08 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: nokogiri
@@ -57,16 +57,32 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 19
+        hash: 47
         segments: 
         - 2
-        - 7
+        - 8
         - 0
-        version: 2.7.0
+        version: 2.8.0
   type: :development
   version_requirements: *id003
-description: Deface is a library that allows you to customize ERB views in a Rails application without editing the underlying view.
-email: brian@railsdog.com
+- !ruby/object:Gem::Dependency 
+  name: haml
+  prerelease: false
+  requirement: &id004 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 11
+        segments: 
+        - 3
+        - 1
+        - 4
+        version: 3.1.4
+  type: :development
+  version_requirements: *id004
+description: Deface is a library that allows you to customize ERB & HAML views in a Rails application without editing the underlying view.
+email: brian@spreecommerce.com
 executables: []
 
 extensions: []
@@ -84,19 +100,27 @@ files:
 - init.rb
 - lib/deface.rb
 - lib/deface/action_view_extensions.rb
+- lib/deface/applicator.rb
 - lib/deface/environment.rb
+- lib/deface/haml_converter.rb
+- lib/deface/original_validator.rb
 - lib/deface/override.rb
 - lib/deface/parser.rb
 - lib/deface/railtie.rb
+- lib/deface/search.rb
 - lib/deface/template_helper.rb
 - spec/assets/admin/posts/index.html.erb
+- spec/assets/shared/_hello.html.haml
 - spec/assets/shared/_post.html.erb
 - spec/assets/shared/person.html.erb
+- spec/assets/shared/pirate.html.haml
+- spec/deface/action_view_template_spec.rb
+- spec/deface/applicator_spec.rb
 - spec/deface/environment_spec.rb
+- spec/deface/haml_converter_spec.rb
 - spec/deface/override_spec.rb
 - spec/deface/parser_spec.rb
 - spec/deface/template_helper_spec.rb
-- spec/deface/template_spec.rb
 - spec/spec_helper.rb
 - tasks/precompile.rake
 - tasks/utils.rake
@@ -129,17 +153,21 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.8.10
+rubygems_version: 1.8.15
 signing_key: 
 specification_version: 3
-summary: Deface is a library that allows you to customize ERB views in Rails
+summary: Deface is a library that allows you to customize ERB & HAML views in Rails
 test_files: 
 - spec/assets/admin/posts/index.html.erb
+- spec/assets/shared/_hello.html.haml
 - spec/assets/shared/_post.html.erb
 - spec/assets/shared/person.html.erb
+- spec/assets/shared/pirate.html.haml
+- spec/deface/action_view_template_spec.rb
+- spec/deface/applicator_spec.rb
 - spec/deface/environment_spec.rb
+- spec/deface/haml_converter_spec.rb
 - spec/deface/override_spec.rb
 - spec/deface/parser_spec.rb
 - spec/deface/template_helper_spec.rb
-- spec/deface/template_spec.rb
 - spec/spec_helper.rb

data/spec/deface/template_spec.rb

@@ -1,286 +0,0 @@
-require 'spec_helper'
-
-module ActionView
-  describe Template do
-    include_context "mock Rails.application"
-
-    describe "with no overrides defined" do
-      before(:each) do
-        @updated_at = Time.now - 600
-        @template = ActionView::Template.new("<p>test</p>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html, :updated_at => @updated_at})
-        #stub for Rails < 3.1
-        unless defined?(@template.updated_at)
-          @template.stub(:updated_at).and_return(@updated_at)
-        end
-      end
-
-      it "should initialize new template object" do
-        @template.is_a?(ActionView::Template).should == true
-      end
-
-      it "should return unmodified source" do
-        @template.source.should == "<p>test</p>"
-      end
-
-      it "should not change updated_at" do
-        @template.updated_at.should == @updated_at
-      end
-
-    end
-
-    describe "with a single remove override defined" do
-      before(:each) do
-        @updated_at = Time.now - 300
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :remove => "p", :text => "<h1>Argh!</h1>")
-        @template = ActionView::Template.new("<p>test</p><%= raw(text) %>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html, :updated_at => @updated_at})
-        #stub for Rails < 3.1
-        unless defined?(@template.updated_at)
-          @template.stub(:updated_at).and_return(@updated_at + 500)
-        end
-      end
-
-      it "should return modified source" do
-        @template.source.should == "<%= raw(text) %>"
-      end
-
-      it "should change updated_at" do
-        @template.updated_at.should > @updated_at
-      end
-    end
-
-    describe "with a single remove override with closing_selector defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :remove => "h1", :closing_selector => "h2")
-        @template = ActionView::Template.new("<h2>I should be safe</h2><span>Before!</span><h1>start</h1><p>some junk</p><div>more junk</div><h2>end</h2><span>After!</span>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.should == "<h2>I should be safe</h2><span>Before!</span><span>After!</span>"
-      end
-    end
-
-    describe "with a single replace override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace => "p", :text => "<h1>Argh!</h1>")
-        @template = ActionView::Template.new("<p>test</p>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.should == "<h1>Argh!</h1>"
-      end
-    end
-
-    describe "with a single replace override with closing_selector defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace => "h1", :closing_selector => "h2", :text => "<span>Argh!</span>")
-        @template = ActionView::Template.new("<h1>start</h1><p>some junk</p><div>more junk</div><h2>end</h2>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.should == "<span>Argh!</span>"
-      end
-    end
-
-    describe "with a single replace_contents override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace_contents => "p", :text => "<h1>Argh!</h1>")
-        @template = ActionView::Template.new("<p><span>Hello</span>I am not a <em>pirate</em></p>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.should == "<p><h1>Argh!</h1></p>"
-      end
-    end
-
-    describe "with a single replace_contents override with closing_selector defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :replace_contents => "h1", :closing_selector => "h2", :text => "<span>Argh!</span>")
-        @template = ActionView::Template.new("<h1>start</h1><p>some junk</p><div>more junk</div><h2>end</h2>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.should == "<h1>start</h1><span>Argh!</span><h2>end</h2>"
-      end
-    end
-
-    describe "with a single insert_after override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :insert_after => "img.button", :text => "<% help %>")
-
-        @template = ActionView::Template.new("<div><img class=\"button\" src=\"path/to/button.png\"></div>",
-                                             "/path/to/file.erb",
-                                             ActionView::Template::Handlers::ERB,
-                                             {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.gsub("\n", "").should == "<div><img class=\"button\" src=\"path/to/button.png\"><% help %></div>"
-      end
-    end
-
-    describe "with a single insert_before override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :insert_after => "ul li:last", :text => "<%= help %>")
-
-        @template = ActionView::Template.new("<ul><li>first</li><li>second</li><li>third</li></ul>",
-                                             "/path/to/file.erb",
-                                             ActionView::Template::Handlers::ERB,
-                                             {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.gsub("\n", "").should == "<ul><li>first</li><li>second</li><li>third</li><%= help %></ul>"
-      end
-    end
-
-    describe "with a single insert_top override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :insert_top => "ul", :text => "<li>me first</li>")
-
-        @template = ActionView::Template.new("<ul><li>first</li><li>second</li><li>third</li></ul>",
-                                             "/path/to/file.erb",
-                                             ActionView::Template::Handlers::ERB,
-                                             {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.gsub("\n", "").should == "<ul><li>me first</li><li>first</li><li>second</li><li>third</li></ul>"
-      end
-    end
-
-    describe "with a single insert_top override defined when targetted elemenet has no children" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :insert_top => "ul", :text => "<li>first</li><li>second</li><li>third</li>")
-
-        @template = ActionView::Template.new("<ul></ul>",
-                                             "/path/to/file.erb",
-                                             ActionView::Template::Handlers::ERB,
-                                             {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.gsub("\n", "").should == "<ul><li>first</li><li>second</li><li>third</li></ul>"
-      end
-    end
-
-    describe "with a single insert_bottom override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :insert_bottom => "ul", :text => "<li>I'm always last</li>")
-
-        @template = ActionView::Template.new("<ul><li>first</li><li>second</li><li>third</li></ul>",
-                                             "/path/to/file.erb",
-                                             ActionView::Template::Handlers::ERB,
-                                             {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.gsub("\n", "").should == "<ul><li>first</li><li>second</li><li>third</li><li>I'm always last</li></ul>"
-      end
-    end
-
-    describe "with a single insert_bottom override defined when targetted elemenet has no children" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :insert_bottom => "ul", :text => "<li>I'm always last</li>")
-
-        @template = ActionView::Template.new("<ul></ul>",
-                                             "/path/to/file.erb",
-                                             ActionView::Template::Handlers::ERB,
-                                             {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.gsub("\n", "").should == "<ul><li>I'm always last</li></ul>"
-      end
-    end
-
-    describe "with a single set_attributes override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :set_attributes => 'img', 
-                                      :attributes => {:class => 'pretty', :alt => 'something interesting'})
-
-        @template = ActionView::Template.new("<div><img class=\"button\" src=\"path/to/button.png\"></div>",
-                                             "/path/to/file.erb",
-                                             ActionView::Template::Handlers::ERB,
-                                             {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.gsub("\n", "").should == "<div><img class=\"pretty\" src=\"path/to/button.png\" alt=\"something interesting\"></div>"
-      end
-    end
-
-
-    describe "with a single html surround override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :surround => "p", :text => "<h1>It's behind you!</h1><div><%= render_original %></div>")
-        @template = ActionView::Template.new("<p>test</p>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.should == "<h1>It's behind you!</h1><div><p>test</p></div>"
-      end
-    end
-
-    describe "with a single erb surround override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :surround => "p", :text => "<% some_method('test') do %><%= render_original %><% end %>")
-        @template = ActionView::Template.new("<span><p>test</p></span>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.gsub("\n", '').should == "<span><% some_method('test') do %><p>test</p><% end %></span>"
-      end
-    end
-
-    describe "with a single html surround_contents override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :surround_contents => "div", :text => "<span><%= render_original %></span>")
-        @template = ActionView::Template.new("<h4>yay!</h4><div><p>test</p></div>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.should == "<h4>yay!</h4><div><span><p>test</p></span></div>"
-      end
-    end
-
-    describe "with a single erb surround_contents override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :surround_contents => "p", :text => "<% if 1==1 %><%= render_original %><% end %>")
-        @template = ActionView::Template.new("<p>test</p>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.should == "<p><% if 1==1 %>test<% end %></p>"
-      end
-    end
-
-    describe "with a single disabled override defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "Posts#index", :remove => "p", :text => "<h1>Argh!</h1>", :disabled => true)
-        @template = ActionView::Template.new("<p>test</p><%= raw(text) %>", "/some/path/to/file.erb", ActionView::Template::Handlers::ERB, {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return unmodified source" do
-        @template.source.should == "<p>test</p><%= raw(text) %>"
-      end
-    end
-
-    describe "with mulitple sequenced overrides defined" do
-      before(:each) do
-        Deface::Override.new(:virtual_path => "posts/index", :name => "third", :insert_after => "li:contains('second')", :text => "<li>third</li>", :sequence => {:after => "second"})
-        Deface::Override.new(:virtual_path => "posts/index", :name => "second", :insert_after => "li", :text => "<li>second</li>", :sequence => {:after => "first"})
-        Deface::Override.new(:virtual_path => "posts/index", :name => "first", :replace => "li", :text => "<li>first</li>")
-
-        @template = ActionView::Template.new("<ul><li>replaced</li></ul>",
-                                             "/path/to/file.erb",
-                                             ActionView::Template::Handlers::ERB,
-                                             {:virtual_path=>"posts/index", :format=>:html})
-      end
-
-      it "should return modified source" do
-        @template.source.gsub("\n", "").should == "<ul><li>first</li><li>second</li><li>third</li></ul>"
-      end
-    end
-
-  end
-end