test: add extra test asserting that unknown keys are allowed

asset-pipe · Jan 31, 2018 · 9113289 · 9113289
1 parent e90832b
commit 9113289
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 4 deletions.
diff --git a/test/__snapshots__/reader.test.js.snap b/test/__snapshots__/reader.test.js.snap
@@ -58949,9 +58949,9 @@ exports[`should dedupe 1`] = `
 "
 `;
 
-exports[`should error if any feed content keys are missing 1`] = `"Expected at least 1 feed (array) to have more than one entry (object)."`;
+exports[`should error if any feed content keys are missing 1`] = `"Expected every item in argument 'feeds' (array) to be a valid feed (object).: child \\"id\\" fails because [\\"id\\" is required]"`;
 
-exports[`should error if any feed content keys are missing 2`] = `"Expected at least 1 feed (array) to have more than one entry (object)."`;
+exports[`should error if any feed content keys are missing 2`] = `"Expected every item in argument 'feeds' (array) to be a valid feed (object).: child \\"source\\" fails because [\\"source\\" is required]"`;
 
 exports[`should error if feed content does not contain at least 1 entrypoint 1`] = `"Expected at least 1 feed (array) to have more than one entry (object)."`;
 

diff --git a/test/reader.test.js b/test/reader.test.js
@@ -210,12 +210,29 @@ test('should error if feed content is non object', async () => {
 });
 
 test('should error if any feed content keys are missing', async () => {
-    await expect(bundleJS([{}])).rejects.toThrowErrorMatchingSnapshot();
+    await expect(bundleJS([[{}]])).rejects.toThrowErrorMatchingSnapshot();
     await expect(
-        bundleJS([{ file: 'asd', deps: {}, id: 'a', entry: true }])
+        bundleJS([[{ file: 'asd', deps: {}, id: 'a', entry: true }]])
     ).rejects.toThrowErrorMatchingSnapshot();
 });
 
+test('should not error if any feed contains extra content keys', async () => {
+    await expect(
+        bundleJS([
+            [
+                {
+                    file: 'asd',
+                    deps: {},
+                    id: 'a',
+                    entry: true,
+                    source: '"use strict";',
+                    somethingWeird: true,
+                },
+            ],
+        ])
+    ).resolves.toBeDefined();
+});
+
 test('should error if feed content does not contain at least 1 entrypoint', async () => {
     await expect(
         bundleJS([