On Jun 29, 2009, at 11:35 AM, Asiri Rathnayake wrote:
Hi,
- return new
File("./chart.png");
+ return new
File("./target/chart.png");
This is very bad. You should never hardcode maven directories
location.
Shall I leave it the way it was? The problem with that was a chart
image
gets left behind after the tests, and since it is not inside the
target/
directory it is not cleaned by maven.
May be override the tearDown() method and cleanup the file? But I'm
not sure
if this is possible with the way rendering tests are done.
No, no.
There are 2 options:
- Best: Do not create an image. Just verify that jfreechart is called
and verify where it would create the image . To be done with some mocks.
- Not as good: pass the location from the pom.xml. Actually it's
already passed by surefire so you just need to retrieve it as a system
property.
-Vincent