Source Merge Request #39: Improve build and test setup (merged)

• Fix compiler warnings
• Delete test file when VM terminates
• Externalize remaining plugin/dependency versions
• Streamline JUnit/Surefire dependency and plugin management
• Adjust CSVParser constructor visibility for subclassing
• Deprecate nullFieldIndicator() and add getNullFieldIndicator()

[Scott Conway]

Sorry it took so long to merge but I had family in this week :) Thank you for all the work. The only thing I found contentious was the deleteOnExit for files created in unit tests because if there is an issue the files are gone. But that test has been around for years without issue so I just merged it - if an issue ever arises we can just comment out the delete.

[Markus Spann]

Thanks for merging.
The file showed as an uncommited resource after execution of the tests. I like writing test temp files to a subdir of target/ and name the file after the test class/method. In a test base class you could control whether to clean up after tests or not etc. If this sound interesting I'll put together a pull request.

[Scott Conway]

No there is no need. I see the use of it but honestly I am happy when someone has unit tests for the features/fixes in their merge requests and don't want to add any sort of real or imagined complexity.