LinkedInAttic / Zopkio

A Functional and Performance Test Framework for Distributed Systems
Apache License 2.0
159 stars 33 forks source link

fixing issue #102 #104

Closed jdehrlich closed 8 years ago

jdehrlich commented 8 years ago

Added setup and teardown in testobj and changed single test execution to run the Ztest setup/teardown function if present instead of the suite teardown/setup function

jdehrlich commented 8 years ago

I'm not sure why the test coverage dropped as all of the added lines are tested, but I can add another test to bring up the coverage

arpras commented 8 years ago

The coverage monitoring was to monitor more drastic changes or catch significant code addition/change without sufficient unit testing

However I think you still need to make changes to Ztest to account for my changes that got pulled in at the top. (moving setup/teardown back into Ztest class instead of ztestsuite)

arpras commented 8 years ago

Thanks Josh. Merry Christmas and happy new year :)

arpras commented 8 years ago

Josh,

I have also created a pull request to merge all changes into zopkio/zopkio from linkedin/zopkio so we can move to zopkio/zopkio as the main branch. Can you please take a look at that ?