yawpa 1.1.0 → 1.2.0

checksums.yaml

@@ -0,0 +1,15 @@
+---
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    Y2ViMTM4ZWVlZDBjNTNjMDUyMjEwNWJjZDMzMzhmODUxYTg4Y2I1YQ==
+  data.tar.gz: !binary |-
+    ZThmNmVjMDYzZTg4NjYxNDVlNjIwODgxYjRiZjc4N2NhYmUwMGY3Yw==
+SHA512:
+  metadata.gz: !binary |-
+    NjU4MDQwOWQ0ODk2MmEyMjVkZWI5NDQxMDg4OTI5NmY3MzcxZDNiNjJkMzNl
+    NDE1ZjFlMDYwYTA0NzA3ODY5M2JjZDBmYTljNjY0MjJhYzA3MzExNDYzYWU2
+    NzdhOGFkYTQ5YWQwMjcxNWE1MWRlOGYzYzFlMWRlNTczZjkyMGU=
+  data.tar.gz: !binary |-
+    YWRhNGM2MTZhMzljODM1Njc0YWE1ZGU4NDgxODMzYzBiYzdmMjA2ZWQ3M2Zm
+    OTFmNjZhNjM4NmRhOWE5YTRiYTY1YzczZWNiN2RkODk3ZGE5MzQ3OGIyYTdi
+    Y2Q3ZmI2N2Y4ZTM1ZWQyNDNiZmI0NzI2OTllMGYxYTMwNDIwMDk=

data/Gemfile.lock

@@ -1,37 +1,37 @@
 PATH
   remote: .
   specs:
-    yawpa (1.1.0)
+    yawpa (1.2.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
     diff-lcs (1.2.5)
     docile (1.1.5)
-    json (1.8.1)
-    multi_json (1.10.1)
-    rake (10.3.2)
-    rdoc (4.1.1)
+    json (1.8.3)
+    rake (10.4.2)
+    rdoc (4.2.2)
       json (~> 1.4)
-    redcarpet (3.1.2)
-    rspec (3.0.0)
-      rspec-core (~> 3.0.0)
-      rspec-expectations (~> 3.0.0)
-      rspec-mocks (~> 3.0.0)
-    rspec-core (3.0.2)
-      rspec-support (~> 3.0.0)
-    rspec-expectations (3.0.2)
+    redcarpet (3.3.4)
+    rspec (3.4.0)
+      rspec-core (~> 3.4.0)
+      rspec-expectations (~> 3.4.0)
+      rspec-mocks (~> 3.4.0)
+    rspec-core (3.4.4)
+      rspec-support (~> 3.4.0)
+    rspec-expectations (3.4.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.0.0)
-    rspec-mocks (3.0.2)
-      rspec-support (~> 3.0.0)
-    rspec-support (3.0.2)
-    simplecov (0.8.2)
+      rspec-support (~> 3.4.0)
+    rspec-mocks (3.4.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.4.0)
+    rspec-support (3.4.1)
+    simplecov (0.11.2)
       docile (~> 1.1.0)
-      multi_json
-      simplecov-html (~> 0.8.0)
-    simplecov-html (0.8.0)
-    yard (0.8.7.4)
+      json (~> 1.8)
+      simplecov-html (~> 0.10.0)
+    simplecov-html (0.10.0)
+    yard (0.8.7.6)
 
 PLATFORMS
   ruby
@@ -44,3 +44,6 @@ DEPENDENCIES
   simplecov
   yard
   yawpa!
+
+BUNDLED WITH
+   1.10.6

data/README.md

@@ -109,6 +109,10 @@ options that were not processed after observing a non-option parameters).
 
 ## Release Notes
 
+### v1.2.0
+
+- Always return non-frozen strings
+
 ### v1.1.0
 
 - Add `:boolean` option flag.

data/lib/yawpa/version.rb

@@ -1,4 +1,4 @@
 module Yawpa
   # gem version
-  VERSION = "1.1.0"
+  VERSION = "1.2.0"
 end

data/lib/yawpa.rb

@@ -13,179 +13,190 @@ require "yawpa/version"
 # - Options can be defined with a range specifying the allowed number of
 #   parameters
 module Yawpa
+
   # Exception class raised when an unknown option is observed.
   class ArgumentParsingException < Exception; end
 
-  # Parse input parameters looking for options according to rules given in
-  # flags.
-  # Syntax:
-  #   opts, args = parse(params, options, flags = {})
-  #
-  # An ArgumentParsingException will be raised if an unknown option is observed
-  # or insufficient arguments are present for an option.
-  #
-  # Example +options+:
-  #
-  #   {
-  #     version: nil,
-  #     verbose: {short: 'v'},
-  #     server: {nargs: (1..2)},
-  #     username: {nargs: 1},
-  #     password: {nargs: 1},
-  #     color: :boolean,
-  #   }
-  #
-  # The keys of the +options+ Hash can be either strings or symbols.
-  #
-  #
-  # @param params [Array]
-  #   List of program parameters to parse.
-  # @param options [Hash]
-  #   Hash containing the long option names as keys, and values containing
-  #   special flags for the options as values (examples above).
-  #   Possible values:
-  #   +nil+:: No special flags for this option (equivalent to +{}+)
-  #   +:boolean+::
-  #     The option is a toggleable boolean option (equivalent to
-  #     +{boolean: true}+)
-  #   Hash::
-  #     Possible option flags:
-  #     - +:short+: specify a short option letter to associate with the long option
-  #     - +:nargs+: specify an exact number or range of possible numbers of
-  #       arguments to the option
-  #     - +:boolean+: if true, specify that the option is a toggleable boolean
-  #       option and allow a prefix of "no" to turn it off.
-  # @param flags [Hash]
-  #   Optional flags dictating how {.parse} should do its job.
-  # @option flags [Boolean] :posix_order
-  #   Stop processing parameters when a non-option argument is seen.
-  #   Set this to +true+ if you want to implement subcommands.
-  #
-  # @return [Array]
-  #   Two-element array containing +opts+ and +args+ return values.
-  #   +opts+::
-  #     The returned +opts+ value will be a Hash with the observed
-  #     options as keys and any option arguments as values.
-  #   +args+::
-  #     The returned +args+ will be an Array of the unprocessed
-  #     parameters (if +:posix_order+ was passed in +flags+, this array might
-  #     contain further options that were not processed after observing a
-  #     non-option parameters).
-  def self.parse(params, options, flags = {})
-    options = _massage_options(options)
-    opts = {}
-    args = []
-    i = 0
-    while i < params.length
-      param = params[i]
-      if param =~ /^--([^=]+)(?:=(.+))?$/
-        param_name, val = $1, $2
-        bool_val = true
-        if options[param_name].nil?
-          if param_name =~ /^no(.*)$/
-            test_param_name = $1
-            if options[test_param_name]
-              param_name = test_param_name
-              bool_val = false
+  class << self
+
+    # Parse input parameters looking for options according to rules given in
+    # flags.
+    # Syntax:
+    #   opts, args = parse(params, options, flags = {})
+    #
+    # An ArgumentParsingException will be raised if an unknown option is
+    # observed or insufficient arguments are present for an option.
+    #
+    # Example +options+:
+    #
+    #   {
+    #     version: nil,
+    #     verbose: {short: 'v'},
+    #     server: {nargs: (1..2)},
+    #     username: {nargs: 1},
+    #     password: {nargs: 1},
+    #     color: :boolean,
+    #   }
+    #
+    # The keys of the +options+ Hash can be either strings or symbols.
+    #
+    #
+    # @param params [Array]
+    #   List of program parameters to parse.
+    # @param options [Hash]
+    #   Hash containing the long option names as keys, and values containing
+    #   special flags for the options as values (examples above).
+    #   Possible values:
+    #   +nil+:: No special flags for this option (equivalent to +{}+)
+    #   +:boolean+::
+    #     The option is a toggleable boolean option (equivalent to
+    #     +{boolean: true}+)
+    #   Hash::
+    #     Possible option flags:
+    #     - +:short+: specify a short option letter to associate with the long
+    #       option
+    #     - +:nargs+: specify an exact number or range of possible numbers of
+    #       arguments to the option
+    #     - +:boolean+: if true, specify that the option is a toggleable
+    #       boolean option and allow a prefix of "no" to turn it off.
+    # @param flags [Hash]
+    #   Optional flags dictating how {.parse} should do its job.
+    # @option flags [Boolean] :posix_order
+    #   Stop processing parameters when a non-option argument is seen.
+    #   Set this to +true+ if you want to implement subcommands.
+    #
+    # @return [Array]
+    #   Two-element array containing +opts+ and +args+ return values.
+    #   +opts+::
+    #     The returned +opts+ value will be a Hash with the observed
+    #     options as keys and any option arguments as values.
+    #   +args+::
+    #     The returned +args+ will be an Array of the unprocessed
+    #     parameters (if +:posix_order+ was passed in +flags+, this array might
+    #     contain further options that were not processed after observing a
+    #     non-option parameters).
+    def parse(params, options, flags = {})
+      options = _massage_options(options)
+      opts = {}
+      args = []
+      i = 0
+      while i < params.length
+        param = params[i]
+        if param =~ /^--([^=]+)(?:=(.+))?$/
+          param_name, val = $1, $2
+          bool_val = true
+          if options[param_name].nil?
+            if param_name =~ /^no(.*)$/
+              test_param_name = $1
+              if options[test_param_name]
+                param_name = test_param_name
+                bool_val = false
+              end
             end
           end
-        end
-        opt_config = options[param_name]
-        raise ArgumentParsingException.new("Unknown option '#{param_name}'") unless opt_config
-        param_key = opt_config[:key]
-        if opt_config[:boolean]
-          opts[param_key] = bool_val
-        elsif opt_config[:nargs].last == 0
-          opts[param_key] = true
-        else
-          opts[param_key] = []
-          i += _gather(opt_config[:nargs], i + 1, params, val, param_key, opts[param_key])
-        end
-      elsif param =~ /^-(.+)$/
-        short_flags = $1
-        short_idx = 0
-        while short_idx < short_flags.length
-          opt_config = _find_opt_config_by_short_name(options, short_flags[short_idx])
-          if opt_config.nil?
-            raise ArgumentParsingException.new("Unknown option '-#{short_flags[short_idx]}'")
-          end
+          opt_config = options[param_name]
+          raise ArgumentParsingException.new("Unknown option '#{param_name}'") unless opt_config
           param_key = opt_config[:key]
-          if opt_config[:nargs].last == 0
+          if opt_config[:boolean]
+            opts[param_key] = bool_val
+          elsif opt_config[:nargs].last == 0
             opts[param_key] = true
           else
             opts[param_key] = []
-            i += _gather(opt_config[:nargs], i + 1, params, short_flags[short_idx + 1, short_flags.length], param_key, opts[param_key])
-            break
+            i += _gather(opt_config[:nargs], i + 1, params, val, param_key, opts[param_key])
           end
-          short_idx += 1
+        elsif param =~ /^-(.+)$/
+          short_flags = $1
+          short_idx = 0
+          while short_idx < short_flags.length
+            opt_config = _find_opt_config_by_short_name(options, short_flags[short_idx])
+            if opt_config.nil?
+              raise ArgumentParsingException.new("Unknown option '-#{short_flags[short_idx]}'")
+            end
+            param_key = opt_config[:key]
+            if opt_config[:nargs].last == 0
+              opts[param_key] = true
+            else
+              opts[param_key] = []
+              i += _gather(opt_config[:nargs],
+                           i + 1,
+                           params,
+                           short_flags[short_idx + 1, short_flags.length],
+                           param_key,
+                           opts[param_key])
+              break
+            end
+            short_idx += 1
+          end
+        elsif flags[:posix_order]
+          args = params[i, params.length].map(&:dup)
+          break
+        else
+          args << params[i].dup
         end
-      elsif flags[:posix_order]
-        args = params[i, params.length]
-        break
-      else
-        args << params[i]
+        i += 1
       end
-      i += 1
-    end
 
-    # Condense 1-element arrays of option values to just the element itself
-    opts.each_key do |k|
-      if opts[k].is_a?(Array) and opts[k].length == 1
-        opts[k] = opts[k].first
+      # Condense 1-element arrays of option values to just the element itself
+      opts.each_key do |k|
+        if opts[k].is_a?(Array) and opts[k].length == 1
+          opts[k] = opts[k].first
+        end
       end
+
+      return [opts, args]
     end
 
-    return [opts, args]
-  end
+    private
 
-  # Internal helper method to gather arguments for an option
-  def self._gather(nargs, start_idx, params, initial, param_key, result)
-    n_gathered = 0
-    if initial and initial != ''
-      result << initial
-      n_gathered += 1
-    end
-    num_indices_used = 0
-    index = start_idx
-    while n_gathered < nargs.last and
-          index < params.length and
-          params[index][0] != '-' do
-      result << params[index]
-      index += 1
-      num_indices_used += 1
-      n_gathered += 1
+    # Internal helper method to gather arguments for an option
+    def _gather(nargs, start_idx, params, initial, param_key, result)
+      n_gathered = 0
+      if initial and initial != ''
+        result << initial
+        n_gathered += 1
+      end
+      num_indices_used = 0
+      index = start_idx
+      while n_gathered < nargs.last and
+            index < params.length and
+            params[index][0] != '-' do
+        result << params[index].dup
+        index += 1
+        num_indices_used += 1
+        n_gathered += 1
+      end
+      if n_gathered < nargs.first
+        raise ArgumentParsingException.new("Not enough arguments supplied for option '#{param_key}'")
+      end
+      num_indices_used
     end
-    if n_gathered < nargs.first
-      raise ArgumentParsingException.new("Not enough arguments supplied for option '#{param_key}'")
+
+    # Internal helper method to format the options in a consistent format
+    def _massage_options(options)
+      {}.tap do |newopts|
+        options.each_pair do |k, v|
+          v = {} if v.nil?
+          v = {boolean: true} if v == :boolean
+          newkey = k.to_s
+          newopts[newkey] = {key: k}
+          nargs = v[:nargs] || 0
+          nargs = (nargs..nargs) if nargs.is_a?(Fixnum)
+          newopts[newkey][:nargs] = nargs
+          newopts[newkey][:short] = v[:short] || ''
+          newopts[newkey][:boolean] = v[:boolean]
+        end
+      end
     end
-    num_indices_used
-  end
-  private_class_method :_gather
 
-  # Internal helper method to format the options in a consistent format
-  def self._massage_options(options)
-    {}.tap do |newopts|
+    # Internal helper method to find an option configuration by short name
+    def _find_opt_config_by_short_name(options, short_name)
       options.each_pair do |k, v|
-        v = {} if v.nil?
-        v = {boolean: true} if v == :boolean
-        newkey = k.to_s
-        newopts[newkey] = {key: k}
-        nargs = v[:nargs] || 0
-        nargs = (nargs..nargs) if nargs.is_a?(Fixnum)
-        newopts[newkey][:nargs] = nargs
-        newopts[newkey][:short] = v[:short] || ''
-        newopts[newkey][:boolean] = v[:boolean]
+        return v if v[:short] == short_name
       end
+      nil
     end
-  end
-  private_class_method :_massage_options
 
-  # Internal helper method to find an option configuration by short name
-  def self._find_opt_config_by_short_name(options, short_name)
-    options.each_pair do |k, v|
-      return v if v[:short] == short_name
-    end
-    nil
   end
-  private_class_method :_find_opt_config_by_short_name
+
 end

data/spec/yawpa_spec.rb

@@ -11,7 +11,7 @@ describe Yawpa do
     it "raises an exception when an invalid option is passed" do
       options = { }
       params = ['one', '--option', 'two']
-      expect { Yawpa.parse(params, options) }.to raise_error
+      expect { Yawpa.parse(params, options) }.to raise_error(Yawpa::ArgumentParsingException, /Unknown option/)
     end
 
     it "returns boolean options which are set" do
@@ -53,7 +53,7 @@ describe Yawpa do
         opt: {nargs: 2},
       }
       params = ['--opt', 'val']
-      expect { Yawpa.parse(params, options) }.to raise_error
+      expect { Yawpa.parse(params, options) }.to raise_error(Yawpa::ArgumentParsingException, /Not enough arguments supplied/)
     end
 
     it "uses --opt=val syntax for an option's value" do
@@ -179,7 +179,7 @@ describe Yawpa do
         a: {short: 'a'},
       }
       params = ['-ab']
-      expect { Yawpa.parse(params, options) }.to raise_error
+      expect { Yawpa.parse(params, options) }.to raise_error(Yawpa::ArgumentParsingException, /Unknown option/)
     end
 
     it "raises an error when not enough arguments are given to short option" do
@@ -187,7 +187,7 @@ describe Yawpa do
         a: {nargs: 1, short: 'a'},
       }
       params = ['-a']
-      expect { Yawpa.parse(params, options) }.to raise_error
+      expect { Yawpa.parse(params, options) }.to raise_error(Yawpa::ArgumentParsingException, /Not enough arguments supplied/)
     end
 
     it "overwrites option value when short option used after long" do
@@ -230,5 +230,32 @@ describe Yawpa do
       expect(opts).to eq(push: false, pull: true)
       expect(args).to eq(%w[arg])
     end
+
+    it "returns non-frozen strings" do
+      options = {
+        o1: {nargs: 1, short: "1"},
+        o2: {nargs: 1, short: "2"},
+        o3: {nargs: 1, short: "3"},
+        o4: {nargs: 1, short: "4"},
+      }
+
+      arguments = %w[--o1=one --o2 two -3 three -4four arg].map(&:freeze)
+
+      opts, args = Yawpa.parse(arguments, options)
+      expect(opts[:o1].frozen?).to be_falsey
+      expect{opts[:o1].sub!(/./, '-')}.to_not raise_error
+      expect(opts[:o2].frozen?).to be_falsey
+      expect{opts[:o2].sub!(/./, '-')}.to_not raise_error
+      expect(opts[:o3].frozen?).to be_falsey
+      expect{opts[:o3].sub!(/./, '-')}.to_not raise_error
+      expect(opts[:o4].frozen?).to be_falsey
+      expect{opts[:o4].sub!(/./, '-')}.to_not raise_error
+      expect(args[0].frozen?).to be_falsey
+      expect{args[0].sub!(/./, '-')}.to_not raise_error
+
+      opts, args = Yawpa.parse(arguments, options, posix_order: true)
+      expect(args[0].frozen?).to be_falsey
+      expect{args[0].sub!(/./, '-')}.to_not raise_error
+    end
   end
 end

metadata

@@ -1,20 +1,18 @@
 --- !ruby/object:Gem::Specification
 name: yawpa
 version: !ruby/object:Gem::Version
-  version: 1.1.0
-  prerelease: 
+  version: 1.2.0
 platform: ruby
 authors:
 - Josh Holtrop
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-25 00:00:00.000000000 Z
+date: 2016-04-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -22,7 +20,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -30,7 +27,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -38,7 +34,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -46,7 +41,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -54,7 +48,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -62,7 +55,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -70,7 +62,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -78,7 +69,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -86,7 +76,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -94,7 +83,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: redcarpet
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -102,7 +90,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -129,33 +116,26 @@ files:
 - yawpa.gemspec
 homepage: ''
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 77772525
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 77772525
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23.2
+rubygems_version: 2.4.8
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Yet Another Way to Parse Arguments
 test_files:
 - spec/spec_helper.rb