search

reverb / swagger-spec

This fork of the swagger-spec is intended for 2.0 planning
Other
19 stars 15 forks source link

Fix and refactor the mocha tests #87

Closed maxlinc closed 10 years ago

maxlinc commented 10 years ago

I caught an error while working on https://github.com/reverb/swagger-spec/pull/86. The loop didn't work properly: it displayed the name of each file but actually tested the same file each time. I fixed that, and also cleaned things up in general.

This is working well for me now, as evidenced by being able to detect and fix all the issues in #86. However, JavaScript isn't my strongest language, so please feel free to improve this further.

maxlinc commented 10 years ago

Everything looks good now.

I suggest merging this first - then I'll rebase #86 so everything will run with the corrected tests and travis settings. Travis should go green, and you'll havea PR to review that has just schema and example changes.

maxlinc commented 10 years ago

@fehguy can you merge this as well? It goes along with #86 but I split it to make the changes easier to review.