From cea4eda6efb799edd86f2a4cc61896796406a0fa Mon Sep 17 00:00:00 2001
From: jpr <jpr>
Date: Fri, 4 Nov 2005 17:05:49 +0000
Subject: [PATCH] Enforce TestValidate, and run it. (Just to have an overwiew
 of oddities in the files. Never breaks the test suite)

---
 Testing/CMakeLists.txt   |  3 ++-
 Testing/TestValidate.cxx | 57 +++++++++++++++++++++++++++++++---------
 2 files changed, 47 insertions(+), 13 deletions(-)

diff --git a/Testing/CMakeLists.txt b/Testing/CMakeLists.txt
index f5670d92..ddf756f2 100644
--- a/Testing/CMakeLists.txt
+++ b/Testing/CMakeLists.txt
@@ -20,7 +20,7 @@ SET(TEST_SOURCES
   TestImageSet.cxx
   TestDicomDirElement.cxx
   TestDicomString.cxx
-  #TestValidate.cxx
+
 )
 IF(UNIX)
   SET(TEST_SOURCES ${TEST_SOURCES}
@@ -48,6 +48,7 @@ IF (GDCM_DATA_ROOT)
     TestMakeDicomDir.cxx             # writes a file named "NewDICOMDIR"
     TestSerieHelper.cxx              # uses gdcmData as a default root directory    
     TestAllVM.cxx
+    TestValidate.cxx                 #Just to have on overviews of oddities ...  
   )
   # add test that require VTK:
   IF(GDCM_VTK)
diff --git a/Testing/TestValidate.cxx b/Testing/TestValidate.cxx
index 721581b4..835d0ec9 100644
--- a/Testing/TestValidate.cxx
+++ b/Testing/TestValidate.cxx
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: TestValidate.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/11/03 16:11:56 $
-  Version:   $Revision: 1.3 $
+  Date:      $Date: 2005/11/04 17:05:49 $
+  Version:   $Revision: 1.4 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -18,23 +18,56 @@
 #include "gdcmFile.h"
 #include "gdcmValidator.h"
 
+//Generated file:
+#include "gdcmDataImages.h"
 
-int TestValidate(int argc, char *argv[])
-{
-  if( argc < 2 )
-  {
-    std::cerr << "ouh les cornes / shame on you" << std::endl;
-    return 1;
-  }
-
-   const char *filename = argv[1];
+int Validate(std::string const &filename);
 
+int Validate(std::string const &filename)
+{
    gdcm::File *input =  gdcm::File::New( );
    input->SetFileName(filename);
    input->Load();
    gdcm::Validator *v = new gdcm::Validator();
    v->SetInput( input );
+   return 1; // allways true (we don't want to break the test suite)
+} 
+
+
+int TestValidate(int argc, char *argv[])
+{
+   if ( argc == 2 )
+   {
+      // The test is specified a specific filename, use it instead of looping
+      // over all images
+      const std::string input = argv[1];
+      return Validate( input );
+   }
+   else if ( argc > 2 || argc == 2 )
+   {
+      std::cout << "   Usage: " << argv[0]
+                << " (no arguments needed)." << std::endl;
+      std::cout << "or   Usage: " << argv[0]
+                << " filename.dcm " << std::endl;
+      return 1;
+   }
+   
+
+   int i =0;
+   int retVal = 0;  //by default this is an error
+   while( gdcmDataImages[i] != 0 )
+   {
+      std::string filename = GDCM_DATA_ROOT;
+      filename += "/";  //doh!
+      filename += gdcmDataImages[i];
+      std::cout << filename << std::endl;
+      if( Validate( filename ) != 0 )
+      {
+         retVal++;
+      }
 
-   return 0;
+      i++;
+   }
+   return 0; // never break the testsuite!
 }
 
-- 
2.45.1