Remove constant declarations
This was a little weird since we were declaring constants inside of a constant array. It works, but the reason it was complaining was because there were duplicates, like `APPROVE` for example. Also, Devin said we aren't actually using the constants to reference these strings, we are just using the strings themselves (though please correct me if I'm wrong about this). So I am pulling out the constant declarations from here. Just keep in mind that these strings should never be changed.
Rspec has a new way to stub methods.
The old `.stub()` and `.stub_chain()` methods are deprecated.

Now you should use:

    allow(object).to receive_message_chain(:method, :chain).and_return(value)
Merge pull request #48 from biola/fix_test_warnings

Fix test warnings