Null reference error when running with MsTest trx results file

[surekhadhal]

Fixed by adding a null check for input parameter "scenario" in method ExecutionIdElement(this XElement scenario) in MsTestElementExtensions.cs file.

[dirkrombauts]

Thank you for spotting and reporting this one. I will add some more information about how to fix this issue, and then put it "up for grabs".

[dirkrombauts]

In order to do this in a test-driven way, the implementer(s) should create a test class MsTestElementExtensionsTests in $/Pickles.TestFrameworks.UnitTests/MsTest, and a test method that calls the method MsTestElementExtensionsTests.ExecutionIdElement with a null argument. That test will fail.

Then the actual method can be changed to make the test pass.

[am071092]

What are the steps to reproduce the defect? I am using the UI to upload the "results-example-mstest.trx" file but I'm not getting the null pointer exception?

[dirkrombauts]

The UI will catch that error, and should display it in the trace grid at the bottom of the screen.

@ankurmalhotra are you trying to solve this issue?

[am071092]

Yes would like to try to solve it if no one else is looking at it. Thanks

[dirkrombauts]

Great! I wrote some instructions earlier in this thread. Do you need additional information?

[thopark]

@ankurmalhotra you still work on this? Maybe it's the same problem as in #375

[am071092]

i am working on it. But if you've already got a fix, then go for it

[dirkrombauts]

Released in version 2.11.0.