Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Drywall needs a testing suite #3

Closed
trystant opened this issue Aug 25, 2015 · 0 comments
Closed

Drywall needs a testing suite #3

trystant opened this issue Aug 25, 2015 · 0 comments
Labels
technical debt

Comments

@trystant
Copy link
Owner

I plan on using the testing strategy I used on the Red Cross project; mocha and Supertest to do "Unit" Testing. I'm familiar with it as end-to-end testing, but having that suite will be helpful. The second stage will be to have that testing suite up and running on Travis or another Continuous Integration server.
trystant pushed a commit that referenced this issue Sep 1, 2015
@mnyon-grandkru
Hello supertest & mocha for #3
21f5872 
Added supertest and mocha for "Unit" testing (AKA end to end testing)
The test suite is invoked with the command "npm test"

- package.json - Added a "test" command which maps to an invocation
  of mocha with the specified directory, in this case
  "test": "NODE_ENV=test mocha specs".

- specs/index-spec.js - Initial spec created that maps to the
  web application home page.
trystant pushed a commit that referenced this issue Sep 1, 2015
@mnyon-grandkru
Added travis.yml file to repo for #3
0eed44a 
Setting up a Travis server for this project

Runs for node.js and io.js for now
trystant pushed a commit that referenced this issue Sep 1, 2015
@mnyon-grandkru
Changed copy on spec for #3
4a11866 
Changed the copy on the 'it' function for the specs/index-spec.js
trystant pushed a commit that referenced this issue Sep 1, 2015
@mnyon-grandkru
Changed travis syntax for #3
115f6de 
Changed the YML syntax of the YAML file
trystant pushed a commit that referenced this issue Sep 1, 2015
@mnyon-grandkru
Added before_script npm install for #3
4e4e778 
npm install added to the before_script attribute to make sure that
the NPM script is installed properly.
trystant pushed a commit that referenced this issue Sep 1, 2015
@mnyon-grandkru
Added documentation & removed unneeded Travis; #3
e00d580 
- .travis.yml - Removed the 'before_script' line since it is not used

- README.md - Added a "Testing" section which describes how testing
  works at a high level and how to invoke the command
@trystant trystant closed this as completed Sep 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
technical debt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@trystant