protip 0.33.0 → 0.35.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: b00d08f38111e8b0182e50c6f4e452787d0f9168
-  data.tar.gz: 963dc1bde387ed8308925a2185a4f9b5d5494631
+SHA256:
+  metadata.gz: 30131e70e41b5199b3d87287c7d04cc29730965b7ee3d526b05c963a2a6782ab
+  data.tar.gz: 07fc8c4685f97553082dd964d6b428378a34ff356dee5dca1f5cf394fdc90164
 SHA512:
-  metadata.gz: 736fe9fc7445f0fa83b608a142cd74001ef30b96304be7297f1653016fa795668cfc7685f0f4e2ca01047ab14fc033b41425e2f5e2ea0c5db8e9454cf850be31
-  data.tar.gz: 0ad6ab1ff91ff83179093f5dbdc71a776632c0147687e88819894525a90aaea5cfcf08637663d3927aa46681b89c62f3ae06545b19a96eda5b77da712dc699b8
+  metadata.gz: 2450adaae3e28b8b711c1d96084fa3caad1e6960e48b5f4bd02fcbf17ca2ae71728fabb1618ef8c5f488211979232abec89f56c18103218d247947719ab96f15
+  data.tar.gz: 3a0667f59aea909f493334949bafa487d9ddf7644f8df4a0431c276b1a470b78c8b05455097b23112994ee6f9eeef650f139101494e6a00abef1cc2435759c47

data/lib/protip.rb

@@ -1,3 +1,4 @@
+require 'protip/version'
 require 'protip/client'
 require 'protip/resource'
 

data/lib/protip/client.rb

@@ -69,27 +69,9 @@ module Protip
     # @param request [Net::HTTPGenericRequest] the raw request object to send
     # @return [Net::HTTPResponse] the response for the given request
     def execute_request(request)
-      http = nil
       uri = request.uri
-      retries = 0
-      max_retries = 3
-
-      begin
-        unless http
-          http = Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == 'https', read_timeout: 120)
-        end
+      Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == 'https', read_timeout: 600) do |http|
         http.request request
-      rescue Net::OpenTimeout, Net::ReadTimeout
-        if (retries += 1) <= max_retries
-          sleep(retries)
-          retry
-        else
-          raise
-        end
-      ensure
-        if http
-          http.finish
-        end
       end
     end
   end

data/lib/protip/decorator.rb

@@ -281,7 +281,7 @@ module Protip
 
     def method_missing_query(name, *args)
       field = message.class.descriptor.lookup(name[0, name.length - 1])
-      raise NoMethodError unless field
+      raise NoMethodError.new("No query method found for: #{name}") unless field
 
       if nil != Protip::Decorator.enum_for_field(field) && args.length == 1
         matches?(field, args[0])

data/lib/protip/resource.rb

@@ -19,7 +19,6 @@ require 'active_model/dirty'
 
 require 'forwardable'
 
-require 'protip'
 require 'protip/client'
 require 'protip/error'
 require 'protip/decorator'
@@ -78,14 +77,14 @@ module Protip
       end
 
       def transformer
-        @transformer || ::Protip.default_transformer
+        defined?(@transformer) ? @transformer : ::Protip.default_transformer
       end
 
       private
 
       # Primary entry point for defining resourceful behavior.
       def resource(actions:, message:, query: nil, nested_resources: {})
-        raise RuntimeError.new('Only one call to `resource` is allowed') if @message
+        raise RuntimeError.new('Only one call to `resource` is allowed') if defined?(@message) && @message
         validate_actions!(actions)
         validate_nested_resources!(nested_resources)
 
@@ -338,6 +337,18 @@ module Protip
       success
     end
 
+    class RecordInvalid < StandardError
+    end
+
+    def save!
+      success = save
+      if !success
+        error_messages = errors.full_messages.join(", ")
+        raise RecordInvalid.new("Validation failed: #{error_messages}")
+      end
+      success
+    end
+
     def persisted?
       id != nil
     end

data/lib/protip/tasks/compile.rake

@@ -1,4 +1,6 @@
 require 'shellwords'
+require 'bundler/setup'
+
 namespace :protip do
   desc 'compile a single .proto file to Ruby'
   task :compile, [:filename, :proto_path, :ruby_path] do |t, args|
@@ -9,7 +11,7 @@ namespace :protip do
 
     filename = args[:filename] || raise(ArgumentError.new 'filename argument is required')
 
-    command = "grpc_tools_ruby_protoc #{proto_path.map{|p| "--proto_path=#{Shellwords.escape p}"}.join ' '} --ruby_out=#{Shellwords.escape ruby_path} #{Shellwords.escape filename}"
+    command = "bundle exec grpc_tools_ruby_protoc #{proto_path.map{|p| "--proto_path=#{Shellwords.escape p}"}.join ' '} --ruby_out=#{Shellwords.escape ruby_path} #{Shellwords.escape filename}"
     puts command
     system command
 

data/lib/protip/transformers/active_support/time_with_zone_transformer.rb

@@ -22,7 +22,7 @@ module Protip
           if !value.is_a?(::ActiveSupport::TimeWithZone) && (value.is_a?(Time) || value.is_a?(DateTime))
             value = ::ActiveSupport::TimeWithZone.new(value.to_time.utc, ::ActiveSupport::TimeZone.new('UTC'))
           end
-          raise ArgumentError unless value.is_a?(::ActiveSupport::TimeWithZone)
+          raise ArgumentError.new("Expected ActiveSupport::TimeWithZone, received #{value.class.name}") unless value.is_a?(::ActiveSupport::TimeWithZone)
 
           message_class.new(
             utc_timestamp: value.to_i,
@@ -32,4 +32,4 @@ module Protip
       end
     end
   end
-end
+end

data/lib/protip/transformers/big_decimal_transformer.rb

@@ -16,7 +16,7 @@ module Protip
             end
 
             def to_message(object, field)
-              object = BigDecimal.new(object)
+              object = BigDecimal(object)
               rational = object.to_r
               field.subtype.msgclass.new(
                 numerator: rational.numerator,
@@ -28,4 +28,4 @@ module Protip
       end
     end
   end
-end
+end

data/lib/protip/transformers/decorating_transformer.rb

@@ -25,7 +25,7 @@ module Protip
           decorator.assign_attributes(object)
           decorator.message
         else
-          raise ArgumentError
+          raise ArgumentError.new("#to_message expected a Hash or an already decorated object, received #{object.class.name} for #{field.type} #{field.name}")
         end
       end
     end

data/lib/protip/transformers/deprecated_transformer.rb

@@ -50,7 +50,7 @@ module Protip
           ::Date.new(message.year, message.month, message.day)
         end
         def to_message(object, field)
-          raise ArgumentError unless object.is_a?(::Date)
+          raise ArgumentError.new("Field #{field.name} expected a date, #{object.class.name} given") unless object.is_a?(::Date)
           field.subtype.msgclass.new(year: object.year, month: object.month, day: object.day)
         end
       end
@@ -77,7 +77,7 @@ module Protip
           if !object.is_a?(::ActiveSupport::TimeWithZone) && (object.is_a?(Time) || object.is_a?(DateTime))
             object = ::ActiveSupport::TimeWithZone.new(object.to_time.utc, ::ActiveSupport::TimeZone.new('UTC'))
           end
-          raise ArgumentError unless object.is_a?(::ActiveSupport::TimeWithZone)
+          raise ArgumentError.new("Expected ActiveSupport::TimeWithZone, received #{object.class.name}") unless object.is_a?(::ActiveSupport::TimeWithZone)
 
           field.subtype.msgclass.new(
             utc_timestamp: object.to_i,

data/lib/protip/version.rb

@@ -0,0 +1,3 @@
+module Protip
+  VERSION = '0.35.4'
+end

data/test/test_helper.rb

@@ -1,6 +1,7 @@
 require 'minitest/autorun'
-require 'mocha/mini_test'
+require 'mocha/minitest'
 require 'webmock/minitest'
 require 'minitest/stub/const'
-
 require 'minitest/pride'
+require 'pry-byebug'
+require 'protip'

data/test/unit/protip/resource/associations/belongs_to_polymorphic_association_test.rb

@@ -46,7 +46,7 @@ describe Protip::Resource::Associations::BelongsToPolymorphicAssociation do
         Protip::Resource::Associations::BelongsToPolymorphicAssociation.new resource_class,
           :reference, [rick_ross_association]
       end
-      assert_match /requires an association to be defined/, error.message
+      assert_match(/requires an association to be defined/, error.message)
     end
 
     it 'raises an error if a belongs-to association is provided for a field outside the oneof' do
@@ -54,7 +54,7 @@ describe Protip::Resource::Associations::BelongsToPolymorphicAssociation do
         Protip::Resource::Associations::BelongsToPolymorphicAssociation.new resource_class,
           :reference, [rick_ross_association, other_association]
       end
-      assert_match /requires an association to be defined/, error.message
+      assert_match(/requires an association to be defined/, error.message)
     end
 
     it 'raises an error if a duplicate belongs-to association is provided' do
@@ -62,7 +62,7 @@ describe Protip::Resource::Associations::BelongsToPolymorphicAssociation do
         Protip::Resource::Associations::BelongsToPolymorphicAssociation.new resource_class,
           :reference, [rick_ross_association, rick_ross_association, fetty_wap_association]
       end
-      assert_match /Duplicate association/, error.message
+      assert_match(/Duplicate association/, error.message)
     end
 
     it 'allows the oneof ID field to be specified' do
@@ -147,4 +147,4 @@ describe Protip::Resource::Associations::BelongsToPolymorphicAssociation do
       end
     end
   end
-end
+end

data/test/unit/protip/resource_test.rb

@@ -1043,7 +1043,7 @@ module Protip::ResourceTest # Namespace for internal constants
               send method, :association_name, bad_option: 'bad', &block
             end
           end
-          assert_match /bad_option/, error.message
+          assert_match(/bad_option/, error.message)
         end
       end
     end

data/test/unit/protip/transformers/big_decimal_transformer_test.rb

@@ -27,14 +27,14 @@ describe Protip::Transformers::BigDecimalTransformer do
       end
     end
     describe 'for BigDecimal arguments' do
-      it_transforms 'integer', BigDecimal.new(104, 1)
-      it_transforms 'fractions', BigDecimal.new(100.5, 5)
-      it_transforms 'rational numbers', BigDecimal.new(Rational(2, 3), 3)
+      it_transforms 'integer', BigDecimal(104, 1)
+      it_transforms 'fractions', BigDecimal(100.5, 5)
+      it_transforms 'rational numbers', BigDecimal(Rational(2, 3), 3)
     end
 
     describe 'for non-BigDecimal arguments' do
-      it_transforms 'integer', 3, BigDecimal.new(3, 1)
-      it_transforms 'string', '3.3', BigDecimal.new(3.3, 2)
+      it_transforms 'integer', 3, BigDecimal(3, 1)
+      it_transforms 'string', '3.3', BigDecimal(3.3, 2)
 
       # Match standard BigDecimal behavior for floats
       it 'raises an argument error for floats' do
@@ -45,4 +45,4 @@ describe Protip::Transformers::BigDecimalTransformer do
       end
     end
   end
-end
+end

data/test/unit/protip/transformers/default_transformer_test.rb

@@ -5,7 +5,7 @@ require 'protip/transformers/default_transformer'
 describe Protip::Transformers::DefaultTransformer do
   describe '#initialize' do
     it 'initializes without errors' do
-      transformer = Protip::Transformers::DefaultTransformer.new
+      Protip::Transformers::DefaultTransformer.new
     end
   end
-end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: protip
 version: !ruby/object:Gem::Version
-  version: 0.33.0
+  version: 0.35.4
 platform: ruby
 authors:
 - AngelList
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-09-11 00:00:00.000000000 Z
+date: 2020-11-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -16,42 +16,40 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.2.0
+        version: 4.2.10
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.2.0
+        version: 4.2.10
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.2.0
+        version: 4.2.10
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.2.0
-- !ruby/object:Gem::Dependency
-  name: google-protobuf
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
+        version: 4.2.10
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: money
   requirement: !ruby/object:Gem::Requirement
@@ -72,20 +70,34 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '7.0'
+- !ruby/object:Gem::Dependency
+  name: google-protobuf
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.7.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.7.1
 - !ruby/object:Gem::Dependency
   name: grpc-tools
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 1.3.4
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 1.3.4
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -106,85 +118,85 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.5'
+        version: '0.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.5'
+        version: '0.6'
 - !ruby/object:Gem::Dependency
   name: mocha
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.11'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.11'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 12.3.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 12.3.3
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.10'
+        version: '0.18'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.10'
+        version: '0.18'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.10'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.10'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.3.0
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.3.0
-description: 
+        version: '3'
+description:
 email:
 - team@angel.co
 executables: []
@@ -243,6 +255,7 @@ files:
 - lib/protip/transformers/enum_transformer.rb
 - lib/protip/transformers/primitives_transformer.rb
 - lib/protip/transformers/timestamp_transformer.rb
+- lib/protip/version.rb
 - test/functional/protip/decorator_test.rb
 - test/functional/protip/resource_test.rb
 - test/test_helper.rb
@@ -264,7 +277,7 @@ homepage: https://github.com/AngelList/protip
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -279,9 +292,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.2.3
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: Resources backed by protobuf messages
 test_files: []