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

Issue: TRX file for VS: 2015: App crash (Fixed) #14

Closed
Praveenpvs opened this issue Jan 21, 2016 · 6 comments
Closed

Issue: TRX file for VS: 2015: App crash (Fixed) #14

Praveenpvs opened this issue Jan 21, 2016 · 6 comments

Comments

@Praveenpvs
Copy link

Hi,
Thanks for this nice tool. Just now tested this with VS 2015 TRX file (generated from VSTest.Console.Exe).

Wanted to share a fix, so that it will help others in using this one.

Application is crashing at line where it is removing the namespace from the assembly name. So, just placed a condition in Trxer.xslt (line#13, RemoveAssemblyName method).

public string RemoveAssemblyName(string asm)
{
if (asm.IndexOf(',') > 0)
return asm.Substring(0,asm.IndexOf(','));
else
return asm;
}

Now, it started generating the file without any problems.

Some other issues might turn up, e.g. Duration not present etc, so just get the project and fix. I believe the fixes might be limited to the XSLT file.

@magestican
Copy link

Thank you man :D

@gnaneswarreddy
Copy link

Can you upload the exe as i was unable to compile it?

@Praveenpvs
Copy link
Author

Hi,
Could not see your request on time, attached is the full package with EXE file. You need not re-compile, just refer to this directory and run the console command as suggested by the author.
TrxerConsole.zip

@gnaneswarreddy
Copy link

I am still seeing APP Crash even after doing the change. Could you please let me know any other changes needs to be done

@Praveenpvs
Copy link
Author

Did you use the package I have shared?

If you are still getting, better you open the project in VS and debug. I believe the error would be in the XSLT file. Some TRX files are a bit different w.r.to the VS version which causes this issue.

In the code, you can remove transformations which you do not need (e.g. Top 5 slowest tests).

@sorianog
Copy link

sorianog commented Mar 2, 2017

@Praveenpvs Have you considered opening a PR for your change to allow support of VS 2015?

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

No branches or pull requests

5 participants