[SUREFIRE-325] when parsing excludedGroups config prop, trim leading and trailing whitespace off of group names

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.3
• Fix Version/s: 2.4
• Component/s: TestNG support
• Assignee: Unassigned
• Reporter: Ian P. Springer
• Created: 03/May/07 09:25
• Updated: 21/Nov/07 03:42
• Resolved: 21/Nov/07 03:42

Description

If I specify:

<excludedGroups>agent-comm, comm-client, native-system</excludedGroups>

it gets parsed into:

"agent-comm", " comm-client", " native-system"

It would be nice if, after tokenizing on commas, leading and trailing whitespace were stripped off of the group names. That is, so results for the above example would be:

"agent-comm", "comm-client", "native-system"

I know that TestNG group names technically could truly contain leading or trailing whitespace, but realistically I don't think anyone would ever do such a thing.