moar plugin tests

src/plugins.spec.js

@@ -86,9 +86,9 @@ describe("resolveUserPlugins", function () {
 });
 
 describe("parseInputFile", function () {
-  it("throws when parseInputFile returns unexpected type", async function () {
+  it("throws when parseInputFile returns unexpected object", async function () {
     const plugins = await loadAllPlugins([
-      "../testfiles/plugins/bad-parse-method.js",
+      "../testfiles/plugins/bad-parse-method1.js",
     ]);
     assert.throws(
       () => parseFile(plugins.allLoadedPlugins, "{}", "foo.json", null),
@@ -99,4 +99,18 @@ describe("parseInputFile", function () {
       },
     );
   });
+
+  it("throws when parseInputFile returns unexpected array", async function () {
+    const plugins = await loadAllPlugins([
+      "../testfiles/plugins/bad-parse-method2.js",
+    ]);
+    assert.throws(
+      () => parseFile(plugins.allLoadedPlugins, "{}", "foo.json", null),
+      {
+        name: "Error",
+        message:
+          "Plugin v8r-plugin-test-bad-parse-method returned an unexpected type from parseInputFile hook. Expected Document, got string",
+      },
+    );
+  });
 });

testfiles/plugins/bad-parse-method.js → testfiles/plugins/bad-parse-method1.js

@@ -1,6 +1,6 @@
 import { BasePlugin } from "v8r";
 
-export default class ValidTestPlugin extends BasePlugin {
+export default class BadParseMethod1TestPlugin extends BasePlugin {
   static name = "v8r-plugin-test-bad-parse-method";
 
   // eslint-disable-next-line no-unused-vars

testfiles/plugins/bad-parse-method2.js

@@ -0,0 +1,12 @@
+import { BasePlugin, Document } from "v8r";
+
+export default class BadParseMethod2TestPlugin extends BasePlugin {
+  static name = "v8r-plugin-test-bad-parse-method";
+
+  // eslint-disable-next-line no-unused-vars
+  parseInputFile(contents, fileLocation, parser) {
+    // if we are returning an array
+    // all objects in the array should be a Document
+    return [new Document({}), "foobar"];
+  }
+}