From 5c0dc811ec19bf32f44bf91aa81c5771023eaddb Mon Sep 17 00:00:00 2001
From: Naomi Seyfer <naomi@meteor.com>
Date: Mon, 18 Feb 2013 00:32:13 -0800
Subject: [PATCH] When switching test sets, treat it like a hot code push

---
 packages/test-in-browser/driver.css  |  1 +
 packages/test-in-browser/driver.html |  6 ++-
 packages/test-in-browser/driver.js   | 64 +++++++++++++---------------
 3 files changed, 35 insertions(+), 36 deletions(-)

diff --git a/packages/test-in-browser/driver.css b/packages/test-in-browser/driver.css
index e09c28b0f8..0421e77c0a 100644
--- a/packages/test-in-browser/driver.css
+++ b/packages/test-in-browser/driver.css
@@ -1,5 +1,6 @@
 body {
     padding-top: 40px;
+    padding-bottom: 40px;
 }
 
 #testProgressBar {
diff --git a/packages/test-in-browser/driver.html b/packages/test-in-browser/driver.html
index 452d3bd539..ab031d5060 100644
--- a/packages/test-in-browser/driver.html
+++ b/packages/test-in-browser/driver.html
@@ -12,7 +12,8 @@
   <div class="navbar navbar-fixed-top navbar-inverse">
     <div class="navbar-inner">
       <div class="row-fluid">
-        <div class="span2 offset1"><a class="brand" href="#">
+        <div class="span3"><a class="brand" href="#">
+          &nbsp;
           {{#if running}}
             Testing in progress...
           {{else}}
@@ -36,7 +37,7 @@
     </div>
   </div>
   {{> groupNav}}
-  <div class="row-fluid"><div class="span11 offset1">
+  <div class="row-fluid"><div class="span12">
   <ul class="failedTests">
     {{#each failedTests}}
       <li>{{this}}</li>
@@ -63,6 +64,7 @@
     <div class="navbar-inner">
       <ul class="nav">
       {{#each groupPaths}}
+        <li class="navbar-text">&nbsp;-&nbsp;</li>
         <li><a class="group" href="#">{{name}}</a></li>
       {{/each}}
       </ul>
diff --git a/packages/test-in-browser/driver.js b/packages/test-in-browser/driver.js
index 82aa3c6323..8f23847a49 100644
--- a/packages/test-in-browser/driver.js
+++ b/packages/test-in-browser/driver.js
@@ -1,42 +1,26 @@
-var running;
+var running = true;
 
-var groupPathDeps;
-var groupPath;
-var resultTree;
-var failedTests;
-var resultDeps;
-var countDeps;
-var totalCount;
-var passedCount;
-var failedCount;
-var resetTests = function () {
-  if (!Session.get("groupPath"))
-    Session.set("groupPath", []);
-  resultTree =  [];
-  failedTests = [];
-  resultDeps = new Meteor.deps._ContextSet;
-  countDeps = new Meteor.deps._ContextSet;
-  totalCount = 0;
-  passedCount = 0;
-  failedCount = 0;
+var resultTree = [];
+var failedTests = [];
+var resultDeps = new Meteor.deps._ContextSet;
+var countDeps = new Meteor.deps._ContextSet;
+var totalCount = 0;
+var passedCount = 0;
+var failedCount = 0;
+
+if (!Session.get("groupPath"))
+  Session.set("groupPath", ["tinytest"]);
 
-  running = true;
-};
-resetTests();
 Meteor.startup(function () {
-  rerunTests();
-});
-
-var rerunTests = function () {
-  countDeps.invalidateAll();
-  _resultsChanged();
+  Meteor.flush();
   Meteor._runTestsEverywhere(reportResults, function () {
     running = false;
     Meteor.onTestsComplete && Meteor.onTestsComplete();
     _resultsChanged();
     Meteor.flush();
   }, Session.get("groupPath"));
-};
+
+});
 
 Template.progressBar.running = function () {
   countDeps.addCurrentContext();
@@ -71,16 +55,22 @@ Template.groupNav.groupPaths = function () {
   return ret;
 };
 
+var changeToPath = function (path) {
+  Session.set("groupPath", path);
+  // pretend there's just been a hot code push
+  // so we run the tests completely fresh.
+  Meteor._reload.reload();
+};
+
 Template.groupNav.events({
   "click .group": function () {
-    console.log("clicked", this);
-    Session.set("groupPath", this.path);
+    changeToPath(this.path);
   }
 });
 
 Template.test_group.events({
   "click .groupname": function () {
-    Session.set("groupPath", this.path);
+    changeToPath(this.path);
   }
 });
 
@@ -321,15 +311,21 @@ var _findTestForResults = function (results) {
   }
 
   var group;
+  var i = 0;
   _.each(groupPath, function(gname) {
     var array = (group ? (group.groups || (group.groups = []))
                  : resultTree);
     var newGroup = _.find(array, function(g) { return g.name === gname; });
     if (! newGroup) {
-      newGroup = {name: gname, parent: (group || null), path: groupPath}; // create group
+      newGroup = {
+        name: gname,
+        parent: (group || null),
+        path: groupPath.slice(0, i+1)
+      }; // create group
       array.push(newGroup);
     }
     group = newGroup;
+    i++;
   });
 
   var testName = results.test;