Expand error documentation for query protocol (#3205)

Author: jterapin

build_tools/aws-sdk-code-generator/lib/aws-sdk-code-generator/error_list.rb

@@ -25,7 +25,8 @@ def initialize(options)
             members: members,
             data_type: "#{@module_name}::Types::#{name}",
             retryable: !!shape['retryable'],
-            throttling: throttling?(shape)
+            throttling: throttling?(shape),
+            custom_error_class: custom_error_class(shape)
           )
         end
         es
@@ -45,6 +46,11 @@ def throttling?(shape)
       shape['retryable'] && shape['retryable'].kind_of?(Hash) && shape['retryable']['throttling']
     end
 
+    def custom_error_class(shape)
+      return unless (custom_error_name = shape.dig('error', 'code'))
+
+      custom_error_name
+    end
 
     class Error
 
@@ -54,6 +60,7 @@ def initialize(options)
         @members = options[:members]
         @retryable = options[:retryable]
         @throttling = options[:throttling]
+        @custom_error_class = options[:custom_error_class]
       end
 
       # @return [String]
@@ -71,6 +78,14 @@ def initialize(options)
       # @return [Boolean]
       attr_reader :throttling
 
+      # @return [String, nil]
+      attr_reader :custom_error_class
+
+      # Covers dynamically created errors by query protocol
+      def custom_error_class?
+        !@custom_error_class.nil? && @name != @custom_error_class
+      end
+
     end
 
   end

build_tools/aws-sdk-code-generator/lib/aws-sdk-code-generator/views/errors_module.rb

@@ -20,6 +20,7 @@ def initialize(options)
       # @return [String|nil]
       def generated_src_warning
         return if @service.protocol == 'api-gateway'
+
         GENERATED_SRC_WARNING
       end
 
@@ -42,6 +43,10 @@ def customization_file_path
         "#{gem_name}/customizations/errors"
       end
 
+      def query_protocol?
+        @service.protocol == 'query'
+      end
+
       private
 
       # @return [String]

build_tools/aws-sdk-code-generator/templates/errors_module.mustache

@@ -27,15 +27,29 @@ module {{module_name}}
   {{=<% %>=}}
   # * {<% name %>}
   <%={{ }}=%>
+  {{#query_protocol?}}
+  {{#custom_error_class?}}
+  #    * This error class is not used. `{{custom_error_class}}` is used during parsing instead.
+  {{/custom_error_class?}}
+  {{/query_protocol?}}
   {{/errors}}
   #
   # Additionally, error classes are dynamically generated for service errors based on the error code
   # if they are not defined above.
+  {{#query_protocol?}}
+  # Some existing error classes may use a different class name than the one documented.
+  {{/query_protocol?}}
   module Errors
 
     extend Aws::Errors::DynamicErrors
 
     {{#errors}}
+    {{#query_protocol?}}
+    {{#custom_error_class?}}
+    # @deprecated This error class is not used during parsing.
+    #   Please use `{{custom_error_class}}` instead.
+    {{/custom_error_class?}}
+    {{/query_protocol?}}
     class {{name}} < ServiceError
 
       # @param [Seahorse::Client::RequestContext] context