jekyll_plugin_support 3.0.0 → 3.1.1

data/lib/jekyll_all_collections/all_collections_tag.rb

@@ -1,4 +1,3 @@
-# require 'jekyll_draft'
 require 'securerandom'
 
 # @author Copyright 2020 Michael Slinn
@@ -15,17 +14,18 @@ module JekyllAllCollections
     # @return [String]
     def render_impl
       parse_arguments # Defines instance variables like @sort_by
-      sort_lambda = init_sort_by @sort_by, @sort_by_param
-      @heading = @helper.parameter_specified?('heading') || default_head(@sort_by)
+      sort_lambda = init_sort_by @sort_by
       generate_output sort_lambda
     rescue StandardError => e
-      JekyllSupport.error_short_trace @logger, e
-      # JekyllSupport.warn_short_trace @logger, e
+      ::JekyllSupport.error_short_trace @logger, e
+      # ::JekyllSupport.warn_short_trace @logger, e
     end
 
-    # Descending sort keys reverse the order of comparison
-    # Example return values:
-    #   "->(a, b) { [a.last_modified] <=> [b.last_modified] }"
+    # Descending sort keys are preceded by a minus sign, and reverse the order of comparison
+    # @param criteria String Examples: 'date', '-date', 'last_modified', '-last_modified',
+    #        ['date', 'last_modified], ['-date', '-last_modified'], ['date', '-last_modified']
+    # @return values:
+    #   "->(a, b) { [a.last_modified] <=> [b.last_modified] }" (ascending)
     #   "->(a, b) { [b.last_modified] <=> [a.last_modified] }" (descending)
     #   "->(a, b) { [a.last_modified, a.date] <=> [b.last_modified, b.date] }" (descending last_modified, ascending date)
     #   "->(a, b) { [a.last_modified, b.date] <=> [b.last_modified, a.date] }" (ascending last_modified, descending date)
@@ -76,43 +76,45 @@ module JekyllAllCollections
       warn_short_trace e.red
     end
 
-    def last_modified_value(apage)
-      @logger.debug do
-        "  apage.last_modified='#{apage.last_modified}'; " \
-          "apage.last_modified_at='#{apage.last_modified_at}'; " \
-          "@date_column='#{@date_column}'"
+    def date_value(apage, field_name)
+      if %i[last_modified last_modified_at].include? field_name
+        apage.field(:last_modified_at, use_default: false) ||
+          apage.field(:last_modified, use_default: false) ||
+          Date.today
+      else
+        apage.date || Time.now
       end
-      last_modified = if @date_column == 'last_modified' && apage.respond_to?(:last_modified)
-                        apage.last_modified
-                      elsif apage.respond_to? :last_modified_at
-                        apage.last_modified_at
-                      else
-                        apage.date
-                      end
-      last_modified ||= apage.date || Date.today
-      last_modified
     end
 
     def generate_output(sort_lambda)
-      id = @id.to_s.strip.empty? ? '' : " id='#{@id}'"
+      id = @id.to_s.strip.empty? ? '' : " id=\"#{@id}\""
       heading = @heading.strip.to_s.empty? ? '' : "<h2#{id}>#{@heading}</h2>"
-      data = case @data_selector
-             when 'all_collections'
-               @site.all_collections
-             when 'all_documents'
-               @site.all_documents
-             when 'everything'
-               @site.everything
-             else
-               raise AllCollectionsError, "Invalid value for @data_selector (#{data_selector})"
-             end
-      collection = data.sort(&sort_lambda)
-      posts = collection.map do |x|
-        last_modified = last_modified_value x
-        date = last_modified.strftime '%Y-%m-%d'
-        draft = x.draft ? DRAFT_HTML : ''
-        href = "<a href='#{x.href}'>#{x.title}</a>"
-        @logger.debug { "  date='#{date}' #{x.title}\n" }
+      apages = case @data_selector
+               when 'all_collections'
+                 @site.all_collections
+               when 'all_documents'
+                 @site.all_documents
+               when 'everything'
+                 @site.everything
+               else
+                 raise AllCollectionsError, "Invalid value for @data_selector (#{data_selector})"
+               end
+      filtered_apages = @collection_name.nil? ? apages : apages.select { |apage| apage.collection_name == @collection_name }
+      sorted_apages = filtered_apages.sort(&sort_lambda)
+      posts = sorted_apages.map do |apage|
+        date_column = @date_column.to_s == 'last_modified' ? :last_modified : :date
+        d = date_value(apage, date_column)
+        if d.respond_to?(:strftime)
+          date = d.strftime '%Y-%m-%d'
+        else
+          @logger.error do
+            "date_value returned a #{d.class} instead of a class with a strftime method like Date and Time; date_column=#{date_column}"
+          end
+        end
+        draft         = apage.draft ? DRAFT_HTML : ''
+        title         = apage.title || apage.href
+        href          = "<a href='#{apage.href}'>#{title}</a>"
+        @logger.debug { "  date='#{date}' #{title}\n" }
         "  <span>#{date}</span><span>#{href}#{draft}</span>"
       end
       <<~END_TEXT
@@ -121,35 +123,50 @@ module JekyllAllCollections
           #{posts.join "\n"}
         </div>
       END_TEXT
-    rescue ArgumentError => e
-      warn_short_trace e
+    rescue NoMethodError || ArgumentError => e
+      ::JekyllSupport.error_short_trace @logger, e
     end
 
     # See https://stackoverflow.com/a/75377832/553865
-    def init_sort_by(sort_by, sort_by_param)
+    def init_sort_by(sort_by)
       sort_lambda_string = AllCollectionsTag.create_lambda_string sort_by
 
       @logger.debug do
-        "#{@page['path']} sort_by_param=#{sort_by_param}  " \
-          "sort_lambda_string = #{sort_lambda_string}\n"
+        "#{@page['path']} sort_lambda_string = #{sort_lambda_string}\n"
       end
 
       evaluate sort_lambda_string
     end
 
+    # @return String defining the parsed sort_by expression
     def parse_arguments
+      @collection_name = @helper.parameter_specified?('collection_name')
       @data_selector = @helper.parameter_specified?('data_selector') || 'all_collections'
       abort "Invalid data_selector #{@data_selector}" unless %w[all_collections all_documents everything].include? @data_selector
+      if (@data_selector != 'all_collections') && @collection_name
+        @logger.warn do
+          "collection_name was specified as '#{@collection_name}', but data_selector is #{@data_selector},
+          which is less effcient than specifying all_collections."
+        end
+      end
+
+      sort_by_param = @helper.parameter_specified? 'sort_by' # Might specify multiple sort fields
 
-      @date_column = @helper.parameter_specified?('date_column') || 'date'
+      # Default to displaying last modified field unless a sort field is specified
+      @date_column = @helper.parameter_specified?('date_column') || 'last_modified'
       unless %w[date last_modified].include?(@date_column)
         raise AllCollectionsError "The date_column attribute must either have value 'date' or 'last_modified', " \
-                                  "but '#{@date_column}' was specified"
+                                  "but '#{@date_column}' was specified instead."
       end
+      @date_column ||= (sort_by_param.include?('last_modified') ? 'last_modified' : 'date') # display the sort date by default
 
       @id = @helper.parameter_specified?('id') || SecureRandom.hex(10)
-      @sort_by_param = @helper.parameter_specified? 'sort_by'
-      @sort_by = (@sort_by_param&.delete(' ')&.split(',') if @sort_by_param != false) || ['-date']
+
+      @sort_by = (sort_by_param&.delete(' ')&.split(',') if sort_by_param != false) || ['-date']
+
+      @heading = @helper.parameter_specified?('heading') || default_head(@sort_by)
+
+      @sort_by
     end
 
     ::JekyllSupport::JekyllPluginHelper.register(self, PLUGIN_NAME)

data/lib/jekyll_plugin_support/jekyll_plugin_support_class.rb

@@ -1,6 +1,7 @@
 # Monkey patch StandardError so a new method called shorten_backtrace is added.
 class StandardError
-  def shorten_backtrace(backtrace_element_count = 3)
+  def shorten_backtrace(backtrace_element_count = 5)
+    set_backtrace backtrace[0..backtrace_element_count]
     # self.backtrace = backtrace[0..backtrace_element_count].map do |x|
     #   raise JekyllPluginSupportError, "backtrace contains a #{x.class} with value '#{x}'." unless x.instance_of? String
 
@@ -14,7 +15,7 @@ module JekyllSupport
 
   def self.error_short_trace(logger, error)
     error.set_backtrace error.backtrace[0..DISPLAYED_CALLS]
-    logger.error { error }
+    logger.error { error.full_message } # Are error and logger.error defined?
     error
   end
 

data/lib/jekyll_plugin_support/version.rb

@@ -1,3 +1,3 @@
 module JekyllPluginSupportVersion
-  VERSION = '3.0.0'.freeze unless defined?(VERSION)
+  VERSION = '3.1.1'.freeze unless defined?(VERSION)
 end

data/lib/jekyll_plugin_support.rb

@@ -28,6 +28,12 @@ module JekyllSupport
   end
 end
 
+module ToString
+  def to_s
+    "#{self}.class.name"
+  end
+end
+
 module NoArgParsing
   attr_accessor :no_arg_parsing
 
@@ -37,17 +43,21 @@ end
 module JekyllSupport
   class JekyllTag
     include JekyllSupportError
+    include ToString
   end
 
   class JekyllTagNoArgParsing
     include JekyllSupportError
+    include ToString
   end
 
   class JekyllBlock
     include JekyllSupportError
+    include ToString
   end
 
   class JekyllBlockNoArgParsing
     include JekyllSupportError
+    include ToString
   end
 end

data/spec/all_collections_tag/all_collections_tag_sort_spec.rb

@@ -1,112 +1,184 @@
-require 'spec_helper'
-require_relative '../../lib/jekyll_all_collections'
+require_relative '../spec_helper'
+require_relative '../../lib/hooks/a_page'
 
-class APageStub
-  attr_reader :date, :last_modified, :label
+class NullBinding < BasicObject
+  include ::Kernel
 
-  def initialize(date, last_modified, label = '')
-    @date = Date.parse date
-    @last_modified = Date.parse last_modified
-    @label = label
-  end
+  # Avoid error message "warning: undefining `object_id' may cause serious problems"
+  # https://stackoverflow.com/a/17791631/553865
+  (
+    ::Kernel.instance_methods(false) +
+    ::Kernel.private_instance_methods(false) -
+    [:binding]
+  ).each { |x| undef_method(x) unless x == :object_id }
 
-  def to_s
-    @label
+  def min_binding
+    binding
   end
 end
 
-def show(lambda_string, result, expected)
-  p "For lambda_string: #{lambda_string}"
-  p "  result: #{result.map(&:label).join(', ')}  <==>  expected: #{expected.map(&:label).join(', ')}"
+def show_dates(label, array)
+  puts "  #{label} actual: #{array.map(&:title).join(', ')}  <==>  expected: #{expected.map(&:title).join(', ')}"
+end
+
+def show(label, lambda_string, actual, expected)
+  puts "#{label} - For lambda_string: #{lambda_string}"
+  puts "  actual: #{actual.map(&:title).join(', ')}"
+  puts "expected: #{expected.map(&:title).join(', ')}"
+  actual_array = actual.map do |x|
+    [
+      (x.date.strftime '%Y-%m-%d' + '/' + x.title), (x.last_modified.strftime '%Y-%m-%d' + '/' + x.title)
+    ].join('/')
+  end
+  expected_array = expected.map do |x|
+    [
+      (x.date.strftime '%Y-%m-%d' + '/' + x.title), (x.last_modified.strftime '%Y-%m-%d' + '/' + x.title)
+    ].join('/')
+  end
+  puts '  actual date/last_modified: ' + actual_array.join(', ')
+  puts 'expected date/last_modified: ' + expected_array.join(', ')
 end
 
+logger = PluginMetaLogger.instance.new_logger(self, PluginMetaLogger.instance.config)
+
 # See https://stackoverflow.com/a/75388137/553865
-RSpec.describe(AllCollectionsTag::AllCollectionsTag) do
-  let(:o1) { APageStub.new('2020-01-01', '2020-01-01', 'a_A') }
-  let(:o2) { APageStub.new('2021-01-01', '2020-01-01', 'b_A') }
-  let(:o3) { APageStub.new('2021-01-01', '2023-01-01', 'b_B') }
-  let(:o4) { APageStub.new('2022-01-01', '2023-01-01', 'c_B') }
+RSpec.describe(JekyllSupport) do
+  let(:o1) do
+    described_class.apage_from(
+      collection_name: '_posts',
+      date:            '2020-01-20',
+      last_modified:   '2020-01-20',
+      logger:          logger,
+      title:           'a_A (o1)'
+    )
+  end
+  let(:o2) do
+    described_class.apage_from(
+      collection_name: '_posts',
+      date:            '2021-01-21',
+      last_modified:   '2021-01-21',
+      logger:          logger,
+      title:           'b_B (o2)'
+    )
+  end
+  let(:o3) do
+    described_class.apage_from(
+      collection_name: '_posts',
+      date:            '2021-01-21',
+      last_modified:   '2022-01-22',
+      logger:          logger,
+      title:           'b_C (o3)'
+    )
+  end
+  let(:o4) do
+    described_class.apage_from(
+      collection_name: '_posts',
+      date:            '2022-01-22',
+      last_modified:   '2022-01-22',
+      logger:          logger,
+      title:           'c_C (o4)'
+    )
+  end
   let(:objs) { [o1, o2, o3, o4] }
 
-  it 'defines sort_by lambda with last_modified' do
+  it '(1) defines sort_by lambda with last_modified (ascending)' do
     sort_lambda = ->(a, b) { [a.last_modified] <=> [b.last_modified] }
-    result = objs.sort(&sort_lambda)
-    expect(result).to eq([o1, o2, o3, o4])
+    actual = objs.sort(&sort_lambda)
+    expected = [o1, o2, o3, o4]
+    show('(1)', '[a.last_modified] <=> [b.last_modified]', actual, expected)
+    expect(actual).to eq(expected)
   end
 
-  it 'makes sort_by lambdas from stringified date' do
-    sort_lambda = eval '->(a, b) { a.last_modified <=> b.last_modified }',
-                       NullBinding.new.min_binding, __FILE__, __LINE__ - 1
-    result = objs.sort(&sort_lambda)
-    expect(result).to eq([o1, o2, o3, o4])
+  it '(2) makes sort_by lambdas from stringified comparison of last_modified (ascending)' do
+    sort_lambda_string = '->(a, b) { a.last_modified <=> b.last_modified }'
+    sort_lambda = eval sort_lambda_string, NullBinding.new.min_binding, __FILE__, __LINE__ - 1
+    actual = objs.sort(&sort_lambda)
+    expected = [o1, o2, o3, o4]
+    show('(2)', sort_lambda_string, actual, expected)
+    expect(actual).to eq(expected)
   end
 
-  it 'makes sort_by lambdas from stringified array of last_modified' do
-    sort_lambda = eval '->(a, b) { [a.last_modified] <=> [b.last_modified] }',
-                       NullBinding.new.min_binding, __FILE__, __LINE__ - 1
-    result = objs.sort(&sort_lambda)
-    expect(result).to eq([o1, o2, o3, o4])
+  it '(3) makes sort_by lambda from stringified array of last_modified (ascending)' do
+    sort_lambda_string = '->(a, b) { [a.last_modified] <=> [b.last_modified] }'
+    sort_lambda = eval sort_lambda_string, NullBinding.new.min_binding, __FILE__, __LINE__ - 1
+    actual = objs.sort(&sort_lambda)
+    expected = [o1, o2, o3, o4]
+    show('(3)', sort_lambda_string, actual, expected)
+    expect(actual).to eq(expected)
   end
 
-  it 'makes sort_by lambdas with descending keys from stringified array of last_modified' do
-    sort_lambda = eval '->(a, b) { [b.last_modified] <=> [a.last_modified] }',
-                       NullBinding.new.min_binding, __FILE__, __LINE__ - 1
-    result = objs.sort(&sort_lambda)
-    expected = [o3, o4, o1, o2]
-    expect(result).to eq(expected)
+  it '(4) makes sort_by lambda with last_modified (descending) from stringified array' do
+    sort_lambda_string = '->(a, b) { [b.last_modified] <=> [a.last_modified] }'
+    sort_lambda = eval sort_lambda_string, NullBinding.new.min_binding, __FILE__, __LINE__ - 1
+    actual = objs.sort(&sort_lambda)
+    expected = [o4, o3, o2, o1]
+    show('(4)', sort_lambda_string, actual, expected)
+    expect([o3, o4]).to include(actual[0]) # The sort might yield o3 or o4 in this position
+    expect([o3, o4]).to include(actual[1]) # The sort might yield o3 or o4 in this position
+    expect(o2).to eq(actual[2])
+    expect(o1).to eq(actual[3])
   end
 
-  it 'create_lambda with 1 date key, descending' do
-    lambda_string = described_class.create_lambda_string('-last_modified')
-    sort_lambda = described_class.evaluate(lambda_string)
-    result = objs.sort(&sort_lambda)
-    expected = [o3, o4, o1, o2]
-    # show(lambda_string, result, expected)
-    expect(result).to eq(expected)
+  it '(5) create_lambda with date (descending)' do
+    lambda_string = JekyllAllCollections::AllCollectionsTag.create_lambda_string('-last_modified')
+    expect(lambda_string).to eq('->(a, b) { [b.last_modified] <=> [a.last_modified] }')
+    sort_lambda = self.eval lambda_string, binding
+    actual = objs.sort(&sort_lambda)
+    expected = [o4, o3, o2, o1]
+    show('(5)', lambda_string, actual, expected)
+    expect([o3, o4]).to include(actual[0]) # The sort might yield o3 or o4 in this position
+    expect([o3, o4]).to include(actual[1]) # The sort might yield o3 or o4 in this position
+    expect(o2).to eq(actual[2])
+    expect(o1).to eq(actual[3])
   end
 
-  it 'create_lambda with 1 date key, ascending' do
-    lambda_string = described_class.create_lambda_string('date')
-    sort_lambda = described_class.evaluate(lambda_string)
-    result = objs.sort(&sort_lambda)
+  it '(6) create_lambda with date (ascending)' do
+    lambda_string = JekyllAllCollections::AllCollectionsTag.create_lambda_string('date')
+    expect(lambda_string).to eq('->(a, b) { [a.date] <=> [b.date] }')
+    sort_lambda = self.eval lambda_string, binding
+    actual = objs.sort(&sort_lambda)
     expected = [o1, o2, o3, o4]
-    # show(lambda_string, result, expected)
-    expect(result).to eq(expected)
+    show('(1)', lambda_string, actual, expected)
+    expect(actual).to eq(expected)
   end
 
-  it 'create_lambda with 2 date keys, both ascending' do
-    lambda_string = described_class.create_lambda_string(%w[date last_modified])
-    sort_lambda = described_class.evaluate(lambda_string)
-    result = objs.sort(&sort_lambda)
+  it '(7) create_lambda with date (ascending) and last_modified (ascending)' do
+    lambda_string = JekyllAllCollections::AllCollectionsTag.create_lambda_string(%w[date last_modified])
+    expect(lambda_string).to eq('->(a, b) { [a.date, a.last_modified] <=> [b.date, b.last_modified] }')
+    sort_lambda = self.eval lambda_string, binding
+    actual = objs.sort(&sort_lambda)
     expected = [o1, o2, o3, o4]
-    # show(lambda_string, result, expected)
-    expect(result).to eq(expected)
+    show('(7)', lambda_string, actual, expected)
+    expect(actual).to eq(expected)
   end
 
-  it 'create_lambda with 2 date keys, both descending' do
-    lambda_string = described_class.create_lambda_string(['-date', '-last_modified'])
-    sort_lambda = described_class.evaluate(lambda_string)
-    result = objs.sort(&sort_lambda)
+  it '(8) create_lambda with date (descending) and last_modified (descending)' do
+    lambda_string = JekyllAllCollections::AllCollectionsTag.create_lambda_string(['-date', '-last_modified'])
+    expect(lambda_string).to eq('->(a, b) { [b.date, b.last_modified] <=> [a.date, a.last_modified] }')
+    sort_lambda = self.eval lambda_string, binding
+    actual = objs.sort(&sort_lambda)
     expected = [o4, o3, o2, o1]
-    # show(lambda_string, result, expected)
-    expect(result).to eq(expected)
+    show('(8)', lambda_string, actual, expected)
+    expect(actual).to eq(expected)
   end
 
-  it 'create_lambda with 2 date keys, first descending and second ascending' do
-    lambda_string = described_class.create_lambda_string(['-date', 'last_modified'])
-    sort_lambda = described_class.evaluate(lambda_string)
-    result = objs.sort(&sort_lambda)
+  it '(9) create_lambda with date (descending) and last_modified (ascending)' do
+    lambda_string = JekyllAllCollections::AllCollectionsTag.create_lambda_string(['-date', 'last_modified'])
+    expect(lambda_string).to eq('->(a, b) { [b.date, a.last_modified] <=> [a.date, b.last_modified] }')
+    sort_lambda = self.eval lambda_string, binding
+    actual = objs.sort(&sort_lambda)
     expected = [o4, o2, o3, o1]
-    # show(lambda_string, result, expected)
-    expect(result).to eq(expected)
+    show('(9)', lambda_string, actual, expected)
+    expect(actual).to eq(expected)
   end
 
-  it 'create_lambda with 2 date keys, first ascending and second descending' do
-    lambda_string = described_class.create_lambda_string(['date', '-last_modified'])
-    sort_lambda = described_class.evaluate(lambda_string)
-    result = objs.sort(&sort_lambda)
+  it '(10) create_lambda with date (ascending) and last_modified (descending)' do
+    lambda_string = JekyllAllCollections::AllCollectionsTag.create_lambda_string(['date', '-last_modified'])
+    expect(lambda_string).to eq('->(a, b) { [a.date, b.last_modified] <=> [b.date, a.last_modified] }')
+    sort_lambda = self.eval lambda_string, binding
+    actual = objs.sort(&sort_lambda)
     expected = [o1, o3, o2, o4]
-    # show(lambda_string, result, expected)
-    expect(result).to eq(expected)
+    show('(10)', lambda_string, actual, expected)
+    expect(actual).to eq(expected)
   end
 end

data/spec/custom_error_spec.rb

@@ -1,5 +1,7 @@
+require 'spec_helper'
 require_relative '../lib/error/jekyll_custom_error'
 require_relative '../lib/jekyll_plugin_support/jekyll_plugin_support_class'
+require_relative '../lib/helper/jekyll_plugin_helper'
 
 class Dummy
   def just_for_testing; end
@@ -14,14 +16,14 @@ class CustomErrorSpec
 
   puts "AnError is a #{AnError.class}; StandardError is a #{StandardError.class}"
   begin
-    raise AnError, 'Oops'
+    raise AnError, 'This error is expected'
   rescue AnError => e
     puts "Caught AnError: #{e.message}"
   rescue ::JekyllSupport::CustomError => e
     puts "Caught CustomError: #{e.message}"
   end
 
-  RSpec.describe JekyllSupport::JekyllPluginHelper do
+  RSpec.describe ::JekyllSupport::JekyllPluginHelper do
     it 'generates messages' do
       msg = described_class.generate_message(Dummy, tag_name, '0.1.0')
       puts msg

data/spec/jekyll_plugin_helper_options_spec.rb

@@ -1,4 +1,6 @@
+require 'jekyll'
 require 'jekyll_plugin_logger'
+require 'spec_helper'
 # require 'rspec/match_ignoring_whitespace'
 require_relative '../lib/jekyll_plugin_support'
 require_relative '../lib/jekyll_plugin_support/jekyll_plugin_support_spec_support'

data/spec/liquid_variable_parsing_spec.rb

@@ -1,4 +1,5 @@
 require 'jekyll_plugin_logger'
+require 'spec_helper'
 # require 'rspec/match_ignoring_whitespace'
 require_relative '../lib/jekyll_plugin_support'
 require_relative '../lib/jekyll_plugin_support/jekyll_plugin_support_spec_support'

data/spec/spec_helper.rb

@@ -4,9 +4,19 @@ require_relative '../lib/jekyll_plugin_support'
 Jekyll.logger.log_level = :info
 
 RSpec.configure do |config|
-  config.filter_run_when_matching focus: true
-  # config.order = 'random'
-
   # See https://relishapp.com/rspec/rspec-core/docs/command-line/only-failures
   config.example_status_persistence_file_path = 'spec/status_persistence.txt'
+
+  # See https://rspec.info/features/3-12/rspec-core/filtering/filter-run-when-matching/
+  # and https://github.com/rspec/rspec/issues/221
+  config.filter_run_when_matching :focus
+
+  # Other values: :progress, :html, :json, CustomFormatterClass
+  config.formatter = :documentation
+
+  # See https://rspec.info/features/3-12/rspec-core/command-line/order/
+  config.order = :defined
+
+  # See https://www.rubydoc.info/github/rspec/rspec-core/RSpec%2FCore%2FConfiguration:pending_failure_output
+  config.pending_failure_output = :skip
 end

data/spec/status_persistence.txt

@@ -1,3 +1,9 @@
-example_id                                      | status | run_time        |
------------------------------------------------ | ------ | --------------- |
-./spec/jekyll_block_plugin_support_spec.rb[1:1] | passed | 0.00062 seconds |
+example_id                                       | status | run_time        |
+------------------------------------------------ | ------ | --------------- |
+./spec/custom_error_spec.rb[1:1]                 | failed | 0.00772 seconds |
+./spec/custom_error_spec.rb[2:1]                 | passed | 0.0011 seconds  |
+./spec/jekyll_plugin_helper_options_spec.rb[1:1] | failed | 0.00005 seconds |
+./spec/jekyll_plugin_helper_options_spec.rb[1:2] | failed | 0.00003 seconds |
+./spec/jekyll_plugin_helper_options_spec.rb[1:3] | failed | 0.00003 seconds |
+./spec/jekyll_plugin_helper_options_spec.rb[1:4] | failed | 0.00003 seconds |
+./spec/liquid_variable_parsing_spec.rb[1:1]      | failed | 0.00003 seconds |

data/spec/testable_spec.rb

@@ -9,6 +9,15 @@ module TestModule
   end
 end
 
+RSpec.describe(TestModule) do
+  extend described_class
+
+  it 'Invokes a_method from module' do
+    result = self.class.a_method
+    expect(result).to eq('a_method says Hi!')
+  end
+end
+
 class TestClass
   extend TestModule # Defines class methods
 
@@ -19,15 +28,6 @@ class TestClass
   end
 end
 
-RSpec.describe(TestModule) do
-  extend described_class
-
-  it 'Invokes a_method from module' do
-    result = self.class.a_method
-    expect(result).to eq('a_method says Hi!')
-  end
-end
-
 RSpec.describe(TestClass) do
   let(:o1) { described_class.new('value1', 'value2') }
 

metadata

@@ -1,15 +1,28 @@
 --- !ruby/object:Gem::Specification
 name: jekyll_plugin_support
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.1.1
 platform: ruby
 authors:
 - Mike Slinn
-autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-03-06 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: bigdecimal
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: facets
   requirement: !ruby/object:Gem::Requirement
@@ -94,7 +107,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description:
 email:
 - mslinn@mslinn.com
 executables: []
@@ -169,8 +181,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.22
-signing_key:
+rubygems_version: 3.7.2
 specification_version: 4
 summary: Provides a framework for writing and testing Jekyll plugins
 test_files: