gabrielg-xultestrunner 0.2.2 → 0.2.3

data/VERSION

@@ -1 +1 @@
-0.2.2
+0.2.3

data/xulapp/application.ini

@@ -1,7 +1,7 @@
 [App]
 Name=XULTestRunner
-Version=0.2.2
-BuildID=859ab1e7-bc8e-42bf-83d0-d84abcba0d48
+Version=0.2.3
+BuildID=badffba7-8ff1-44cc-9ee2-832aa7332e27
 ID=XULTestRunner@conflagrationjs.org
 Vendor=Conflagration JS
 

data/xulapp/chrome/content/lib/xultestcase.js

@@ -28,7 +28,7 @@ Object.extend(XULTestCase, {
       if (this.__LAST_DEFINED_TESTCASE__) {
         return this.__LAST_DEFINED_TESTCASE__;
       } else {
-        throw("wuh oh");
+        throw("Wuh Oh");
       }
     } catch (e) {
       this._handleTestLoadError(fileURI, e);
@@ -38,7 +38,10 @@ Object.extend(XULTestCase, {
   },
   
   _handleTestLoadError: function(fileURI, e) {
-    puts("There was a problem loading the test case from '" + fileURI + "': " + e);
+    // Notify observers we failed and hope for the best
+    // FIXME - this sucks error reporting wise
+    var observerService = Cc["@mozilla.org/observer-service;1"].getService(Ci.nsIObserverService);
+    observerService.notifyObservers(null, "xultestrunner-testloaderror", Object.toJSON({file: fileURI, exception: e.toString()}));
   }
   
 });

data/xulapp/chrome/content/lib/xultestrunner.js

@@ -29,8 +29,13 @@ var XULTestRunner = Class.create({
   },
   
   observe: function(subject, topic, data) {
-    // Need to pass between observers as JSON to avoid crashes. Wee.
-    this._testResults.push(data.evalJSON());
+    // Need to pass between observers as JSON to avoid XPCOM crashes. Wee.
+    var data = data.evalJSON();
+    if (topic == "xultestrunner-testdone") {
+      this._testResults.push(data);
+    } else if (topic == "xultestrunner-testloaderror") {
+      puts("There was an error loading test '#{file}': #{exception}".interpolate(data));
+    }
     this._runNextTest();
   },
   
@@ -84,6 +89,7 @@ var XULTestRunner = Class.create({
   _registerObservers: function() {
     var observerService = Cc["@mozilla.org/observer-service;1"].getService(Ci.nsIObserverService);
     observerService.addObserver(this, "xultestrunner-testdone", false);
+    observerService.addObserver(this, "xultestrunner-testloaderror", false);
   }
 });
 

data/xulapp/components/bootstrap.js

@@ -1,4 +1,5 @@
 (function(outerScope){
+  // FIXME - make the code stop making my eyes bleed.
   COMPONENTS = [];
   
   var loader = Components.classes["@mozilla.org/moz/jssubscript-loader;1"].getService(Components.interfaces.mozIJSSubScriptLoader);

data/xultestrunner.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{xultestrunner}
-  s.version = "0.2.2"
+  s.version = "0.2.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Gabriel Gironda"]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: gabrielg-xultestrunner
 version: !ruby/object:Gem::Version 
-  version: 0.2.2
+  version: 0.2.3
 platform: ruby
 authors: 
 - Gabriel Gironda