Full unit test coverage on package fsm. #8
Conversation
|
I'm not sure what's going on with this check failure. |
|
Can you trigger a rerun of the tests? |
|
I think the only way I can trigger a re-run of the tests is to close this pull request and then re-open it. I don't see a way for me to trigger a re-run of the tests directly in wercker. I see that the merge for pull request #9 failed with the same error as mine, Let me know if I should close this pull request and open it again to trigger a re-run of the tests. |
|
If you rebase and push, will it re run the tests then? I have merged a minor PR since this. |
Full unit test coverage on package fsm.
|
I was able to rerun the tests, thanks for the contribution! |
I use your FSM package. The branch I am requesting to be merged in to your master branch brings the package to full unit test coverage. The key changes were:
FSM'sTransitionfunction into an interface, so that I could provide an implementation that would throw an internal error.errors.go. This mostly just checks return strings, but it is nice to have coverage at 100%.