Permalink
Browse files

Remove TODO in AccessPolicyIntegrationTest, because the suggestion is…

… not necessary.

REmove TODO in IntegrationTestBase, because suggestion would make tests less robust.
For #593
  • Loading branch information...
1 parent 8435e5e commit 1e0beaa0441b3093eca8d8b39f4f1c4bc55f0f9f @jcookems jcookems committed Jan 8, 2013
@@ -82,8 +82,6 @@ public void canCreateAccessPolicyWithReadPermissions() throws Exception {
verifyPolicyProperties("policy", testName, duration, AccessPolicyPermission.READ, policy);
}
- // TODO: Null name or duration?
-
@Test
public void canGetSinglePolicyById() throws Exception {
String expectedName = testPolicyPrefix + "GetOne";
@@ -81,9 +81,6 @@ public static void cleanup() throws Exception {
}
private static void cleanupEnvironment() {
- // TODO: This should be removed once cascade delete is implemented for Assets.
- // But for now, trying to delete an asset with fail if there are any
- // existing Locators associated with it.
removeAllTestLocators();
removeAllTestAssets();
removeAllTestAccessPolicies();

0 comments on commit 1e0beaa

Please sign in to comment.