paked / messenger

Package messenger is used for making bots for use with Facebook messenger
MIT License
274 stars 74 forks source link

Add tests for event dispatching #40

Closed dgellow closed 6 years ago

dgellow commented 6 years ago

This pull request implements unit tests for the dispatching of events by covering (*Messenger).classify and (*Messenger).dispatch.

dgellow commented 6 years ago

Relates to #1. I guess that covers the quote:

I think it should be relatively easy to get some basic tests going

paked commented 6 years ago

Do you think you would be able to set up Travis CI for this (or Circle, I don't really mind)?

dgellow commented 6 years ago

Sure

dgellow commented 6 years ago

I don't have the permissions to create the travis job, you will have to do it yourself :) But this config file should be enough.

dgellow commented 6 years ago

You can see the travis job for my fork, for the branch related to this PR: https://travis-ci.org/dgellow/messenger/builds/354007431

paked commented 6 years ago

Merged!