fix: exception while trying docs validate

README.md

@@ -66,7 +66,7 @@ Options:
 - Title is right-aligned to reveal the common part
 - Validation of some columns (status, request time, description length)
 
-### Fields list (19.04.2020):
+### Fields list (20.04.2020):
 - url
 - mixed_content_url
 - canonical
@@ -89,6 +89,7 @@ Options:
 - images
 - images_without_alt
 - images_alt_empty
+- images_outer
 - links
 - links_inner
 - links_outer

src/scrap-site.js

@@ -194,16 +194,6 @@ module.exports = async (baseUrl, options = {}) => {
     onSuccess: result => {
       if (!result.result) return;
 
-      // console validate output
-      const msgs = [];
-      const validate = validateResults(result, fields); // TODO: fields declared implicitly
-      for(let name in validate) {
-        const res = validate[name];
-        const msgColor = { warning: color.yellow, error: color.red }[res.type];
-        msgs.push(`${name}: ${msgColor}${res.msg}${color.reset}`);
-      }
-      if(msgs.length > 0) console.log(msgs.join(', '));
-
       if (result.result.error) console.error(`${color.red}Error collect page data: result.result.error${color.reset}`);
       // console.log(`html_size: ${result.result.html_size}`);
     },
@@ -273,6 +263,17 @@ module.exports = async (baseUrl, options = {}) => {
 
       result.result.mixed_content_url = mixedContentUrl;
 
+      // console validate output
+      // was in onSuccess(), but causes exception on docs
+      const msgs = [];
+      const validate = validateResults(result, fields); // TODO: fields declared implicitly
+      for(let name in validate) {
+        const res = validate[name];
+        const msgColor = { warning: color.yellow, error: color.red }[res.type];
+        msgs.push(`${name}: ${msgColor}${res.msg}${color.reset}`);
+      }
+      if(msgs.length > 0) console.log(msgs.join(', '));
+
       // You can access the page object after requests
       result.content = await page.content();
       // You need to extend and return the crawled result