inversion 1.3.1 → 1.4.0

data/lib/inversion/template/publishtag.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'inversion/template' unless defined?( Inversion::Template )
@@ -31,7 +30,7 @@ class Inversion::Template::PublishTag < Inversion::Template::Tag
 	include Inversion::Template::ContainerTag
 
 
-	### Create a new PublishTag with the given +body+.
+	### Create a new PublishTag with the given `body`.
 	def initialize( body, line=nil, column=nil )
 		super
 
@@ -50,7 +49,7 @@ class Inversion::Template::PublishTag < Inversion::Template::Tag
 	attr_reader :key
 
 
-	### Render the published subnodes in the context of the given +renderstate+, but
+	### Render the published subnodes in the context of the given `renderstate`, but
 	### save them for publication after the render is done.
 	def render( renderstate )
 		self.log.debug "Publishing %d nodes as %s" % [ self.subnodes.length, self.key ]

data/lib/inversion/template/rescuetag.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'inversion/template' unless defined?( Inversion::Template )
@@ -72,7 +71,7 @@ class Inversion::Template::RescueTag < Inversion::Template::Tag
 	private
 	#######
 
-	### Parse one or more exception classes from the given +rescuespec+ and return them.
+	### Parse one or more exception classes from the given `rescuespec` and return them.
 	def parse_exception_types( rescuespec )
 		return [ ::RuntimeError ] if rescuespec.nil? || rescuespec == ''
 

data/lib/inversion/template/subscribetag.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'inversion/template' unless defined?( Inversion::Template )
@@ -29,7 +28,7 @@ require 'inversion/template/tag'
 #
 class Inversion::Template::SubscribeTag < Inversion::Template::Tag
 
-	### Create a new SubscribeTag with the given +body+.
+	### Create a new SubscribeTag with the given `body`.
 	def initialize( body, line=nil, column=nil )
 		super
 
@@ -60,7 +59,7 @@ class Inversion::Template::SubscribeTag < Inversion::Template::Tag
 	attr_reader :content
 
 
-	### Tell the +renderstate+ that this tag is interested in nodes that are published with
+	### Tell the `renderstate` that this tag is interested in nodes that are published with
 	### its key.
 	def before_rendering( renderstate )
 		@content.clear
@@ -74,7 +73,7 @@ class Inversion::Template::SubscribeTag < Inversion::Template::Tag
 	end
 
 
-	### Pub/sub callback. Called from the RenderState when a PublishTag publishes +nodes+
+	### Pub/sub callback. Called from the RenderState when a PublishTag publishes `nodes`
 	### with the same key as the current tag.
 	def publish( *nodes )
 		self.log.debug "Adding published nodes %p to %p" % [ nodes, @content ]

data/lib/inversion/template/tag.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'loggability'
@@ -105,7 +104,7 @@ class Inversion::Template::Tag < Inversion::Template::Node
 	end
 
 
-	### Safely load the specified +tagfile+.
+	### Safely load the specified `tagfile`.
 	def self::load( tagfile )
 		tagrequire = tagfile[ %r{inversion/template/\w+tag} ] or
 			raise "tag file %p doesn't look like a tag plugin" % [ tagfile ]
@@ -118,7 +117,7 @@ class Inversion::Template::Tag < Inversion::Template::Node
 	end
 
 
-	### Create a new Inversion::Template::Tag from the specified +tagname+ and +body+.
+	### Create a new Inversion::Template::Tag from the specified `tagname` and `body`.
 	def self::create( tagname, body, linenum=nil, colnum=nil )
 		tagname =~ /^(\w+)$/i or raise ArgumentError, "invalid tag name %p" % [ tagname ]
 		tagtype = $1.downcase
@@ -137,7 +136,7 @@ class Inversion::Template::Tag < Inversion::Template::Node
 	### I N S T A N C E   M E T H O D S
 	########################################################################
 
-	### Create a new Inversion::Template::Tag with the specified +body+.
+	### Create a new Inversion::Template::Tag with the specified `body`.
 	def initialize( body, linenum=nil, colnum=nil )
 		super
 		@body = body.to_s.strip

data/lib/inversion/template/textnode.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'inversion/template' unless defined?( Inversion::Template )
@@ -8,7 +7,7 @@ require 'inversion/template/node'
 # Inversion text node class -- container for static content in templates between tags.
 class Inversion::Template::TextNode < Inversion::Template::Node
 
-	### Create a new TextNode with the specified +source+.
+	### Create a new TextNode with the specified `source`.
 	def initialize( body, linenum=nil, colnum=nil )
 		@body = body
 		super

data/lib/inversion/template/timedeltatag.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'uri'
@@ -74,7 +73,7 @@ class Inversion::Template::TimeDeltaTag < Inversion::Template::AttrTag
 	private
 	#######
 
-	### Return a string describing +seconds+ as an approximate interval of time.
+	### Return a string describing `seconds` as an approximate interval of time.
 	def timeperiod( seconds )
 		return case
 			when seconds < MINUTES - 5

data/lib/inversion/template/unlesstag.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'inversion/mixins'

data/lib/inversion/template/uriencodetag.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'uri'
@@ -19,7 +18,7 @@ class Inversion::Template::UriencodeTag < Inversion::Template::AttrTag
 	include Inversion::Escaping
 
 
-	### Render the method chains against the attributes of the specified +render_state+
+	### Render the method chains against the attributes of the specified `render_state`
 	### and return them.
 	def render( render_state )
 		raw = super

data/lib/inversion/template/yieldtag.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'pathname'

data/lib/inversion/template.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'loggability'
@@ -28,23 +27,23 @@ end
 #
 # == Template Options
 #
-# Inversion supports the {Configurability}[http://rubygems.org/gems/configurability]
-# API, and registers itself with the +templates+ key. This means you can either add
-# a +templates+ section to your Configurability config, or call
+# Inversion supports the [Configurability](http://rubygems.org/gems/configurability)
+# API, and registers itself with the `templates` key. This means you can either add
+# a `templates` section to your Configurability config, or call
 # ::configure yourself with a config Hash (or something that quacks like one).
 #
 # To set options on a per-template basis, you can pass an options hash to either
 # Inversion::Template::load or Inversion::Template::new, or set them from within the template
-# itself using the {config tag}[rdoc-ref:Tags@config].
+# itself using the [config tag](rdoc-ref:Tags@config).
 #
 # The available options are:
 #
 # [:ignore_unknown_tags]
 #   Setting to false causes unknown tags used in templates to raise an
-#   Inversion::ParseError. Defaults to +true+.
+#   Inversion::ParseError. Defaults to `true`.
 #
 # [:on_render_error]
-#   Dictates the behavior of exceptions during rendering. Defaults to +:comment+.
+#   Dictates the behavior of exceptions during rendering. Defaults to `:comment`.
 #
 #   [:ignore]
 #     Exceptions are silently ignored.
@@ -55,7 +54,7 @@ end
 #
 #
 # [:debugging_comments]
-#   Insert various Inversion parse and render statements while rendering. Defaults to +false+.
+#   Insert various Inversion parse and render statements while rendering. Defaults to `false`.
 #
 # [:comment_start]
 #   When rendering debugging comments, the comment is started with these characters.
@@ -71,16 +70,16 @@ end
 #   last checked member of this. Defaults to <code>[]</code>.
 #
 # [:escape_format]
-#   The escaping used by tags such as +escape+ and +pp+. Default: +:html+.
+#   The escaping used by tags such as `escape` and `pp`. Default: `:html`.
 #
 # [:strip_tag_lines]
 #   If a tag's presence introduces a blank line into the output, this option
-#   removes it. Defaults to +true+.
+#   removes it. Defaults to `true`.
 #
 # [:stat_delay]
 #   Templates know when they've been altered on disk, and can dynamically
 #   reload themselves in long running applications.  Setting this option creates
-#   a purposeful delay between reloads for busy servers. Defaults to +0+
+#   a purposeful delay between reloads for busy servers. Defaults to `0`
 #   (disabled).
 #
 # [:strict_attributes]
@@ -111,9 +110,6 @@ class Inversion::Template
 	require 'inversion/template/tag'
 	require 'inversion/renderstate'
 
-	# Alias to maintain backward compatibility with <0.2.0 code
-	Parser = Inversion::Parser
-
 	# Valid actions for 'on_render_error'
 	VALID_ERROR_ACTIONS = [
 		:ignore,
@@ -165,7 +161,7 @@ class Inversion::Template
 	end
 
 
-	### Read a template object from the specified +path+.
+	### Read a template object from the specified `path`.
 	def self::load( path, parsestate=nil, opts={} )
 
 		# Shift the options hash over if there isn't a parse state
@@ -210,10 +206,10 @@ class Inversion::Template
 	end
 
 
-	### Add one or more extension +modules+ to Inversion::Template. This allows tags to decorate
+	### Add one or more extension `modules` to Inversion::Template. This allows tags to decorate
 	### the template class with new functionality.
 	###
-	### Each one of the given +modules+ will be included as a mixin, and if it also
+	### Each one of the given `modules` will be included as a mixin, and if it also
 	### contains a constant called ClassMethods and/or PrependedMethods, it will
 	### also be extended/prepended (respectively) with it.
 	###
@@ -260,7 +256,7 @@ class Inversion::Template
 	end
 
 
-	### Create a new Inversion:Template with the given +source+.
+	### Create a new Inversion:Template with the given `source`.
 	def initialize( source, parsestate=nil, opts={} )
 		if parsestate.is_a?( Hash )
 			# self.log.debug "Shifting template options: %p" % [ parsestate ]
@@ -325,7 +321,7 @@ class Inversion::Template
 	end
 
 
-	### Returns +true+ if the template was loaded from a file and the file's mtime
+	### Returns `true` if the template was loaded from a file and the file's mtime
 	### is after the time the template was created.
 	def changed?
 		return false unless file = self.source_file
@@ -408,7 +404,7 @@ class Inversion::Template
 	end
 
 
-	### Parse the given +source+ into the template node tree.
+	### Parse the given `source` into the template node tree.
 	def parse( source, parsestate=nil )
 		opts = self.class.config.merge( self.options )
 		parser = Inversion::Parser.new( self, opts )
@@ -442,7 +438,7 @@ class Inversion::Template
 	end
 
 
-	### Add attributes for the given +node+'s identifiers.
+	### Add attributes for the given `node`'s identifiers.
 	def add_attributes_from_node( node )
 		if node.respond_to?( :identifiers )
 			node.identifiers.each do |id|
@@ -453,7 +449,7 @@ class Inversion::Template
 	end
 
 
-	### Install reader and writer methods for the attribute associated with the specified +key+.
+	### Install reader and writer methods for the attribute associated with the specified `key`.
 	def install_accessors( key )
 		reader, writer = self.make_attribute_accessors( key )
 

data/lib/inversion/tilt.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'tilt'
@@ -36,7 +35,7 @@ if defined?( ::Tilt )
 			self.evaluate( *args )
 		end
 
-		### Tilt::Template API: render the template with the given +scope+, +locals+, and +block+.
+		### Tilt::Template API: render the template with the given `scope`, `locals`, and `block`.
 		def evaluate( scope, locals, &block )
 			@template.attributes.merge!( scope.to_h ) if scope.respond_to?( :to_h )
 			@template.attributes.merge!( locals )

data/lib/inversion.rb

@@ -1,5 +1,4 @@
 # -*- ruby -*-
-# frozen_string_literal: true
 # vim: set noet nosta sw=4 ts=4 :
 
 require 'loggability'
@@ -26,7 +25,7 @@ module Inversion
 	warn ">>> Inversion requires Ruby 2.4.0 or later. <<<" if RUBY_VERSION < '2.4.0'
 
 	# Library version constant
-	VERSION = '1.3.1'
+	VERSION = '1.4.0'
 
 	# Version-control revision constant
 	REVISION = %q$Revision$
@@ -40,8 +39,8 @@ module Inversion
 	end
 
 	require 'inversion/exceptions'
+	require 'inversion/refinements'
 	require 'inversion/mixins'
-	require 'inversion/monkeypatches'
 	require 'inversion/template'
 
 end # module Inversion

data/spec/helpers.rb

@@ -11,7 +11,7 @@ require 'loggability/spechelpers'
 
 require 'inversion'
 
-Inversion::Template::Tag.load_all
+# Inversion::Template::Tag.load_all
 
 
 ### RSpec helper functions.
@@ -21,14 +21,14 @@ module Inversion::SpecHelpers
 	module_function
 	###############
 
-	### Make an easily-comparable version vector out of +ver+ and return it.
+	### Make an easily-comparable version vector out of `ver` and return it.
 	def vvec( ver )
 		return ver.split('.').collect {|char| char.to_i }.pack('N*')
 	end
 
 
-	### Create a string containing an XML Processing Instruction with the given +name+
-	### and +data+.
+	### Create a string containing an XML Processing Instruction with the given `name`
+	### and `data`.
 	def create_pi( name, data )
 		return "<?#{name} #{data} ?>"
 	end
@@ -37,19 +37,25 @@ end
 
 
 ### Mock with RSpec
-RSpec.configure do |c|
+RSpec.configure do |config|
 
-	c.run_all_when_everything_filtered = true
-	c.filter_run :focus
-	c.order = 'random'
-	c.mock_with( :rspec ) do |mock|
+	config.mock_with( :rspec ) do |mock|
 		mock.syntax = :expect
 	end
-	c.wait_timeout = 3
-	c.example_status_persistence_file_path = 'spec/.state'
 
-	c.include( Inversion::SpecHelpers )
-	c.include( Loggability::SpecHelpers )
+	config.disable_monkey_patching!
+	config.example_status_persistence_file_path = "spec/.status"
+	config.filter_run :focus
+	config.filter_run_when_matching :focus
+	config.order = :random
+	config.profile_examples = 5
+	config.run_all_when_everything_filtered = true
+	config.shared_context_metadata_behavior = :apply_to_host_groups
+	config.wait_timeout = 3
+	# config.warnings = true
+
+	config.include( Inversion::SpecHelpers )
+	config.include( Loggability::SpecHelpers )
 end
 
 # vim: set nosta noet ts=4 sw=4:

data/spec/inversion/mixins_spec.rb

@@ -7,7 +7,7 @@ require 'set'
 require 'inversion/mixins'
 
 
-describe Inversion, "mixins" do
+RSpec.describe Inversion, "mixins" do
 
 	describe Inversion::HashUtilities do
 		it "includes a function for stringifying Hash keys" do

data/spec/inversion/monkeypatches_spec.rb

@@ -8,7 +8,7 @@ require 'ripper'
 require 'inversion/monkeypatches'
 
 
-describe Inversion, "monkeypatches" do
+RSpec.describe Inversion, "monkeypatches" do
 
 	describe Inversion::RipperAdditions do
 

data/spec/inversion/parser_spec.rb

@@ -5,7 +5,7 @@ require_relative '../helpers'
 
 require 'inversion/parser'
 
-describe Inversion::Parser do
+RSpec.describe Inversion::Parser do
 
 	before( :all ) do
 		Inversion::Template::Tag.load_all

data/spec/inversion/renderstate_spec.rb

@@ -11,7 +11,7 @@ require 'inversion/template/fragmenttag'
 require 'inversion/template/subscribetag'
 require 'inversion/template/publishtag'
 
-describe Inversion::RenderState do
+RSpec.describe Inversion::RenderState do
 
 
 	it "provides access to the block it was constructed with if there was one" do

data/spec/inversion/sinatra_spec.rb

@@ -13,7 +13,7 @@ rescue LoadError => err
 	$sinatra_support = false
 end
 
-describe "Sinatra support", :if => $sinatra_support do
+RSpec.describe "Sinatra support", :if => $sinatra_support do
 	include Rack::Test::Methods if defined?( ::Rack )
 
 	before( :each ) do

data/spec/inversion/template/attrtag_spec.rb

@@ -5,7 +5,7 @@ require_relative '../../helpers'
 
 require 'inversion/template/attrtag'
 
-describe Inversion::Template::AttrTag do
+RSpec.describe Inversion::Template::AttrTag do
 
 	describe "parsing" do
 

data/spec/inversion/template/begintag_spec.rb

@@ -11,7 +11,7 @@ require 'inversion/template/rescuetag'
 require 'inversion/template/endtag'
 require 'inversion/renderstate'
 
-describe Inversion::Template::BeginTag do
+RSpec.describe Inversion::Template::BeginTag do
 
 
 	context "without any rescue clauses" do

data/spec/inversion/template/calltag_spec.rb

@@ -5,7 +5,7 @@ require_relative '../../helpers'
 
 require 'inversion/template/calltag'
 
-describe Inversion::Template::CallTag do
+RSpec.describe Inversion::Template::CallTag do
 
 	# Since call is just an alias, the attrtag_spec already tests all of
 	# this.

data/spec/inversion/template/codetag_spec.rb

@@ -5,7 +5,7 @@ require_relative '../../helpers'
 
 require 'inversion/template/codetag'
 
-describe Inversion::Template::CodeTag do
+RSpec.describe Inversion::Template::CodeTag do
 
 
 	it "is an abstract class" do
@@ -70,4 +70,27 @@ describe Inversion::Template::CodeTag do
 			}.to raise_exception( ScriptError, /patterns already exist/i )
 		end
 	end
+
+
+	it "throws an error if trying to declare a tag pattern with an invalid character" do
+		expect {
+			Class.new( described_class ) do
+				tag_pattern "°°°" do |tag, match|
+					:foo
+				end
+			end
+		}.to raise_error( Ripper::TokenPattern::CompileError, /invalid char/i )
+	end
+
+
+	it "throws an error if given an invalid tag pattern" do
+		expect {
+			Class.new( described_class ) do
+				tag_pattern "$(ident)$(" do |tag, match|
+					:foo
+				end
+			end
+		}.to raise_error( Ripper::TokenPattern::CompileError, /unmatched paren/i )
+	end
+
 end

data/spec/inversion/template/commenttag_spec.rb

@@ -8,7 +8,7 @@ require 'inversion/template/attrtag'
 require 'inversion/template/textnode'
 require 'inversion/renderstate'
 
-describe Inversion::Template::CommentTag do
+RSpec.describe Inversion::Template::CommentTag do
 
 	it "allows any free-form text in its body" do
 		# <?comment Disabled for now ?>...<?end?>

data/spec/inversion/template/configtag_spec.rb

@@ -5,7 +5,7 @@ require_relative '../../helpers'
 
 require 'inversion/template/configtag'
 
-describe Inversion::Template::ConfigTag do
+RSpec.describe Inversion::Template::ConfigTag do
 
 	# <?config comment_start: /* ?>
 	# <?config comment_end: */ ?>

data/spec/inversion/template/containertag_spec.rb

@@ -5,7 +5,7 @@ require_relative '../../helpers'
 
 require 'inversion/template/containertag'
 
-describe Inversion::Template::ContainerTag do
+RSpec.describe Inversion::Template::ContainerTag do
 
 	context "instances of including classes" do
 

data/spec/inversion/template/defaulttag_spec.rb

@@ -5,7 +5,7 @@ require_relative '../../helpers'
 
 require 'inversion/template/defaulttag'
 
-describe Inversion::Template::DefaultTag do
+RSpec.describe Inversion::Template::DefaultTag do
 
 
 	it "sets a template attribute to a default value" do

data/spec/inversion/template/elsetag_spec.rb

@@ -10,7 +10,7 @@ require 'inversion/template/commenttag'
 require 'inversion/template/fortag'
 require 'inversion/renderstate'
 
-describe Inversion::Template::ElseTag do
+RSpec.describe Inversion::Template::ElseTag do
 
 	it "handles a non-existant body" do
 		Inversion::Template::ElseTag.new( nil )

data/spec/inversion/template/elsiftag_spec.rb

@@ -9,7 +9,7 @@ require 'inversion/template/unlesstag'
 require 'inversion/template/textnode'
 require 'inversion/renderstate'
 
-describe Inversion::Template::ElsifTag do
+RSpec.describe Inversion::Template::ElsifTag do
 
 
 	it "can be appended to an 'if' tag" do

data/spec/inversion/template/endtag_spec.rb

@@ -8,7 +8,7 @@ require 'inversion/template/textnode'
 require 'inversion/template/endtag'
 require 'inversion/renderstate'
 
-describe Inversion::Template::EndTag do
+RSpec.describe Inversion::Template::EndTag do
 
 	before( :each ) do
 		@tag = Inversion::Template::EndTag.new

data/spec/inversion/template/escapetag_spec.rb

@@ -5,7 +5,7 @@ require_relative '../../helpers'
 
 require 'inversion/template/escapetag'
 
-describe Inversion::Template::EscapeTag do
+RSpec.describe Inversion::Template::EscapeTag do
 
 	before( :each ) do
 		@attribute_object = double( "template attribute" )

data/spec/inversion/template/fortag_spec.rb

@@ -9,7 +9,7 @@ require 'inversion/template/attrtag'
 require 'inversion/template/textnode'
 require 'inversion/renderstate'
 
-describe Inversion::Template::ForTag do
+RSpec.describe Inversion::Template::ForTag do
 
 	it "knows which identifiers should be added to the template" do
 		tag = Inversion::Template::ForTag.new( 'foo in bar' )

data/spec/inversion/template/fragmenttag_spec.rb

@@ -8,7 +8,7 @@ require 'inversion/template/attrtag'
 require 'inversion/template/textnode'
 require 'inversion/renderstate'
 
-describe Inversion::Template::FragmentTag do
+RSpec.describe Inversion::Template::FragmentTag do
 
 	it "raises a parse error if the body isn't a simple attribute" do
 		expect {

data/spec/inversion/template/iftag_spec.rb

@@ -7,7 +7,7 @@ require 'inversion/template/iftag'
 require 'inversion/template/textnode'
 require 'inversion/renderstate'
 
-describe Inversion::Template::IfTag do
+RSpec.describe Inversion::Template::IfTag do
 
 
 	it "renders its contents if its attribute is true" do

data/spec/inversion/template/importtag_spec.rb

@@ -5,7 +5,7 @@ require_relative '../../helpers'
 
 require 'inversion/template/importtag'
 
-describe Inversion::Template::ImportTag do
+RSpec.describe Inversion::Template::ImportTag do
 
 	it "can import a single attribute" do
 		tag = Inversion::Template::ImportTag.new( 'txn ' )

data/spec/inversion/template/includetag_spec.rb

@@ -7,7 +7,7 @@ require 'timeout'
 require 'inversion/template'
 require 'inversion/template/includetag'
 
-describe Inversion::Template::IncludeTag do
+RSpec.describe Inversion::Template::IncludeTag do
 
 	it "renders the IncludeTag as an empty string" do
 		included_path = Pathname.pwd + 'included.tmpl'