diff options
author | dai_y <dai_y@ae88bc3d-4319-0410-8dbf-d08b4c9d3795> | 2005-12-09 23:49:11 +0000 |
---|---|---|
committer | dai_y <dai_y@ae88bc3d-4319-0410-8dbf-d08b4c9d3795> | 2005-12-09 23:49:11 +0000 |
commit | f17da25d30d6467df7de817e4214829911d29335 (patch) | |
tree | a0b6556e54be29f3cbd7454fda210948133bf996 | |
parent | f2c900324bf816418b59d57e2477b7416b718e09 (diff) | |
download | ATCD-f17da25d30d6467df7de817e4214829911d29335.tar.gz |
Fri Dec 9 16:36:27 USMST 2005 Yan Dai <dai_y@ociweb.com>
-rw-r--r-- | TAO/ChangeLog | 6 | ||||
-rw-r--r-- | TAO/tests/CSD_Strategy_Tests/TP_Foo_B/Foo_B_Statistics.cpp | 28 |
2 files changed, 25 insertions, 9 deletions
diff --git a/TAO/ChangeLog b/TAO/ChangeLog index 338bafc0fe0..6268d97d981 100644 --- a/TAO/ChangeLog +++ b/TAO/ChangeLog @@ -1,3 +1,9 @@ +Fri Dec 9 16:36:27 USMST 2005 Yan Dai <dai_y@ociweb.com> + + * tests/CSD_Strategy_Tests/TP_Foo_B/Foo_B_Statistics.cpp: + + Made actual_vs_expected() dump more details with the failure. + Fri Dec 9 21:48:48 UTC 2005 Jeff Parsons <j.parsons@vanderbilt.edu> * tao/Array_VarOut_T.h: diff --git a/TAO/tests/CSD_Strategy_Tests/TP_Foo_B/Foo_B_Statistics.cpp b/TAO/tests/CSD_Strategy_Tests/TP_Foo_B/Foo_B_Statistics.cpp index c2306c2a250..6cf19e95125 100644 --- a/TAO/tests/CSD_Strategy_Tests/TP_Foo_B/Foo_B_Statistics.cpp +++ b/TAO/tests/CSD_Strategy_Tests/TP_Foo_B/Foo_B_Statistics.cpp @@ -87,12 +87,15 @@ Foo_B_Statistics::servant_error_count (unsigned error_count) bool Foo_B_Statistics::actual_vs_expected() { + bool ret = true; + // Verify the checking results in servant operation code. if (this->servant_error_count_ > 0) { ACE_ERROR((LM_ERROR, "(%P|%t)Foo_B_Statistics::actual_vs_expected " "servant_error_count_=%u\n", servant_error_count_)); - return false; + if (ret) + ret = false; } // Verify the number of callbacks received for the collocated @@ -102,7 +105,8 @@ Foo_B_Statistics::actual_vs_expected() ACE_ERROR((LM_ERROR, "(%P|%t)Foo_B_Statistics::actual_vs_expected " "actual_callbacks_=%u expected_callbacks_=%u \n", actual_callbacks_, expected_callbacks_)); - return false; + if (ret) + ret = false; } // Verify the number of operations server received. @@ -113,7 +117,8 @@ Foo_B_Statistics::actual_vs_expected() ACE_ERROR((LM_ERROR, "(%P|%t)Foo_B_Statistics::actual_vs_expected " "expected_op_count_[%u]=%u actual_op_count_[%u]=%u \n", z, expected_op_count_[z], z, actual_op_count_[z])); - return false; + if (ret) + ret = false; } } @@ -134,7 +139,8 @@ Foo_B_Statistics::actual_vs_expected() { ACE_ERROR((LM_ERROR, "(%P|%t)Foo_B_Statistics::actual_vs_expected " "actual_size=%u expected_size=%u\n", actual_size, expected_size)); - return false; + if (ret) + ret = false; } sort (this->actual_in_long_[i]); @@ -145,7 +151,8 @@ Foo_B_Statistics::actual_vs_expected() ACE_ERROR((LM_ERROR, "(%P|%t)Foo_B_Statistics::actual_vs_expected " "actual_in_long_[%u][%u]=%d actual_in_long_[%u][%u]=%d\n", i, j, actual_in_long_[i][j], i, j+1, actual_in_long_[i][j + 1])); - return false; + if (ret) + ret = false; } } } @@ -161,7 +168,8 @@ Foo_B_Statistics::actual_vs_expected() { ACE_ERROR((LM_ERROR, "(%P|%t)Foo_B_Statistics::actual_vs_expected " "actual_size=%u expected_size=%u\n", actual_size, expected_size)); - return false; + if (ret) + ret = false; } LongVector lvec; @@ -176,7 +184,8 @@ Foo_B_Statistics::actual_vs_expected() ACE_ERROR((LM_ERROR, "(%P|%t)Foo_B_Statistics::actual_vs_expected " "actual_in_string_[%u][%u]=%s \n", i, k, actual_in_string_[i][k].c_str())); - return false; + if (ret) + ret = false; } lvec.push_back (client_id); } @@ -190,7 +199,8 @@ Foo_B_Statistics::actual_vs_expected() ACE_ERROR((LM_ERROR, "(%P|%t)Foo_B_Statistics::actual_vs_expected " "lvec[%u]=%d lvec[%u]=%d\n", j, lvec[j], j+1, lvec[j + 1])); - return false; + if (ret) + ret = false; } } } @@ -203,6 +213,6 @@ Foo_B_Statistics::actual_vs_expected() } - return true; + return ret; } |