fix: resolve PR comments

Author: starfy84

packages/typed-express-router/src/errors.ts

@@ -1,4 +1,5 @@
 import * as PathReporter from 'io-ts/lib/PathReporter';
+
 import type {
   DecodeErrorFormatterFn,
   EncodeErrorFormatterFn,

packages/typed-express-router/src/index.ts

@@ -3,9 +3,11 @@
  */
 
 import { ApiSpec, HttpRoute, KeyToHttpStatus } from '@api-ts/io-ts-http';
+import type { Span } from '@opentelemetry/api';
 import express from 'express';
 import * as E from 'fp-ts/Either';
 import { pipe } from 'fp-ts/pipeable';
+
 import {
   defaultDecodeErrorFormatter,
   defaultEncodeErrorFormatter,
@@ -34,8 +36,6 @@ import {
   WrappedRouterOptions,
 } from './types';
 
-import type { Span } from '@opentelemetry/api';
-
 export type {
   AfterEncodedResponseSentFn,
   DecodeErrorFormatterFn,

packages/typed-express-router/test/server.test.ts

@@ -440,6 +440,27 @@ test('should send a 500 when response type does not match', async () => {
   assert.equal(response.original.status, 500);
 });
 
+test('should invoke default encode error function when response type does not match', async () => {
+  const router = createRouter(TestApiSpec);
+
+  router.get('hello.world', [
+    (_req, res) => {
+      res.sendEncoded(200, { what: 'is this parameter?' } as any);
+    },
+  ]);
+
+  const app = express();
+  app.use(express.json());
+  app.use(router);
+
+  const server = supertest(app);
+  const apiClient = buildApiClient(supertestRequestFactory(server), TestApiSpec);
+  const response = await apiClient['hello.world'].get({ id: '1234' }).decode();
+
+  assert.equal(response.original.status, 500);
+  assert.deepStrictEqual(response.body, {});
+});
+
 test('should invoke custom encode error function when response type does not match', async () => {
   const router = createRouter(TestApiSpec, {
     encodeErrorFormatter: (_err, _req) => {