Updating issue8test.js - Githubissues

startersacademy / fullstack-project-01

Learning Management System

MIT License

0 stars 5 forks source link

Closed goldlilys closed 9 years ago

goldlilys commented 9 years ago

2nd pull request without merging

jeffreytu commented 9 years ago

Code is clean and has helpful comments. Casper file is good.

treyhunner commented 9 years ago

Those two new HTML files make all the tests pass for me.

:+1: for merging

jadedsurfer commented 9 years ago

A couple of things on the integration test:

When you have followed a link from courses.html to another page, instead of using a link to go back to courses.html, you can us the .back() method.
You'll want to test all of the links in the header.
After you followed a link to another page, you were testing the title of the page but you used this.getTitle(). This test will never fail. When testing the page title, you should provide a string for the expected title.