Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Bug #100 (closed)

Opened 18 years ago

Closed 18 years ago

Last modified 18 years ago

TestNG haltOnSkipped is broken

Reported by: jamoore Owned by: jamoore
Priority: minor Cc:
Sprint: n.a.
Total Remaining Time: n.a.

Description

For some unexplained (and possibly unexplainable) reason, the testng ant task has this for the haltOnSkipped setter:

   public void setHaltOnSkipped(boolean value)
   {
     m_haltOnSkipped = true;
   }

Ha! Therefore, currently all builds run without "quick" will fail with the error, "There are SKIPPED tests". This is a problem. Several possible solutions:

  1. (Short-term) Use java omero quick build when you want all artifacts.
  2. Simply get all the tests to pass (working on this anyway).
  3. Get TestNG to fix this. (working on that, too).


Change History (3)

comment:1 Changed 18 years ago by jmoore

  • Resolution set to fixed
  • Status changed from new to closed

r677 includes a patched testng jar which sets m_haltOnSkipped = value. Now, by default failing & skipped tests will still allow for a passing build. This will change. After that, using quick or -Dfail.on.error=false (or similar) will be necessary. Stay tuned.

comment:2 Changed 18 years ago by jmoore

comment:3 Changed 18 years ago by jmoore

  • Keywords set to iteration2,testing
Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.69670 sec.)

We're Hiring!