haveapi-client 0.13.3 → 0.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e9d415ea5340264305787c329a3f8bff4f8e90da4fc2d3b9a187c340131438c9
-  data.tar.gz: 678d54f49e930d20c9b033a4a2725bd7bda8a0f6efb5edd5d177b8ffdb0424c0
+  metadata.gz: 4ab49346d0209df996a1832992ad1f11bf7bb2f7de6f1f7cd49fb92e4bbc62c0
+  data.tar.gz: a49dc688a464d90e4188d93abcda7d27f252f0c4d44ce7769fb773b68b08e11b
 SHA512:
-  metadata.gz: c77561672d6b40484361db5a46e7dacd21be5afa6c9c1609194c3d9e38f3ee0a3547aabb2b0e11ba7db1d3384f89e347df52bcc8ab14901fd941737dd00c570a
-  data.tar.gz: a6314cae9ee7f0a63a0f164f3033b071c303fd57545361525f3860d5d0444963a14ed303aa8421535256ec88c7391ead84ac7fae2b835e5136b046465db2763b
+  metadata.gz: b39a2abebdc41d1480c2cba71ff4440ac320532c1d22885d92aba8d383f3e9e4d2a055114821c76322f90d1c0d135c5233b80f95a3ac366483a8ee6fa6577756
+  data.tar.gz: 768c5ec61c25e4dcfbc99f79f0a788d2237789e8316065fecc766fe60b832852609954ee1bf2c21256aa80116f901aef55314825e4fb42df23c69cbf84dab7d0

data/haveapi-client.gemspec

@@ -23,8 +23,8 @@ Gem::Specification.new do |spec|
 
   spec.add_runtime_dependency 'activesupport', '>= 4.0'
   spec.add_runtime_dependency 'require_all', '~> 2.0.0'
-  spec.add_runtime_dependency 'rest-client', '~> 2.0.2'
+  spec.add_runtime_dependency 'rest-client', '~> 2.1.0'
   spec.add_runtime_dependency 'json'
-  spec.add_runtime_dependency 'highline', '~> 1.7.8'
-  spec.add_runtime_dependency 'ruby-progressbar', '~> 1.7.5'
+  spec.add_runtime_dependency 'highline', '~> 2.0.3'
+  spec.add_runtime_dependency 'ruby-progressbar', '~> 1.11.0'
 end

data/lib/haveapi/cli/output_formatter.rb

@@ -5,8 +5,8 @@ module HaveAPI::CLI
       f.format
     end
 
-    def self.print(*args)
-      f = new(*args)
+    def self.print(*args, **kwargs)
+      f = new(*args, **kwargs)
       f.print
     end
 
@@ -84,7 +84,7 @@ module HaveAPI::CLI
     def generate
       return if @cols.empty?
       prepare
-      
+
       case @layout
       when :columns
         columns
@@ -113,7 +113,7 @@ module HaveAPI::CLI
         i += 1
         ret
       end.join('  ')
-      
+
       line sprintf(formatters, * @cols.map { |c| c[:label] }) if @header
 
       @str_objects.each do |o|
@@ -128,7 +128,7 @@ module HaveAPI::CLI
       @str_objects.each do |o|
         @cols.each_index do |i|
           c = @cols[i]
-        
+
           if o[i].is_a?(::String) && o[i].index("\n")
             lines = o[i].split("\n")
             v = ([lines.first] + lines[1..-1].map { |l| (' ' * (w+3)) + l }).join("\n")
@@ -139,7 +139,7 @@ module HaveAPI::CLI
 
           line sprintf("%#{w}s:  %s", c[:label], v)
         end
-        
+
         line
       end
     end
@@ -155,10 +155,10 @@ module HaveAPI::CLI
 
     def prepare
       @str_objects = []
-      
+
       each_object do |o|
         arr = []
-        
+
         @cols.each do |c|
           v = o[ c[:name] ]
           str = (c[:display] ? c[:display].call(v) : v)
@@ -190,7 +190,7 @@ module HaveAPI::CLI
 
     def col_width(i, c)
       w = c[:label].to_s.length
-      
+
       @str_objects.each do |o|
         len = o[i].to_s.length
         w = len if len > w

data/lib/haveapi/client/action.rb

@@ -12,8 +12,8 @@ module HaveAPI::Client
       apply_args(args)
     end
 
-    def execute(data, *_)
-      args = [self]
+    def execute(data)
+      params_arg = {}
 
       if input
         params = Params.new(self, data)
@@ -22,10 +22,10 @@ module HaveAPI::Client
           raise ValidationError.new(self, params.errors)
         end
 
-        args << params.to_api << {}
+        params_arg = params.to_api
       end
 
-      ret = @api.call(*args)
+      ret = @api.call(self, params_arg, **{})
       reset
       ret
     end

data/lib/haveapi/client/client.rb

@@ -49,8 +49,8 @@ class HaveAPI::Client::Client
   end
 
   # See Communicator#authenticate.
-  def authenticate(*args, &block)
-    @api.authenticate(*args, &block)
+  def authenticate(auth_method, **options, &block)
+    @api.authenticate(auth_method, options, &block)
   end
 
   # Get uthentication provider

data/lib/haveapi/client/resource.rb

@@ -76,7 +76,7 @@ module HaveAPI::Client
       action.aliases(true).each do |name|
         next unless define_method?(action, name)
 
-        define_singleton_method(name) do |*args, &block|
+        define_singleton_method(name) do |*args, **kwargs, &block|
           client_opts = @client.opts(:block, :block_interval, :block_timeout)
           all_args = @prepared_args + args
 
@@ -89,28 +89,42 @@ module HaveAPI::Client
             end
 
             if action.unresolved_args?
-              raise ArgumentError.new('One or more object ids missing')
+              raise ArgumentError, 'one or more object ids missing'
             end
           end
 
-          if all_args.empty?
-            all_args << default_action_input_params(action)
+          if all_args.length > 1 || (kwargs.any? && all_args.any?)
+            raise ArgumentError, "too many arguments for action #{@name}##{action.name}"
+          end
+
+          arg = all_args.shift
 
-          elsif all_args.last.is_a?(Hash)
-            last = all_args.pop
+          user_params =
+            if kwargs.any? && arg
+              raise ArgumentError,
+                    'pass the input parameters either as a hash or keyword arguments'
+            elsif kwargs.any?
+              kwargs
+            elsif arg
+              arg
+            end
 
-            if last.has_key?(:meta)
-              meta = last[:meta]
+          if user_params.nil?
+            input_params = default_action_input_params(action)
+
+          else
+            if user_params.has_key?(:meta)
+              meta = user_params[:meta]
 
               %i(block block_interval block_timeout).each do |p|
                 client_opts[p] = meta.delete(p) if meta.has_key?(p)
               end
             end
 
-            all_args << default_action_input_params(action).update(last)
+            input_params = default_action_input_params(action).update(user_params)
           end
 
-          ret = Response.new(action, action.execute(*all_args))
+          ret = Response.new(action, action.execute(input_params))
 
           raise ActionFailed.new(ret) unless ret.ok?
 
@@ -137,8 +151,8 @@ module HaveAPI::Client
             }.each do |k, v|
               wait_opts[v] = client_opts[k] if client_opts.has_key?(k)
             end
-            
-            ret.wait_for_completion(wait_opts) do |state|
+
+            ret.wait_for_completion(**wait_opts) do |state|
               block.call(return_value, state) if block
             end
           end

data/lib/haveapi/client/response.rb

@@ -61,10 +61,10 @@ class HaveAPI::Client::Response
   # it is regularly called with the action's state.
   #
   # @see HaveAPI::Client::Action#wait_for_completion
-  def wait_for_completion(*args, &block)
+  def wait_for_completion(**kwargs, &block)
     id = meta[:action_state_id]
     return nil unless id
 
-    HaveAPI::Client::Action.wait_for_completion(@action.client, id, *args, &block)
+    HaveAPI::Client::Action.wait_for_completion(@action.client, id, **kwargs, &block)
   end
 end

data/lib/haveapi/client/version.rb

@@ -1,6 +1,6 @@
 module HaveAPI
   module Client
     PROTOCOL_VERSION = '2.0'
-    VERSION = '0.13.3'
+    VERSION = '0.15.0'
   end
 end

data/shell.nix

@@ -6,7 +6,7 @@ in stdenv.mkDerivation rec {
   name = "haveapi-client";
 
   buildInputs = with pkgs;[
-    ruby
+    ruby_2_7
     git
     openssl
   ];

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: haveapi-client
 version: !ruby/object:Gem::Version
-  version: 0.13.3
+  version: 0.15.0
 platform: ruby
 authors:
 - Jakub Skokan
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 1970-01-01 00:00:00.000000000 Z
+date: 1980-01-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -72,14 +72,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.2
+        version: 2.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.2
+        version: 2.1.0
 - !ruby/object:Gem::Dependency
   name: json
   requirement: !ruby/object:Gem::Requirement
@@ -100,28 +100,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.7.8
+        version: 2.0.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.7.8
+        version: 2.0.3
 - !ruby/object:Gem::Dependency
   name: ruby-progressbar
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.7.5
+        version: 1.11.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.7.5
+        version: 1.11.0
 description: Ruby API and CLI for HaveAPI
 email:
 - jakub.skokan@vpsfree.cz
@@ -183,7 +183,7 @@ homepage: ''
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -198,8 +198,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
-signing_key: 
+rubygems_version: 3.2.26
+signing_key:
 specification_version: 4
 summary: Ruby API and CLI for HaveAPI
 test_files: []