raise MissingDependencyError instead of aborting on missing dependency

Reverts a0acb69 Fixes #240
gjtorikian · Jan 14, 2016 · 3edf0c9 · 3edf0c9
1 parent 5bf0502
commit 3edf0c9
Show file tree

Hide file tree

Showing 9 changed files with 11 additions and 9 deletions.
diff --git a/lib/html/pipeline.rb b/lib/html/pipeline.rb
@@ -42,6 +42,8 @@ class Pipeline
     autoload :TextileFilter,         'html/pipeline/textile_filter'
     autoload :TableOfContentsFilter, 'html/pipeline/toc_filter'
     autoload :TextFilter,            'html/pipeline/text_filter'
+
+    MissingDependencyError = Class.new(LoadError)
 
     # Our DOM implementation.
     DocumentFragment = Nokogiri::HTML::DocumentFragment

diff --git a/lib/html/pipeline/autolink_filter.rb b/lib/html/pipeline/autolink_filter.rb
@@ -1,7 +1,7 @@
 begin
   require "rinku"
 rescue LoadError => _
-  abort "Missing dependency 'rinku' for AutolinkFilter. See README.md for details."
+  raise MissingDependencyError, "Missing dependency 'rinku' for AutolinkFilter. See README.md for details."
 end
 
 module HTML

diff --git a/lib/html/pipeline/email_reply_filter.rb b/lib/html/pipeline/email_reply_filter.rb
@@ -1,13 +1,13 @@
 begin
   require "escape_utils"
 rescue LoadError => _
-  abort "Missing dependency 'escape_utils' for EmailReplyFilter. See README.md for details."
+  raise MissingDependencyError, "Missing dependency 'escape_utils' for EmailReplyFilter. See README.md for details."
 end
 
 begin
   require "email_reply_parser"
 rescue LoadError => _
-  abort "Missing dependency 'email_reply_parser' for EmailReplyFilter. See README.md for details."
+  raise MissingDependencyError, "Missing dependency 'email_reply_parser' for EmailReplyFilter. See README.md for details."
 end
 
 module HTML

diff --git a/lib/html/pipeline/emoji_filter.rb b/lib/html/pipeline/emoji_filter.rb
@@ -3,7 +3,7 @@
 begin
   require "gemoji"
 rescue LoadError => _
-  abort "Missing dependency 'gemoji' for EmojiFilter. See README.md for details."
+  raise MissingDependencyError, "Missing dependency 'gemoji' for EmojiFilter. See README.md for details."
 end
 
 module HTML

diff --git a/lib/html/pipeline/markdown_filter.rb b/lib/html/pipeline/markdown_filter.rb
@@ -1,7 +1,7 @@
 begin
   require "github/markdown"
 rescue LoadError => _
-  abort "Missing dependency 'github-markdown' for MarkdownFilter. See README.md for details."
+  raise MissingDependencyError, "Missing dependency 'github-markdown' for MarkdownFilter. See README.md for details."
 end
 
 module HTML

diff --git a/lib/html/pipeline/plain_text_input_filter.rb b/lib/html/pipeline/plain_text_input_filter.rb
@@ -1,7 +1,7 @@
 begin
   require "escape_utils"
 rescue LoadError => _
-  abort "Missing dependency 'escape_utils' for PlainTextInputFilter. See README.md for details."
+  raise MissingDependencyError, "Missing dependency 'escape_utils' for PlainTextInputFilter. See README.md for details."
 end
 
 module HTML

diff --git a/lib/html/pipeline/sanitization_filter.rb b/lib/html/pipeline/sanitization_filter.rb
@@ -1,7 +1,7 @@
 begin
   require "sanitize"
 rescue LoadError => _
-  abort "Missing dependency 'sanitize' for SanitizationFilter. See README.md for details."
+  raise MissingDependencyError, "Missing dependency 'sanitize' for SanitizationFilter. See README.md for details."
 end
 
 module HTML

diff --git a/lib/html/pipeline/syntax_highlight_filter.rb b/lib/html/pipeline/syntax_highlight_filter.rb
@@ -1,7 +1,7 @@
 begin
   require "linguist"
 rescue LoadError => _
-  abort "Missing dependency 'github-linguist' for SyntaxHighlightFilter. See README.md for details."
+  raise MissingDependencyError, "Missing dependency 'github-linguist' for SyntaxHighlightFilter. See README.md for details."
 end
 
 module HTML

diff --git a/lib/html/pipeline/textile_filter.rb b/lib/html/pipeline/textile_filter.rb
@@ -1,7 +1,7 @@
 begin
   require "redcloth"
 rescue LoadError => _
-  abort "Missing dependency 'RedCloth' for TextileFilter. See README.md for details."
+  raise MissingDependencyError, "Missing dependency 'RedCloth' for TextileFilter. See README.md for details."
 end
 
 module HTML