From e7f8d11e45e97ec72fcdd6eef953d292387002c9 Mon Sep 17 00:00:00 2001 From: Henry Oswald Date: Wed, 12 Mar 2014 22:29:44 +0000 Subject: [PATCH] added checks on error and response status code for smoke tests to make them easier to follow in logs --- services/web/test/smoke/coffee/SmokeTests.coffee | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/services/web/test/smoke/coffee/SmokeTests.coffee b/services/web/test/smoke/coffee/SmokeTests.coffee index 04116e1da3..683892a496 100644 --- a/services/web/test/smoke/coffee/SmokeTests.coffee +++ b/services/web/test/smoke/coffee/SmokeTests.coffee @@ -37,9 +37,11 @@ describe "Opening", -> headers: "X-Forwarded-Proto": "https" }, (error, response, body) -> + expect(error, "smoke test: error in getting project").to.not.exist + expect(response.statusCode, "smoke test: response code is not 200 getting project").to.equal(200) # Check that the project id is present in the javascript that loads up the project match = !!body.match("\"project_id\":\"#{Settings.smokeTest.projectId}\"") - expect(match).to.equal true + expect(match, "smoke test: project page html does not have project_id").to.equal true done() it "the project list", (done) -> @@ -49,7 +51,9 @@ describe "Opening", -> headers: "X-Forwarded-Proto": "https" }, (error, response, body) -> - expect(!!body.match("Your Projects - Online LaTeX Editor ShareLaTeX")).to.equal true - expect(!!body.match("

Projects

")).to.equal true + expect(error, "smoke test: error returned in getting project list").to.not.exist + expect(response.statusCode, "smoke test: response code is not 200 getting project life").to.equal(200) + expect(!!body.match("Your Projects - Online LaTeX Editor ShareLaTeX"), "smoke test: body does not have correct title").to.equal true + expect(!!body.match("

Projects

"), "smoke test: body does not have correct h1").to.equal true done()