Merge pull request #5 from DerekStride/caching

Cache converted content to prevent slow livereload

Author: DerekStride

.gitignore

@@ -6,3 +6,4 @@
 /pkg/
 /spec/reports/
 /tmp/
+Gemfile.lock

Gemfile.lock

@@ -2,6 +2,6 @@
 
 module Jekyll
   module Graphviz
-    VERSION = "0.1.3"
+    VERSION = "0.1.4"
   end
 end

lib/jekyll-graphviz/version.rb

@@ -6,9 +6,12 @@ class Graphviz < Converter
       safe true
       priority :low
 
+      attr_accessor :cache
+
       def setup
         require 'ruby-graphviz'
         @format = (@config.dig("graphviz", "format") || :svg).to_sym
+        @cache = {}
         @setup = true
       end
 
@@ -23,7 +26,7 @@ def output_ext(ext)
 
       def convert(content)
         setup unless @setup
-        GraphViz.parse_string(content).output(@format => String)
+        @cache[content] ||= GraphViz.parse_string(content).output(@format => String)
       end
     end
   end

lib/jekyll/converters/graphviz.rb

@@ -31,6 +31,12 @@ def test_output_to_another_output_type
     assert_equal(Encoding::ASCII_8BIT, output.encoding)
   end
 
+  def test_files_are_cached
+    content = "digraph x { a -> b }"
+    converter.convert(content)
+    assert(converter.cache.key?(content))
+  end
+
   def converter(opts = {})
     config = Jekyll::Configuration::DEFAULTS.merge(opts)
     @converter ||= Jekyll::Converters::Graphviz.new(config)