test: add test to getFilePath

Author: liady

test/library.spec.js

@@ -1,4 +1,5 @@
 const nodeExternals = require('../index.js');
+const path = require('path');
 const utils = require('../utils.js');
 const testUtils = require('./test-utils.js');
 const mockNodeModules = testUtils.mockNodeModules;
@@ -326,3 +327,11 @@ describe('validate options', function () {
         expect(results[0].correctTerm).to.be.equal('modulesDir');
     });
 });
+
+describe('file path', function() {
+    it('should return the correct file path', function() {
+        expect(utils.getFilePath({ fileName: 'file.json' })).to.be.equal(path.resolve(process.cwd(), 'file.json'));
+        expect(utils.getFilePath({})).to.be.equal(path.resolve(process.cwd(), 'package.json'));
+        expect(utils.getFilePath({ fileName: 'c:\\test\\test.js' })).to.be.equal('c:\\test\\test.js');
+    })
+})

utils.js

@@ -38,6 +38,11 @@ exports.readDir = function readDir(dirName) {
     }
 };
 
+function getFilePath(options) {
+    return path.resolve(process.cwd(), options.fileName || 'package.json');
+}
+exports.getFilePath = getFilePath;
+
 exports.readFromPackageJson = function readFromPackageJson(options) {
     if (typeof options !== 'object') {
         options = {};
@@ -48,11 +53,7 @@ exports.readFromPackageJson = function readFromPackageJson(options) {
     // read the file
     let packageJson;
     try {
-        const fileName = options.fileName || 'package.json';
-        const packageJsonString = fs.readFileSync(
-            path.resolve(process.cwd(), fileName),
-            'utf8'
-        );
+        const packageJsonString = fs.readFileSync(getFilePath(options), 'utf8');
         packageJson = JSON.parse(packageJsonString);
     } catch (e) {
         return [];