Skip to content

Commit 5a65083

Browse files
Readability Improvements
Pull Request Cleanup. Clarifying the capital letters are ASCII specific. Applying better whitespace formatting for readability.
1 parent b468673 commit 5a65083

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/test/java/org/owasp/esapi/codecs/percent/PercentCodecStringTest.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public class PercentCodecStringTest extends AbstractCodecStringTest {
3636
* It is being replicated here to allow the test to reasonably expect the correct state back.
3737
*/
3838
List<Character> immune = new ArrayList<>();
39-
// 65 - 90 (capital letters) 97 - 122 lower case 48 - 57 digits
39+
// 65 - 90 (capital letters in ASCII) 97 - 122 lower case 48 - 57 digits
4040
//numbers
4141
for (int index = 48 ; index < 58; index ++) {
4242
immune.add((char)index);
@@ -57,11 +57,9 @@ public class PercentCodecStringTest extends AbstractCodecStringTest {
5757
@Parameters(name = "{0}")
5858
public static Collection<Object[]> buildTests() {
5959
Collection<Object[]> tests = new ArrayList<>();
60-
6160
List<CodecStringTestTuple> tuples = new ArrayList<>();
61+
6262
tuples.add(newTuple("%3C", "<"));
63-
64-
//CODEPOINT tuples.add(newTuple("%C4%80", (char) 0x100));
6563
tuples.add(newTuple("%00", Character.MIN_VALUE));
6664
tuples.add(newTuple("%3D", '='));
6765
tuples.add(newTuple("%26", '&'));

0 commit comments

Comments
 (0)