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

--loggerOutput option #243

Closed
TheRealPiotrP opened this issue Nov 24, 2016 · 4 comments
Closed

--loggerOutput option #243

TheRealPiotrP opened this issue Nov 24, 2016 · 4 comments

Comments

@TheRealPiotrP
Copy link
Contributor

Of course, the name is just a suggestion

Users should be able to specify the output path for loggers. This would allow aggregating all logs in one artifact folder. This, in turn, makes it much easier to scan logs for failures.

/cc @blackdwarf @livarcocc

@codito
Copy link
Contributor

codito commented Nov 24, 2016

Elaborating bit more to clarify expectation. Thoughts?

  • User will specify the directory for all result logs or should she specify the result file name. I prefer later for automation friendliness :)
  • If user has 2 target frameworks, the results are merged in the same trx
  • If user specifies multiple test projects, the results are merged in the same trx

/cc @pvlakshm

@TheRealPiotrP
Copy link
Contributor Author

That sounds great. If I want to avoid merging I can do smaller invocations. Makes sense!

@pvlakshm
Copy link
Contributor

Triage:

  • User will specify the filename using /logger:"trx;logfile=<filename>"
  • If user has 2 target frameworks, each will be a separate .trx file. If we can figure out that the targets involved, then the targetname will be appended to the filename specified. If we cannot figure out the targetname, then we will append (1), (2) etc. to the filename specified.
  • If user specifies multiple test projects, then trx file(s) generated by subsequent csproj's will clobber those already created. If any of the csprojs is targeting mulitple frameworks, we will use the above semantics to disambiguate.

@smadala
Copy link
Contributor

smadala commented Dec 26, 2016

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

4 participants