infinitered · GantMan · Feb 26, 2018 · Feb 26, 2018 · Feb 26, 2018 · Feb 26, 2018
diff --git a/.vscode/cSpell.json b/.vscode/cSpell.json
@@ -18,6 +18,7 @@
         "config",
         "configs",
         "custom",
+        "delineator",
         "docsify",
         "e.g.",
         "env",

diff --git a/__tests__/__mocks__/mockContext.ts b/__tests__/__mocks__/mockContext.ts
@@ -9,13 +9,17 @@ const noConfigSolidarity = {
   setSolidaritySettings: jest.fn(),
   updateRequirement: jest.fn(),
   updateVersions: jest.fn(() => Promise.resolve()),
+  getLineWithVersion: jest.fn(),
+  removeNonVersionCharacters: jest.fn(),
 }
 
 const mockContext = {
   ...realThing,
   outputMode: undefined,
   system: {
     startTimer: jest.fn(() => jest.fn()),
+    run: jest.fn(() => '12'),
+    which: jest.fn((name) => 'usr/local/bin/${name}'),
   },
   template: {
     generate: jest.fn(),

diff --git a/__tests__/command_helpers/checkCLI.ts b/__tests__/command_helpers/checkCLI.ts
@@ -9,6 +9,11 @@ const alwaysExistCLI = {
   binary: 'node',
 }
 
+const badSemver = {
+  binary: 'node',
+  semver: 'wtfbbq!!!11',
+}
+
 const outOfDateCLI = {
   binary: 'node',
   semver: '10.99',
@@ -24,6 +29,10 @@ test('fine on existing binary', async () => {
   expect(await checkCLI(alwaysExistCLI, context)).toBe(undefined)
 })
 
+test('errors with message when an improper semver is sent', async () => {
+  expect(await checkCLI(badSemver, context)).toBe(`Invalid semver rule ${badSemver.semver}`)
+})
+
 test('returns message on improper version', async () => {
   solidarityExtension(context)
   context.solidarity.getVersion = () => '1'

diff --git a/__tests__/command_helpers/quirksNodeModules.ts b/__tests__/command_helpers/quirksNodeModules.ts
@@ -0,0 +1,32 @@
+const path = require('path')
+const delineator = process.platform === 'win32' ? ';' : ':'
+test('quirks moves node_modules to back', () => {
+  // key is that it has `node_modules` + path.sep
+  const injectedStuff = `node_modules${path.sep}testOnly`
+  // prepend to PATH
+  process.env.PATH = injectedStuff + delineator + process.env.PATH
+  const pathAsArray = process.env.PATH.split(delineator)
+  // Yup it is prepended to front
+  expect(pathAsArray[0]).toBe(injectedStuff)
+  // require mutates PATH
+  require('../../src/extensions/functions/quirksNodeModules')
+  const newPathAsArray = process.env.PATH.split(delineator)
+  // Not in the front
+  expect(newPathAsArray[0]).not.toBe(injectedStuff)
+  // still there though (moved to back)
+  expect(process.env.PATH.match(injectedStuff)).toBeTruthy()
+})
+
+test('quirks does not move just any injected path to back', () => {
+  const injectedStuff = `taco${path.sep}testOnly`
+  // prepend to PATH
+  process.env.PATH = injectedStuff + delineator + process.env.PATH
+  const pathAsArray = process.env.PATH.split(delineator)
+  // Yup it is prepended to front
+  expect(pathAsArray[0]).toBe(injectedStuff)
+  // require does not mutate PATH this time
+  require('../../src/extensions/functions/quirksNodeModules')
+  const newPathAsArray = process.env.PATH.split(delineator)
+  // Not in the front
+  expect(newPathAsArray[0]).toBe(injectedStuff)
+})
diff --git a/src/extensions/functions/checkCLI.ts b/src/extensions/functions/checkCLI.ts
@@ -2,6 +2,8 @@ import { SolidarityRunContext, CLIRule } from '../../types'
 module.exports = async (rule: CLIRule, context: SolidarityRunContext): Promise<string | undefined> => {
   const { semver, solidarity } = context
   const binaryExists = require('./binaryExists')
+  // Node Modules do strange things
+  require('./quirksNodeModules')
 
   // First check for binary
   if (!binaryExists(rule.binary, context)) {

diff --git a/src/extensions/functions/quirksNodeModules.ts b/src/extensions/functions/quirksNodeModules.ts
@@ -0,0 +1,11 @@
+import {
+  reject, contains, concat, difference
+} from 'ramda'
+import path from 'path'
+const delineator = process.platform === 'win32' ? ';' : ':'
+
+// Node mutates path by adding to the front, move that to the back if it exists
+const originalPath = process.env.PATH || ''
+const originalArray = originalPath.split(delineator)
+const cleanArray = reject(contains('node_modules' + path.sep), originalArray)
+process.env.PATH = concat(cleanArray, difference(originalArray, cleanArray)).join(delineator)