You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by br...@apache.org on 2015/04/19 11:48:41 UTC

[5/5] [text] Merge branch 'refactor-nameparsertest'

Merge branch 'refactor-nameparsertest'

Use Commons CSV to parse the testNames.txt file instead of implementing
a parsing algorithm in the test.
The input file has been adjusted for easier use in the test: A header
record has been added for easier retrieval of the fields during validation.
Furthermore the unnecessary spaces in testNames.txt have been removed.
This way we don't need to call trim on every field, making the test code
better readable.


Project: http://git-wip-us.apache.org/repos/asf/commons-text/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-text/commit/65852f80
Tree: http://git-wip-us.apache.org/repos/asf/commons-text/tree/65852f80
Diff: http://git-wip-us.apache.org/repos/asf/commons-text/diff/65852f80

Branch: refs/heads/master
Commit: 65852f808ce5b7d0652b479cc237e7531a7eb395
Parents: c4e8a3e 1ccc7b3
Author: Benedikt Ritter <br...@apache.org>
Authored: Sun Apr 19 11:45:00 2015 +0200
Committer: Benedikt Ritter <br...@apache.org>
Committed: Sun Apr 19 11:45:00 2015 +0200

----------------------------------------------------------------------
 pom.xml                                         |   7 ++
 .../commons/text/names/HumanNameParserTest.java |  92 +++++++++++++++++
 .../apache/commons/text/names/ParserTest.java   | 102 -------------------
 .../org/apache/commons/text/names/testNames.txt |  63 ++++++------
 4 files changed, 131 insertions(+), 133 deletions(-)
----------------------------------------------------------------------