Fix BaseController.ok on non string content (#423)

* refactor: update JsonResult with no generic

* fix: update OkNegotiatedContentResult

update OkNegotiatedContentResult to rely on JsonContent when non string content is provided

Author: notaphplover

CHANGELOG.md

@@ -12,8 +12,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 - Updated `BaseMiddleware.handler` to allow async handlers.
 - Updated `Middleware` to allow include any `ServiceIdentifier`.
+- Updated `JsonContent` with no generic.
 
 ### Fixed
+- Updated `BaseController.ok` to no longer return `text/plain` responses when non string content is passed.
 
 ## [6.4.10]
 

package-lock.json

@@ -52,6 +52,7 @@
     "@types/supertest": "6.0.2",
     "@typescript-eslint/eslint-plugin": "8.20.0",
     "@typescript-eslint/parser": "8.20.0",
+    "body-parser": "1.20.3",
     "cookie-parser": "1.4.7",
     "eslint": "9.18.0",
     "eslint-config-prettier": "10.0.1",

package.json

@@ -79,10 +79,10 @@ export class BaseHttpController {
     return new StatusCodeResult(statusCode);
   }
 
-  protected json<T extends Record<string, unknown>>(
-    content: T | T[],
+  protected json(
+    content: unknown,
     statusCode: number = StatusCodes.OK,
-  ): JsonResult<T> {
+  ): JsonResult {
     return new JsonResult(content, statusCode);
   }
 

src/base_http_controller.ts

@@ -1,5 +1,4 @@
 import type { OutgoingHttpHeaders } from 'node:http';
-import type { Readable } from 'node:stream';
 
 export abstract class HttpContent {
   private readonly _headers: OutgoingHttpHeaders = {};
@@ -8,7 +7,5 @@ export abstract class HttpContent {
     return this._headers;
   }
 
-  public abstract readAsync(): Promise<
-    string | Record<string, unknown> | Record<string, unknown>[] | Readable
-  >;
+  public abstract readAsync(): Promise<unknown>;
 }

src/content/httpContent.ts

@@ -2,16 +2,14 @@ import { HttpContent } from './httpContent';
 
 const DEFAULT_MEDIA_TYPE: string = 'application/json';
 
-export class JsonContent<
-  T extends Record<string, unknown>,
-> extends HttpContent {
-  constructor(private readonly content: T | T[]) {
+export class JsonContent extends HttpContent {
+  constructor(private readonly content: unknown) {
     super();
 
     this.headers['content-type'] = DEFAULT_MEDIA_TYPE;
   }
 
-  public async readAsync(): Promise<T | T[]> {
+  public async readAsync(): Promise<unknown> {
     return this.content;
   }
 }

src/content/jsonContent.ts

@@ -2,19 +2,17 @@ import { JsonContent } from '../content/jsonContent';
 import { HttpResponseMessage } from '../httpResponseMessage';
 import type { IHttpActionResult } from '../interfaces';
 
-export class JsonResult<T extends Record<string, unknown>>
-  implements IHttpActionResult
-{
+export class JsonResult implements IHttpActionResult {
   constructor(
-    public readonly json: T | T[],
+    public readonly json: unknown,
     public readonly statusCode: number,
   ) {}
 
   public async executeAsync(): Promise<HttpResponseMessage> {
     const response: HttpResponseMessage = new HttpResponseMessage(
       this.statusCode,
     );
-    response.content = new JsonContent<T>(this.json);
+    response.content = new JsonContent(this.json);
 
     return response;
   }

src/results/JsonResult.ts

@@ -1,5 +1,6 @@
 import { StatusCodes } from 'http-status-codes';
 
+import { JsonContent } from '../content/jsonContent';
 import { StringContent } from '../content/stringContent';
 import { HttpResponseMessage } from '../httpResponseMessage';
 import type { IHttpActionResult } from '../interfaces';
@@ -11,7 +12,12 @@ export class OkNegotiatedContentResult<T> implements IHttpActionResult {
     const response: HttpResponseMessage = new HttpResponseMessage(
       StatusCodes.OK,
     );
-    response.content = new StringContent(JSON.stringify(this.content));
+
+    if (typeof this.content === 'string') {
+      response.content = new StringContent(this.content);
+    } else {
+      response.content = new JsonContent(this.content);
+    }
 
     return response;
   }

src/results/OkNegotiatedContentResult.ts

@@ -41,9 +41,7 @@ describe('ActionResults', () => {
         await actionResult.executeAsync();
 
       expect(responseMessage.statusCode).toBe(StatusCodes.OK);
-      expect(await responseMessage.content.readAsync()).toBe(
-        JSON.stringify(content),
-      );
+      expect(await responseMessage.content.readAsync()).toStrictEqual(content);
     });
   });
 

src/test/action_result.test.ts

@@ -4,7 +4,7 @@ import { JsonContent } from '../../content/jsonContent';
 
 describe('JsonContent', () => {
   it('should have application/json as the default media type', () => {
-    const content: JsonContent<Record<string, unknown>> = new JsonContent({});
+    const content: JsonContent = new JsonContent({});
     expect(content.headers['content-type']).toBe('application/json');
   });