Merge pull request #23 from Financial-Times/no-more-heroku

No more heroku

lib/smoke/index.js

@@ -10,11 +10,7 @@ module.exports.run = async (opts, sets) => {
 		throw new Error(`Config file for smoke test does not exist at ${configFile}. Either create a config file at ./test/smoke.js, or pass in a path using --config.`);
 	} else {
 
-		let host = opts.host || 'http://local.ft.com:3002';
-
-		if (!/:|\./.test(host)) {
-			host += '.herokuapp.com';
-		}
+		let host = opts.host || 'http://localhost:3002';
 
 		if (!/https?\:\/\//.test(host)) {
 			host = 'http://' + host;

lib/smoke/verify-url.js

@@ -67,6 +67,7 @@ const tests = {
 							result: count === assertion
 						});
 					} else if(typeof assertion === 'string') {
+							await testPage.page.waitForSelector(selector);
 							const elText = await testPage.page.$eval(selector, el => el.innerText);
 							results.push({
 								expected: `element with selector selector ${selector} should contain text ${assertion}`,

tasks/open.js

@@ -9,5 +9,5 @@ module.exports = (program) => {
 		.option('-H, --host [value]', 'Set the hostname to use for all tests')
 
 		.description('Starts an instance of Chromium with all the URLs from the smoke test open')
-		.action((sets, opts) => { openSmokeTests(opts, sets) });
+		.action((sets, opts) => { openSmokeTests(opts, sets); });
 };