Merge pull request #86 from Financial-Times/jsdom-fix

Fix Jest security error coming from jsdom
Financial-Times · Sep 4, 2018 · 9c773c7 · 9c773c7
2 parents 9d43e1b + e6ad21d
commit 9c773c7
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/Makefile b/Makefile
@@ -9,6 +9,6 @@ node_modules/@financial-times/n-gage/index.mk:
 unit-test:
 	export TEST_SESSIONS_URL=https://fuhn0pye67.execute-api.eu-west-1.amazonaws.com/prod; \
 	export TEST_SESSIONS_API_KEY=mock-api-key; \
-	jest test/tasks/*.js --forceExit $(if $(CI), --ci --runInBand --testResultsProcessor="jest-junit", )
+	jest test/tasks/*.js --testURL="http://localhost/" --forceExit $(if $(CI), --ci --runInBand --testResultsProcessor="jest-junit", )
 
 test: verify unit-test
diff --git a/secret-squirrel.js b/secret-squirrel.js
@@ -0,0 +1,12 @@
+module.exports = {
+	files: {
+		allow: [],
+		allowOverrides: []
+	},
+	strings: {
+		deny: [],
+		denyOverrides: [
+			'nextpremium@ftqa\\.org' // README.md:302
+		]
+	}
+};