Skip to content

Commit

Permalink
Removing suggestion for where to add tests for future features. We sh…
Browse files Browse the repository at this point in the history
…ould make those decisions in the future when we add teh features.

Also, cleaning out an implicit TODO that is out of date.
For Azure#593
  • Loading branch information
jcookems committed Jan 8, 2013
1 parent f86ea9c commit c07e6f7
Showing 1 changed file with 0 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -189,8 +189,6 @@ public void cancelTaskSuccess() throws ServiceException, InterruptedException {
//Assert
List<TaskInfo> taskInfos = service.list(Task.list(cancellingJobInfo.getTasksLink()));
for (TaskInfo taskInfo : taskInfos) {
// Replace with enums
// https://github.com/WindowsAzure/azure-sdk-for-java-pr/issues/522
verifyTaskPropertiesNoEncryption("canceled task", mediaProcessorId, TaskOption.None, taskBody,
configuration, name, 0, new Date(), null, 0.0, 0.0, null, TaskState.Canceled, taskInfo);
}
Expand Down Expand Up @@ -225,9 +223,6 @@ private void verifyTaskProperties(String message, String mediaProcessorId, TaskO
assertNotNull(message + " getErrorDetails", actual.getErrorDetails());
assertEquals(message + " getErrorDetails.size", 0, actual.getErrorDetails().size());

// TODO: Add tests for Historical events when added
// https://github.com/WindowsAzure/azure-sdk-for-java-pr/issues/494

ListResult<AssetInfo> inputAssets = service.list(Asset.list(actual.getInputAssetsLink()));
ListResult<AssetInfo> outputAssets = service.list(Asset.list(actual.getOutputAssetsLink()));

Expand Down

0 comments on commit c07e6f7

Please sign in to comment.