Fixup unit tests for isSerializableProps

vercel · Mar 15, 2021 · 55b8d8d · 55b8d8d
1 parent e896570
commit 55b8d8d
Showing 1 changed file with 24 additions and 4 deletions.
diff --git a/test/unit/is-serializable-props.unit.test.js b/test/unit/is-serializable-props.unit.test.js
@@ -6,14 +6,34 @@ describe('isSerializableProps', () => {
     expect(() => isSerializableProps('/', 'test', null))
       .toThrowErrorMatchingInlineSnapshot(`
 "Error serializing props returned from \`test\` in \\"/\\".
-Reason: Props must be returned as a plain object from test: \`{ props: { ... } }\`."
-`)
+Reason: Props must be returned as a plain object from test: \`{ props: { ... } }\` (received: \`[object Null]\`)."
+    `)
 
     expect(() => isSerializableProps('/', 'test', undefined))
       .toThrowErrorMatchingInlineSnapshot(`
 "Error serializing props returned from \`test\` in \\"/\\".
-Reason: Props must be returned as a plain object from test: \`{ props: { ... } }\`."
-`)
+Reason: Props must be returned as a plain object from test: \`{ props: { ... } }\` (received: \`[object Undefined]\`)."
+    `)
+  })
+
+  it('handles non-plain object props', () => {
+    expect(() => isSerializableProps('/', 'test', []))
+      .toThrowErrorMatchingInlineSnapshot(`
+"Error serializing props returned from \`test\` in \\"/\\".
+Reason: Props must be returned as a plain object from test: \`{ props: { ... } }\` (received: \`[object Array]\`)."
+    `)
+
+    expect(() => isSerializableProps('/', 'test', class Foobar {}))
+      .toThrowErrorMatchingInlineSnapshot(`
+"Error serializing props returned from \`test\` in \\"/\\".
+Reason: Props must be returned as a plain object from test: \`{ props: { ... } }\` (received: \`[object Function]\`)."
+    `)
+
+    expect(() => isSerializableProps('/', 'test', function Foobar() {}))
+      .toThrowErrorMatchingInlineSnapshot(`
+"Error serializing props returned from \`test\` in \\"/\\".
+Reason: Props must be returned as a plain object from test: \`{ props: { ... } }\` (received: \`[object Function]\`)."
+    `)
   })
 
   it('allows empty props', () => {