mustermann 1.0.3 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 0f9565d31c5b29a990840eb04f933be370358de6
-  data.tar.gz: 5f4dc020bd50b6d11f787dc555e0fcab6252d486
+SHA256:
+  metadata.gz: 66586a7576074e21ef117955d2af89012bbe08d48a36ac85c42574d671350292
+  data.tar.gz: e08f66d8a4d5b33760fe0ee3c328ef6aa016207f1f248e64a79742c344d98195
 SHA512:
-  metadata.gz: 7bf3d61ccac65e24d83b5551188ae5d9836b7664dad722fd077264d9b1b6e3097bc57b474c09acc3b250c009bd7852378b3049ee5630c28706335111ec651e60
-  data.tar.gz: b4ab4965dd93dae14ddf44c60116711a488ead67e0bf1109582c79ed86bf81c70315c1b9b4b6258978093515f4813e8f01d0aeca302de883289d65ccbcf5e870
+  metadata.gz: f82b3641946356fad899d72b3ad3bdfde241030fbc25829f2e1e5a3b3f7da8cc642147d2e9021cb6bb7f88a2268360983b6cab1e07423115c3f3281282066777
+  data.tar.gz: eb6bcb7b642b03d83c3c463d72f8997755af3919dbcd2a2df212ee95c3b5cb51dc6fa0bf2cb5e5539a2ad1aa47d26c905f87d3ffcc15a925c48dcd8cf664d5a9

data/lib/mustermann/ast/compiler.rb

@@ -40,7 +40,7 @@ module Mustermann
 
         # @!visibility private
         def translate(**options)
-          return pattern(options) if options[:no_captures]
+          return pattern(**options) if options[:no_captures]
           "(?<#{name}>#{translate(no_captures: true, **options)})"
         end
 

data/lib/mustermann/ast/expander.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 require 'mustermann/ast/translator'
 require 'mustermann/ast/compiler'
+require 'ruby2_keywords'
 
 module Mustermann
   module AST
@@ -92,11 +93,11 @@ module Mustermann
       # @see Mustermann::Pattern#expand
       # @!visibility private
       def expand(values)
-        values = values.each_with_object({}){ |(key, value), new_hash|
+        adjusted = values.each_with_object({}){ |(key, value), new_hash|
           new_hash[value.instance_of?(Array) ? [key] * value.length : key] = value }
-        keys, pattern, filters = mappings.fetch(values.keys.flatten.sort) { error_for(values) }
-        filters.each { |key, filter| values[key] &&= escape(values[key], also_escape: filter) }
-        pattern % (values[keys] || values.values_at(*keys))
+        keys, pattern, filters = mappings.fetch(adjusted.keys.flatten.sort) { error_for(values) }
+        filters.each { |key, filter| adjusted[key] &&= escape(adjusted[key], also_escape: filter) }
+        pattern % (adjusted[keys] || adjusted.values_at(*keys))
       end
 
       # @see Mustermann::Pattern#expandable?
@@ -122,7 +123,7 @@ module Mustermann
 
       # @see Mustermann::AST::Translator#expand
       # @!visibility private
-      def escape(string, *args)
+      ruby2_keywords def escape(string, *args)
         # URI::Parser is pretty slow, let's not send every string to it, even if it's unnecessary
         string =~ /\A\w*\Z/ ? string : super
       end
@@ -137,7 +138,7 @@ module Mustermann
       # @!visibility private
       def add_to(list, result)
         list << [[], ""] if list.empty?
-        list.inject([]) { |l, (k1, p1, f1)| l + result.map { |k2, p2, f2| [k1+k2, p1+p2, **f1, **f2] } }
+        list.inject([]) { |l, (k1, p1, f1)| l + result.map { |k2, p2, f2| [k1+k2, p1+p2, f1.merge(f2)] } }
       end
     end
   end

data/lib/mustermann/ast/node.rb

@@ -108,7 +108,7 @@ module Mustermann
         # @see Mustermann::AST::Node#parse
         # @!visibility private
         def parse
-          self.payload ||= ""
+          self.payload ||= String.new
           super
         end
 

data/lib/mustermann/ast/parser.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 require 'mustermann/ast/node'
 require 'forwardable'
+require 'ruby2_keywords'
 require 'strscan'
 
 module Mustermann
@@ -64,7 +65,7 @@ module Mustermann
       # @param [Symbol] type node type
       # @return [Mustermann::AST::Node]
       # @!visibility private
-      def node(type, *args, &block)
+      ruby2_keywords def node(type, *args, &block)
         type  = Node[type] unless type.respond_to? :new
         start = pos
         node  = block ? type.parse(*args, &block) : type.new(*args)
@@ -153,7 +154,7 @@ module Mustermann
       def read_brackets(open, close, char: nil, escape: ?\\, quote: false, **options)
         result = String.new
         escape = false if escape.nil?
-        while current = getch
+        while (current = getch)
           case current
           when close  then return result
           when open   then result << open   << read_brackets(open, close) << close

data/lib/mustermann/ast/translator.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 require 'mustermann/ast/node'
 require 'mustermann/error'
+require 'ruby2_keywords'
 require 'delegate'
 
 module Mustermann
@@ -36,7 +37,7 @@ module Mustermann
 
         # shorthand for translating a nested object
         # @!visibility private
-        def t(*args, &block)
+        ruby2_keywords def t(*args, &block)
           return translator unless args.any?
           translator.translate(*args, &block)
         end
@@ -109,7 +110,7 @@ module Mustermann
 
       # Start the translation dance for a (sub)tree.
       # @!visibility private
-      def translate(node, *args, &block)
+      ruby2_keywords def translate(node, *args, &block)
         result = decorator_for(node).translate(*args, &block)
         result = result.node while result.is_a? NodeTranslator
         result
@@ -119,7 +120,7 @@ module Mustermann
       # @!visibility private
       def escape(char, parser: URI::DEFAULT_PARSER, escape: parser.regexp[:UNSAFE], also_escape: nil)
         escape = Regexp.union(also_escape, escape) if also_escape
-        char =~ escape ? parser.escape(char, Regexp.union(*escape)) : char
+        char.to_s =~ escape ? parser.escape(char, Regexp.union(*escape)) : char
       end
     end
   end

data/lib/mustermann/composite.rb

@@ -102,9 +102,9 @@ module Mustermann
     end
 
     # @!visibility private
-    def patterns_from(pattern, options = nil)
+    def patterns_from(pattern, **options)
       return pattern.patterns if pattern.is_a? Composite and pattern.operator == self.operator
-      [options ? Mustermann.new(pattern, **options) : pattern]
+      [options.empty? && pattern.is_a?(Pattern) ? pattern : Mustermann.new(pattern, **options)]
     end
 
     private :with_matching, :patterns_from

data/lib/mustermann/concat.rb

@@ -40,7 +40,7 @@ module Mustermann
 
     # Should not be used directly.
     # @!visibility private
-    def initialize(*)
+    def initialize(*, **)
       super
       AST::Validation.validate(combined_ast) if respond_to? :expand
     end

data/lib/mustermann/mapper.rb

@@ -40,11 +40,7 @@ module Mustermann
     #
     #   @example map before options
     #     require 'mustermann/mapper'
-    #     Mustermann::Mapper.new("/:foo" => "/:foo.html", type: :rails)
-    #
-    #   @example map after options
-    #     require 'mustermann/mapper'
-    #     Mustermann::Mapper.new(type: :rails, "/:foo" => "/:foo.html")
+    #     Mustermann::Mapper.new({"/:foo" => "/:foo.html"}, type: :rails)
     def initialize(map = {}, additional_values: :ignore, **options, &block)
       @map               = []
       @options           = options

data/lib/mustermann/pattern.rb

@@ -56,7 +56,7 @@ module Mustermann
       end
 
       @map ||= EqualityMap.new
-      @map.fetch([string, options]) { super(string, options) { options } }
+      @map.fetch([string, options]) { super(string, **options) { options } }
     end
 
     supported_options :uri_decode, :ignore_unknown_options

data/lib/mustermann/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module Mustermann
-  VERSION ||= '1.0.3'
+  VERSION ||= '1.1.2'
 end

data/lib/mustermann.rb

@@ -75,8 +75,8 @@ module Mustermann
     end
   end
 
-  @mutex ||= Mutex.new
-  @types ||= {}
+  @mutex = Mutex.new
+  @types = {}
 
   # Maps a type to its factory.
   #

data/mustermann.gemspec

@@ -14,4 +14,6 @@ Gem::Specification.new do |s|
   s.files                 = `git ls-files`.split("\n")
   s.test_files            = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.executables           = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
+
+  s.add_runtime_dependency('ruby2_keywords', '~> 0.0.1')
 end

data/spec/expander_spec.rb

@@ -64,6 +64,8 @@ describe Mustermann::Expander do
       example { expander.expand(a: ?a).should be == '/a' }
       example { expander.expand(a: ?a, b: ?b).should be == '/a' }
       example { expect { expander.expand(b: ?b) }.to raise_error(Mustermann::ExpandError) }
+      example { expect { expander.expand(b: ?b, c: []) }.to raise_error(Mustermann::ExpandError) }
+      example { expect { expander.expand(b: ?b, c: [], d: ?d) }.to raise_error(Mustermann::ExpandError) }
     end
 
     context :append do

data/spec/mapper_spec.rb

@@ -19,21 +19,14 @@ describe Mustermann::Mapper do
     end
 
     context 'accepts mappings followed by options' do
-      subject(:mapper) { Mustermann::Mapper.new("/foo" => "/bar", additional_values: :raise) }
-      its(:to_h) { should be == { Mustermann.new("/foo") => Mustermann::Expander.new("/bar") } }
-      example { mapper['/foo'].should be == '/bar' }
-      example { mapper['/fox'].should be == '/fox' }
-    end
-
-    context 'accepts options followed by mappings' do
-      subject(:mapper) { Mustermann::Mapper.new(additional_values: :raise, "/foo" => "/bar") }
+      subject(:mapper) { Mustermann::Mapper.new({ "/foo" => "/bar" }, additional_values: :raise) }
       its(:to_h) { should be == { Mustermann.new("/foo") => Mustermann::Expander.new("/bar") } }
       example { mapper['/foo'].should be == '/bar' }
       example { mapper['/fox'].should be == '/fox' }
     end
 
     context 'allows specifying type' do
-      subject(:mapper) { Mustermann::Mapper.new(additional_values: :raise, type: :rails, "/foo" => "/bar") }
+      subject(:mapper) { Mustermann::Mapper.new({ "/foo" => "/bar" }, additional_values: :raise, type: :rails) }
       its(:to_h) { should be == { Mustermann.new("/foo", type: :rails) => Mustermann::Expander.new("/bar", type: :rails) } }
       example { mapper['/foo'].should be == '/bar' }
       example { mapper['/fox'].should be == '/fox' }

metadata

@@ -1,16 +1,30 @@
 --- !ruby/object:Gem::Specification
 name: mustermann
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.1.2
 platform: ruby
 authors:
 - Konstantin Haase
 - Zachary Scott
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-08-16 00:00:00.000000000 Z
-dependencies: []
+date: 2022-07-17 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: ruby2_keywords
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.0.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.0.1
 description: A library implementing patterns that behave like regular expressions.
 email: sinatrarb@googlegroups.com
 executables: []
@@ -76,7 +90,7 @@ homepage: https://github.com/sinatra/mustermann
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -91,9 +105,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.8
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: Your personal string matching expert.
 test_files: