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

@@ -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');
   });
 

src/test/content/jsonContent.test.ts

@@ -0,0 +1,57 @@
+import { beforeEach, describe, expect, it } from '@jest/globals';
+import * as bodyParser from 'body-parser';
+import { Application, Request } from 'express';
+import { Container } from 'inversify';
+import supertest from 'supertest';
+import TestAgent from 'supertest/lib/agent';
+
+import { BaseHttpController } from '../../base_http_controller';
+import { controller, httpPut, request } from '../../decorators';
+import { InversifyExpressServer } from '../../server';
+import { cleanUpMetadata } from '../../utils';
+
+describe('Issue 420', () => {
+  beforeEach(() => {
+    cleanUpMetadata();
+  });
+
+  it('should work with no url params', async () => {
+    @controller('/controller')
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    class TestController extends BaseHttpController {
+      @httpPut('/test')
+      public async updateTest(
+        @request()
+        req: Request<unknown, unknown, { test: string }, void>,
+      ) {
+        return this.ok({ message: req.body.test });
+      }
+    }
+
+    const container: Container = new Container();
+
+    const server: InversifyExpressServer = new InversifyExpressServer(
+      container,
+    );
+
+    server.setConfig((app: Application) => {
+      app.use(
+        bodyParser.urlencoded({
+          extended: true,
+        }),
+      );
+      app.use(bodyParser.json());
+    });
+
+    const agent: TestAgent<supertest.Test> = supertest(server.build());
+
+    const response: supertest.Response = await agent
+      .put('/controller/test')
+      .send({ test: 'test' })
+      .set('Content-Type', 'application/json')
+      .set('Accept', 'application/json');
+
+    expect(response.status).toBe(200);
+    expect(response.body).toStrictEqual({ message: 'test' });
+  });
+});