doh.registerGroup setUp and tearDown doesn't work

[mumme]

Hi !

When trying to figure out how the new unit test system works, I could never get registerGroup setUp and tearDown args to work as documented.

Trying the sample from: http://dojotoolkit.org/book/dojo-porting-guide-0-4-x-0-9/test-system-changes

// file located in core at:
//      tests/fullGroupTest.js
dojo.provide("tests.fullGroupTest");
dojo.require("tests.runner");

tests.registerGroup("tests.fullGroupTest",
        [
                // single test, no test fixture
                function assertTrueTest(t){ t.t(true); },
                // string variant of the same:
                "tests.t(true);",
                // test that uses variable set up by group
                function assertTrueTest(t){
                        t.t(tests.fullGroupTest._localVariable);
                },
                // ...
        ],
        function(){ // setUp
                tests.fullGroupTest._localVariable = true;
        },
        function(){ // tearDown
                tests.fullGroupTest._localVariable = false;
        }
);

Fails because setUp is never called.

Regards Fredrik Johansson

[Jared Jurkiewicz]

Simple patch for this problem.

[Jared Jurkiewicz]

Problem was that the group level execution merely wasn't calling the group level setup and teardowns. Easy enough to fix.

[Jared Jurkiewicz]

(In [9929]) Fixing minor test harness issue. fixes #3504