Page MenuHomePhabricator

Add VerifyDICOMDump tests for all reader configurations
Closed, WontfixPublic

Description

Status currently we only thest the ClassicDICOMReader with the VerifyDumpInfrastructure, we should extend the tests to realy check all configurations given in our source code to detect any regression for configuration options.

Tests are defined in Modules\DICOMTesting\test\CMakeLists.txt

What to do:

  • Extend VerifyDump application that you can choose via argument if a certain configuration is used.
  • Extend the Cmake script for test generation to also iterate over all possible configurations
  • Add reference dumps for all configurations

Event Timeline

floca triaged this task as High priority.May 11 2020, 1:57 PM
floca created this task.
kislinsk added a project: Auto-closed.
kislinsk added a subscriber: kislinsk.

Hi there! 🙂

This task was auto-closed according to our Task Lifecycle Management.
Please follow this link for more information and don't forget that you are encouraged to reasonable re-open tasks to revive them. 🚑

Best wishes,
The MITK devs