html-pipeline-linuxfr 0.14.23 → 0.14.24

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8f4ef09e99d77078e0ac176f875a641731457b61
-  data.tar.gz: e2c4db8771edbed3c8686091dfa8da8de533049f
+  metadata.gz: bb682966ec238f8fab36cad469d5ce4150e4d7ca
+  data.tar.gz: 20244031571553790cc97cd5d633c702419188b2
 SHA512:
-  metadata.gz: b5eb66b70d8b5e11dc356e7125d7dfc7409a356a5577206bd017abd80dd8a345d23058ffe63b380d36edc91dde5e3b4f425bcf1a69bd9171468ae71fc0f9a2f6
-  data.tar.gz: dfd4ab1ac0defd572c6dc7f684398e4ba5aefbd48195fb81ce3ab0dbeaca6f3fc6930f6e0c908e5a0086bafc8d803d34a8972c891225cf8f125897ecc0c7bd12
+  metadata.gz: e3e46ea4cb7bcf90f9d6af31f7442ab6417f56bea353580f119de09f21931bf8b14506da95ccb08435fdb2c95f107c811f494bd7fbf3dc94c75a876d15993080
+  data.tar.gz: 91cbc215b6699a01893180f018d0604d0f5fd3958fa6fa9d45dc0e094ed054a181b74760b572cda2639af2119c78ecab91b87fa1f5186b48c0708bc701a4cb2f

data/.gitignore

@@ -1,19 +1,2 @@
-*.gem
-*.rbc
-.bundle
-.config
-.yardoc
 Gemfile.lock
-InstalledFiles
-_yardoc
-coverage
-doc/
-lib/bundler/man
-pkg
-rdoc
-spec/reports
-test/tmp
-test/version_tmp
-tmp
-exec/*
-vendor/gems
+/pkg

data/html-pipeline-linuxfr.gemspec

@@ -15,11 +15,13 @@ Gem::Specification.new do |gem|
   gem.test_files    = gem.files.grep(%r{^test})
   gem.require_paths = ["lib"]
 
-  gem.add_dependency "nokogiri",        "~> 1.4"
-  gem.add_dependency "redcarpet",       "~> 3.1"
+  gem.add_dependency "nokogiri",        "~> 1.6"
+  gem.add_dependency "redcarpet",       "~> 3.2"
   gem.add_dependency "pygments.rb",     "~> 0.5"
-  gem.add_dependency "sanitize",        "~> 2.0"
-  gem.add_dependency "escape_utils",    "~> 1.0"
-  gem.add_dependency "activesupport",   "~> 4.0"
+  gem.add_dependency "sanitize",        "~> 3.1"
+  gem.add_dependency "escape_utils",    "~> 1.1"
+  gem.add_dependency "activesupport",   "~> 4.2"
   gem.add_dependency "patron",          "~> 0.4"
+
+  gem.add_development_dependency "test-unit"
 end

data/lib/html/pipeline/sanitization_filter.rb

@@ -23,7 +23,6 @@ module HTML
       # The main sanitization whitelist. Only these elements and attributes are
       # allowed through by default.
       WHITELIST = {
-        :output => :xhtml,
         :elements => %w(a abbr b blockquote br cite code dd del dfn div dl dt em
                         h1 h2 h3 h4 h5 h6 hr i img ins kbd li mark meter ol p pre
                         q s samp small source span strong sub sup table tbody td
@@ -70,7 +69,7 @@ module HTML
 
       # Sanitize markup using the Sanitize library.
       def call
-        encode_mb4 Sanitize.clean_node!(doc, whitelist)
+        encode_mb4 Sanitize.node!(doc, whitelist)
       end
 
       # The whitelist to use when sanitizing. This can be passed in the context

data/lib/html/pipeline/version.rb

@@ -1,5 +1,5 @@
 module HTML
   class Pipeline
-    VERSION = "0.14.23"
+    VERSION = "0.14.24"
   end
 end

data/test/html/pipeline/markdown_filter_test.rb

@@ -30,24 +30,11 @@ class HTML::Pipeline::MarkdownFilterTest < Test::Unit::TestCase
     assert_equal 2, doc.search('br').size
   end
 
-  def test_disabling_gfm
-    doc = MarkdownFilter.to_document(@haiku, :gfm => false)
-    assert doc.kind_of?(HTML::Pipeline::DocumentFragment)
-    assert_equal 0, doc.search('br').size
-  end
-
   def test_fenced_code_blocks
     doc = MarkdownFilter.to_document(@code)
     assert doc.kind_of?(HTML::Pipeline::DocumentFragment)
     assert_equal 1, doc.search('pre').size
   end
-
-  def test_fenced_code_blocks_with_language
-    doc = MarkdownFilter.to_document(@code.sub("```", "``` ruby"))
-    assert doc.kind_of?(HTML::Pipeline::DocumentFragment)
-    assert_equal 1, doc.search('pre').size
-    assert_equal 'ruby', doc.search('pre').first['lang']
-  end
 end
 
 class GFMTest < Test::Unit::TestCase
@@ -56,46 +43,41 @@ class GFMTest < Test::Unit::TestCase
   end
 
   def test_not_touch_single_underscores_inside_words
-    assert_equal "<p>foo_bar</p>",
+    assert_equal "<p>foo_bar</p>\n",
                  gfm("foo_bar")
   end
 
   def test_not_touch_underscores_in_code_blocks
-    assert_equal "<pre><code>foo_bar_baz\n</code></pre>",
+    assert_equal "<pre><code>foo_bar_baz\n</code></pre>\n",
                  gfm("    foo_bar_baz")
   end
 
-  def test_not_touch_underscores_in_pre_blocks
-    assert_equal "<pre>\nfoo_bar_baz\n</pre>",
-                 gfm("<pre>\nfoo_bar_baz\n</pre>")
-  end
-
   def test_not_touch_two_or_more_underscores_inside_words
-    assert_equal "<p>foo_bar_baz</p>",
+    assert_equal "<p>foo_bar_baz</p>\n",
                  gfm("foo_bar_baz")
   end
 
   def test_turn_newlines_into_br_tags_in_simple_cases
-    assert_equal "<p>foo<br>\nbar</p>",
+    assert_equal "<p>foo<br/>\nbar</p>\n",
                  gfm("foo\nbar")
   end
 
   def test_convert_newlines_in_all_groups
-    assert_equal "<p>apple<br>\npear<br>\norange</p>\n\n" +
-                 "<p>ruby<br>\npython<br>\nerlang</p>",
+    assert_equal "<p>apple<br/>\npear<br/>\norange</p>\n\n" +
+                 "<p>ruby<br/>\npython<br/>\nerlang</p>\n",
                  gfm("apple\npear\norange\n\nruby\npython\nerlang")
   end
 
   def test_convert_newlines_in_even_long_groups
-    assert_equal "<p>apple<br>\npear<br>\norange<br>\nbanana</p>\n\n" +
-                 "<p>ruby<br>\npython<br>\nerlang</p>",
+    assert_equal "<p>apple<br/>\npear<br/>\norange<br/>\nbanana</p>\n\n" +
+                 "<p>ruby<br/>\npython<br/>\nerlang</p>\n",
                  gfm("apple\npear\norange\nbanana\n\nruby\npython\nerlang")
   end
 
   def test_not_convert_newlines_in_lists
-    assert_equal "<h1>foo</h1>\n\n<h1>bar</h1>",
+    assert_equal "<h2>foo</h2>\n<h2>bar</h2>\n",
                  gfm("# foo\n# bar")
-    assert_equal "<ul>\n<li>foo</li>\n<li>bar</li>\n</ul>",
+    assert_equal "<ul>\n<li>foo</li>\n<li>bar</li>\n</ul>\n",
                  gfm("* foo\n* bar")
   end
 end

data/test/html/pipeline/sanitization_filter_test.rb

@@ -29,22 +29,11 @@ class HTML::Pipeline::SanitizationFilterTest < Test::Unit::TestCase
     assert_no_match /onclick/, html
   end
 
-  def test_sanitizes_li_elements_not_contained_in_ul_or_ol
-    stuff = "a\n<li>b</li>\nc"
-    html  = SanitizationFilter.call(stuff).to_s
-    assert_equal "a\nb\nc", html
-  end
-
   def test_does_not_sanitize_li_elements_contained_in_ul_or_ol
     stuff = "a\n<ul><li>b</li></ul>\nc"
     assert_equal stuff, SanitizationFilter.call(stuff).to_s
   end
 
-  def test_github_specific_protocols_are_not_removed
-    stuff = '<a href="github-windows://spillthelog">Spill this yo</a> and so on'
-    assert_equal stuff, SanitizationFilter.call(stuff).to_s
-  end
-
   def test_script_contents_are_removed
     orig = '<script>JavaScript!</script>'
     assert_equal "", SanitizationFilter.call(orig).to_s

data/test/html/pipeline/toc_filter_test.rb

@@ -2,15 +2,16 @@ require "test_helper"
 
 class HTML::Pipeline::TableOfContentsFilterTest < Test::Unit::TestCase
   TocFilter = HTML::Pipeline::TableOfContentsFilter
+  CONTEXT = { toc_minimal_length: 1, toc_header: 'Table of contents' }
 
   def test_anchors_are_added_properly
-    orig = %(<h1>Ice cube</h1><p>Will swarm on any motherfucker in a blue uniform</p>)
-    assert_includes '<a name=', TocFilter.call(orig).to_s
+    orig = %(<h2>Ice cube</h2><p>Will swarm on any motherfucker in a blue uniform</p>)
+    assert_includes '<h2 id=', TocFilter.call(orig, CONTEXT).to_s
   end
 
   def test_anchors_have_sane_names
-    orig = %(<h1>Dr Dre</h1><h1>Ice Cube</h1><h1>Eazy-E</h1><h1>MC Ren</h1>)
-    result = TocFilter.call(orig).to_s
+    orig = %(<h2>Dr Dre</h2><h2>Ice Cube</h2><h2>Eazy-E</h2><h2>MC Ren</h2>)
+    result = TocFilter.call(orig, CONTEXT).to_s
 
     assert_includes '"dr-dre"', result
     assert_includes '"ice-cube"', result
@@ -19,29 +20,27 @@ class HTML::Pipeline::TableOfContentsFilterTest < Test::Unit::TestCase
   end
 
   def test_dupe_headers_have_unique_trailing_identifiers
-    orig = %(<h1>Straight Outta Compton</h1>
-             <h2>Dopeman</h2>
-             <h3>Express Yourself</h3>
-             <h1>Dopeman</h1>)
+    orig = %(<h2>Straight Outta Compton</h2>
+             <h3>Dopeman</h3>
+             <h4>Express Yourself</h4>
+             <h2>Dopeman</h2>)
 
-    result = TocFilter.call(orig).to_s
+    result = TocFilter.call(orig, CONTEXT).to_s
 
     assert_includes '"dopeman"', result
     assert_includes '"dopeman-1"', result
   end
 
   def test_all_header_tags_are_found_when_adding_anchors
-    orig = %(<h1>"Funky President" by James Brown</h1>
-             <h2>"It's My Thing" by Marva Whitney</h2>
-             <h3>"Boogie Back" by Roy Ayers</h3>
-             <h4>"Feel Good" by Fancy</h4>
-             <h5>"Funky Drummer" by James Brown</h5>
-             <h6>"Ruthless Villain" by Eazy-E</h6>
-             <h7>"Be Thankful for What You Got" by William DeVaughn</h7>)
-
-    doc = TocFilter.call(orig)
-    assert_equal 6, doc.search('a').size
+    orig = %(<h2>"Funky President" by James Brown</h2>
+             <h3>"It's My Thing" by Marva Whitney</h3>
+             <h4>"Boogie Back" by Roy Ayers</h4>
+             <h5>"Feel Good" by Fancy</h5>
+             <h6>"Funky Drummer" by James Brown</h6>
+             <h7>"Ruthless Villain" by Eazy-E</h7>
+             <h8>"Be Thankful for What You Got" by William DeVaughn</h8>)
+
+    doc = TocFilter.call(orig, CONTEXT)
+    assert_equal 5, doc.search('a').size
   end
 end
-
-

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: html-pipeline-linuxfr
 version: !ruby/object:Gem::Version
-  version: 0.14.23
+  version: 0.14.24
 platform: ruby
 authors:
 - Ryan Tomayko
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-02 00:00:00.000000000 Z
+date: 2015-04-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -18,28 +18,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.4'
+        version: '1.6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.4'
+        version: '1.6'
 - !ruby/object:Gem::Dependency
   name: redcarpet
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '3.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '3.2'
 - !ruby/object:Gem::Dependency
   name: pygments.rb
   requirement: !ruby/object:Gem::Requirement
@@ -60,42 +60,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '3.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '3.1'
 - !ruby/object:Gem::Dependency
   name: escape_utils
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '4.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '4.2'
 - !ruby/object:Gem::Dependency
   name: patron
   requirement: !ruby/object:Gem::Requirement
@@ -110,6 +110,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.4'
+- !ruby/object:Gem::Dependency
+  name: test-unit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: LinuxFr.org HTML processing filters and utilities, adapted from those
   of GitHub
 email:
@@ -121,13 +135,11 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
-- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - LICENSE
 - README.md
 - Rakefile
-- bin/html-pipeline
 - html-pipeline-linuxfr.gemspec
 - lib/html/pipeline.rb
 - lib/html/pipeline/custom_links_filter.rb
@@ -142,12 +154,7 @@ files:
 - lib/html/pipeline/toc_filter.rb
 - lib/html/pipeline/version.rb
 - test/helpers/mocked_instrumentation_service.rb
-- test/html/pipeline/absolute_source_filter_test.rb
-- test/html/pipeline/camo_filter_test.rb
-- test/html/pipeline/image_max_width_filter_test.rb
 - test/html/pipeline/markdown_filter_test.rb
-- test/html/pipeline/mention_filter_test.rb
-- test/html/pipeline/plain_text_input_filter_test.rb
 - test/html/pipeline/sanitization_filter_test.rb
 - test/html/pipeline/toc_filter_test.rb
 - test/html/pipeline_test.rb
@@ -172,18 +179,13 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Helpers for processing content through a chain of filters
 test_files:
 - test/helpers/mocked_instrumentation_service.rb
-- test/html/pipeline/absolute_source_filter_test.rb
-- test/html/pipeline/camo_filter_test.rb
-- test/html/pipeline/image_max_width_filter_test.rb
 - test/html/pipeline/markdown_filter_test.rb
-- test/html/pipeline/mention_filter_test.rb
-- test/html/pipeline/plain_text_input_filter_test.rb
 - test/html/pipeline/sanitization_filter_test.rb
 - test/html/pipeline/toc_filter_test.rb
 - test/html/pipeline_test.rb

data/.travis.yml

@@ -1,13 +0,0 @@
-language: ruby
-
-before_install:
-  - sudo apt-get update -qq
-  - sudo apt-get install -qq libicu-dev
-
-script: "bundle exec rake"
-
-rvm:
-  - 1.8.7
-  - 1.9.2
-  - 1.9.3
-  - ree

data/bin/html-pipeline

@@ -1,80 +0,0 @@
-#!/usr/bin/env ruby
-require 'html/pipeline'
-
-require 'optparse'
-
-# Accept "help", too
-ARGV.map!{|a| a == "help" ? "--help" : a }
-
-OptionParser.new do |opts|
-  opts.banner = <<-HELP.gsub(/^    /, '')
-    Usage: html-pipeline [-h] [-f]
-           html-pipeline [FILTER [FILTER [...]]] < file.md
-           cat file.md | html-pipeline [FILTER [FILTER [...]]]
-  HELP
-
-  opts.separator "Options:"
-
-  opts.on("-f", "--filters", "List the available filters") do
-    filters = HTML::Pipeline.constants.grep(/\w+Filter$/).
-      map{|f| f.to_s.gsub(/Filter$/,'') }
-
-    # Text filter doesn't work, no call method
-    filters -= ["Text"]
-
-    abort <<-HELP.gsub(/^      /, '')
-      Available filters:
-        #{filters.join("\n        ")}
-    HELP
-  end
-end.parse!
-
-# Default to a GitHub-ish pipeline
-if ARGV.empty?
-
-  filters = [
-    HTML::Pipeline::MarkdownFilter,
-    HTML::Pipeline::SanitizationFilter,
-    HTML::Pipeline::ImageMaxWidthFilter,
-    HTML::Pipeline::EmojiFilter,
-    HTML::Pipeline::AutolinkFilter,
-    HTML::Pipeline::TableOfContentsFilter,
-  ]
-
-  # Add syntax highlighting if linguist is present
-  begin
-    require 'linguist'
-    filters << HTML::Pipeline::SyntaxHighlightFilter
-  rescue LoadError
-  end
-
-else
-
-  def filter_named(name)
-    case name
-    when "Text"
-      raise NameError # Text filter doesn't work, no call method
-    when "Textile"
-      require "RedCloth" # Textile filter doesn't require RedCloth
-    end
-
-    HTML::Pipeline.const_get("#{name}Filter")
-  rescue NameError => e
-    abort "Unknown filter '#{name}'. List filters with the -f option."
-  end
-
-  filters = []
-  until ARGV.empty?
-    name = ARGV.shift
-    filters << filter_named(name)
-  end
-
-end
-
-context = {
-  :asset_root => "/assets",
-  :base_url   => "/",
-  :gfm        => true
-}
-
-puts HTML::Pipeline.new(filters, context).call(ARGF.read)[:output]

data/test/html/pipeline/absolute_source_filter_test.rb

@@ -1,56 +0,0 @@
-require "test_helper"
-
-class HTML::Pipeline::AbsoluteSourceFilterTest < Test::Unit::TestCase
-  AbsoluteSourceFilter = HTML::Pipeline::AbsoluteSourceFilter
-
-  def setup
-    @image_base_url = 'http://assets.example.com'
-    @image_subpage_url = 'http://blog.example.com/a/post'
-    @options = {
-      :image_base_url    => @image_base_url,
-      :image_subpage_url => @image_subpage_url
-    }
-  end
-
-  def test_rewrites_root_relative_urls
-    orig = %(<p><img src="/img.png"></p>)
-    puts AbsoluteSourceFilter.call(orig, @options).to_s
-    assert_equal "<p><img src=\"#{@image_base_url}/img.png\"></p>",
-      AbsoluteSourceFilter.call(orig, @options).to_s
-  end
-
-  def test_rewrites_root_relative_urls
-    orig = %(<p><img src="post/img.png"></p>)
-    assert_equal "<p><img src=\"#{@image_subpage_url}/img.png\"></p>",
-      AbsoluteSourceFilter.call(orig, @options).to_s
-  end
-
-  def test_does_not_rewrite_absolute_urls
-    orig = %(<p><img src="http://other.example.com/img.png"></p>)
-    result = AbsoluteSourceFilter.call(orig, @options).to_s
-    assert_no_match /@image_base_url/, result
-    assert_no_match /@image_subpage_url/, result
-  end
-
-  def test_fails_when_context_is_missing
-    assert_raise RuntimeError do
-      AbsoluteSourceFilter.call("<img src=\"img.png\">", {})
-    end
-    assert_raise RuntimeError do
-      AbsoluteSourceFilter.call("<img src=\"/img.png\">", {})
-    end
-  end
-  
-  def test_tells_you_where_context_is_required
-    exception = assert_raise(RuntimeError) { 
-      AbsoluteSourceFilter.call("<img src=\"img.png\">", {}) 
-    }
-    assert_match 'HTML::Pipeline::AbsoluteSourceFilter', exception.message
-
-    exception = assert_raise(RuntimeError) { 
-      AbsoluteSourceFilter.call("<img src=\"/img.png\">", {}) 
-    }
-    assert_match 'HTML::Pipeline::AbsoluteSourceFilter', exception.message
-  end
-
-end

data/test/html/pipeline/camo_filter_test.rb

@@ -1,47 +0,0 @@
-require "test_helper"
-
-class HTML::Pipeline::CamoFilterTest < Test::Unit::TestCase
-  CamoFilter = HTML::Pipeline::CamoFilter
-
-  def setup
-    @asset_proxy_url        = 'https//assets.example.org'
-    @asset_proxy_secret_key = 'ssssh-secret'
-    @options = {
-      :asset_proxy            => @asset_proxy_url,
-      :asset_proxy_secret_key => @asset_proxy_secret_key
-    }
-  end
-
-  def test_camouflaging_http_image_urls
-    orig = %(<p><img src="http://twitter.com/img.png"></p>)
-    assert_includes 'img src="' + @asset_proxy_url,
-      CamoFilter.call(orig, @options).to_s
-  end
-
-  def test_rewrites_dotcom_image_urls
-    orig = %(<p><img src="http://github.com/img.png"></p>)
-    assert_equal "<p><img src=\"https://github.com/img.png\"></p>",
-      CamoFilter.call(orig, @options).to_s
-  end
-
-  def test_not_camouflaging_https_image_urls
-    orig = %(<p><img src="https://foo.com/img.png"></p>)
-    assert_doesnt_include 'img src="' + @asset_proxy_url,
-      CamoFilter.call(orig, @options).to_s
-  end
-
-  def test_handling_images_with_no_src_attribute
-    orig = %(<p><img></p>)
-    assert_nothing_raised do
-      CamoFilter.call(orig, @options).to_s
-    end
-  end
-    
-  def test_required_context_validation
-    exception = assert_raise(ArgumentError) { 
-      CamoFilter.call("", {}) 
-    }
-    assert_match /:asset_proxy[^_]/, exception.message
-    assert_match /:asset_proxy_secret_key/, exception.message
-  end
-end

data/test/html/pipeline/image_max_width_filter_test.rb

@@ -1,50 +0,0 @@
-require "test_helper"
-
-class HTML::Pipeline::ImageMaxWidthFilterTest < Test::Unit::TestCase
-  def filter(html)
-    HTML::Pipeline::ImageMaxWidthFilter.call(html)
-  end
-
-  def test_rewrites_image_style_tags
-    body = "<p>Screenshot: <img src='screenshot.png'></p>"
-    doc  = Nokogiri::HTML::DocumentFragment.parse(body)
-
-    res = filter(doc)
-    assert_equal_html %q(<p>Screenshot: <a target="_blank" href="screenshot.png"><img src="screenshot.png" style="max-width:100%;"></a></p>),
-      res.to_html
-  end
-
-  def test_leaves_existing_image_style_tags_alone
-    body = "<p><img src='screenshot.png' style='width:100px;'></p>"
-    doc  = Nokogiri::HTML::DocumentFragment.parse(body)
-
-    res = filter(doc)
-    assert_equal_html '<p><img src="screenshot.png" style="width:100px;"></p>',
-      res.to_html
-  end
-
-  def test_links_to_image
-    body = "<p>Screenshot: <img src='screenshot.png'></p>"
-    doc  = Nokogiri::HTML::DocumentFragment.parse(body)
-
-    res = filter(doc)
-    assert_equal_html '<p>Screenshot: <a target="_blank" href="screenshot.png"><img src="screenshot.png" style="max-width:100%;"></a></p>',
-      res.to_html
-  end
-
-  def test_doesnt_link_to_image_when_already_linked
-    body = "<p>Screenshot: <a href='blah.png'><img src='screenshot.png'></a></p>"
-    doc  = Nokogiri::HTML::DocumentFragment.parse(body)
-
-    res = filter(doc)
-    assert_equal_html %q(<p>Screenshot: <a href="blah.png"><img src="screenshot.png" style="max-width:100%;"></a></p>),
-      res.to_html
-  end
-
-  def test_doesnt_screw_up_inlined_images
-    body = "<p>Screenshot <img src='screenshot.png'>, yes, this is a <b>screenshot</b> indeed.</p>"
-    doc  = Nokogiri::HTML::DocumentFragment.parse(body)
-
-    assert_equal_html %q(<p>Screenshot <a target="_blank" href="screenshot.png"><img src="screenshot.png" style="max-width:100%;"></a>, yes, this is a <b>screenshot</b> indeed.</p>), filter(doc).to_html
-  end
-end

data/test/html/pipeline/mention_filter_test.rb

@@ -1,156 +0,0 @@
-require "test_helper"
-
-class HTML::Pipeline::MentionFilterTest < Test::Unit::TestCase
-  def filter(html, base_url='/', info_url=nil)
-    HTML::Pipeline::MentionFilter.call(html, :base_url => base_url, :info_url => info_url)
-  end
-
-  def test_filtering_a_documentfragment
-    body = "<p>@kneath: check it out.</p>"
-    doc  = Nokogiri::HTML::DocumentFragment.parse(body)
-
-    res  = filter(doc, '/')
-    assert_same doc, res
-
-    link = "<a href=\"/kneath\" class=\"user-mention\">@kneath</a>"
-    assert_equal "<p>#{link}: check it out.</p>",
-      res.to_html
-  end
-
-  def test_filtering_plain_text
-    body = "<p>@kneath: check it out.</p>"
-    res  = filter(body, '/')
-
-    link = "<a href=\"/kneath\" class=\"user-mention\">@kneath</a>"
-    assert_equal "<p>#{link}: check it out.</p>",
-      res.to_html
-  end
-
-  def test_not_replacing_mentions_in_pre_tags
-    body = "<pre>@kneath: okay</pre>"
-    assert_equal body, filter(body).to_html
-  end
-
-  def test_not_replacing_mentions_in_code_tags
-    body = "<p><code>@kneath:</code> okay</p>"
-    assert_equal body, filter(body).to_html
-  end
-
-  def test_not_replacing_mentions_in_links
-    body = "<p><a>@kneath</a> okay</p>"
-    assert_equal body, filter(body).to_html
-  end
-
-  def test_entity_encoding_and_whatnot
-    body = "<p>@&#x6b;neath what's up</p>"
-    link = "<a href=\"/kneath\" class=\"user-mention\">@kneath</a>"
-    assert_equal "<p>#{link} what's up</p>", filter(body, '/').to_html
-  end
-
-  def test_html_injection
-    body = "<p>@kneath &lt;script>alert(0)&lt;/script></p>"
-    link = "<a href=\"/kneath\" class=\"user-mention\">@kneath</a>"
-    assert_equal "<p>#{link} &lt;script&gt;alert(0)&lt;/script&gt;</p>",
-      filter(body, '/').to_html
-  end
-
-  def test_links_to_nothing_when_no_info_url_given
-    body = "<p>How do I @mention someone?</p>"
-    assert_equal "<p>How do I @mention someone?</p>",
-      filter(body, '/').to_html
-  end
-
-  def test_links_to_more_info_when_info_url_given
-    body = "<p>How do I @mention someone?</p>"
-    link = "<a href=\"https://github.com/blog/821\" class=\"user-mention\">@mention</a>"
-    assert_equal "<p>How do I #{link} someone?</p>",
-      filter(body, '/', 'https://github.com/blog/821').to_html
-  end
-
-  MarkdownPipeline =
-    HTML::Pipeline.new [
-      HTML::Pipeline::MarkdownFilter,
-      HTML::Pipeline::MentionFilter
-    ]
-
-  def mentioned_usernames
-    result = {}
-    MarkdownPipeline.call(@body, {}, result)
-    result[:mentioned_usernames]
-  end
-
-  def test_matches_usernames_in_body
-    @body = "@test how are you?"
-    assert_equal %w[test], mentioned_usernames
-  end
-
-  def test_matches_usernames_with_dashes
-    @body = "hi @some-user"
-    assert_equal %w[some-user], mentioned_usernames
-  end
-
-  def test_matches_usernames_followed_by_a_single_dot
-    @body = "okay @some-user."
-    assert_equal %w[some-user], mentioned_usernames
-  end
-
-  def test_matches_usernames_followed_by_multiple_dots
-    @body = "okay @some-user..."
-    assert_equal %w[some-user], mentioned_usernames
-  end
-
-  def test_does_not_match_email_addresses
-    @body = "aman@tmm1.net"
-    assert_equal [], mentioned_usernames
-  end
-
-  def test_does_not_match_domain_name_looking_things
-    @body = "we need a @github.com email"
-    assert_equal [], mentioned_usernames
-  end
-
-  def test_does_not_match_organization_team_mentions
-    @body = "we need to @github/enterprise know"
-    assert_equal [], mentioned_usernames
-  end
-
-  def test_matches_colon_suffixed_names
-    @body = "@tmm1: what do you think?"
-    assert_equal %w[tmm1], mentioned_usernames
-  end
-
-  def test_matches_list_of_names
-    @body = "@defunkt @atmos @kneath"
-    assert_equal %w[defunkt atmos kneath], mentioned_usernames
-  end
-
-  def test_matches_list_of_names_with_commas
-    @body = "/cc @defunkt, @atmos, @kneath"
-    assert_equal %w[defunkt atmos kneath], mentioned_usernames
-  end
-
-  def test_matches_inside_brackets
-    @body = "(@mislav) and [@rtomayko]"
-    assert_equal %w[mislav rtomayko], mentioned_usernames
-  end
-
-  def test_doesnt_ignore_invalid_users
-    @body = "@defunkt @mojombo and @somedude"
-    assert_equal ['defunkt', 'mojombo', 'somedude'], mentioned_usernames
-  end
-
-  def test_returns_distinct_set
-    @body = "/cc @defunkt, @atmos, @kneath, @defunkt, @defunkt"
-    assert_equal %w[defunkt atmos kneath], mentioned_usernames
-  end
-
-  def test_does_not_match_inline_code_block_with_multiple_code_blocks
-    @body = "something\n\n`/cc @defunkt @atmos @kneath` `/cc @atmos/atmos`"
-    assert_equal %w[], mentioned_usernames
-  end
-
-  def test_mention_at_end_of_parenthetical_sentence
-    @body = "(We're talking 'bout @ymendel.)"
-    assert_equal %w[ymendel], mentioned_usernames
-  end
-end

data/test/html/pipeline/plain_text_input_filter_test.rb

@@ -1,22 +0,0 @@
-require "test_helper"
-
-class HTML::Pipeline::PlainTextInputFilterTest < Test::Unit::TestCase
-  PlainTextInputFilter = HTML::Pipeline::PlainTextInputFilter
-
-  def test_fails_when_given_a_documentfragment
-    body = "<p>heyo</p>"
-    doc  = Nokogiri::HTML::DocumentFragment.parse(body)
-    assert_raise(TypeError) { PlainTextInputFilter.call(doc, {}) }
-  end
-
-  def test_wraps_input_in_a_div_element
-    doc = PlainTextInputFilter.call("howdy pahtner", {})
-    assert_equal "<div>howdy pahtner</div>", doc.to_s
-  end
-
-  def test_html_escapes_plain_text_input
-    doc = PlainTextInputFilter.call("See: <http://example.org>", {})
-    assert_equal "<div>See: &lt;http://example.org&gt;</div>",
-      doc.to_s
-  end
-end