search

OneBusAway / onebusaway-alexa

An Java-based app to communicate with Amazon Alexa for devices such as the Amazon Echo
Other
52 stars 18 forks source link

Add test for repeat intent #56

Closed barbeau closed 8 years ago

barbeau commented 8 years ago

Alright - here we are again :).

Last assert currently fails - I believe the issue is with the obaDao mocking.

philipmw commented 8 years ago

I wonder if you rebase this atop of master now, with the change to ObaDao's mocking, if this test will start working.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+2.4%) to 52.732% when pulling ee9604d0d7c6ae10a8d4b0b0f14d47dd382f4553 on repeat-test into e5abf56e918de6f87083662d9355b5f9fefbe5bb on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+2.4%) to 52.732% when pulling ee9604d0d7c6ae10a8d4b0b0f14d47dd382f4553 on repeat-test into e5abf56e918de6f87083662d9355b5f9fefbe5bb on master.

philipmw commented 8 years ago

HEY HEY! I simply rebased atop of master, and your test runs beautifully.

We're up to a fabulous 53% coverage.

barbeau commented 8 years ago

:+1: Merge away :)