markdown_views 2.1.2 → 2.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 28a2126be795fb88f32731b85c5fbdede07be1f7599497a8b9dead6d1c777050
-  data.tar.gz: 764aa9f22d74e6bccaeabbbd091821d9bdf17ec9bb94e495fa8acf8eb233ac20
+  metadata.gz: 25a0fb5b00066a055d0b4c233759673fcd610526e49f2032895fe2fddc3eaf28
+  data.tar.gz: 151a7d490fef3d447bf5e696974fbd3f74bb53264a908bf778636a2aafb877ce
 SHA512:
-  metadata.gz: 2ba620b6720c78b88c368952af15fd7fca3b04ef0c653646e803c339b9a30be5d2bc3b6b15d46496250edcd09f05a792e9ff3c994dd8b31b45116120d320858f
-  data.tar.gz: 95a238c3c16111c483b49f7e8f7651600c326626d62266b9a903adf8d2dbe3c2daa9b8f5961a42d583f2b324cee1fa790a0b7fdff57ebdb0704b8c126f8cdb68
+  metadata.gz: '009af435d553af463e18ecf712701e90bdd061658640606fa0243d9bd6207235eef7dffeacb5519730416bafc2f34272478919efb2965bfc63950686af43e9e7'
+  data.tar.gz: b7a7393c0a961f4ecb5ac4caf490209aff862cae5e7ceb2876a847394ed3b2cb905f380fbaaaf505b5e6a964949fa97ab61efd178046ade2304541e6f784668c

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2014-2019 thomas morgan
+Copyright (c) 2014-2020 thomas morgan
 
 MIT License
 

data/Rakefile

@@ -1,4 +1,5 @@
 require "bundler/gem_tasks"
+require "rake/testtask"
 
 task :generate_stylesheet do
   %w( base16.light base16.dark base16.solarized.light base16.solarized.dark
@@ -9,3 +10,12 @@ task :generate_stylesheet do
     `rougify style #{theme} --scope .rouge-highlight > app/assets/stylesheets/rouge.#{theme}.css`
   end
 end
+
+
+Rake::TestTask.new(:test) do |t|
+  t.libs << "test"
+  t.libs << "lib"
+  t.test_files = FileList["test/**/*_test.rb"]
+end
+
+task :default => :test

data/lib/markdown_views.rb

@@ -1,5 +1,6 @@
 require 'action_view'
 require 'commonmarker'
+require 'rails'
 require 'rouge'
 
 %w(config engine handler renderer version).each do |f|

data/lib/markdown_views/renderer.rb

@@ -4,9 +4,9 @@ module MarkdownViews
 
       def render(template)
         out = template.to_s
-        strip_comments!(out) if MarkdownViews.strip_comments
+        out = strip_comments(out) if MarkdownViews.strip_comments
         out = render_md(out)
-        strip_comments!(out) if MarkdownViews.strip_comments
+        out = strip_comments(out) if MarkdownViews.strip_comments
         out.html_safe
       end
 
@@ -24,8 +24,31 @@ module MarkdownViews
         MarkdownViews.rouge_opts[:formatter] || Rouge::Formatters::HTML.new
       end
 
-      def strip_comments!(input)
-        input.gsub!(/[ \t\r\n\f]*<!--(.*?)-->*/m, '')
+      # removes single & multi-line comments
+      #   if any content besides comment & whitespace is on same line(s), strips just the comment.
+      #   if no other content, strips the lines & whitespace too.
+      def strip_comments(input)
+        # ^[ \t]*(<!--.*?-->)++[ \t]*\r?\n    lines with just comments
+        # |                                   or
+        # <!--.*?-->                          comments on lines with other content
+        #
+        # ^                 start of line
+        # [ \t]*            optional spaces or tabs
+        # (<!--.*?-->)++
+        #   <!--            start of html comment
+        #   .*?             any char, incl linefeed (for multi-line comments)
+        #                     lazy (non-greedy): *?
+        #   -->             end of html comment
+        #   ++              possessive match - prevents a match across comment boundaries
+        #                     ie: prevent matching this: <!-- a --> keep <!-- b -->
+        #                     explanation: initially .*? will refuse to match --> because it's
+        #                       non-greedy. but, in search of pre/post whitespace, the regex engine
+        #                       could backtrack and ask .*? to match an --> as long as there's
+        #                       another --> later. possessive disables the backtracking.
+        #                     can combine <!-- a --><!-- b --> into one match, which is of no harm.
+        # [ \t]*            optional spaces or tabs
+        # \r?\n             end of line (either unix or windows style)
+        input.gsub(/^[ \t]*(<!--.*?-->)++[ \t]*\r?\n|<!--.*?-->/m, '')
       end
 
       def transform_code_blocks(doc)

data/lib/markdown_views/version.rb

@@ -1,3 +1,3 @@
 module MarkdownViews
-  VERSION = '2.1.2'
+  VERSION = '2.2.1'
 end

data/markdown_views.gemspec

@@ -18,10 +18,11 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_dependency 'commonmarker', '>= 0.18.2', '< 0.21'
+  spec.add_dependency 'commonmarker', '>= 0.18.2', '< 0.22'
+  spec.add_dependency 'rails', '>= 5.0', '< 6.2'
   spec.add_dependency 'rouge', '~> 3.3'
-  spec.add_dependency 'actionpack', '>= 5.0', '< 6.1'
 
   spec.add_development_dependency "bundler", "~> 1.5"
+  spec.add_development_dependency "minitest-reporters"
   spec.add_development_dependency "rake"
 end

data/test/strip_comments_test.rb

@@ -0,0 +1,67 @@
+require "test_helper"
+
+class StripCommentsTest < Minitest::Test
+
+  def test_not_greedy
+    assert_equal "ace", strip_comments("a<!-- b -->c<!-- d -->e")
+    assert_equal "a\nc\nd\nf", strip_comments("a\n<!-- b -->c\nd<!-- e -->\nf")
+  end
+
+  # only comment should vanish, without absorbing any outside whitespace
+  def test_intra_line_comments
+    assert_equal "ac", strip_comments("a<!-- b -->c"), 'no whitespace'
+    assert_equal "\nac\n", strip_comments("\na<!-- b -->c\n"), 'no whitespace, with linefeeds'
+    assert_equal "a c", strip_comments("a <!-- b -->c"), 'left whitespace'
+    assert_equal "a c", strip_comments("a<!-- b --> c"), 'right whitespace'
+    assert_equal "a  c", strip_comments("a <!-- b --> c"), 'double whitespace'
+    assert_equal "ac", strip_comments("a<!--b-->c"), 'comment w/o whitespace'
+    assert_equal "a \nc", strip_comments("a <!-- b -->\nc"), 'trailing comment'
+      # could drop trailing space, but is unnecessary
+    assert_equal "a\n c", strip_comments("a\n<!-- b --> c"), 'leading comment'
+  end
+
+  # only comment should vanish, without absorbing outside whitespace
+  def test_shared_line_multi_line_comments
+    assert_equal "14", strip_comments("1<!-- 2\n3 -->4"), 'no whitespace'
+    assert_equal "1  4", strip_comments("1 <!-- 2\n3 --> 4"), 'with whitespace'
+    assert_equal "1\n  4", strip_comments("1\n <!-- 2\n3 --> 4"), 'leading whitespace on comment'
+      #                                       ^
+      # should this absorb whitespace (preceeding line 2) on an otherwise empty line? ... debatable
+    assert_equal "1  \n4", strip_comments("1 <!-- 2\n3 --> \n4"), 'trailing whitespace on comment'
+      #                                                   ^
+      # this one too (the space following 3)?
+    assert_equal "1 \n 4", strip_comments("1 \n<!-- 2\n3 --> 4"), 'trailing whitespace on line'
+    assert_equal "1 \n 4", strip_comments("1 <!-- 2\n3 -->\n 4"), 'leading whitespace on line'
+  end
+
+  # should make line vanish, including outside whitespace
+  def test_whole_line_single_line_comments
+    assert_equal "1\n3\n", strip_comments("1\n<!-- 2 -->\n3\n"), 'adjacent lines'
+    assert_equal "1\r\n3\r\n", strip_comments("1\r\n<!-- 2 -->\r\n3\r\n"), 'adjacent lines w/windows LF'
+    assert_equal "1\n\n\n3\n", strip_comments("1\n\n<!-- 2 -->\n\n3\n"), 'spaced lines'
+    assert_equal "  1\n  3\n", strip_comments("  1\n  <!-- 2 -->\n  3\n"), 'adjacent lines indented w/spaces'
+    assert_equal "\t1\n\t3\n", strip_comments("\t1\n\t<!-- 2 -->\n\t3\n"), 'adjacent lines indented w/tabs'
+    assert_equal "  1\n\n\n  3\n", strip_comments("  1\n\n  <!-- 2 -->\n\n  3\n"), 'spaced lines indented w/spaces'
+    assert_equal "1  \n  \n4\t\n", strip_comments("1  \n  \n<!-- 2 -->  \n<!--3-->\t\n4\t\n"), 'spaced lines w/trailing whitespace'
+  end
+
+  # should make all lines vanish, including outside whitespace on same lines
+  def test_whole_line_multi_line_comments
+    assert_equal "1\n4\n", strip_comments("1\n<!-- 2\n3 -->\n4\n"), 'adjacent lines'
+    assert_equal "1\n\n\n4\n", strip_comments("1\n\n<!-- 2\n3 -->\n\n4\n"), 'spaced lines'
+    assert_equal "  1\n  4\n", strip_comments("  1\n  <!-- 2\n  3 -->\n  4\n"), 'adjacent lines indented w/spaces'
+    assert_equal "1  \n  \n4\t\n", strip_comments("1  \n  \n<!-- 2  \n3 -->\t\n4\t\n"), 'spaced lines w/trailing whitespace'
+  end
+
+
+  def strip_comments(*args)
+    MarkdownViews::Renderer.strip_comments(*args)
+  end
+
+  # hack to make diffs more understandable
+  # otherwise minitest sometimes truncates multi-lines diffs, making them unusable
+  def mu_pp(obj)
+    super obj.gsub("\n", '\n').gsub("\t", '\t')
+  end
+
+end

data/test/test_helper.rb

@@ -0,0 +1,7 @@
+require 'minitest/reporters'
+Minitest::Reporters.use! Minitest::Reporters::SpecReporter.new
+
+$LOAD_PATH.unshift File.expand_path("../../lib", __FILE__)
+require "markdown_views"
+
+require "minitest/autorun"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: markdown_views
 version: !ruby/object:Gem::Version
-  version: 2.1.2
+  version: 2.2.1
 platform: ruby
 authors:
 - thomas morgan
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-08-08 00:00:00.000000000 Z
+date: 2020-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: commonmarker
@@ -19,7 +19,7 @@ dependencies:
         version: 0.18.2
     - - "<"
       - !ruby/object:Gem::Version
-        version: '0.21'
+        version: '0.22'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -29,41 +29,41 @@ dependencies:
         version: 0.18.2
     - - "<"
       - !ruby/object:Gem::Version
-        version: '0.21'
+        version: '0.22'
 - !ruby/object:Gem::Dependency
-  name: rouge
+  name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.3'
+        version: '5.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '6.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.3'
-- !ruby/object:Gem::Dependency
-  name: actionpack
-  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '6.2'
+- !ruby/object:Gem::Dependency
+  name: rouge
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '5.0'
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '3.3'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -78,6 +78,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.5'
+- !ruby/object:Gem::Dependency
+  name: minitest-reporters
+  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'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -130,11 +144,13 @@ files:
 - lib/markdown_views/renderer.rb
 - lib/markdown_views/version.rb
 - markdown_views.gemspec
+- test/strip_comments_test.rb
+- test/test_helper.rb
 homepage: https://github.com/zarqman/markdown_views
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -149,8 +165,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.0.8
+signing_key:
 specification_version: 4
 summary: Add .md template handler to Rails.
-test_files: []
+test_files:
+- test/strip_comments_test.rb
+- test/test_helper.rb