patch(vest): disallow bad include input

Author: ealush

packages/vest/src/hooks/__tests__/include.test.ts

@@ -8,6 +8,15 @@ describe('include', () => {
     cb2 = jest.fn(() => false);
   });
 
+  describe('When not passing a string fieldName', () => {
+    it('Should throw an error', () => {
+      // @ts-ignore
+      expect(() => vest.include({})).toThrow();
+      // @ts-ignore
+      expect(() => vest.include(undefined)).toThrow();
+    });
+  });
+
   describe('There is an `onlyd` field', () => {
     describe('`include` is run as-is without modifiers', () => {
       it('Should run the included test along with the onlyd test', () => {

packages/vest/src/hooks/include.ts

@@ -1,5 +1,6 @@
 import defaultTo from 'defaultTo';
 import hasOwnProperty from 'hasOwnProperty';
+import invariant from 'invariant';
 import { isStringValue } from 'isStringValue';
 import optionalFunctionValue from 'optionalFunctionValue';
 
@@ -14,9 +15,7 @@ export default function include(fieldName: string): {
   const context = ctx.useX();
   const { inclusion, exclusion } = context;
 
-  if (!fieldName) {
-    return { when };
-  }
+  invariant(isStringValue(fieldName));
 
   inclusion[fieldName] = defaultTo(exclusion.tests[fieldName], true);