Better middleware array check

Author: compulim

packages/api/src/middleware/private/templateMiddleware.check.test.tsx

@@ -1,5 +1,17 @@
 import templateMiddleware from './templateMiddleware';
 
+test('should warn if middleware is not an array of function', () => {
+  const warn = jest.fn();
+  const template = templateMiddleware('Check');
+
+  jest.spyOn(console, 'warn').mockImplementation(warn);
+
+  template.extractMiddleware(1 as any);
+
+  expect(warn).toHaveBeenCalledTimes(1);
+  expect(warn).toHaveBeenNthCalledWith(1, expect.stringContaining('must be an array of function'));
+});
+
 test('should warn if middleware did not return function', () => {
   const warn = jest.fn();
   const template = templateMiddleware('Check');

packages/api/src/middleware/private/templateMiddleware.ts

@@ -28,7 +28,7 @@ function templateMiddleware<Request = any, Props extends {} = EmptyObject>(name:
     return factory;
   };
 
-  const warnInvalid = warnOnce(`"${name}" prop is invalid`);
+  const warnInvalid = warnOnce(`"${name}" middleware prop is must be an array of function`);
 
   const extractMiddleware = (
     middleware: readonly MiddlewareWithInit<ComponentMiddleware<unknown, unknown>, unknown>[] | undefined