fix: Fix non-top-level requires (#37)

revert @babel/plugin-transform-modules-commonjs to babel-plugin-add-module-exports

Author: chrisbreiding

.node-version

@@ -1 +1 @@
-8.2.1
+12.8.1

index.js

@@ -29,7 +29,7 @@ const defaultOptions = {
           babelrc: false,
           plugins: [
             ...[
-              '@babel/plugin-transform-modules-commonjs',
+              'babel-plugin-add-module-exports',
               '@babel/plugin-proposal-class-properties',
               '@babel/plugin-proposal-object-rest-spread',
             ].map(require.resolve),

package-lock.json

@@ -55,17 +55,17 @@
     "semantic-release": "15.13.15",
     "sinon": "7.2.3",
     "sinon-chai": "3.3.0",
-    "snap-shot-it": "6.2.9"
+    "snap-shot-it": "7.9.2"
   },
   "dependencies": {
     "@babel/core": "7.4.5",
     "@babel/plugin-proposal-class-properties": "7.3.0",
     "@babel/plugin-proposal-object-rest-spread": "7.3.2",
-    "@babel/plugin-transform-modules-commonjs": "7.8.3",
     "@babel/plugin-transform-runtime": "7.2.0",
     "@babel/preset-env": "7.4.5",
     "@babel/preset-react": "7.0.0",
     "@babel/runtime": "7.3.1",
+    "babel-plugin-add-module-exports": "1.0.2",
     "babelify": "10.0.0",
     "bluebird": "3.5.3",
     "browserify": "16.2.3",

package.json

@@ -74,4 +74,11 @@ describe('imports and exports', () => {
       eval(output)
     })
   })
+
+  it('handles non-top-level require', () => {
+    return bundle('require_spec.js').then((output) => {
+      // check that bundled tests work
+      eval(output)
+    })
+  })
 })

test/e2e/e2e_spec.js

@@ -1,4 +1,4 @@
-const dom = require('./dom').default
+const dom = require('./dom')
 
 context('imports default string', function () {
   it('works', () => {

test/fixtures/dom_spec.js

@@ -0,0 +1,14 @@
+context('non-top-level requires', function () {
+  const math = require('./math')
+  const dom = require('./dom')
+
+  it('imports proper types of values', () => {
+    expect(math.add, 'add').to.be.a('function')
+    expect(dom, 'dom').to.be.a('string')
+  })
+
+  it('values are correct', function () {
+    expect(math.add(1, 2)).to.eq(3)
+    expect(dom, 'dom').to.equal('dom')
+  })
+})