borisdan/CommManager — pull request #1 by mmaracic (2013-12-02 13:37:32+00:00)
Title: WiSpear Java Developer Position - CommManager tests

Mr. Danilovich,

I have finished writing tests for the code you provided. I wrote three JUnit tests, one similar to the test you provided (ResponseTest) and two new ones testing Publisher pattern (Publisher test) and Router-Dealer pattern (Dealer test).
Despite the large changes specified by git in CommManager (at least it looks so on my end) I changed only one line in CommManager - the publish method specified wrong socket for its send command (i think it was subscriber and I changed it to publisher). My changes in test were also minute, only a few outputs to System.out.

I wrote the code originally in NetBeans (I use it because of better and more transparent Maven support) and tried it in Eclipse and everything worked well.

Please notify me if you received the code changes successfully. Contact me if any problems appear and I will do my best to assist.

Best regards,
Marijo Maračić