Convert abstract text TextFilterPlugin descendants to modules

This change makes all actual descendants of TextFilterPlugin real filters.

lib/publify_core/text_filter/markdown.rb

@@ -4,7 +4,9 @@
 require "commonmarker"
 
 module PublifyCore::TextFilter
-  class Markdown < TextFilterPlugin::Markup
+  class Markdown < TextFilterPlugin
+    include TextFilterPlugin::Markup
+
     plugin_display_name "Markdown"
     plugin_description "Markdown markup language from" \
                        ' <a href="http://daringfireball.com/">Daring Fireball</a>'

lib/publify_core/text_filter/none.rb

@@ -3,7 +3,9 @@
 require "text_filter_plugin"
 
 module PublifyCore::TextFilter
-  class None < TextFilterPlugin::Markup
+  class None < TextFilterPlugin
+    include TextFilterPlugin::Markup
+
     plugin_display_name "None"
     plugin_description "Raw HTML only"
 

lib/publify_core/text_filter/smartypants.rb

@@ -3,7 +3,9 @@
 require "rubypants"
 
 module PublifyCore::TextFilter
-  class Smartypants < TextFilterPlugin::PostProcess
+  class Smartypants < TextFilterPlugin
+    include TextFilterPlugin::PostProcess
+
     plugin_display_name "Smartypants"
     plugin_description "Converts HTML to use typographically correct quotes and dashes"
 

lib/publify_core/text_filter/twitterfilter.rb

@@ -5,7 +5,9 @@
 require "html/pipeline/hashtag/hashtag_filter"
 
 module PublifyCore::TextFilter
-  class Twitterfilter < TextFilterPlugin::PostProcess
+  class Twitterfilter < TextFilterPlugin
+    include TextFilterPlugin::PostProcess
+
     plugin_display_name "HTML Filter"
     plugin_description "Strip HTML tags"
 

lib/text_filter_plugin.rb

@@ -118,16 +118,16 @@ def self.logger
   end
 end
 
-class TextFilterPlugin::PostProcess < TextFilterPlugin
+module TextFilterPlugin::PostProcess
   def self.filter_type
     "postprocess"
   end
 end
 
-class TextFilterPlugin::Macro < TextFilterPlugin
+module TextFilterPlugin::MacroMethods
   # Utility function -- hand it a XML string like <a href="foo" title="bar">
   # and it'll give you back { "href" => "foo", "title" => "bar" }
-  def self.attributes_parse(string)
+  def attributes_parse(string)
     attributes = {}
 
     string.gsub(/([^ =]+="[^"]*")/) do |match|
@@ -143,7 +143,7 @@ def self.attributes_parse(string)
     attributes
   end
 
-  def self.filtertext(text)
+  def filtertext(text)
     regex1 = %r{<publify:#{short_name}(?:[ \t][^>]*)?/>}
     regex2 = %r{<publify:#{short_name}([ \t][^>]*)?>(.*?)</publify:#{short_name}>}m
 
@@ -157,20 +157,40 @@ def self.filtertext(text)
   end
 end
 
-class TextFilterPlugin::MacroPre < TextFilterPlugin::Macro
-  def self.filter_type
-    "macropre"
+module TextFilterPlugin::MacroPre
+  def self.included(base)
+    base.extend ClassMethods
+    base.extend TextFilterPlugin::MacroMethods
+  end
+
+  module ClassMethods
+    def filter_type
+      "macropre"
+    end
   end
 end
 
-class TextFilterPlugin::MacroPost < TextFilterPlugin::Macro
-  def self.filter_type
-    "macropost"
+module TextFilterPlugin::MacroPost
+  def self.included(base)
+    base.extend ClassMethods
+    base.extend TextFilterPlugin::MacroMethods
+  end
+
+  module ClassMethods
+    def filter_type
+      "macropost"
+    end
   end
 end
 
-class TextFilterPlugin::Markup < TextFilterPlugin
-  def self.filter_type
-    "markup"
+module TextFilterPlugin::Markup
+  def self.included(base)
+    base.extend ClassMethods
+  end
+
+  module ClassMethods
+    def filter_type
+      "markup"
+    end
   end
 end

spec/lib/text_filter_plugin_spec.rb

@@ -20,17 +20,51 @@
     end
   end
 
-  describe described_class::Macro do
-    describe "#self.attributes_parse" do
+  shared_examples "a macro module" do
+    describe ".attributes_parse" do
       it 'parses lang="ruby" to {"lang" => "ruby"}' do
-        expect(described_class.attributes_parse('<publify:code lang="ruby">'))
+        expect(derived_class.attributes_parse('<publify:code lang="ruby">'))
           .to eq("lang" => "ruby")
       end
 
       it "parses lang='ruby' to {'lang' => 'ruby'}" do
-        expect(described_class.attributes_parse("<publify:code lang='ruby'>"))
+        expect(derived_class.attributes_parse("<publify:code lang='ruby'>"))
           .to eq("lang" => "ruby")
       end
     end
   end
+
+  describe described_class::MacroPre, "when included" do
+    let(:derived_class) do
+      Class.new.tap { |klass| klass.include described_class }
+    end
+
+    it_behaves_like "a macro module"
+
+    it "declares including classes to be macropre filters" do
+      expect(derived_class.filter_type).to eq "macropre"
+    end
+  end
+
+  describe described_class::MacroPost, "when included" do
+    let(:derived_class) do
+      Class.new.tap { |klass| klass.include described_class }
+    end
+
+    it_behaves_like "a macro module"
+
+    it "declares including classes to be macropost filters" do
+      expect(derived_class.filter_type).to eq "macropost"
+    end
+  end
+
+  describe described_class::Markup, "when included" do
+    let(:derived_class) do
+      Class.new.tap { |klass| klass.include described_class }
+    end
+
+    it "declares including classes to be markup filters" do
+      expect(derived_class.filter_type).to eq "markup"
+    end
+  end
 end