Fix tapioca compiler behaviour for custom return types

Author: Verseth

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    shale-builder (0.5.1)
+    shale-builder (0.5.2)
       booleans (>= 0.1)
       shale (< 2.0)
       sorbet-runtime (> 0.5)

lib/shale/builder/version.rb

@@ -2,6 +2,6 @@
 
 module Shale
   module Builder
-    VERSION = '0.5.1'
+    VERSION = '0.5.2'
   end
 end

lib/tapioca/dsl/compilers/shale.rb

@@ -40,6 +40,7 @@ def decorate
             attribute = constant.attributes[attribute_name] #: ::Shale::Attribute
             if (type = attribute.return_type)
               return_type = type
+              nilable = true
             else
               return_type, nilable = shale_type_to_sorbet_return_type(attribute)
             end
@@ -58,6 +59,7 @@ def decorate
 
             if (type = attribute.return_type || attribute.setter_type)
               setter_type = type
+              nilable = true
             else
               setter_type, nilable = shale_type_to_sorbet_setter_type(attribute)
             end

test/shale/builder_test.rb

@@ -26,6 +26,7 @@ class TestTransactionResponseType < ::Shale::Mapper
     attribute :cvv_code, ::Shale::Type::String
     attribute :amount, TestAmountType
     attribute :success, ::Shale::Type::Boolean
+    attribute :aux, ::Shale::Type::Value, return_type: Integer
   end
 
   class TestTransactionType < ::Shale::Mapper