Skip to content

Commit 7943011

Browse files
committed
Comment out previously proposed test changes. See comments in GitHub issue history for reason.
1 parent f6d7a61 commit 7943011

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

src/test/java/org/owasp/esapi/reference/UserTest.java

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -163,13 +163,14 @@ public void testChangePassword() throws Exception {
163163
} catch( AuthenticationException e ) {
164164
// expected
165165
}
166-
try {
167-
// Test for GitHub issue 288
168-
user.changePassword(password2, oldPassword, oldPassword);
169-
fail("Shouldn't be able to reuse original (initial) password.");
170-
} catch( AuthenticationException e ) {
171-
// expected
172-
}
166+
// Invalid test until we implement an actual password history!
167+
// try {
168+
// // Test for GitHub issue 288
169+
// user.changePassword(password2, oldPassword, oldPassword);
170+
// fail("Shouldn't be able to reuse original (initial) password.");
171+
// } catch( AuthenticationException e ) {
172+
// // expected
173+
// }
173174
assertTrue(user.verifyPassword(password2));
174175
assertFalse(user.verifyPassword("badpass"));
175176
}

0 commit comments

Comments
 (0)