[Webkit-unassigned] [Bug 61088] run-api-tests should run one test per process

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=61088





--- Comment #7 from Daniel Bates <dbates at webkit.org>  2011-05-19 14:22:56 PST ---
(From update of attachment 94115)
View in context: https://bugs.webkit.org/attachment.cgi?id=94115&action=review

> Tools/Scripts/run-api-tests:39
> +use Term::ANSIColor qw(:constants);

Is this part of the standard Perl distribution?  From briefly googling, I came across <http://perldoc.perl.org/perlfaq8.html#How-do-I-do-fancy-stuff-with-the-keyboard%2fscreen%2fmouse%3f>, which states that it isn't. That being said, Mac OS 10 10.6.7 includes this Perl module. We should also look to see if the WebKit Cgywin distribution (for WebKit Windows development) downloads and installs this module. I take it from Adam Roben's review that this module is included in our Cgywin distribution. Even better, can we make this module optional since it only affects the visual appearance of the output?

> Tools/Scripts/run-api-tests:45
> -sub runTestTool(@);
> +sub dumpAllTests();
> +sub runAllTests();
> +sub runAllTestsInSuite($);
> +sub runTest($$);
> +sub populateTests();

Nit: Can we sort these?

> Tools/Scripts/run-api-tests:86
> +if (runAllTests()) {
> +    exit 1;
> +}
> +else {
> +    exit 0;
> +}

I would write this in one-line:

exit(runAllTests() > 0);

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.