Make sure mockOnCancellation is checked on one test, reorder a couple checks b/c I felt like it

Author: benrr101

src/Microsoft.Data.SqlClient/tests/UnitTests/Microsoft/Data/SqlClient/Utilities/AsyncHelperTest.cs

@@ -86,7 +86,7 @@ public async Task ContinueTask_TaskCompletesHandlerThrows()
             Assert.Equal(TaskStatus.Faulted, taskCompletionSource.Task.Status);
             mockOnSuccess.Verify(action => action(), Times.Once);
             mockOnFailure.VerifyNeverCalled();
-            mockOnFailure.VerifyNeverCalled();
+            mockOnCancellation.VerifyNeverCalled();
         }
 
         [Theory]
@@ -185,8 +185,8 @@ public async Task ContinueTask_TaskFaults(bool handlerShouldThrow)
             Assert.Equal(TaskStatus.Faulted, taskCompletionSource.Task.Status);
 
             mockOnSuccess.VerifyNeverCalled();
-            mockOnCancellation.VerifyNeverCalled();
             mockOnFailure.Verify(action => action(It.IsAny<Exception>()), Times.Once);
+            mockOnCancellation.VerifyNeverCalled();
         }
 
         [Fact]
@@ -394,8 +394,8 @@ public async Task ContinueTaskWithState_1Generic_TaskFaults(bool handlerShouldTh
             Assert.Equal(TaskStatus.Faulted, taskCompletionSource.Task.Status);
 
             mockOnSuccess.VerifyNeverCalled();
-            mockOnCancellation.VerifyNeverCalled();
             mockOnFailure.Verify(action => action(state1, It.IsAny<Exception>()), Times.Once);
+            mockOnCancellation.VerifyNeverCalled();
         }
 
         [Fact]