Merge pull request #53 from kbrock/config_require

Allow config file to specify requires for filters

Author: jedcn

features/generate-with-config.feature

@@ -94,6 +94,41 @@ Feature: The capabilities of config.yml
     transition: 'page'
     """
 
+  Scenario: Generating slides with custom filters and requires
+    Given a file named "config.yml" with:
+    """
+    filters:
+      - 'HTML::Pipeline::ItsFilter'
+    requires:
+      - './its_filter'
+    """
+    And a file named "its_filter.rb" with:
+    """
+    module HTML
+      class Pipeline
+        class ItsFilter < TextFilter
+          def call
+            # Certain people prefer to say "It is" rather than "It's"
+            @text.gsub(/It&#39;s/, 'It is')
+          end
+        end
+      end
+    end
+    """
+    And a file named "slides.md" with:
+    """
+    It's not professional.
+    """
+    When I run `reveal-ck generate`
+    Then the exit status should be 0
+    And the output should contain exactly "Generating slides for 'slides.md'..\n"
+    And the following files should exist:
+    | slides/index.html  |
+    And the file "slides/index.html" should contain:
+    """
+    <p>It is not professional.</p>
+    """
+
   Scenario: Referencing config data with a templating language
     Given a file named "config.yml" with:
     """

lib/reveal-ck/builders/create_slides_html.rb

@@ -36,6 +36,7 @@ def presentation_html
       end
 
       def apply_filters_to(html)
+        load_dependencies(config.requires)
         filters = get_classes_from_array(config.filters)
         pipeline = HTML::Pipeline.new(filters)
         filtered_html_string = FilteredHtmlString.new(html: html,
@@ -44,6 +45,10 @@ def apply_filters_to(html)
         filtered_html_string.render
       end
 
+      def load_dependencies(array_of_includes)
+        array_of_includes.each { |name| require name }
+      end
+
       def get_classes_from_array(array_of_names)
         array_of_names.map do |name|
           name.split('::').reduce(Object) { |a, e| a.const_get(e) }

lib/reveal-ck/config.rb

@@ -50,7 +50,8 @@ def filter_defaults
                       'HTML::Pipeline::MentionFilter',
                       'HTML::Pipeline::AutolinkFilter'],
         'asset_root' => 'https://assets-cdn.github.com/images/icons/',
-        'base_url' => 'https://github.com'
+        'base_url' => 'https://github.com',
+        'requires' => []
       }
     end
   end

spec/lib/reveal-ck/builders/create_slides_html_spec.rb

@@ -54,6 +54,31 @@ module Builders
           end
         end
       end
+
+      it 'requires the entries in config.filters' do
+        Dir.mktmpdir do |dir|
+          Dir.chdir(dir) do
+            slides_file_initial = 'slides-initial.md'
+            File.open(slides_file_initial, 'w') do |file|
+              file.puts('# Slides')
+            end
+
+            config = Config.new
+            config.requires = %w(json time)
+            application = Rake::Application.new
+            slides_html =
+              CreateSlidesHtml.new(slides_file: slides_file_initial,
+                                   output_dir: dir,
+                                   config: config,
+                                   application: application)
+
+            slides_html.prepare
+            expect(slides_html).to receive(:require).with('json')
+            expect(slides_html).to receive(:require).with('time')
+            application['create_slides_html'].invoke
+          end
+        end
+      end
     end
   end
 end