haveapi 0.27.1 → 0.27.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c1d65a2661353ff707b5b33f0bf8df446bfd4dcc6c669b5af8fdc46dbbf8a0a5
-  data.tar.gz: ddb0c92670d615f22468c91c96f8e83e34cf53a3680cf55c588936aa95a34df7
+  metadata.gz: 33a06b199d36331bf03d273d528e62fad7a0fa96e78667afa3d8411734e8d966
+  data.tar.gz: 94c99822c1b06670ea2623b9faa8c9d22bc7fdfd1be9e3a7d4fb09928f812395
 SHA512:
-  metadata.gz: 908348858c2829a57531813ceb696c47768ec9a5e29133e8fb64e2c33613949ef70dd52468480a5a2e33c25c9c204541946ccfb982a17cec27656f4b184c2724
-  data.tar.gz: 82e5b873d368caf4914895b24fa4e57d6b0b39c55357910c307496c535cacbdfebb35cd4036b2d07bb1627625a951f5e04fa440416f1eaa141980a22e943468e
+  metadata.gz: e473cd23de9122f5e817bafbaf24147a2c67b5d311a37c690d7b474d09647e7b4b0ea6f7104f99377ca8534e352b274dcaa7403a67b3dce2dfd510a7ab75fe7a
+  data.tar.gz: c7f8b6f1ffbcf9b8f299e8ce487b8959cc631b98ac3c7b53e508ada38a3191d687b5772c8fd84eac0a14359b70ac182c98859976d81e0cacebfb2a5c436e6b5e

data/haveapi.gemspec

@@ -15,7 +15,7 @@ Gem::Specification.new do |s|
   s.required_ruby_version = ">= #{File.read('../../.ruby-version').strip}"
 
   s.add_dependency 'activesupport', '>= 7.1'
-  s.add_dependency 'haveapi-client', '~> 0.27.1'
+  s.add_dependency 'haveapi-client', '~> 0.27.2'
   s.add_dependency 'json'
   s.add_dependency 'mail'
   s.add_dependency 'nesty', '~> 1.0'

data/lib/haveapi/parameters/typed.rb

@@ -75,6 +75,11 @@ module HaveAPI::Parameters
     def clean(raw)
       return instance_exec(raw, &@clean) if @clean
 
+      if raw.is_a?(String)
+        stripped = raw.strip
+        return nil if stripped.empty? && optional?
+      end
+
       if raw.nil?
         @default
 

data/lib/haveapi/version.rb

@@ -1,4 +1,4 @@
 module HaveAPI
   PROTOCOL_VERSION = '2.0'.freeze
-  VERSION = '0.27.1'.freeze
+  VERSION = '0.27.2'.freeze
 end

data/spec/parameters/typed_spec.rb

@@ -78,22 +78,28 @@ describe 'Parameters::Typed' do
     expect(p.clean(12.0)).to eq(12)
     expect { p.clean('abc') }.to raise_error(HaveAPI::ValidationError)
     expect { p.clean('12abc') }.to raise_error(HaveAPI::ValidationError)
-    expect { p.clean('') }.to raise_error(HaveAPI::ValidationError)
+    expect(p.clean('')).to be_nil
     expect { p.clean('12.0') }.to raise_error(HaveAPI::ValidationError)
     expect { p.clean(12.3) }.to raise_error(HaveAPI::ValidationError)
     expect { p.clean(true) }.to raise_error(HaveAPI::ValidationError)
 
+    p = p_arg(type: Integer, required: true)
+    expect { p.clean('') }.to raise_error(HaveAPI::ValidationError)
+
     # Float
     p = p_type(Float)
     expect(p.clean('3.1456')).to eq(3.1456)
     expect(p.clean('1e3')).to eq(1000.0)
     expect(p.clean(3)).to eq(3.0)
     expect { p.clean('abc') }.to raise_error(HaveAPI::ValidationError)
-    expect { p.clean('') }.to raise_error(HaveAPI::ValidationError)
+    expect(p.clean('')).to be_nil
     expect { p.clean('NaN') }.to raise_error(HaveAPI::ValidationError)
     expect { p.clean(Float::NAN) }.to raise_error(HaveAPI::ValidationError)
     expect { p.clean(Float::INFINITY) }.to raise_error(HaveAPI::ValidationError)
 
+    p = p_arg(type: Float, required: true)
+    expect { p.clean('') }.to raise_error(HaveAPI::ValidationError)
+
     # Boolean
     p = p_type(Boolean)
 
@@ -109,9 +115,12 @@ describe 'Parameters::Typed' do
     expect(p.clean(1)).to be true
     expect(p.clean('  YES ')).to be true
     expect { p.clean('maybe') }.to raise_error(HaveAPI::ValidationError)
-    expect { p.clean('') }.to raise_error(HaveAPI::ValidationError)
+    expect(p.clean('')).to be_nil
     expect { p.clean(2) }.to raise_error(HaveAPI::ValidationError)
 
+    p = p_arg(type: Boolean, required: true)
+    expect { p.clean('') }.to raise_error(HaveAPI::ValidationError)
+
     # Datetime
     p = p_type(Datetime)
     t = Time.now
@@ -119,6 +128,9 @@ describe 'Parameters::Typed' do
 
     expect(p.clean(t.iso8601)).to eq(t2)
     expect { p.clean('bzz') }.to raise_error(HaveAPI::ValidationError)
+    expect(p.clean('')).to be_nil
+
+    p = p_arg(type: Datetime, required: true)
     expect { p.clean('') }.to raise_error(HaveAPI::ValidationError)
 
     # String, Text

data/test_support/client_test_api.rb

@@ -457,6 +457,54 @@ module HaveAPI
           end
         end
 
+        define_action(:EchoOptional) do
+          extend DocFilter
+          route 'echo_optional'
+          http_method :post
+          input(:hash) do
+            datetime :dt, required: false
+          end
+          output(:hash) do
+            bool :dt_provided, required: true
+            bool :dt_nil, required: true
+            datetime :dt
+          end
+          authorize { allow }
+
+          def exec
+            ret = {
+              dt_provided: input.has_key?(:dt),
+              dt_nil: input[:dt].nil?
+            }
+            ret[:dt] = input[:dt] unless input[:dt].nil?
+            ret
+          end
+        end
+
+        define_action(:EchoOptionalGet) do
+          extend DocFilter
+          route 'echo_optional_get'
+          http_method :get
+          input(:hash) do
+            datetime :dt, required: false
+          end
+          output(:hash) do
+            bool :dt_provided, required: true
+            bool :dt_nil, required: true
+            datetime :dt
+          end
+          authorize { allow }
+
+          def exec
+            ret = {
+              dt_provided: input.has_key?(:dt),
+              dt_nil: input[:dt].nil?
+            }
+            ret[:dt] = input[:dt] unless input[:dt].nil?
+            ret
+          end
+        end
+
         define_action(:EchoResource) do
           extend DocFilter
           route 'echo_resource'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: haveapi
 version: !ruby/object:Gem::Version
-  version: 0.27.1
+  version: 0.27.2
 platform: ruby
 authors:
 - Jakub Skokan
@@ -29,14 +29,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.27.1
+        version: 0.27.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.27.1
+        version: 0.27.2
 - !ruby/object:Gem::Dependency
   name: json
   requirement: !ruby/object:Gem::Requirement