Skip to content
This repository has been archived by the owner on Jan 23, 2024. It is now read-only.

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

Closed
wikier opened this issue Sep 11, 2014 · 8 comments
Closed

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

wikier opened this issue Sep 11, 2014 · 8 comments
Assignees

Comments

@wikier
Copy link
Member

wikier commented Sep 11, 2014

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

@wikier wikier self-assigned this Sep 11, 2014
@wikier
Copy link
Member Author

wikier commented Sep 17, 2014

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
Copy link
Contributor

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

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

@sspeiche
Copy link
Contributor

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

wikier added a commit that referenced this issue Sep 18, 2014
@wikier
Copy link
Member Author

wikier commented Sep 18, 2014

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
Copy link
Contributor

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
Copy link
Member Author

wikier commented Sep 18, 2014

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
Copy link
Member Author

wikier commented Sep 18, 2014

OK, now it looks much better :-)

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

@wikier
Copy link
Member Author

wikier commented Sep 18, 2014

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants