Repository: reef
Updated Branches:
  refs/heads/master 6ef63c196 -> 75f78ac8d


[REEF-1473] Add check for strStatus=null to TestBridgeClient

This change adds check for strStatus being null in TestBridgeClient.
It doesn't fix the root cause of failures, which can't be reproduced now,
but slightly improves troubleshooting in case the failures repeat.

JIRA:
  [REEF-1473](https://issues.apache.org/jira/browse/REEF-1473)

Pull request:
  This closes #1139


Project: http://git-wip-us.apache.org/repos/asf/reef/repo
Commit: http://git-wip-us.apache.org/repos/asf/reef/commit/75f78ac8
Tree: http://git-wip-us.apache.org/repos/asf/reef/tree/75f78ac8
Diff: http://git-wip-us.apache.org/repos/asf/reef/diff/75f78ac8

Branch: refs/heads/master
Commit: 75f78ac8dfbcfa76bf4bde82948761d0fd1372db
Parents: 6ef63c1
Author: Mariia Mykhailova <[email protected]>
Authored: Wed Sep 28 15:20:02 2016 -0700
Committer: Markus Weimer <[email protected]>
Committed: Wed Sep 28 17:11:22 2016 -0700

----------------------------------------------------------------------
 .../cs/Org.Apache.REEF.Tests/Functional/Bridge/TestBridgeClient.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/reef/blob/75f78ac8/lang/cs/Org.Apache.REEF.Tests/Functional/Bridge/TestBridgeClient.cs
----------------------------------------------------------------------
diff --git 
a/lang/cs/Org.Apache.REEF.Tests/Functional/Bridge/TestBridgeClient.cs 
b/lang/cs/Org.Apache.REEF.Tests/Functional/Bridge/TestBridgeClient.cs
index f72399b..66fd513 100644
--- a/lang/cs/Org.Apache.REEF.Tests/Functional/Bridge/TestBridgeClient.cs
+++ b/lang/cs/Org.Apache.REEF.Tests/Functional/Bridge/TestBridgeClient.cs
@@ -15,7 +15,6 @@
 // specific language governing permissions and limitations
 // under the License.
 
-using System;
 using System.Threading.Tasks;
 using Org.Apache.REEF.Client.Common;
 using Org.Apache.REEF.Examples.AllHandlers;
@@ -57,6 +56,7 @@ namespace Org.Apache.REEF.Tests.Functional.Bridge
 
             var uri = driverHttpEndpoint.DriverUrl + "NRT/status?a=1&b=2";
             var strStatus = driverHttpEndpoint.GetUrlResult(uri);
+            Assert.NotNull(strStatus);
             Assert.True(strStatus.Equals("Byte array returned from 
HelloHttpHandler in CLR!!!\r\n"));
 
             
await((JobSubmissionResult)driverHttpEndpoint).TryUntilNoConnection(uri);

Reply via email to