mirror of
https://github.com/graphhopper/jsprit.git
synced 2020-01-24 07:45:05 +01:00
reproduce and fix #190
This commit is contained in:
parent
bbcaa863ee
commit
eb5b9cca9a
2 changed files with 27 additions and 1 deletions
|
|
@ -47,7 +47,7 @@ public class Skills {
|
||||||
* @return builder
|
* @return builder
|
||||||
*/
|
*/
|
||||||
public Builder addSkill(String skill) {
|
public Builder addSkill(String skill) {
|
||||||
skills.add(skill.toLowerCase());
|
skills.add(skill.trim().toLowerCase());
|
||||||
return this;
|
return this;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -63,4 +63,30 @@ public class SkillsTest {
|
||||||
assertTrue(skills.containsSkill("skill2"));
|
assertTrue(skills.containsSkill("skill2"));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void whenSkillsAddedPrecedingWhitespaceShouldNotMatter() {
|
||||||
|
Set<String> skillSet = new HashSet<String>();
|
||||||
|
skillSet.add(" skill1");
|
||||||
|
skillSet.add("Skill2");
|
||||||
|
Skills skills = Skills.Builder.newInstance().addAllSkills(skillSet).build();
|
||||||
|
assertTrue(skills.containsSkill("skill1"));
|
||||||
|
assertTrue(skills.containsSkill("skill2"));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void whenSkillsAddedTrailingWhitespaceShouldNotMatter() {
|
||||||
|
Set<String> skillSet = new HashSet<String>();
|
||||||
|
skillSet.add("skill1 ");
|
||||||
|
skillSet.add("Skill2");
|
||||||
|
Skills skills = Skills.Builder.newInstance().addAllSkills(skillSet).build();
|
||||||
|
assertTrue(skills.containsSkill("skill1"));
|
||||||
|
assertTrue(skills.containsSkill("skill2"));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void whenSkillsAddedTrailingWhitespaceShouldNotMatter2() {
|
||||||
|
Skills skills = Skills.Builder.newInstance().addSkill("skill1 ").build();
|
||||||
|
assertTrue(skills.containsSkill("skill1"));
|
||||||
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue