search

mesos / storm

Storm on Mesos!
Apache License 2.0
139 stars 66 forks source link

Add test case for CommandLineShim class. #187

Closed fuji-151a closed 7 years ago

fuji-151a commented 7 years ago

There is no test class in CommandLineShim. So, I created CommnadLineShimTest class and I wrote the test case. Please check this pull request.

Thanks.

fuji-151a commented 7 years ago

@erikdw Thank you for review. I rewrote code based on your review. Please check again.

erikdw commented 7 years ago

@fuji-151a : great, thank you for making those changes, and for your contributions to the tests!