Bug 36078 – new-run-webkit-tests fails java/lc3 on a clean checkout

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Dirk Pranke]

Apparently someone recently introduced a java class for some tests and run-webkit-tests compiles this class as part of the test setup, but new-run-webkit-tests does not. So, all of the tests under java/lc3 will fail.

[Alexey Proskuryakov]

Do only java/lc3 tests fail? Other ones in java directory also need to compile java sources.

[Eric Seidel (no email)]

I ran new-run-webkit-tests again last night.  Wow.  Very impressive.

But I ran into this bug.

if (isAppleMacWebKit()) {
    print STDERR "Compiling Java tests\n";
    my $javaTestsDirectory = catdir($testDirectory, "java");

    if (system("/usr/bin/make", "-C", "$javaTestsDirectory")) {
        exit 1;
    }
}

Seems to be the perl code used for the fix.  I'll prepare a patch.

[Eric Seidel (no email)]

Created attachment 51774 [details]
Patch

[Eric Seidel (no email)]

Comment on attachment 51774 [details]
Patch

I did not do a full new-run-webkit-tests run with this change.  My laptop takes an hour to build webkit.  I did however verify that the build part seems to be working. :)

[Dirk Pranke]

Comment on attachment 51774 [details]
Patch

LGTM (although I'm not a reviewer).

[Adam Barth]

Comment on attachment 51774 [details]
Patch

ok

[Alexey Proskuryakov]

You didn't port this though:

    print STDERR "Compiling Java tests\n";

Does new-run-webkit-tests build quietly?

[Eric Seidel (no email)]

new-run-webkit-tests in its default mode does everything quietly.  (Using one line which overwrites itself until it finally spits out results.)  I expect we'll eventually customize the logging to the greater webkit's whims and desires. :)

I don't know enough about all the various logging modes to know how to spit that part out in only the right modes.

Dirk would know more.

[Dirk Pranke]

new-run-webkit-tests uses the "MeteredStream" class to provide the one-line status updates. The Port class doesn't actually get passed the handle to the meter, but it would be a good enhancement if it did.

I'll add that and add separate lines for the build steps in a different patch.

The general rule of thumbs I'm trying to follow for output are:

(1) By default, if everything works, we produce one line of output ("all tests completed successfully")

(2) Status updates are provided for every step that takes more than a second or two to run (so that you at least know what stage we're in). Only the current stage is displayed.

(3) Where possible, long-running stages actually produce a progress-bar style update (which is what meter.update() implements), so that you can tell that things aren't hung. Where possible, updates happen every few seconds or more frequently, up to ~10 times per second for a smooth animation). Currently only the actual running of the tests does this.
 
(I should probably document some of this stuff somewhere ...)

[WebKit Commit Bot]

Comment on attachment 51774 [details]
Patch

Rejecting patch 51774 from commit-queue.

Failed to run "['/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', '--reviewer', 'Adam Barth', '--force']" exit_code: 1
patching file WebKitTools/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file WebKitTools/Scripts/webkitpy/layout_tests/port/mac.py
Hunk #1 succeeded at 77 (offset 1 line).
Hunk #2 FAILED at 85.
1 out of 2 hunks FAILED -- saving rejects to file WebKitTools/Scripts/webkitpy/layout_tests/port/mac.py.rej

Full output: http://webkit-commit-queue.appspot.com/results/1578005

[Eric Seidel (no email)]

Attachment 51774 [details] was posted by a committer and has review+, assigning to Eric Seidel for commit.

[Eric Seidel (no email)]

Committed r56743: <http://trac.webkit.org/changeset/56743>