npmfiles - npm Package Compare versions

Comparing version 0.1.0 to 0.1.1

.travis.yml

+language: node_js
+os:
+    - linux
+    - windows
+    - osx
+node_js:
+    - "stable"
+    - "6.4.0"
+    - "6.2.2"
+    - "4.5.0"
+    - "4.0.0"
+sudo: false
+script:
+    - npm test
+after_success:
+    - npm run coverage

.vscode/settings.json

+// Place your settings in this file to overwrite default and user settings.
+{
+    "files.exclude": {
+        "**/node_modules": false,
+        "node_modules": true
+    }
+}

test/__tests__/npmfiles.test.js

+'use strict';
+
+const npmfiles = require('../../index');
+const pResolve = require('path').resolve;
+const fixturePath = pResolve(__dirname + '/../fixtures');
+
+const expectedResults = {
+    simple: [
+        `${fixturePath}/simple/node_modules/first/first.js`,
+        `${fixturePath}/simple/node_modules/second/second.js`
+    ],
+    arrayfiles: [
+        `${fixturePath}/arrayfiles/node_modules/first/first1.js`,
+        `${fixturePath}/arrayfiles/node_modules/first/first2.js`,
+        `${fixturePath}/arrayfiles/node_modules/second/second1.js`,
+        `${fixturePath}/arrayfiles/node_modules/second/second2.js`
+    ],
+    'incl-dev': [
+        `${fixturePath}/incl-dev/node_modules/first/first.js`,
+        `${fixturePath}/incl-dev/node_modules/second/second.js`
+    ],
+};
+
+const getConfig = function getConfig(folder){
+    return {nodeModulesPath: `${fixturePath}/${folder}/node_modules`, packageJsonPath: `${fixturePath}/${folder}/package.json`};
+};
+
+describe('NPM files', () => {
+    it('Returns main file declared in the modules own package.json', () => {
+        const testType = 'simple';
+        let resultFiles = npmfiles(getConfig(testType));
+
+        expect(resultFiles).toEqual(expectedResults[testType]);
+    });
+    it('Doesn\'t return non existing main entries', () => {
+        const testType = 'noexist';
+        let resultFiles = npmfiles(getConfig(testType));
+
+        expect(resultFiles).toEqual([]);
+    });
+    it('Returns main file declared in the modules own package.json as array', () => {
+        const testType = 'arrayfiles';
+        let resultFiles = npmfiles(getConfig(testType));
+
+        expect(resultFiles).toEqual(expectedResults[testType]);
+    });
+    it('Returns devDependendencies as well if specified as an option', () => {
+        const testType = 'incl-dev';
+        let conf = getConfig(testType);
+        conf.includeDev = true;
+        let resultFiles = npmfiles(conf);
+
+        expect(resultFiles).toEqual(expectedResults[testType]);
+    });
+});

test/fixtures/arrayfiles/package.json

+{
+  "dependencies": {
+    "first": "*",
+    "second": "*"
+  },
+  "overrides": {
+    "first": {
+      "main": [
+        "first1.js", "first2.js"
+      ]
+    },
+    "second": {
+      "main": [
+        "second1.js", "second2.js"
+      ]
+    }
+  }
+}

test/fixtures/incl-dev/package.json

+{
+  "dependencies": {
+    "first": "*"
+  },
+  "devDependencies": {
+    "second": "*"
+  }
+}

test/fixtures/noexist/package.json

+{
+  "dependencies": {
+    "first": "*",
+    "second": "*"
+  }
+}

test/fixtures/simple/package.json

+{
+  "dependencies": {
+    "first": "*",
+    "second": "*"
+  }
+}

index.js

@@ -84,3 +84,5 @@ var fs = require('fs');
     } else if(json.main){
-        files.push(path.resolve(modulePath + "/" + json.main));
+        files = files.concat(
+            glob.sync(path.resolve(modulePath + "/" + json.main))
+        );
     }
@@ -87,0 +89,0 @@

package.json

 {
   "name": "npmfiles",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Get all main files for your project's NPM modules",
   "main": "index.js",
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1"
+    "test": "jest --coverage",
+    "coverage": "cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js"
   },
@@ -28,3 +29,7 @@ "repository": {
     "glob": "^7.1.1"
+  },
+  "devDependencies": {
+    "coveralls": "^2.13.1",
+    "jest": "^20.0.4"
   }
 }

README.md

 npmfiles
 ================
 
+[![Build Status](https://travis-ci.org/cosminlupu/npmfiles.svg?branch=master)](https://travis-ci.org/cosminlupu/npmfiles) [![Coverage Status](https://coveralls.io/repos/github/cosminlupu/npmfiles/badge.svg?branch=master)](https://coveralls.io/github/cosminlupu/npmfiles?branch=master)
+
 - [Inspiration](#inspiration)
@@ -5,0 +7,0 @@ - [Usage](#usage)

Fixed alerts

No tests

Quality

Package does not have any tests. This is a strong signal of a poorly maintained or low quality package.

Improved metrics

Total package byte prevSize

10400

46.46%

Number of package files

11

175%

Lines of code

126

75%

Number of low quality alerts

1

-50%

Number of lines in readme file

154

1.32%

Worsened metrics

Dev dependency count

2

Infinity%

No dependency changes