Merge #235 - Travis CI implementation

[mnquintana]

See #235

TODO:

• Verify build after merge, since it was failing on the schema.rb (see Current db/schema.rb broken? #636)

[dgoerger]

^ this is new.

[mnquintana]

Indeed - now you know it's implemented!

[mnquintana]

OH WAIT I think Travis CI is trying to build the merged PR - let's wait and see what happens.

[orenyk]

So it runs on each pull request?

[mnquintana]

Yup, that's the beauty of it - we won't have to manually check if tests or builds are failing anymore!

[dgoerger]

this file should be added to the git-ignore file, and copied to a database.travis.yml.example file (just like database.yml), and this info added to appropriate documentation. database setups will vary and we don't want to assume mysql2 with these default settings. :)

[mnquintana]

Unfortunately that's not how it works. This file is for Travis CI's VMs, and is necessary for automatic builds. There can't be a .example file because this isn't for human setup, it's only for Travis CI.

[dgoerger]

ah, I see, I didn't realize Travis CI is a hosted service and doesn't use our databases. Can we add that to a comment in the top line, for documentation/clarity?

[orenyk]

That is so cool :-)

[dgoerger]

is this a placeholder for #598?

[mnquintana]

Oh that was a test, sorry, I can get rid of that.

[orenyk]

Can we have the CodeClimate integration set up in this PR as well, since it's relevant?

[mnquintana]

@orenyk Sure, I'll add that, although I'm not sure if it'll work until the build passes? We'll see.

As I suspected, the build is hanging indefinitely because it's calling rake app:setup, and waiting for user input. Any suggestions about how to get around that?

[orenyk]

It's actually running rake db:seed, maybe we can modify it to default to minimal with some dummy netID?

[dgoerger]

can we call the seed script minimal=true? we might be able to program a time-out for the user NetID, such that if wait-time is greater than say 30 seconds it uses the NetID "guest" or "catest"? that should get it to pass.