Skip to content

Commit 65b3ec0

Browse files
refined assertion in canCRUDTest and canInstallVMExtension
1 parent 59e9eb3 commit 65b3ec0

File tree

3 files changed

+142
-164
lines changed

3 files changed

+142
-164
lines changed

sdk/batch/microsoft-azure-batch/src/test/java/com/microsoft/azure/batch/PoolTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ public void canCRUDLowPriIaaSPool() throws Exception {
110110
Assert.assertEquals(POOL_LOW_PRI_VM_COUNT, (long) pool.currentLowPriorityNodes());
111111
Assert.assertNotNull("CurrentNodeCommunicationMode should be defined for pool with more than one target dedicated node", pool.currentNodeCommunicationMode());
112112
Assert.assertEquals(NodeCommunicationMode.DEFAULT, pool.targetNodeCommunicationMode());
113-
Assert.assertNotNull("Should have option on whether to use accelerated networking",pool.networkConfiguration().enableAcceleratedNetworking());
113+
Assert.assertTrue(pool.networkConfiguration().enableAcceleratedNetworking());
114114

115115
List<ComputeNode> computeNodes = batchClient.computeNodeOperations().listComputeNodes(poolId);
116116
List<InboundEndpoint> inboundEndpoints = computeNodes.get(0).endpointConfiguration().inboundEndpoints();
@@ -242,7 +242,7 @@ public void canInstallVMExtension() throws Exception {
242242
for(ComputeNode node : computeNodes){
243243
NodeVMExtension nodeVMExtension = batchClient.protocolLayer().computeNodeExtensions().get(poolId, node.id(), VM_EXTENSION_NAME);
244244
Assert.assertNotNull(nodeVMExtension);
245-
Assert.assertNotNull("Should have the option on whether to enable vm extension auto upgrade",nodeVMExtension.vmExtension().enableAutomaticUpgrade());
245+
Assert.assertTrue(nodeVMExtension.vmExtension().enableAutomaticUpgrade());
246246
}
247247

248248
// DELETE

0 commit comments

Comments
 (0)