feat: remove old API

BREAKING CHANGE: the `entry` named export no longer exists, so users
should use the API as documented in the README.

See #8.

Author: eventualbuddha

index.js

@@ -1,8 +1,10 @@
-type Config = string | Array<string> | { include: Array<string>, exclude: Array<string> };
+/* @flow */
+
+type Config = string | Array<string> | { include?: Array<string>, exclude?: Array<string>, exports?: boolean };
 
 import glob from 'glob';
 
-export const entry = '\0rollup-plugin-multi-entry:entry-point';
+const entry = '\0rollup-plugin-multi-entry:entry-point';
 
 export default function multiEntry(config: ?Config=null) {
   let include = [];
@@ -15,7 +17,8 @@ export default function multiEntry(config: ?Config=null) {
     } else if (Array.isArray(config)) {
       include = config;
     } else {
-      ({ include = [], exclude = [] } = config);
+      include = config.include || [];
+      exclude = config.exclude || [];
       if (config.exports === false) {
         exporter = path => `import ${JSON.stringify(path)};`;
       }
@@ -27,7 +30,7 @@ export default function multiEntry(config: ?Config=null) {
   }
 
   return {
-    options(options: {entry: string}) {
+    options(options: { entry: ?string }) {
       if (options.entry && options.entry !== entry) {
         configure(options.entry);
       }

test/test.js

@@ -1,4 +1,4 @@
-import multiEntry, { entry } from '../';
+import multiEntry from '../';
 import { ok } from 'assert';
 import { rollup } from 'rollup';
 
@@ -14,48 +14,42 @@ function doesNotInclude(string, substring) {
   }
 }
 
-function makeBundle(entries, useOldAPI) {
-  return rollup(
-      !useOldAPI ?
-      { entry: entries, plugins: [multiEntry()] } :
-      { entry, plugins: [multiEntry(entries)] }
-  );
+function makeBundle(entries) {
+  return rollup({ entry: entries, plugins: [multiEntry()] });
 }
 
-function setupTests(options) {
-  const useOldAPI = options ? options.useOldAPI : false;
-
+describe('rollup-plugin-multi-entry', () => {
   it('takes a single file as input', () =>
-    makeBundle('test/fixtures/0.js', useOldAPI).then(bundle => {
+    makeBundle('test/fixtures/0.js').then(bundle => {
       includes(bundle.generate({ format: 'cjs' }).code, 'exports.zero = zero;');
     })
   );
 
   it('takes an array of files as input', () =>
-    makeBundle(['test/fixtures/0.js', 'test/fixtures/1.js'], useOldAPI).then(bundle => {
+    makeBundle(['test/fixtures/0.js', 'test/fixtures/1.js']).then(bundle => {
       const code = bundle.generate({ format: 'cjs' }).code;
       includes(code, 'exports.zero = zero;');
       includes(code, 'exports.one = one;');
     })
   );
 
   it('allows an empty array as input', () =>
-    makeBundle([], useOldAPI).then(bundle => {
+    makeBundle([]).then(bundle => {
       const code = bundle.generate({ format: 'cjs' }).code;
       doesNotInclude(code, 'exports');
     })
   );
 
   it('takes a glob as input', () =>
-    makeBundle('test/fixtures/{0,1}.js', useOldAPI).then(bundle => {
+    makeBundle('test/fixtures/{0,1}.js').then(bundle => {
       const code = bundle.generate({ format: 'cjs' }).code;
       includes(code, 'exports.zero = zero;');
       includes(code, 'exports.one = one;');
     })
   );
 
   it('takes an array of globs as input', () =>
-    makeBundle(['test/fixtures/{0,}.js', 'test/fixtures/{1,}.js'], useOldAPI).then(bundle => {
+    makeBundle(['test/fixtures/{0,}.js', 'test/fixtures/{1,}.js']).then(bundle => {
       const code = bundle.generate({ format: 'cjs' }).code;
       includes(code, 'exports.zero = zero;');
       includes(code, 'exports.one = one;');
@@ -64,8 +58,7 @@ function setupTests(options) {
 
   it('takes an {include,exclude} object as input', () =>
     makeBundle(
-      { include: ['test/fixtures/*.js'], exclude: ['test/fixtures/1.js'] },
-      useOldAPI
+      { include: ['test/fixtures/*.js'], exclude: ['test/fixtures/1.js'] }
     ).then(bundle => {
       const code = bundle.generate({ format: 'cjs' }).code;
       includes(code, 'exports.zero = zero;');
@@ -75,21 +68,12 @@ function setupTests(options) {
 
   it('allows to prevent exporting', () =>
     makeBundle(
-      { include: ['test/fixtures/*.js'], exports: false },
-      useOldAPI
+      { include: ['test/fixtures/*.js'], exports: false }
     ).then(bundle => {
       const code = bundle.generate({ format: 'iife' }).code;
       includes(code, `console.log('Hello, 2');`)
       doesNotInclude(code, 'zero');
       doesNotInclude(code, 'one');
     })
   );
-}
-
-describe('rollup-plugin-multi-entry', () => {
-  setupTests({ useOldAPI: false });
-
-  describe('with old API', () => {
-    setupTests({ useOldAPI: true });
-  });
 });