From 8cdd6cd13115a0c6abd05652314e4611b3ac9f3a Mon Sep 17 00:00:00 2001 From: John-David Dalton Date: Wed, 29 Jan 2014 08:54:38 -0800 Subject: [PATCH] Fix test/test.js in lodash-node and carry fixes over to perf/perf.js. --- perf/perf.js | 43 ++++++++++++++++++++++--------------------- test/test.js | 8 ++++++-- 2 files changed, 28 insertions(+), 23 deletions(-) diff --git a/perf/perf.js b/perf/perf.js index e11d0a9f4..736e66e07 100644 --- a/perf/perf.js +++ b/perf/perf.js @@ -15,14 +15,6 @@ params = root.arguments, system = root.system; - /** Detect if running in Java */ - var isJava = !document && !!root.java; - - /** Use a single "load" function */ - var load = (typeof require == 'function' && !amd) - ? require - : (isJava && root.load) || noop; - /** The file path of the Lo-Dash file to test */ var filePath = (function() { var min = 0, @@ -44,12 +36,33 @@ if (!amd) { try { - return require.resolve(result); + result = require('fs').realpathSync(result); + } catch(e) { } + + try { + result = require.resolve(result); } catch(e) { } } return result; }()); + /** The `ui` object */ + var ui = root.ui || (root.ui = { + 'buildPath': basename(filePath, '.js'), + 'otherPath': 'underscore' + }); + + /** Detect if in a browser environment */ + var isBrowser = isHostType(root, 'document') && isHostType(root, 'navigator'); + + /** Detect if in a Java environment */ + var isJava = !isBrowser && /Java/.test(toString.call(root.java)); + + /** Use a single "load" function */ + var load = (typeof require == 'function' && !amd) + ? require + : (isJava && root.load) || noop; + /** Load Lo-Dash */ var lodash = root.lodash || (root.lodash = ( lodash = load(filePath) || root._, @@ -92,12 +105,6 @@ /** Used to resolve a value's internal [[Class]] */ var toString = Object.prototype.toString; - /** The `ui` object */ - var ui = root.ui || (root.ui = { - 'buildPath': basename(filePath, '.js'), - 'otherPath': 'underscore' - }); - /** The Lo-Dash build basename */ var buildName = root.buildName = basename(ui.buildPath, '.js'); @@ -107,12 +114,6 @@ return result + (result == buildName ? ' (2)' : ''); }()); - /** Detect if in a browser environment */ - var isBrowser = isHostType(root, 'document') && isHostType(root, 'navigator'); - - /** Detect Java environment */ - var isJava = !isBrowser && /Java/.test(toString.call(root.java)); - /** Add `console.log()` support for Narwhal, Rhino, and RingoJS */ var console = root.console || (root.console = { 'log': root.print }); diff --git a/test/test.js b/test/test.js index acf5d1315..39f052631 100644 --- a/test/test.js +++ b/test/test.js @@ -49,7 +49,11 @@ if (!amd) { try { - return require.resolve(result); + result = require('fs').realpathSync(result); + } catch(e) { } + + try { + result = require.resolve(result); } catch(e) { } } return result; @@ -65,7 +69,7 @@ /** The basename of the Lo-Dash file to test */ var basename = /[\w.-]+$/.exec(filePath)[0]; - /** Detect if running in Java */ + /** Detect if in a Java environment */ var isJava = !document && !!root.java; /** Used to indicate testing a modularized build */