fix(matchers): avoid TypeError in toContainAllKeys when received is nullish

.changeset/tidy-keys-recover.md

@@ -0,0 +1,5 @@
+---
+'jest-extended': patch
+---
+
+Fix `toContainAllKeys` throwing a `TypeError` instead of producing an assertion failure message when the received value is `null` or `undefined`

src/matchers/toContainAllKeys.ts

@@ -13,6 +13,8 @@ export function toContainAllKeys<E = unknown>(actual: unknown, expected: readonl
       expected.every(key => contains((a, b) => this.equals(a, b, this.customTesters), objectKeys, key));
   }
 
+  const received = actual == null ? actual : Object.keys(actual as Record<string, unknown>);
+
   return {
     pass,
     message: () =>
@@ -22,12 +24,12 @@ export function toContainAllKeys<E = unknown>(actual: unknown, expected: readonl
           'Expected object to not contain all keys:\n' +
           `  ${printExpected(expected)}\n` +
           'Received:\n' +
-          `  ${printReceived(Object.keys(actual as Record<string, unknown>))}`
+          `  ${printReceived(received)}`
         : matcherHint('.toContainAllKeys') +
           '\n\n' +
           'Expected object to contain all keys:\n' +
           `  ${printExpected(expected)}\n` +
           'Received:\n' +
-          `  ${printReceived(Object.keys(actual as Record<string, unknown>))}`,
+          `  ${printReceived(received)}`,
   };
 }

test/matchers/__snapshots__/toContainAllKeys.test.ts.snap

@@ -9,11 +9,27 @@ Received:
   <red>["a", "b"]</color>"
 `;
 
-exports[`.toContainAllKeys fails when actual is not an object 1`] = `"Cannot convert undefined or null to object"`;
+exports[`.toContainAllKeys fails when actual is not an object 1`] = `
+"<dim>expect(</intensity><red>received</color><dim>).toContainAllKeys(</intensity><green>expected</color><dim>)</intensity>
+
+Expected object to contain all keys:
+  <green>["a", "b"]</color>
+Received:
+  <red>null</color>"
+`;
 
 exports[`.toContainAllKeys fails when actual is not an object 2`] = `
 "<dim>expect(</intensity><red>received</color><dim>).toContainAllKeys(</intensity><green>expected</color><dim>)</intensity>
 
+Expected object to contain all keys:
+  <green>["a", "b"]</color>
+Received:
+  <red>undefined</color>"
+`;
+
+exports[`.toContainAllKeys fails when actual is not an object 3`] = `
+"<dim>expect(</intensity><red>received</color><dim>).toContainAllKeys(</intensity><green>expected</color><dim>)</intensity>
+
 Expected object to contain all keys:
   <green>["a", "b"]</color>
 Received:

test/matchers/toContainAllKeys.test.ts

@@ -23,6 +23,7 @@ describe('.toContainAllKeys', () => {
 
   test('fails when actual is not an object', () => {
     expect(() => expect(null).toContainAllKeys(['a', 'b'])).toThrowErrorMatchingSnapshot();
+    expect(() => expect(undefined).toContainAllKeys(['a', 'b'])).toThrowErrorMatchingSnapshot();
     expect(() => expect(42).toContainAllKeys(['a', 'b'])).toThrowErrorMatchingSnapshot();
   });
 });