Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configurable output file location for midje.emission.plugins.junit #367

Open
ghost opened this issue Aug 15, 2016 · 2 comments
Open

Configurable output file location for midje.emission.plugins.junit #367

ghost opened this issue Aug 15, 2016 · 2 comments

Comments

@ghost
Copy link

ghost commented Aug 15, 2016

It would be great if midje.emission.plugins.junit would allow me to configure the output file location, which is currently hard coded in make-report-filename.

@marick
Copy link
Owner

marick commented Aug 21, 2016

The whole emissions namespace is under-thought. I took a stab at supporting junit output format, knowing almost nothing about it, but I think I did a poor job at it. Any pull requests you have would be accepted, given that (almost?) everyone uses the default output.

@pershyn
Copy link

pershyn commented Feb 3, 2017

The target path can be redefined in leiningen. In my opinion, very rare use-case though.

:target-path "/tmp/target/%s"
:clean-targets ^{:protect false} ["/tmp/target/%s"]

Would be nice to take this :target-path from project.clj as default, since the test run will fail otherwise, because ./target folder does not exists.

I am looking forward to try this out and make a MR.

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

No branches or pull requests

2 participants