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"

Task #319 (closed)

Opened 18 years ago

Closed 18 years ago

Review all failing tests as well as those marked "Broken" or "Ignore"

Reported by: jamoore Owned by: jamoore
Priority: critical Milestone: 3.0-M3
Component: Deployment Version: 3.0-M3
Keywords: iteration5,testng Cc:
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

These should be checked one time thoroughly before we get out of M3. Perhaps we need another marker for when these things should be reviewed.

Change History (3)

comment:1 Changed 18 years ago by jmoore

  • Summary changed from Review all tests marked "Broken" or "Ignore" to Review all failing tests as well as those marked "Broken" or "Ignore"

r960 reviews the server-side tests. Now all tests are:

  • passing
  • marked "broken" with ticket number
  • marked "ignore" with explanation ("experimenter","incomplete",...)

No broken tests pass, though some ignored tests pass. The next server-side phase would be to take more tests out of the "broken" group.

comment:2 Changed 18 years ago by jmoore

r965 contains various fixes to the client-side tests. They are not yet as clean as the server-side tests, but there are a few bugs (like #366) which need to be addressed before the review can continue.

comment:3 Changed 18 years ago by jmoore

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

r984 completes the test review. All tests now:

  • pass
  • are marked "broken" with ticket number
  • are marked "ignore" with explanation.

Further, "test.with.fail" has been set to true for the entire build. No tests should be allowed to fail (add "broken" or "ignore"; review often).

Misc:

  • Client now also has a broken.testng.xml suite.
  • Minor classpath errors fixed for testng.
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.64173 sec.)

© 2000-2014 University of Dundee & Open Microscopy Environment. Creative Commons Attribution 3.0 Unported License
OME source code is available under the GNU General public license or through commercial license from Glencoe Software

We're Hiring!