module-definition - npm Package Compare versions

Comparing version 2.2.0 to 2.2.1

index.js

@@ -56,2 +56,8 @@ var Walker  = require('node-source-walk'),
 
+  // ES6 features are so unique that we can
+  // eagerly exit. (#19)
+  if (isES6) {
+    return 'es6';
+  }
+
   if (isAMD) {
@@ -65,6 +71,2 @@ return 'amd';
 
-  if (isES6) {
-    return 'es6';
-  }
-
   return 'none';
@@ -71,0 +73,0 @@ }

package.json

 {
   "name": "module-definition",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "description": "Determines if a file is using a CommonJS or AMD module definition",
@@ -20,3 +20,3 @@ "main": "index.js",
   "dependencies": {
-    "ast-module-types": "~2.2.1",
+    "ast-module-types": "~2.2.2",
     "node-source-walk": "~1.4.0"
@@ -23,0 +23,0 @@ },

test/test.js

 var getModuleType = require('../'),
     path = require('path'),
     fs = require('fs'),
-    assert = require('assert');
+    assert = require('assert'),
+    amdAST = require('./amdAST');
 
 describe('module-definition', function() {
   var expected = {
-    './cjsExport.js': 'commonjs',
-    './cjsRequire.js': 'commonjs',
-    './amdNoDep.js': 'amd',
-    './iife.js': 'none',
-    './amdFactory.js': 'amd',
-    './amdDeps.js': 'amd',
-    './cjsTopRequire.js': 'commonjs',
-    './empty.js': 'none',
-    './amdREM.js': 'amd',
-    './es6Import.js': 'es6',
-    './es6Export.js': 'es6'
-  },
-
-  amdAST = {
-    type: 'Program',
-    body: [{
-      type: 'ExpressionStatement',
-      expression: {
-        type: 'CallExpression',
-        callee: {
-            type: 'Identifier',
-            name: 'define'
-        },
-        arguments: [
-          {
-            type: 'ArrayExpression',
-            elements: []
-          },
-          {
-            type: 'FunctionExpression',
-            id: null,
-            params: [],
-            defaults: [],
-            body: {
-              type: 'BlockStatement',
-              body: []
-            },
-            rest: null,
-            generator: false,
-            expression: false
-        }]
-      }
-    }]
+    cjsExport: 'commonjs',
+    cjsRequire: 'commonjs',
+    amdNoDep: 'amd',
+    iife: 'none',
+    amdFactory: 'amd',
+    amdDeps: 'amd',
+    cjsTopRequire: 'commonjs',
+    empty: 'none',
+    amdREM: 'amd',
+    es6Import: 'es6',
+    es6Export: 'es6',
+    es6WithRequire: 'es6'
   };
@@ -55,3 +25,3 @@
     Object.keys(expected).forEach(function(file) {
-      method(file, expected[file]);
+      method('./' + file + '.js', expected[file]);
     });
@@ -58,0 +28,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

9089

increased by0.6%

Number of package files

21

increased by10.53%

Lines of code

260

increased by1.96%

Dependency changes

• Updatedast-module-types@~2.2.2