Explorar el Código

added checks on error and response status code for smoke tests to make them easier to follow in logs

Henry Oswald hace 12 años
padre
commit
e7f8d11e45
Se han modificado 1 ficheros con 7 adiciones y 3 borrados
  1. 7 3
      services/web/test/smoke/coffee/SmokeTests.coffee

+ 7 - 3
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("<title>Your Projects - Online LaTeX Editor ShareLaTeX</title>")).to.equal true
-			expect(!!body.match("<h1>Projects</h1>")).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("<title>Your Projects - Online LaTeX Editor ShareLaTeX</title>"), "smoke test: body does not have correct title").to.equal true
+			expect(!!body.match("<h1>Projects</h1>"), "smoke test: body does not have correct h1").to.equal true
 			done()