You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The files CoherenceEnhancingDiffusionCommandLine.h and LinearAnisotropicDiffusionCommandLine.h are not actually needed for the module functionality, and only serve for testing purposes. They should be moved into the test directory, and tests should be refactored to follow ITK testing conventions (i.e. their contents should be in a *.cxx implementation file).
The ITK testing macros should be used (e.g. TRY_EXPECT_NO_EXCEPTION, EXERCISE_BASIC_OBJECT_METHODS, etc.) to avoid boilerplate code.
The tests should run as long as possible and return the status code at the end (i.e. use a testStatus in and change its value as tests fail: requires changing the test design).
Improve the argument check message.
Remove the unnecessary message prints.
The coding style should conform to the ITK coding style (Appendix Three).
The text was updated successfully, but these errors were encountered:
The existing tests should be improved:
CoherenceEnhancingDiffusionCommandLine.h
andLinearAnisotropicDiffusionCommandLine.h
are not actually needed for the module functionality, and only serve for testing purposes. They should be moved into thetest
directory, and tests should be refactored to follow ITK testing conventions (i.e. their contents should be in a*.cxx
implementation file).TRY_EXPECT_NO_EXCEPTION
,EXERCISE_BASIC_OBJECT_METHODS
, etc.) to avoid boilerplate code.testStatus
in and change its value as tests fail: requires changing the test design).The text was updated successfully, but these errors were encountered: