[
https://issues.apache.org/jira/browse/LANG-701?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14258781#comment-14258781
]
Benedikt Ritter commented on LANG-701:
--------------------------------------
{quote}
The issue I have with the @Expected annotation, is that there is no regression
testing for the messages; which it is not uncommon to see people assert over in
their code (I know not good practice but people still do it). I am willing as
part of this or another issue to migrate the entire test class to this
structure, and/or as lang4 do the rest of the codebase.
{quote}
The messages of exceptions are not part of the API we provide. It is just for
informational purposes. If you really have the spare time to work though all
the tests and change this, I won't hinder you. But my feeling is, that there
are much more important things than this :-)
> StringUtils join with var args
> ------------------------------
>
> Key: LANG-701
> URL: https://issues.apache.org/jira/browse/LANG-701
> Project: Commons Lang
> Issue Type: New Feature
> Components: lang.*
> Reporter: Gabriele Kahlout
> Priority: Minor
> Fix For: Review Patch
>
> Attachments: VarArgJoinTest.java
>
>
> {code:java}
> final String apple = "apple";
> final String banana = "banana";
> final String orange = "orange";
> final String expected = apple + SPACE + banana + SPACE + orange;
> final String actual = join(SPACE, apple, banana, orange);
> final int expLength = expected.length();
> final int actLength = actual.length();
> assertEquals(expLength, actLength);
> assertEquals(expected, actual);
> {code}
> I missed the functionality of joining strings with the space separator in one
> line. BTW, why space ' ' not a constant field anywhere?
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
