[Laszlo-checkins] r11970 - openlaszlo/trunk/lps/components/lzunit

[ptw@openlaszlo.org]

Author: ptw
Date: 2008-12-03 13:17:15 -0800 (Wed, 03 Dec 2008)
New Revision: 11970

Modified:
   openlaszlo/trunk/lps/components/lzunit/lzunit.lzx
Log:
Change 20081202-ptw-a by ptw at dueling-banjos.home on 2008-12-02 17:40:55 EST
    in /Users/ptw/OpenLaszlo/trunk
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: lzunit: report failing case in addition to test

Bugs Fixed:
LPP-7426  lzunit no longer reports test case that is failing

Technical Reviewer: a.bargull at intensis.de (pending)
QA Reviewer: hminsky (pending)

Details:
    Use formatToString to print case and test (insulating ourselves
    from debugger internals).

Tests:
    style/metasuite now reports:
    Tests: 541 Failures: 1 Errors: 0
    TestFailure: ?\194?\171<SimpleTestCase>?\194?\187/testBasics failed: Equals:  expected 50 got 279



Modified: openlaszlo/trunk/lps/components/lzunit/lzunit.lzx
===================================================================
--- openlaszlo/trunk/lps/components/lzunit/lzunit.lzx	2008-12-03 21:14:23 UTC (rev 11969)
+++ openlaszlo/trunk/lps/components/lzunit/lzunit.lzx	2008-12-03 21:17:15 UTC (rev 11970)
@@ -631,7 +631,7 @@
       @keywords constructor
      -->
     <method name="construct" args="parent, args">
-        this.name = jsTrue(args["testName"]) ? args.testName : this.constructor.tagname;
+        if (jsTrue(args["testName"])) { this.name = args.testName; delete args.testName; }
 
         super.construct(parent, args);
 
@@ -652,10 +652,7 @@
     <method name="run" args="theTestResult, theTestName">  <![CDATA[
         dw("TestCase.run(", theTestName, ")");
         setResult(theTestResult);
-        var dn = (this._dbg_name is Function) ? this._dbg_name() : this._dbg_name;
-        var tn = (this._dbg_typename is Function) ? this._dbg_typename() : this._dbg_typename;
-        var prefix = (dn != tn) ? dn + ' >>> ' : ''
-        this.result.startTest(prefix + theTestName);
+        this.result.startTest(this.formatter.formatToString("%w/%s", this, theTestName));
         setUp();
 
         // NOTE: [2008-10-26 ptw] Intercepting source warnings is only