Merge pull request #1284 from jokj624/fix/HttpStatusCodeLiteral

Fix: Add HttpStatusCodeLiteral 451
lukeautry · Jul 25, 2022 · 0d48d2b · 0d48d2b
2 parents 6299d90 + bdd25f5
commit 0d48d2b
Show file tree

Hide file tree

Showing 6 changed files with 52 additions and 2 deletions.
diff --git a/packages/runtime/src/interfaces/response.ts b/packages/runtime/src/interfaces/response.ts
@@ -48,6 +48,7 @@ export type HttpStatusCodeLiteral =
   | 428
   | 429
   | 431
+  | 451
   | 500
   | 501
   | 502

diff --git a/tests/fixtures/controllers/testController.ts b/tests/fixtures/controllers/testController.ts
@@ -1,5 +1,5 @@
-import { Controller, Get, Route, SuccessResponse } from '@tsoa/runtime';
-import { TestModel } from '../testModel';
+import { Controller, Get, Route, SuccessResponse, Response } from '@tsoa/runtime';
+import { ErrorResponseModel, TestModel } from '../testModel';
 import { NonFactoryDecorator } from '../custom/non-factory-decorator';
 import { ModelService } from '../services/modelService';
 
@@ -55,4 +55,13 @@ export class TestController extends Controller {
   public async getSuccessResponse(): Promise<void> {
     return;
   }
+
+  @Get('unavailableForLegalReasonsStatusCode')
+  @Response(451)
+  public async getUnavailableForLegalReasonsError(): Promise<ErrorResponseModel> {
+    throw {
+      status: 451,
+      messgage: 'error'
+    };
+  }
 }
diff --git a/tests/integration/dynamic-controllers-express-server.spec.ts b/tests/integration/dynamic-controllers-express-server.spec.ts
@@ -336,6 +336,16 @@ describe('Express Server', () => {
         204,
       );
     });
+
+    it('should unavailable for legal reasons status code', () => {
+      return verifyGetRequest(
+        basePath + `/Controller/unavailableForLegalReasonsStatusCode`,
+        (_err, res) => {
+          expect(res.status).to.equal(451);
+        },
+        451,
+      );
+    });
   });
 
   describe('Validate', () => {

diff --git a/tests/integration/express-server.spec.ts b/tests/integration/express-server.spec.ts
@@ -452,6 +452,16 @@ describe('Express Server', () => {
         204,
       );
     });
+
+    it('should unavailable for legal reasons status code', () => {
+      return verifyGetRequest(
+        basePath + `/Controller/unavailableForLegalReasonsStatusCode`,
+        (_err, res) => {
+          expect(res.status).to.equal(451);
+        },
+        451,
+      );
+    });
   });
 
   describe('NoExtends', () => {

diff --git a/tests/integration/hapi-server.spec.ts b/tests/integration/hapi-server.spec.ts
@@ -336,6 +336,16 @@ describe('Hapi Server', () => {
         204,
       );
     });
+
+    it('should unavailable for legal reasons status code', () => {
+      return verifyGetRequest(
+        basePath + `/Controller/unavailableForLegalReasonsStatusCode`,
+        (_err, res) => {
+          expect(res.status).to.equal(451);
+        },
+        451,
+      );
+    });
   });
 
   describe('NoExtends', () => {

diff --git a/tests/integration/koa-server.spec.ts b/tests/integration/koa-server.spec.ts
@@ -314,6 +314,16 @@ describe('Koa Server', () => {
         204,
       );
     });
+
+    it('should unavailable for legal reasons status code', () => {
+      return verifyGetRequest(
+        basePath + `/Controller/unavailableForLegalReasonsStatusCode`,
+        (_err, res) => {
+          expect(res.status).to.equal(451);
+        },
+        451,
+      );
+    });
   });
 
   describe('Custom Content-Type', () => {