testing only with ok documents.

lemire · jkeiser · commit 8a91cecf4139 · 2020-03-21T18:12:16.000-07:00
diff --git a/tests/parse_many_test.cpp b/tests/parse_many_test.cpp
@@ -101,11 +101,13 @@ bool validate(const char *dirname) {
                 printf("size of file in bytes: %zu \n", json.size());
                 everything_fine = false;
             }
-            // issue 570, we just want to check for segfault
-            simdjson::document::parser parser;
-            for (const simdjson::document &doc : parser.load_many(fullpath)) {
-              auto iter = simdjson::document::iterator(doc);
-              //do something
+            if(!error) {
+              // issue 570, we just want to check for segfault
+              simdjson::document::parser parser;
+              for (const simdjson::document &doc : parser.load_many(fullpath)) {
+                auto iter = simdjson::document::iterator(doc);
+                //do something
+              }
             }
             free(fullpath);
         }

Original file line number	Diff line number	Diff line change
`@@ -101,11 +101,13 @@ bool validate(const char *dirname) {`
`101`	`101`	`printf("size of file in bytes: %zu \n", json.size());`
`102`	`102`	`everything_fine = false;`
`103`	`103`	`}`
`104`		`- // issue 570, we just want to check for segfault`
`105`		`- simdjson::document::parser parser;`
`106`		`- for (const simdjson::document &doc : parser.load_many(fullpath)) {`
`107`		`- auto iter = simdjson::document::iterator(doc);`
`108`		`- //do something`
	`104`	`+ if(!error) {`
	`105`	`+ // issue 570, we just want to check for segfault`
	`106`	`+ simdjson::document::parser parser;`
	`107`	`+ for (const simdjson::document &doc : parser.load_many(fullpath)) {`
	`108`	`+ auto iter = simdjson::document::iterator(doc);`
	`109`	`+ //do something`
	`110`	`+ }`
`109`	`111`	`}`
`110`	`112`	`free(fullpath);`
`111`	`113`	`}`