Changeset 7879

Timestamp:

03/31/07 14:27:09 (22 months ago)

Author:

alex

Message:

keep the system from throwing errors when a test page that includes dojo but doesn't have a parent harness loads it. We still get success/failure info at the console. Refs #2550

Files:

• dojo/trunk/tests/_browserRunner.js (modified) (3 diffs)

dojo/trunk/tests/_browserRunner.js

@@ -188 +188 @@
                 tests._groupStarted = function(group){
                         // console.debug("_groupStarted", group);
-                        getGroupNode(group).className = "inProgress";
+                        var gn = getGroupNode(group);
+                        if(gn){
+                                gn.className = "inProgress";
+                        }
                 }
 
                 tests._groupFinished = function(group, success){
                         // console.debug("_groupFinished", group);
-                        getGroupNode(group).className = (success) ? "success" : "failure";
+                        var gn = getGroupNode(group);
+                        if(gn){
+                                gn.className = (success) ? "success" : "failure";
+                        }
                 }
 
                 tests._testStarted = function(group, fixture){
                         // console.debug("_testStarted", group, fixture.name);
-                        getFixtureNode(group, fixture).className = "inProgress";
+                        var fn = getFixtureNode(group, fixture);
+                        if(fn){
+                                fn.className = "inProgress";
+                        }
                         fixture.startTime = new Date();
                 }
@@ -204 +213 @@
                 tests._testFinished = function(group, fixture, success){
                         var fn = getFixtureNode(group, fixture);
-                        fn.getElementsByTagName("td")[2].innerHTML = ((new Date())-fixture.startTime)+"ms";
-                        fn.className = (success) ? "success" : "failure";
+                        if(fn){
+                                fn.getElementsByTagName("td")[2].innerHTML = ((new Date())-fixture.startTime)+"ms";
+                                fn.className = (success) ? "success" : "failure";
+                        }
                         this.debug(((success) ? "PASSED" : "FAILED"), "test:", fixture.name);
                 }
@@ -269 +280 @@
                         // would be easier w/ query...sigh
                         runAll = (!runAll);
+                        if(!dojo.byId("testList")){ return; }
                         var tb = byId("testList").tBodies[0];
                         var inputs = tb.getElementsByTagName("input");