protip 0.33.0 → 0.35.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: b00d08f38111e8b0182e50c6f4e452787d0f9168
-  data.tar.gz: 963dc1bde387ed8308925a2185a4f9b5d5494631
+SHA256:
+  metadata.gz: e4f44a0af07ee5f3839ebb17387a687388f9829629017b84d73797d2f2e98bbe
+  data.tar.gz: 5495fdb816a03f86b227f67a5b5a2c9f87849bc702c8de9382d887b7eafbbb71
 SHA512:
-  metadata.gz: 736fe9fc7445f0fa83b608a142cd74001ef30b96304be7297f1653016fa795668cfc7685f0f4e2ca01047ab14fc033b41425e2f5e2ea0c5db8e9454cf850be31
-  data.tar.gz: 0ad6ab1ff91ff83179093f5dbdc71a776632c0147687e88819894525a90aaea5cfcf08637663d3927aa46681b89c62f3ae06545b19a96eda5b77da712dc699b8
+  metadata.gz: 2ced23785cbd438817f3c07a2cb8e68d2eebcf7deeb126235ca12bbcbcc25432a1e6373985650d8814d27a50ec717b2c7fd403aca77f510089d8be6947ff3748
+  data.tar.gz: 87a05ad4ec2e849cdc84ab197e613a04fdcdae609393b4fde5f7d121c1b503670749e8233f65f3bc1a40bc75fb3d31f2652c31f22e88da03af5ab5c877b061aa

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/messages/active_support/time_with_zone_pb.rb

@@ -4,9 +4,11 @@
 require 'google/protobuf'
 
 Google::Protobuf::DescriptorPool.generated_pool.build do
-  add_message "protip.messages.ActiveSupport.TimeWithZone" do
-    optional :utc_timestamp, :int64, 1
-    optional :time_zone_name, :string, 2
+  add_file("protip/messages/active_support/time_with_zone.proto", :syntax => :proto3) do
+    add_message "protip.messages.ActiveSupport.TimeWithZone" do
+      optional :utc_timestamp, :int64, 1
+      optional :time_zone_name, :string, 2
+    end
   end
 end
 

data/lib/protip/messages/array_pb.rb

@@ -4,8 +4,10 @@
 require 'google/protobuf'
 
 Google::Protobuf::DescriptorPool.generated_pool.build do
-  add_message "protip.messages.Array" do
-    repeated :messages, :bytes, 1
+  add_file("protip/messages/array.proto", :syntax => :proto3) do
+    add_message "protip.messages.Array" do
+      repeated :messages, :bytes, 1
+    end
   end
 end
 

data/lib/protip/messages/big_decimal_value_pb.rb

@@ -4,10 +4,12 @@
 require 'google/protobuf'
 
 Google::Protobuf::DescriptorPool.generated_pool.build do
-  add_message "protip.messages.BigDecimalValue" do
-    optional :numerator, :int64, 1
-    optional :denominator, :uint64, 2
-    optional :precision, :uint32, 3
+  add_file("protip/messages/big_decimal_value.proto", :syntax => :proto3) do
+    add_message "protip.messages.BigDecimalValue" do
+      optional :numerator, :int64, 1
+      optional :denominator, :uint64, 2
+      optional :precision, :uint32, 3
+    end
   end
 end
 

data/lib/protip/messages/currency_pb.rb

@@ -4,183 +4,185 @@
 require 'google/protobuf'
 
 Google::Protobuf::DescriptorPool.generated_pool.build do
-  add_message "protip.messages.Currency" do
-    optional :currency_code, :enum, 1, "protip.messages.Currency.CurrencyCode"
-  end
-  add_enum "protip.messages.Currency.CurrencyCode" do
-    value :AED, 0
-    value :AFN, 1
-    value :ALL, 2
-    value :AMD, 3
-    value :ANG, 4
-    value :AOA, 5
-    value :ARS, 6
-    value :AUD, 7
-    value :AWG, 8
-    value :AZN, 9
-    value :BAM, 10
-    value :BBD, 11
-    value :BDT, 12
-    value :BGN, 13
-    value :BHD, 14
-    value :BIF, 15
-    value :BMD, 16
-    value :BND, 17
-    value :BOB, 18
-    value :BRL, 19
-    value :BSD, 20
-    value :BTN, 21
-    value :BWP, 22
-    value :BYR, 23
-    value :BZD, 24
-    value :CAD, 25
-    value :CDF, 26
-    value :CHF, 27
-    value :CLF, 28
-    value :CLP, 29
-    value :CNY, 30
-    value :COP, 31
-    value :CRC, 32
-    value :CUC, 33
-    value :CUP, 34
-    value :CVE, 35
-    value :CZK, 36
-    value :DJF, 37
-    value :DKK, 38
-    value :DOP, 39
-    value :DZD, 40
-    value :EGP, 41
-    value :ERN, 42
-    value :ETB, 43
-    value :EUR, 44
-    value :FJD, 45
-    value :FKP, 46
-    value :GBP, 47
-    value :GEL, 48
-    value :GHS, 49
-    value :GIP, 50
-    value :GMD, 51
-    value :GNF, 52
-    value :GTQ, 53
-    value :GYD, 54
-    value :HKD, 55
-    value :HNL, 56
-    value :HRK, 57
-    value :HTG, 58
-    value :HUF, 59
-    value :IDR, 60
-    value :ILS, 61
-    value :INR, 62
-    value :IQD, 63
-    value :IRR, 64
-    value :ISK, 65
-    value :JMD, 66
-    value :JOD, 67
-    value :JPY, 68
-    value :KES, 69
-    value :KGS, 70
-    value :KHR, 71
-    value :KMF, 72
-    value :KPW, 73
-    value :KRW, 74
-    value :KWD, 75
-    value :KYD, 76
-    value :KZT, 77
-    value :LAK, 78
-    value :LBP, 79
-    value :LKR, 80
-    value :LRD, 81
-    value :LSL, 82
-    value :LTL, 83
-    value :LVL, 84
-    value :LYD, 85
-    value :MAD, 86
-    value :MDL, 87
-    value :MGA, 88
-    value :MKD, 89
-    value :MMK, 90
-    value :MNT, 91
-    value :MOP, 92
-    value :MRO, 93
-    value :MUR, 94
-    value :MVR, 95
-    value :MWK, 96
-    value :MXN, 97
-    value :MYR, 98
-    value :MZN, 99
-    value :NAD, 100
-    value :NGN, 101
-    value :NIO, 102
-    value :NOK, 103
-    value :NPR, 104
-    value :NZD, 105
-    value :OMR, 106
-    value :PAB, 107
-    value :PEN, 108
-    value :PGK, 109
-    value :PHP, 110
-    value :PKR, 111
-    value :PLN, 112
-    value :PYG, 113
-    value :QAR, 114
-    value :RON, 115
-    value :RSD, 116
-    value :RUB, 117
-    value :RWF, 118
-    value :SAR, 119
-    value :SBD, 120
-    value :SCR, 121
-    value :SDG, 122
-    value :SEK, 123
-    value :SGD, 124
-    value :SHP, 125
-    value :SKK, 126
-    value :SLL, 127
-    value :SOS, 128
-    value :SRD, 129
-    value :SSP, 130
-    value :STD, 131
-    value :SVC, 132
-    value :SYP, 133
-    value :SZL, 134
-    value :THB, 135
-    value :TJS, 136
-    value :TMT, 137
-    value :TND, 138
-    value :TOP, 139
-    value :TRY, 140
-    value :TTD, 141
-    value :TWD, 142
-    value :TZS, 143
-    value :UAH, 144
-    value :UGX, 145
-    value :USD, 146
-    value :UYU, 147
-    value :UZS, 148
-    value :VEF, 149
-    value :VND, 150
-    value :VUV, 151
-    value :WST, 152
-    value :XAF, 153
-    value :XAG, 154
-    value :XAU, 155
-    value :XCD, 156
-    value :XDR, 157
-    value :XOF, 158
-    value :XPF, 159
-    value :YER, 160
-    value :ZAR, 161
-    value :ZMK, 162
-    value :ZMW, 163
-    value :BTC, 164
-    value :JEP, 165
-    value :EEK, 166
-    value :MTL, 167
-    value :TMM, 168
-    value :ZWD, 169
-    value :ZWL, 170
-    value :ZWN, 171
-    value :ZWR, 172
+  add_file("protip/messages/currency.proto", :syntax => :proto3) do
+    add_message "protip.messages.Currency" do
+      optional :currency_code, :enum, 1, "protip.messages.Currency.CurrencyCode"
+    end
+    add_enum "protip.messages.Currency.CurrencyCode" do
+      value :AED, 0
+      value :AFN, 1
+      value :ALL, 2
+      value :AMD, 3
+      value :ANG, 4
+      value :AOA, 5
+      value :ARS, 6
+      value :AUD, 7
+      value :AWG, 8
+      value :AZN, 9
+      value :BAM, 10
+      value :BBD, 11
+      value :BDT, 12
+      value :BGN, 13
+      value :BHD, 14
+      value :BIF, 15
+      value :BMD, 16
+      value :BND, 17
+      value :BOB, 18
+      value :BRL, 19
+      value :BSD, 20
+      value :BTN, 21
+      value :BWP, 22
+      value :BYR, 23
+      value :BZD, 24
+      value :CAD, 25
+      value :CDF, 26
+      value :CHF, 27
+      value :CLF, 28
+      value :CLP, 29
+      value :CNY, 30
+      value :COP, 31
+      value :CRC, 32
+      value :CUC, 33
+      value :CUP, 34
+      value :CVE, 35
+      value :CZK, 36
+      value :DJF, 37
+      value :DKK, 38
+      value :DOP, 39
+      value :DZD, 40
+      value :EGP, 41
+      value :ERN, 42
+      value :ETB, 43
+      value :EUR, 44
+      value :FJD, 45
+      value :FKP, 46
+      value :GBP, 47
+      value :GEL, 48
+      value :GHS, 49
+      value :GIP, 50
+      value :GMD, 51
+      value :GNF, 52
+      value :GTQ, 53
+      value :GYD, 54
+      value :HKD, 55
+      value :HNL, 56
+      value :HRK, 57
+      value :HTG, 58
+      value :HUF, 59
+      value :IDR, 60
+      value :ILS, 61
+      value :INR, 62
+      value :IQD, 63
+      value :IRR, 64
+      value :ISK, 65
+      value :JMD, 66
+      value :JOD, 67
+      value :JPY, 68
+      value :KES, 69
+      value :KGS, 70
+      value :KHR, 71
+      value :KMF, 72
+      value :KPW, 73
+      value :KRW, 74
+      value :KWD, 75
+      value :KYD, 76
+      value :KZT, 77
+      value :LAK, 78
+      value :LBP, 79
+      value :LKR, 80
+      value :LRD, 81
+      value :LSL, 82
+      value :LTL, 83
+      value :LVL, 84
+      value :LYD, 85
+      value :MAD, 86
+      value :MDL, 87
+      value :MGA, 88
+      value :MKD, 89
+      value :MMK, 90
+      value :MNT, 91
+      value :MOP, 92
+      value :MRO, 93
+      value :MUR, 94
+      value :MVR, 95
+      value :MWK, 96
+      value :MXN, 97
+      value :MYR, 98
+      value :MZN, 99
+      value :NAD, 100
+      value :NGN, 101
+      value :NIO, 102
+      value :NOK, 103
+      value :NPR, 104
+      value :NZD, 105
+      value :OMR, 106
+      value :PAB, 107
+      value :PEN, 108
+      value :PGK, 109
+      value :PHP, 110
+      value :PKR, 111
+      value :PLN, 112
+      value :PYG, 113
+      value :QAR, 114
+      value :RON, 115
+      value :RSD, 116
+      value :RUB, 117
+      value :RWF, 118
+      value :SAR, 119
+      value :SBD, 120
+      value :SCR, 121
+      value :SDG, 122
+      value :SEK, 123
+      value :SGD, 124
+      value :SHP, 125
+      value :SKK, 126
+      value :SLL, 127
+      value :SOS, 128
+      value :SRD, 129
+      value :SSP, 130
+      value :STD, 131
+      value :SVC, 132
+      value :SYP, 133
+      value :SZL, 134
+      value :THB, 135
+      value :TJS, 136
+      value :TMT, 137
+      value :TND, 138
+      value :TOP, 139
+      value :TRY, 140
+      value :TTD, 141
+      value :TWD, 142
+      value :TZS, 143
+      value :UAH, 144
+      value :UGX, 145
+      value :USD, 146
+      value :UYU, 147
+      value :UZS, 148
+      value :VEF, 149
+      value :VND, 150
+      value :VUV, 151
+      value :WST, 152
+      value :XAF, 153
+      value :XAG, 154
+      value :XAU, 155
+      value :XCD, 156
+      value :XDR, 157
+      value :XOF, 158
+      value :XPF, 159
+      value :YER, 160
+      value :ZAR, 161
+      value :ZMK, 162
+      value :ZMW, 163
+      value :BTC, 164
+      value :JEP, 165
+      value :EEK, 166
+      value :MTL, 167
+      value :TMM, 168
+      value :ZWD, 169
+      value :ZWL, 170
+      value :ZWN, 171
+      value :ZWR, 172
+    end
   end
 end
 

data/lib/protip/messages/errors_pb.rb

@@ -4,13 +4,15 @@
 require 'google/protobuf'
 
 Google::Protobuf::DescriptorPool.generated_pool.build do
-  add_message "protip.messages.Errors" do
-    repeated :messages, :string, 1
-    repeated :field_errors, :message, 2, "protip.messages.FieldError"
-  end
-  add_message "protip.messages.FieldError" do
-    optional :field, :string, 1
-    optional :message, :string, 2
+  add_file("protip/messages/errors.proto", :syntax => :proto3) do
+    add_message "protip.messages.Errors" do
+      repeated :messages, :string, 1
+      repeated :field_errors, :message, 2, "protip.messages.FieldError"
+    end
+    add_message "protip.messages.FieldError" do
+      optional :field, :string, 1
+      optional :message, :string, 2
+    end
   end
 end
 

data/lib/protip/messages/money_pb.rb

@@ -5,9 +5,11 @@ require 'google/protobuf'
 
 require 'protip/messages/currency_pb'
 Google::Protobuf::DescriptorPool.generated_pool.build do
-  add_message "protip.messages.Money" do
-    optional :amount_cents, :int64, 1
-    optional :currency, :message, 2, "protip.messages.Currency"
+  add_file("protip/messages/money.proto", :syntax => :proto3) do
+    add_message "protip.messages.Money" do
+      optional :amount_cents, :int64, 1
+      optional :currency, :message, 2, "protip.messages.Currency"
+    end
   end
 end
 

data/lib/protip/messages/range_pb.rb

@@ -4,9 +4,11 @@
 require 'google/protobuf'
 
 Google::Protobuf::DescriptorPool.generated_pool.build do
-  add_message "protip.messages.Range" do
-    optional :begin, :int64, 1
-    optional :end, :int64, 2
+  add_file("protip/messages/range.proto", :syntax => :proto3) do
+    add_message "protip.messages.Range" do
+      optional :begin, :int64, 1
+      optional :end, :int64, 2
+    end
   end
 end
 

data/lib/protip/messages/repeated_wrappers_pb.rb

@@ -5,35 +5,37 @@ require 'google/protobuf'
 
 require 'protip/extensions_pb'
 Google::Protobuf::DescriptorPool.generated_pool.build do
-  add_message "protip.messages.RepeatedDouble" do
-    repeated :values, :double, 1
-  end
-  add_message "protip.messages.RepeatedFloat" do
-    repeated :values, :float, 1
-  end
-  add_message "protip.messages.RepeatedInt64" do
-    repeated :values, :int64, 1
-  end
-  add_message "protip.messages.RepeatedUInt64" do
-    repeated :values, :uint64, 1
-  end
-  add_message "protip.messages.RepeatedInt32" do
-    repeated :values, :int32, 1
-  end
-  add_message "protip.messages.RepeatedUInt32" do
-    repeated :values, :uint32, 1
-  end
-  add_message "protip.messages.RepeatedBool" do
-    repeated :values, :bool, 1
-  end
-  add_message "protip.messages.RepeatedString" do
-    repeated :values, :string, 1
-  end
-  add_message "protip.messages.RepeatedBytes" do
-    repeated :values, :bytes, 1
-  end
-  add_message "protip.messages.RepeatedEnum" do
-    repeated :values, :int32, 1
+  add_file("protip/messages/repeated_wrappers.proto", :syntax => :proto3) do
+    add_message "protip.messages.RepeatedDouble" do
+      repeated :values, :double, 1
+    end
+    add_message "protip.messages.RepeatedFloat" do
+      repeated :values, :float, 1
+    end
+    add_message "protip.messages.RepeatedInt64" do
+      repeated :values, :int64, 1
+    end
+    add_message "protip.messages.RepeatedUInt64" do
+      repeated :values, :uint64, 1
+    end
+    add_message "protip.messages.RepeatedInt32" do
+      repeated :values, :int32, 1
+    end
+    add_message "protip.messages.RepeatedUInt32" do
+      repeated :values, :uint32, 1
+    end
+    add_message "protip.messages.RepeatedBool" do
+      repeated :values, :bool, 1
+    end
+    add_message "protip.messages.RepeatedString" do
+      repeated :values, :string, 1
+    end
+    add_message "protip.messages.RepeatedBytes" do
+      repeated :values, :bytes, 1
+    end
+    add_message "protip.messages.RepeatedEnum" do
+      repeated :values, :int32, 1
+    end
   end
 end
 

data/lib/protip/messages/test_pb.rb

@@ -6,13 +6,15 @@ require 'google/protobuf'
 require 'protip/messages/repeated_wrappers_pb'
 require 'protip/messages/wrappers_pb'
 Google::Protobuf::DescriptorPool.generated_pool.build do
-  add_message "protip.messages.EnumTest" do
-    optional :enum, :message, 1, "protip.messages.EnumValue"
-    optional :repeated_enums, :message, 2, "protip.messages.RepeatedEnum"
-  end
-  add_enum "protip.messages.EnumTest.Enum" do
-    value :ZERO, 0
-    value :ONE, 1
+  add_file("protip/messages/test.proto", :syntax => :proto3) do
+    add_message "protip.messages.EnumTest" do
+      optional :enum, :message, 1, "protip.messages.EnumValue"
+      optional :repeated_enums, :message, 2, "protip.messages.RepeatedEnum"
+    end
+    add_enum "protip.messages.EnumTest.Enum" do
+      value :ZERO, 0
+      value :ONE, 1
+    end
   end
 end
 

data/lib/protip/messages/types_pb.rb

@@ -4,10 +4,12 @@
 require 'google/protobuf'
 
 Google::Protobuf::DescriptorPool.generated_pool.build do
-  add_message "protip.messages.Date" do
-    optional :year, :int64, 1
-    optional :month, :uint32, 2
-    optional :day, :uint32, 3
+  add_file("protip/messages/types.proto", :syntax => :proto3) do
+    add_message "protip.messages.Date" do
+      optional :year, :int64, 1
+      optional :month, :uint32, 2
+      optional :day, :uint32, 3
+    end
   end
 end
 

data/lib/protip/messages/wrappers_pb.rb

@@ -5,8 +5,10 @@ require 'google/protobuf'
 
 require 'protip/extensions_pb'
 Google::Protobuf::DescriptorPool.generated_pool.build do
-  add_message "protip.messages.EnumValue" do
-    optional :value, :int32, 1
+  add_file("protip/messages/wrappers.proto", :syntax => :proto3) do
+    add_message "protip.messages.EnumValue" do
+      optional :value, :int32, 1
+    end
   end
 end
 

data/lib/protip/messages_pb.rb

@@ -14,5 +14,7 @@ require 'protip/messages/types_pb'
 require 'protip/messages/wrappers_pb'
 require 'protip/messages/active_support/time_with_zone_pb'
 Google::Protobuf::DescriptorPool.generated_pool.build do
+  add_file("protip/messages.proto", :syntax => :proto3) do
+  end
 end
 

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.0'
+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.0
 platform: ruby
 authors:
 - AngelList
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-09-11 00:00:00.000000000 Z
+date: 2020-03-25 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.9.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.9.1
 - !ruby/object:Gem::Dependency
   name: grpc-tools
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 1.23.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 1.23.0
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -106,84 +118,84 @@ 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
+        version: '3'
 description: 
 email:
 - team@angel.co
@@ -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
@@ -279,8 +292,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.2.3
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Resources backed by protobuf messages