New stop() method stops the service but doesn't kill the process

This makes it easier to embbed Finatra and run Integration Tests.
The shutdown() method is kept, so this won't break any users, but
we should consider deprecating it.

Our own tests were also being killed before completion --besides
not testing anything...--, fixed.

Signed-off-by: Julio Capote <julio@twitter.com>