Page MenuHomePhabricator
Create Task
Maniphest T2430

enhance pointsettest to test multiple timesteps
Closed, ResolvedPublic
Actions

Description

Currently, mitkPointSetTest only tests a pointset with a single timestep. All tests should be repeated for multiple timesteps.

Also, it would be nice, if the test gets refactored to use the MITK testing macros.

Related Objects

Event Timeline

neuhaus added projects: MITK, Bugzilla.Aug 27 2009, 3:04 PM
• schmidts added a subscriber: • schmidts.Sep 3 2009, 5:16 PM

the test uses MITK testing macros now. Testing with timesteps is still missing

• schmidts added a comment.Sep 3 2009, 5:23 PM

change request

ChangeRequest.txt345 BDownload

maleike added a subscriber: maleike.Sep 4 2009, 3:22 PM

change request ok. for testing I'd just watch the dashboard results the night after checking in

maleike added a comment.Feb 24 2010, 4:01 PM

Sabrina, is this already done?

• schmidts added a comment.Feb 24 2010, 4:38 PM

The MITK testing macros are added, but timesteps are still missing.
I think I will not have the time to do this in my practical (the next 2 weeks).

• vongruen added a subscriber: • vongruen.Mar 10 2010, 4:11 PM

The Test should be correct now

neuhaus added a subscriber: neuhaus.Mar 10 2010, 5:00 PM

Please do not close bugs before the fix is committed and the Dartclient results are uploaded to CDash.

• vongruen added a comment.Apr 1 2010, 5:54 PM

[SVN revision 22004]
COMP (#2430): Fixed adding missing Test

• vongruen added a comment.Apr 7 2010, 4:44 PM

[SVN revision 22089]
FIX (#2430): Fixed PointsetLocale Test, which failed at the last commit, described in T1571. Also removed a warning in PointSetFileIO Test.

nolden added a subscriber: nolden.Jun 22 2010, 10:23 AM

What is the status here?

nolden added a comment.Oct 16 2013, 1:58 PM

After a very quick review I would say there is some coverage of time steps in the testing code.

MITK · MIC · IMSY · DKFZ