regru-premailer 1.7.4

data/test/test_links.rb

@@ -0,0 +1,185 @@
+# encoding: UTF-8
+require File.expand_path(File.dirname(__FILE__)) + '/helper'
+
+class TestLinks < Premailer::TestCase
+  def test_empty_query_string
+    assert_nothing_raised do
+      premailer = Premailer.new('<p>Test</p>', :with_html_string => true, :link_query_string => ' ')
+      premailer.to_inline_css
+    end
+  end
+
+  def test_appending_link_query_string
+    qs = 'utm_source=1234&tracking=good&amp;doublescape'
+    opts = {:base_url => 'http://example.com/',  :link_query_string => qs, :with_html_string => true, :adapter => :hpricot}
+    
+    appendable = [
+      '/', 
+      opts[:base_url], 
+      'https://example.com/tester',
+      'images/',
+      "#{opts[:base_url]}test.html?cn=tf&amp;c=20&amp;ord=random",
+      '?query=string'
+    ]
+      
+    not_appendable = [
+      '%DONOTCONVERT%',
+      '{DONOTCONVERT}',
+      '[DONOTCONVERT]',
+      '<DONOTCONVERT>',
+      '{@msg-txturl}',
+      '[[!unsubscribe]]',
+      '#relative', 
+      'tel:5555551212',
+      'http://example.net/',
+      'mailto:premailer@example.com',
+      'ftp://example.com',
+      'gopher://gopher.floodgap.com/1/fun/twitpher'
+    ]
+    
+    html = appendable.collect {|url| "<a href='#{url}'>Link</a>" }
+
+    premailer = Premailer.new(html.to_s, opts)
+    premailer.to_inline_css
+    
+    premailer.processed_doc.search('a').each do |el|
+      href = el.attributes['href'].to_s
+      next if href.nil? or href.empty?
+      uri = URI.parse(href)
+      assert_match qs, uri.query, "missing query string for #{el.to_s}"
+    end
+
+    html = not_appendable.collect {|url| "<a href='#{url}'>Link</a>" }
+
+    premailer = Premailer.new(html.to_s, opts)
+    premailer.to_inline_css
+    
+    premailer.processed_doc.search('a').each do |el|
+      href = el['href']
+      next if href.nil? or href.empty?
+      assert not_appendable.include?(href), "link #{href} should not be converted: see #{not_appendable.to_s}"
+    end
+  end
+
+  def test_stripping_extra_question_marks_from_query_string
+    qs = '??utm_source=1234'
+
+    premailer = Premailer.new("<a href='/test/?'>Link</a> <a href='/test/'>Link</a>", :link_query_string => qs, :with_html_string => true)
+    premailer.to_inline_css
+    
+    premailer.processed_doc.search('a').each do |a|
+      assert_equal '/test/?utm_source=1234', a['href'].to_s
+    end
+
+    premailer = Premailer.new("<a href='/test/?123&456'>Link</a>", :link_query_string => qs, :with_html_string => true)
+    premailer.to_inline_css
+    
+    assert_equal '/test/?123&456&amp;utm_source=1234', premailer.processed_doc.at('a')['href']
+  end
+
+
+  def test_preserving_links
+    html = "<a href='http://example.com/index.php?pram1=one&pram2=two'>Link</a>"
+    premailer = Premailer.new(html.to_s, :link_query_string => '', :with_html_string => true)
+    premailer.to_inline_css
+    
+    assert_equal 'http://example.com/index.php?pram1=one&pram2=two', premailer.processed_doc.at('a')['href']
+
+    html = "<a href='http://example.com/index.php?pram1=one&pram2=two'>Link</a>"
+    premailer = Premailer.new(html.to_s, :link_query_string => 'qs', :with_html_string => true)
+    premailer.to_inline_css
+    
+    assert_equal 'http://example.com/index.php?pram1=one&pram2=two&amp;qs', premailer.processed_doc.at('a')['href']
+
+  end
+
+  def test_resolving_urls_from_string
+    ['test.html', '/test.html', './test.html', 
+     'test/../test.html', 'test/../test/../test.html'].each do |q|
+      assert_equal 'http://example.com/test.html', Premailer.resolve_link(q, 'http://example.com/'), q
+    end
+
+    assert_equal 'https://example.net:80/~basedir/test.html?var=1#anchor', Premailer.resolve_link('test/../test/../test.html?var=1#anchor', 'https://example.net:80/~basedir/')
+  end
+
+  def test_resolving_urls_from_uri
+    base_uri = URI.parse('http://example.com/')
+    ['test.html', '/test.html', './test.html', 
+     'test/../test.html', 'test/../test/../test.html'].each do |q|
+      assert_equal 'http://example.com/test.html', Premailer.resolve_link(q, base_uri), q
+    end
+
+    base_uri = URI.parse('https://example.net:80/~basedir/')
+    assert_equal 'https://example.net:80/~basedir/test.html?var=1#anchor', Premailer.resolve_link('test/../test/../test.html?var=1#anchor', base_uri)
+    
+    # base URI with a query string
+    base_uri = URI.parse('http://example.com/dir/index.cfm?newsletterID=16')
+    assert_equal 'http://example.com/dir/index.cfm?link=15', Premailer.resolve_link('?link=15', base_uri)
+    
+    # URI preceded by a space
+    base_uri = URI.parse('http://example.com/')
+    assert_equal 'http://example.com/path', Premailer.resolve_link(' path', base_uri)
+  end
+
+  def test_resolving_urls_in_doc
+    # force Nokogiri since this consistenly segfaults with Hpricot
+    base_file = File.dirname(__FILE__) + '/files/base.html'
+    base_url = 'https://my.example.com:8080/test-path.html'
+    premailer = Premailer.new(base_file, :base_url => base_url, :adapter => :nokogiri)
+    premailer.to_inline_css
+    pdoc = premailer.processed_doc
+    doc = premailer.doc
+
+    # unchanged links
+    ['#l02', '#l03', '#l05', '#l06', '#l07', '#l08', 
+     '#l09', '#l10', '#l11', '#l12', '#l13'].each do |link_id|
+      assert_equal doc.at(link_id).attributes['href'], pdoc.at(link_id).attributes['href'], link_id
+    end
+    
+    assert_equal 'https://my.example.com:8080/', pdoc.at('#l01').attributes['href'].to_s
+    assert_equal 'https://my.example.com:8080/images/', pdoc.at('#l04').attributes['href'].to_s
+  end
+
+  def test_convertable_inline_links
+    convertable = [
+      'my/path/to',
+      'other/path',
+      '/'
+    ]
+
+    html = convertable.collect {|url| "<a href='#{url}'>Link</a>" }
+    premailer = Premailer.new(html.to_s, :base_url => "http://example.com", :with_html_string => true)
+    
+    premailer.processed_doc.search('a').each do |el|
+      href = el.attributes['href'].to_s
+      assert(href =~ /http:\/\/example.com/, "link #{href} is not absolute")
+    end
+  end
+
+  def test_non_convertable_inline_links
+    not_convertable = [
+      '%DONOTCONVERT%',
+      '{DONOTCONVERT}',
+      '[DONOTCONVERT]',
+      '<DONOTCONVERT>',
+      '{@msg-txturl}',
+      '[[!unsubscribe]]',
+      '#relative', 
+      'tel:5555551212',
+      'mailto:premailer@example.com',
+      'ftp://example.com',
+      'gopher://gopher.floodgap.com/1/fun/twitpher',
+      'cid:13443452066.10392logo.jpeg@inline_attachment'
+    ]
+    
+    html = not_convertable.collect {|url| "<a href='#{url}'>Link</a>" }
+
+    premailer = Premailer.new(html.to_s, :base_url => "example.com", :with_html_string => true)
+    premailer.to_inline_css
+    
+    premailer.processed_doc.search('a').each do |el|
+      href = el.attributes['href'].to_s
+      assert not_convertable.include?(href), "link #{href} should not be converted: see #{not_convertable.inspect}"
+    end
+  end
+end

data/test/test_misc.rb

@@ -0,0 +1,278 @@
+# encoding: UTF-8
+require File.expand_path(File.dirname(__FILE__)) + '/helper'
+
+# Random tests for specific issues.
+#
+# The test suite will be cleaned up at some point soon.
+class TestMisc < Premailer::TestCase
+
+  # in response to http://github.com/alexdunae/premailer/issues#issue/4
+  #
+  # NB: 2010-11-16 -- after reverting to Hpricot this test can no longer pass.
+  # It's too much of an edge case to get any dev time.
+  def test_parsing_extra_quotes
+    io = StringIO.new('<p></p>
+    <h3 "id="WAR"><a name="WAR"></a>Writes and Resources</h3>
+    <table></table>')
+    premailer = Premailer.new(io, :adapter => :nokogiri)
+    assert_match /<h3>[\s]*<a name="WAR">[\s]*<\/a>[\s]*Writes and Resources[\s]*<\/h3>/i, premailer.to_inline_css
+  end
+
+  def test_styles_in_the_body
+    html = <<END_HTML
+    <html> 
+    <body> 
+    <style type="text/css"> p { color: red; } </style>
+		<p>Test</p> 
+		</body>
+		</html>
+END_HTML
+
+		premailer = Premailer.new(html, :with_html_string => true)
+		premailer.to_inline_css
+
+	  assert_match /color\: red/i,  premailer.processed_doc.at('p')['style']
+  end
+  
+  def test_commented_out_styles_in_the_body
+    html = <<END_HTML
+    <html> 
+    <body> 
+    <style type="text/css"> <!-- p { color: red; } --> </style>
+		<p>Test</p> 
+		</body>
+		</html>
+END_HTML
+
+		premailer = Premailer.new(html, :with_html_string => true)
+		premailer.to_inline_css
+
+	  assert_match /color\: red/i,  premailer.processed_doc.at('p')['style']
+  end
+
+  def test_not_applying_styles_to_the_head
+    html = <<END_HTML
+    <html> 
+    <head>
+    <title>Title</title>
+    <style type="text/css"> * { color: red; } </style>
+    </head>
+    <body> 
+		<p><a>Test</a></p> 
+		</body>
+		</html>
+END_HTML
+
+    [:nokogiri, :hpricot].each do |adapter|
+  		premailer = Premailer.new(html, :with_html_string => true, :adapter => adapter)
+  		premailer.to_inline_css
+
+  	  h = premailer.processed_doc.at('head')
+  	  assert_nil h['style']
+
+  	  t = premailer.processed_doc.at('title')
+  	  assert_nil t['style']
+    end
+  end
+
+  def test_multiple_identical_ids
+    html = <<-END_HTML
+    <html> 
+    <head>
+    <style type="text/css"> #the_id { color: red; } </style>
+    </head>
+    <body> 
+		<p id="the_id">Test</p> 
+		<p id="the_id">Test</p> 
+		</body>
+		</html>
+    END_HTML
+
+		premailer = Premailer.new(html, :with_html_string => true)
+		premailer.to_inline_css
+    premailer.processed_doc.search('p').each do |el|
+      assert_match /red/i, el['style']
+    end
+  end
+
+  def test_preserving_styles
+    html = <<END_HTML
+    <html> 
+    <head>
+    <link rel="stylesheet" href="#"/>
+    <style type="text/css"> a:hover { color: red; } </style>
+    </head>
+    <body> 
+		<p><a>Test</a></p> 
+		</body>
+		</html>
+END_HTML
+    [:nokogiri, :hpricot].each do |adapter|
+  		premailer = Premailer.new(html, :with_html_string => true, :preserve_styles => true,  :adapter => adapter)
+  		premailer.to_inline_css
+  	  assert_equal 1, premailer.processed_doc.search('head link').length
+  	  assert_equal 1, premailer.processed_doc.search('head style').length
+
+  		premailer = Premailer.new(html, :with_html_string => true, :preserve_styles => false, :adapter => adapter)
+  		premailer.to_inline_css
+  	  assert_nil premailer.processed_doc.at('head link')
+
+      # should be preserved as unmergeable
+  	  assert_match /red !important/i, premailer.processed_doc.at('body style').inner_html
+  	end
+  end
+
+  def test_unmergable_rules
+    html = <<END_HTML
+    <html> <head> <style type="text/css"> a { color:blue; } a:hover { color: red; } </style> </head>
+		<p><a>Test</a></p> 
+		</body> </html>
+END_HTML
+
+		premailer = Premailer.new(html, :with_html_string => true, :verbose => true)
+		premailer.to_inline_css
+	  assert_match /a\:hover[\s]*\{[\s]*color\:[\s]*red[\s]*!important;[\s]*\}/i, premailer.processed_doc.at('body style').inner_html
+  end
+
+  def test_unmergable_rules_with_no_body
+    html = <<END_HTML
+    <html> 
+    <style type="text/css"> a:hover { color: red; } </style>
+		<p><a>Test</a></p> 
+		</html>
+END_HTML
+
+		premailer = Premailer.new(html, :with_html_string => true)
+    assert_nothing_raised do
+		  premailer.to_inline_css
+	  end
+	  assert_match /red !important/i, premailer.processed_doc.at('style').inner_html
+  end
+
+  # in response to https://github.com/alexdunae/premailer/issues#issue/7
+  def test_ignoring_link_pseudo_selectors
+    html = <<END_HTML
+    <html>
+    <style type="text/css"> td a:link.top_links { color: red; } </style>
+    <body>
+		<td><a class="top_links">Test</a></td>
+		</body>
+		</html>
+END_HTML
+
+		premailer = Premailer.new(html, :with_html_string => true)
+    assert_nothing_raised do
+		  premailer.to_inline_css
+	  end
+	  assert_match /color: red/, premailer.processed_doc.at('a').attributes['style'].to_s
+  end
+
+  # in response to https://github.com/alexdunae/premailer/issues#issue/7
+  #
+  # fails sometimes in JRuby, see https://github.com/alexdunae/premailer/issues/79
+  def test_parsing_bad_markup_around_tables
+    html = <<END_HTML
+    <html>
+    <style type="text/css"> 
+      .style3 { font-size: xx-large; }
+      .style5 { background-color: #000080; } 
+    </style>
+		<tr>
+						<td valign="top" class="style3">
+						<!-- MSCellType="ContentHead" -->
+						<strong>PROMOCION CURSOS PRESENCIALES</strong></td>
+						<strong>
+						<td valign="top" style="height: 125px" class="style5">
+						<!-- MSCellType="DecArea" -->
+						<img alt="" src="../../images/CertisegGold.GIF" width="608" height="87" /></td>
+		</tr>
+END_HTML
+
+		premailer = Premailer.new(html, :with_html_string => true)
+		premailer.to_inline_css
+	  assert_match /font-size: xx-large/, premailer.processed_doc.search('.style3').first.attributes['style'].to_s
+	  assert_match /background-color: #000080/, premailer.processed_doc.search('.style5').first.attributes['style'].to_s		
+  end
+
+  # in response to https://github.com/alexdunae/premailer/issues/56
+  def test_inline_important
+    html = <<END_HTML
+    <html>
+    <style type="text/css"> 
+      p { color: red !important; }
+    </style>
+    <body>
+      <p style='color: green !important;'>test</p></div>
+    </body>
+    </html>
+END_HTML
+
+    premailer = Premailer.new(html, :with_html_string => true, :adapter => :nokogiri)
+  	premailer.to_inline_css
+    assert_equal 'color: green !important', premailer.processed_doc.search('p').first.attributes['style'].to_s
+  end
+
+  # in response to https://github.com/alexdunae/premailer/issues/28
+  def test_handling_shorthand_auto_properties
+    html = <<END_HTML
+    <html>
+    <style type="text/css"> 
+      #page { margin: 0; margin-left: auto; margin-right: auto; }
+      p { border: 1px solid black; border-right: none; }
+      
+    </style>
+    <body>
+      <div id='page'><p>test</p></div>
+    </body>
+    </html>
+END_HTML
+
+    premailer = Premailer.new(html, :with_html_string => true)
+  	premailer.to_inline_css
+    assert_match /margin: 0 auto;/, premailer.processed_doc.search('#page').first.attributes['style'].to_s
+    assert_match /border-style: solid none solid solid;/, premailer.processed_doc.search('p').first.attributes['style'].to_s
+  end
+
+  def test_sorting_style_attributes
+    html = <<END_HTML
+    <html>
+    <style type="text/css">
+      #page { right: 10px; left: 5px }
+    </style>
+    <body>
+      <div id='page'>test</div>
+    </body>
+    </html>
+END_HTML
+
+    premailer = Premailer.new(html, :with_html_string => true)
+    premailer.to_inline_css
+    assert_equal "left: 5px; right: 10px", premailer.processed_doc.search('#page').first.attributes['style'].to_s
+  end
+
+  def test_removing_scripts
+    html = <<END_HTML
+    <html>
+    <head>
+      <script>script to be removed</script>
+    </head>
+    <body>
+      content
+    </body>
+    </html>
+END_HTML
+
+    [:nokogiri, :hpricot].each do |adapter|
+      premailer = Premailer.new(html, :with_html_string => true, :remove_scripts => true, :adapter => adapter)
+      premailer.to_inline_css
+      assert_equal 0, premailer.processed_doc.search('script').length
+    end
+
+    [:nokogiri, :hpricot].each do |adapter|
+      premailer = Premailer.new(html, :with_html_string => true, :remove_scripts => false, :adapter => adapter)
+      premailer.to_inline_css
+      assert_equal 1, premailer.processed_doc.search('script').length
+    end
+  end
+
+end