Adds redirection response deserialization as non-error (#152)

* Adds redirection response deserialization as non-error

* Adds redirection response sample + doc feature

* Sample regen, test added

Author: spacether

docs/generators/java.md

@@ -285,9 +285,10 @@ These options may be applied as additional-properties (cli) or configOptions (pl
 ### Operation Feature
 | Name | Supported | Defined By |
 | ---- | --------- | ---------- |
-|Responses_HttpStatusCode|✓|OAS2,OAS3
-|Responses_RangedResponseCodes|✗|OAS2,OAS3
-|Responses_Default|✓|OAS2,OAS3
+|Responses_HttpStatusCode|✓|OAS3
+|Responses_RangedResponseCodes|✗|OAS3
+|Responses_Default|✓|OAS3
+|Responses_RedirectionResponse|✗|OAS3
 
 ### Parameter Feature
 | Name | Supported | Defined By |

docs/generators/jaxrs-jersey.md

@@ -268,9 +268,10 @@ These options may be applied as additional-properties (cli) or configOptions (pl
 ### Operation Feature
 | Name | Supported | Defined By |
 | ---- | --------- | ---------- |
-|Responses_HttpStatusCode|✓|OAS2,OAS3
-|Responses_RangedResponseCodes|✗|OAS2,OAS3
-|Responses_Default|✓|OAS2,OAS3
+|Responses_HttpStatusCode|✓|OAS3
+|Responses_RangedResponseCodes|✗|OAS3
+|Responses_Default|✓|OAS3
+|Responses_RedirectionResponse|✗|OAS3
 
 ### Parameter Feature
 | Name | Supported | Defined By |

docs/generators/jmeter.md

@@ -127,9 +127,10 @@ These options may be applied as additional-properties (cli) or configOptions (pl
 ### Operation Feature
 | Name | Supported | Defined By |
 | ---- | --------- | ---------- |
-|Responses_HttpStatusCode|✓|OAS2,OAS3
-|Responses_RangedResponseCodes|✗|OAS2,OAS3
-|Responses_Default|✓|OAS2,OAS3
+|Responses_HttpStatusCode|✓|OAS3
+|Responses_RangedResponseCodes|✗|OAS3
+|Responses_Default|✓|OAS3
+|Responses_RedirectionResponse|✗|OAS3
 
 ### Parameter Feature
 | Name | Supported | Defined By |

docs/generators/kotlin.md

@@ -237,9 +237,10 @@ These options may be applied as additional-properties (cli) or configOptions (pl
 ### Operation Feature
 | Name | Supported | Defined By |
 | ---- | --------- | ---------- |
-|Responses_HttpStatusCode|✓|OAS2,OAS3
-|Responses_RangedResponseCodes|✗|OAS2,OAS3
-|Responses_Default|✓|OAS2,OAS3
+|Responses_HttpStatusCode|✓|OAS3
+|Responses_RangedResponseCodes|✗|OAS3
+|Responses_Default|✓|OAS3
+|Responses_RedirectionResponse|✗|OAS3
 
 ### Parameter Feature
 | Name | Supported | Defined By |

docs/generators/python.md

@@ -196,9 +196,10 @@ These options may be applied as additional-properties (cli) or configOptions (pl
 ### Operation Feature
 | Name | Supported | Defined By |
 | ---- | --------- | ---------- |
-|Responses_HttpStatusCode|✓|OAS2,OAS3
-|Responses_RangedResponseCodes|✓|OAS2,OAS3
-|Responses_Default|✓|OAS2,OAS3
+|Responses_HttpStatusCode|✓|OAS3
+|Responses_RangedResponseCodes|✓|OAS3
+|Responses_Default|✓|OAS3
+|Responses_RedirectionResponse|✓|OAS3
 
 ### Parameter Feature
 | Name | Supported | Defined By |

modules/openapi-json-schema-generator-core/src/main/java/org/openapijsonschematools/codegen/meta/features/OperationFeature.java

@@ -23,15 +23,15 @@
  * Defines Operation features supported in the generated code.
  */
 public enum OperationFeature {
-    /**
-     * Supports header-based basic http auth.
-     */
-    @OAS2 @OAS3
+    @OAS3
     Responses_HttpStatusCode,
 
-    @OAS2 @OAS3
+    @OAS3
     Responses_RangedResponseCodes,
 
-    @OAS2 @OAS3
+    @OAS3
     Responses_Default,
+
+    @OAS3
+    Responses_RedirectionResponse,
 }

modules/openapi-json-schema-generator/src/main/java/org/openapijsonschematools/codegen/languages/PythonClientCodegen.java

@@ -228,7 +228,8 @@ public PythonClientCodegen() {
                         ParameterFeature.Content
                 )
                 .includeOperationFeatures(
-                        OperationFeature.Responses_RangedResponseCodes
+                        OperationFeature.Responses_RangedResponseCodes,
+                        OperationFeature.Responses_RedirectionResponse
                 )
                 .excludeParameterFeatures(
                         ParameterFeature.In_Cookie

modules/openapi-json-schema-generator/src/main/java/org/openapijsonschematools/codegen/model/CodegenOperation.java

@@ -85,14 +85,14 @@ public Map<String, CodegenResponse> getNonErrorResponses() {
         HashMap<String,CodegenResponse> nonErrorResponses = new HashMap<>();
         if (statusCodeResponses != null) {
             for (Map.Entry<Integer, CodegenResponse> entry: statusCodeResponses.entrySet()) {
-                if (entry.getKey() >= 200 && entry.getKey() <= 299) {
+                if (entry.getKey() >= 200 && entry.getKey() <= 399) {
                     nonErrorResponses.put(entry.getKey().toString(), entry.getValue());
                 }
             }
         }
         if (wildcardCodeResponses != null) {
             for (Map.Entry<Integer, CodegenResponse> entry: wildcardCodeResponses.entrySet()) {
-                if (entry.getKey() == 2) {
+                if (entry.getKey() == 2 || entry.getKey() == 3) {
                     nonErrorResponses.put(entry.getKey().toString(), entry.getValue());
                 }
             }

modules/openapi-json-schema-generator/src/main/resources/python/paths/path/verb/operation.hbs

@@ -405,7 +405,7 @@ class BaseApi(api_client.Api):
     {{/if}}
 {{/if}}
 
-        if not 200 <= response.status <= 299:
+        if not 200 <= response.status <= 399:
             raise exceptions.ApiException(
                 status=response.status,
                 reason=response.reason,

modules/openapi-json-schema-generator/src/test/resources/3_0/python/petstore_customized.yaml

@@ -1681,6 +1681,17 @@ paths:
           content:
             application/json:
               schema: {}
+  /fake/redirection:
+    get:
+      operationId: redirection
+      summary: operation with redirection responses
+      tags:
+        - fake
+      responses:
+        303:
+          description: see other
+        3XX:
+          description: 3XX response
 servers:
   - url: 'http://{server}.swagger.io:{port}/v2'
     description: petstore server