tests for directories

Author: Nathan Mahdavi

__tests__/commands/list.js

@@ -3,7 +3,7 @@ const fs = require('fs');
 const list = require('../../commands/list');
 const directories = require('../../directories');
 
-describe('Command: list', function () {
+describe('Command: list', () => {
 
   const mockNpmUserBasePath = '/mock/npm/user/';
   const mockAliases = [
@@ -21,7 +21,7 @@ describe('Command: list', function () {
     jest.restoreAllMocks();
   });
 
-  test('reads users from the correct directory', function () {
+  test('reads users from the correct directory', () => {
     list();
 
     expect(fs.readdirSync).toHaveBeenCalledWith(mockNpmUserBasePath);

__tests__/commands/remove.js

@@ -3,7 +3,7 @@ const fs = require('fs');
 const directories = require('../../directories');
 const remove = require('../../commands/remove');
 
-describe('Command: remove <alias>', function () {
+describe('Command: remove <alias>', () => {
 
   const mockAlias = 'test-alias';
   const mockNpmUserBasePath = '/mock/npm/user/';

__tests__/directories.js

@@ -0,0 +1,23 @@
+const os = require('os');
+
+const directories = require('../directories');
+
+describe('directories', () => {
+
+  beforeEach(() => {
+    jest.spyOn(os, 'homedir').mockReturnValue('/mock/home');
+  });
+
+  afterEach(() => {
+    jest.restoreAllMocks();
+  });
+
+  test('getNpmUsersHomeDirectory() return the correct value', () => {
+    expect(directories.getNpmUsersHomeDirectory()).toEqual('/mock/home/.npm-users/');
+  });
+
+  test('getNpmRc() return the correct value', () => {
+    expect(directories.getNpmRc()).toEqual('/mock/home/.npmrc');
+  });
+
+});

__tests__/setup.js

@@ -17,7 +17,7 @@ describe('setup()', () => {
     jest.restoreAllMocks();
   });
 
-  describe('setupHomeDirectory()', function () {
+  describe('setupHomeDirectory()', () => {
 
     test('creates the npm user directory if it does not exist', () => {
       setup();

directories.js

@@ -1,10 +1,6 @@
 const os = require('os');
 
-const userHomeDirectory = os.homedir();
-const npmUsersRootDirectory = `${userHomeDirectory}/.npm-users/`;
-const npmRc = `${userHomeDirectory}/.npmrc`;
-
 module.exports = {
-  getNpmUsersHomeDirectory: () => npmUsersRootDirectory,
-  getNpmRc: () => npmRc
+  getNpmUsersHomeDirectory: () => `${os.homedir()}/.npm-users/`,
+  getNpmRc: () => `${os.homedir()}/.npmrc`
 };