You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Patrick Hunt (JIRA)" <ji...@apache.org> on 2008/08/06 21:14:44 UTC

[jira] Updated: (ZOOKEEPER-111) significant cleanup of existing tests

     [ https://issues.apache.org/jira/browse/ZOOKEEPER-111?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Hunt updated ZOOKEEPER-111:
-----------------------------------

    Attachment: ZOOKEEPER-111.patch

This patch is tested/passing on three machines:

1) my personal 1cpu 1.2ghz machine ubuntu
2) hudson zones machine (dual core x86 solaris)
3) 2cpu dualcore (4cores) 2ghz rhel

Note: I've comment out testReuse in SessionTest for the time being. The bug in ZOOKEEPER-63 is causing this test to fail (intermittently). I've updated that jira to note
this issue and as part of fixing that bug I will re-enable this test. Otw all tests are maintained from the mainline.

> significant cleanup of existing tests
> -------------------------------------
>
>                 Key: ZOOKEEPER-111
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-111
>             Project: Zookeeper
>          Issue Type: Improvement
>          Components: tests
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>         Attachments: ZOOKEEPER-111.patch
>
>
> About to submit a patch that significantly cleans up existing tests.
> 1) removed the need for "sleep" calls in the tests, instead I monitor the status of the server (using the client socket "stat" command) which provides a barrier to the client test operations. I use this barrier both when starting and ending the test.
> 2) also fixed a number of problems in the tests where the test itself was broken.
> 3) general cleanup and some refactoring to make it easier to write new tests, maintain old, and track down issues if the test does find a problem.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.