Skip to content

Commit 161c101

Browse files
User parameterized with ignoring case, fix bug.
1 parent ffc05d6 commit 161c101

2 files changed

Lines changed: 11 additions & 1 deletion

File tree

AndroidListenerExamples/src/androidTest/java/com/designatednerd/androidlistenerexamples/test/ParameterTestExample.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,4 +79,10 @@ public void findCommonLetters() {
7979
String foundCommonLetters = mLetterFinder.commonLetters(mFirstString, mSecondString);
8080
assertEquals(mExpectedCommonLetters, foundCommonLetters);
8181
}
82+
83+
@Test
84+
public void findCommonLettersIgnoringCase() {
85+
String foundCommonLetters = mLetterFinder.commonLettersCaseInsensitive(mFirstString, mSecondString);
86+
assertEquals(mExpectedCommonLetters, foundCommonLetters);
87+
}
8288
}

AndroidListenerExamples/src/main/java/com/designatednerd/androidlistenerexamples/model/CommonLetterFinder.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,10 @@ public String commonLetters(String aFirstString, String aSecondString) {
3131
}
3232

3333
public String commonLettersCaseInsensitive(String aFirstString, String aSecondString) {
34-
return commonLetters(aFirstString.toLowerCase(), aSecondString.toLowerCase());
34+
if (aFirstString != null && aSecondString != null) {
35+
return commonLetters(aFirstString.toLowerCase(), aSecondString.toLowerCase());
36+
} else {
37+
return null;
38+
}
3539
}
3640
}

0 commit comments

Comments
 (0)