Updating unit tests in EventFactoryTest #256

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

aliabbasrizvi merged 1 commit into master from ali/update_tests

Jan 15, 2019

Contributor

aliabbasrizvi commented Jan 15, 2019

Summary

Follow up from easy event tracking change to update unit tests.


          Updating unit tests in EventFactoryTest

a11248b

aliabbasrizvi requested a review from thomaszurkan-optimizely

January 15, 2019 20:15

coveralls commented Jan 15, 2019 •

edited

Loading

Pull Request Test Coverage Report for Build 837

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.616%

Totals
Change from base Build 836:	0.0%
Covered Lines:	2684
Relevant Lines:	2995

💛 - Coveralls

thomaszurkan-optimizely approved these changes

View reviewed changes

Contributor

thomaszurkan-optimizely left a comment

LGTM

aliabbasrizvi merged commit 43a343a into master

aliabbasrizvi deleted the ali/update_tests branch

January 15, 2019 21:09

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet