Changeset 2429

Timestamp:

Nov 24, 2004, 10:16:00 AM (21 years ago)

Author:

evanalst

Message:

Check return value for function psMetadataSetIterator.

Location:

trunk/psLib/test

Files:

• astronomy/tst_psMetadata_05.c (modified) (3 diffs)
• collections/tst_psMetadata_05.c (modified) (3 diffs)

trunk/psLib/test/astronomy/tst_psMetadata_05.c

@@ -22 +22 @@
 *  @author  Ross Harman, MHPCC
 *
-*  @version $Revision: 1.10 $  $Name: not supported by cvs2svn $
-*  @date  $Date: 2004-11-22 20:54:50 $
+*  @version $Revision: 1.11 $  $Name: not supported by cvs2svn $
+*  @date  $Date: 2004-11-24 20:16:00 $
 *
 *  Copyright 2004 Maui High Performance Computing Center, University of Hawaii
@@ -110 +110 @@
     // Test B - Set iterator at second index
     printPositiveTestHeader(stdout, "psMetadata", "Test B - Set iterator at second index");
-    psMetadataSetIterator(metadata, 2);
+    if ( !psMetadataSetIterator(metadata, 2)) {
+        psError(PS_ERR_UNKNOWN,true,"Failed to set iterator");
+        return 100;
+    }
     printFooter(stdout, "psMetadata", "Test B - Set iterator at second index", true);
 
@@ -201 +204 @@
     printNegativeTestHeader(stdout,"psMetadata", "Test G - Attempt to use null metadata with setIterator",
                             "Null metadata collection not allowed", 0);
-    psMetadataSetIterator(NULL, 0);
+    if( psMetadataSetIterator(NULL, 0)) {
+        psError(PS_ERR_UNKNOWN,true,"Set iterator did not detect invalid parameter.");
+        return 101;
+    }
     tmpList = metadata->list;
     metadata->list = NULL;
-    psMetadataSetIterator(metadata,0);
+    if( psMetadataSetIterator(metadata,0)) {
+        psError(PS_ERR_UNKNOWN,true,"Set iterator did not detect null list.");
+        return 102;
+    }
     metadata->list = tmpList;
     printFooter(stdout, "psMetadata", "Test G - Attempt to use null metadata with setIterator", true);

trunk/psLib/test/collections/tst_psMetadata_05.c

@@ -22 +22 @@
 *  @author  Ross Harman, MHPCC
 *
-*  @version $Revision: 1.10 $  $Name: not supported by cvs2svn $
-*  @date  $Date: 2004-11-22 20:54:50 $
+*  @version $Revision: 1.11 $  $Name: not supported by cvs2svn $
+*  @date  $Date: 2004-11-24 20:16:00 $
 *
 *  Copyright 2004 Maui High Performance Computing Center, University of Hawaii
@@ -110 +110 @@
     // Test B - Set iterator at second index
     printPositiveTestHeader(stdout, "psMetadata", "Test B - Set iterator at second index");
-    psMetadataSetIterator(metadata, 2);
+    if ( !psMetadataSetIterator(metadata, 2)) {
+        psError(PS_ERR_UNKNOWN,true,"Failed to set iterator");
+        return 100;
+    }
     printFooter(stdout, "psMetadata", "Test B - Set iterator at second index", true);
 
@@ -201 +204 @@
     printNegativeTestHeader(stdout,"psMetadata", "Test G - Attempt to use null metadata with setIterator",
                             "Null metadata collection not allowed", 0);
-    psMetadataSetIterator(NULL, 0);
+    if( psMetadataSetIterator(NULL, 0)) {
+        psError(PS_ERR_UNKNOWN,true,"Set iterator did not detect invalid parameter.");
+        return 101;
+    }
     tmpList = metadata->list;
     metadata->list = NULL;
-    psMetadataSetIterator(metadata,0);
+    if( psMetadataSetIterator(metadata,0)) {
+        psError(PS_ERR_UNKNOWN,true,"Set iterator did not detect null list.");
+        return 102;
+    }
     metadata->list = tmpList;
     printFooter(stdout, "psMetadata", "Test G - Attempt to use null metadata with setIterator", true);