From 0b0a34908197a0a58c5d16765661000bd24cb39c Mon Sep 17 00:00:00 2001
From: Tom Hunkapiller <tom@furycodes.com>
Date: Fri, 10 Apr 2015 20:19:26 -0500
Subject: [PATCH] clearer comments about the path handling behavior

---
 src/node/hooks/express/tests.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/node/hooks/express/tests.js b/src/node/hooks/express/tests.js
index dfe02c2af..151c99fab 100644
--- a/src/node/hooks/express/tests.js
+++ b/src/node/hooks/express/tests.js
@@ -23,7 +23,10 @@ exports.expressCreateServer = function (hook_name, args, cb) {
 
   });
 
-  var rootTestFolder = path.join(npm.root, "..", "/tests/frontend/");
+
+  // path.join seems to normalize by default, but we'll just be explicit
+  var rootTestFolder = path.normalize(path.join(npm.root, "../tests/frontend/"));
+
   var url2FilePath = function(url){
     var subPath = url.substr("/tests/frontend".length);
     if (subPath == ""){
@@ -34,7 +37,7 @@ exports.expressCreateServer = function (hook_name, args, cb) {
     var filePath = path.normalize(path.join(rootTestFolder, subPath));
     // make sure we jail the paths to the test folder, otherwise serve index
     if (filePath.indexOf(rootTestFolder) !== 0) {
-      filePath = path.normalize(path.join(rootTestFolder, "index.html"));
+      filePath = path.join(rootTestFolder, "index.html");
     }
     return filePath;
   }