search

BlueHotDog / sails-migrations

The missing, migrations, arm of the octopus
MIT License
157 stars 36 forks source link

"Travis passes even though tests fail" fix #52

Closed bakavic closed 9 years ago

bakavic commented 9 years ago

Fixed:

  1. The databases not being created before test start
  2. Tests failing should cause Travis to note the failure
  3. Speed up the npm setup for the different sails versions slightly.

Tests should be failing now.

bakavic commented 9 years ago

The overall time for the build seems to be faster compared to current test builds, but this is highly dependent on travis-ci's load at the time.

The main purpose of this change is to reduce the number of repeated lines in the travis.yml (as I don't think npm will be supporting concurrent updates to npm cache soon...)

BlueHotDog commented 9 years ago

Looks awesome! merged! thanks!