Merge pull request #203 from Financial-Times/remove-get-pixels-dep

Delete visibleContent check to remove get-pixels dependency to resolve security advisory

Author: joelcarr

README.md

@@ -107,10 +107,6 @@ urls: {
 		content: (content) => {
 			return content.includes('some-text');
 		},
-		visibleContent: {
-		  contentSelector: '.headline, .image, .standfirst'
-		  threshold: 30 // % of viewport that should be visible content
-		},
 		performance: true, //checks firstPaint/firstContentfulPaint against baseline. default = 2000, or can specify.
 		description: 'Each test may have an optional description. It will display when the test result is reported',
 	}

lib/checks/index.js

@@ -8,6 +8,5 @@ module.exports = {
 	pageErrors: require('./page-errors'),
 	performance: require('./performance'),
 	responseHeaders: require('./response-headers'),
-	screenshot: require('./screenshot'),
-	visibleContent: require('./visible-content')
+	screenshot: require('./screenshot')
 };