[#LPP-3465] lztest fails, but build returns success

OpenLaszlo

lztest fails, but build returns success

Created: 23/Jan/07 10:30 AM Updated: 27/Sep/07 08:55 AM

Component/s:

Build System

Affects Version/s:

4.0.0

Fix Version/s:

RingDing (4.1)

Time Tracking:

Not Specified

Severity:	Blocker
Runtime:	N/A
Fix in hand:	False

Description

« Hide

When lztest fails a test, the build as a whole should fail. In build 3484-openlaszlo-branches-legals, lztest fails (see ~~LPP-3464~~) but the build says it was successful. This is very dangerous; test failures should be loud!

All

Comments

Work Log

Change History

Sort Order:

Ascending order - Click to sort in descending order

[ Permlink | « Hide ]

Benjamin Shine - 23/Jan/07 10:42 AM

The build mail does say
SUCCESS for 3484-openlaszlo-branches-legals on linux-builder.corp.laszlosystems.com at Tue Jan 23 01:02:39 PST 2007 (Megatest failed)

but megatest failure should be more than parenthetical.

[ Permlink | « Hide ]

Henry Minsky - 09/Mar/07 03:53 PM

on a related note, a fix for lztest was checked in so it doesn't fail.

The code in build.xml exec's shell scripts which do in fact return non-zero unix exit codes, and
the ant process is supposed to bubble those up into build failures. I need to log on to builder and
kick of a run from a terminal to see what's happening. The megatest uses browser resources however, so I'm not sure how this can be done remotely, need to talk to build meisters...

[ Permlink | « Hide ]

Benjamin Shine - 09/Mar/07 05:11 PM

ant -Dheadless=true megatest turns off the browser-based testing, and only does rhino testing.

[ Permlink | « Hide ]

Jim Grandy - 13/Mar/07 01:12 PM

Let's just make sure lztest doesn't fail. We can hand-hold the builder through this release, but we need lztest to pass.

[ Permlink | « Hide ]

Benjamin Shine - 17/Jul/07 12:51 PM

This has been fixed for a while.