Merge pull request #198 from Financial-Times/rem-old-flag

Remove old flag

.pa11yci.js

@@ -23,10 +23,8 @@ const urls = [
 
 const config = {
 	defaults: {
-		page: {
-			headers: {
-				'Cookie': 'next-flags=ads:off,cookieMessage:off; secure=true'
-			}
+		headers: {
+			'Cookie': 'next-flags=ads:off,cookieMessage:off; secure=true'
 		},
 		timeout: 25000,
 		rules: ['Principle1.Guideline1_3.1_3_1_AAA']
@@ -41,9 +39,7 @@ for (const viewport of viewports) {
 
 		config.urls.push({
 			url: url,
-			page: {
-				viewport: viewport
-			},
+			viewport: viewport,
 			screenCapture: `./pa11y_screenCapture/${path}.png`
 		});
 	}

package.json

@@ -13,7 +13,7 @@
     "date-fns": "^1.29.0"
   },
   "devDependencies": {
-    "@financial-times/n-gage": "^1.5.2",
+    "@financial-times/n-gage": "^2.0.0",
     "@financial-times/n-internal-tool": "^1.2.3",
     "bower": "^1.7.9",
     "chai": "^3.2.0",
@@ -23,7 +23,7 @@
     "mocha": "^5.2.0",
     "node-sass": "^4.0.0",
     "npm-prepublish": "^1.2.1",
-    "pa11y-ci": "^1.1.1"
+    "pa11y-ci": "^2.1.1"
   },
   "config": {},
   "scripts": {

src/presenters/teaser-presenter.js

@@ -379,8 +379,6 @@ class TeaserPresenter {
 
 		return Boolean(
 			!!this.data.enablePlayableVideo
-			&& this.data.flags
-			&& this.data.flags.insituVideoTeaser
 			&& (this.data.type && this.data.type.toLowerCase() === 'video')
 			&& ((isTopStory && !isBigStory) || (isHeavy && isLarge))
 		);