Updated fault tolerance tests to correctly destroy the communicator created

    by the client. This has exposed a bug in the Ice run time. With the way
    the test now works, the client receives a ConnectionRefusedException
    whereas, previously, the adapter was deactivated on the server side.
    Depending on how endpoints are shuffled in Reference.cpp, this
    occasionally causes a test failure. It appears that we need to change
    the run time to
- recognize when an endpoint has gone dead and react correctly
- avoid re-trying endpoints that have previously failed
As is, the test gets slower and slower towards the end because it keeps
    trying endpoints of servers that were killed in previous iterations.

1 files changed, 6 insertions, 0 deletions

diff --git a/cpp/test/Ice/faultTolerance/Client.cpp b/cpp/test/Ice/faultTolerance/Client.cpp
index 054fa0b9c46..96a11f9d0c8 100644
--- a/cpp/test/Ice/faultTolerance/Client.cpp
+++ b/cpp/test/Ice/faultTolerance/Client.cpp
@@ -22,6 +22,12 @@ usage(const char* n)
 int
 run(int argc, char* argv[], const Ice::CommunicatorPtr& communicator)
 {
+    if(argc < 3)
+    {
+        usage(argv[0]);
+	return 1;
+    }
+
     vector<int> ports;
     for(int i = 1; i < argc; ++i)
     {