search

w3c / ldp-testsuite

(UNMAINTAINED) Tests for Linked Data Platform (LDP)
Other
22 stars 29 forks source link

Allow to specify the base path where all files will be generated #182

Closed wikier closed 10 years ago

wikier commented 10 years ago

Currently we're using the current working directory, but in some scenarios we may need to provide another path.

wikier commented 10 years ago

I started to look on this, but I have to admit I do not understand how the test suite calls the reporters. Any hint, @sspeiche and @spadgett?

spadgett commented 10 years ago

@wikier LdpTestSuite.setupSuite() adds them as listeners, and TestNG calls them after the suite runs.

http://testng.org/javadoc/org/testng/IReporter.html

sspeiche commented 10 years ago

@wikier they are also explicitly listed in the testng.xml config file as listeners

wikier commented 10 years ago

I got that. That's what I tried to do on the patch you can see at the topic branch for this issue.

But the problem I do not know who/how calls LdpTestCaseReporter.main(), because line 148 currently makes this not configurable.

spadgett commented 10 years ago

That one you run manually from the command line :)

There's a Maven test-manifest profile that will run it during the build also (see pom.xml), but it's off by default.

wikier commented 10 years ago

I'm mainly thinking about a scenario where the test suite runs embedded, as we do in marmotta, where I'd like to have more control where the tests write.

I'll continue checking what's possible, and let's see. Thanks, @spadgett.

wikier commented 10 years ago

OK, now it looks much better :-)

I'll some some more testing before getting it out as 0.1.1.

wikier commented 10 years ago

I consider this implemented, enough for the scenario we need in Marmotta.