Build 1754: MsTest action throws NRE

Continua CI Discussion
21

so no exception details in the release for me? not that good

22

I’m not sure if the mstest action itself was changed, however the exception logging for all actions was changed this week to make it clearer. I can only suggest that you try it again.

23

as expected: i’m still getting the “An Error occurred parsing results file : …” message. i think this is because you introduced some Try … Catch for this message some time ago but the original exception is not rethrown (as it was when opening this thread), so i have no benefits of this new exception logging stuff.

24

It should be logging the exception with a stack trace :


catch (Exception ex){ 

context.SendLogMessage(new Message(MessageStatusType.Error,String.Format(“An Error occurred parsing results file : {0}.”, this._resultsFile,ex.ToDetailedString()))); 

return false; 

} 

public static string ToDetailedString(this Exception exception)
{ 

StringBuilder result = new StringBuilder();var currentException = exception; 
while (currentException != null)
{ 

result.AppendFormat(“Exception: {0}{1}”, currentException.GetType().Name, Environment.NewLine);
result.AppendFormat(“Message: {0}{1}”, currentException.Message, Environment.NewLine);
result.AppendFormat(“Stack Trace: {0}{1}”, currentException.StackTrace, Environment.NewLine);
result.AppendLine();currentException = currentException.InnerException;
} 

return result.ToString(); 

} 


I will do some more testing tomorrow.
25

Arrhhhgggg these forums are driving me mad… keeps messing up my formatting.

26

i know what you mean am i blind or is the “{1}” missing after the “{0}”?

27

Doh… I think I need a holiday. I’ll post a link to a test build here tomorrow.

28

daily bugs so lets see what happens tomorrow. thank you so far.

29

Hi Christian

This builds have the exception logging changes, let us know how it goes :

http://downloads.finalbuilder.com/downloads/continua/1.0/ContinuaCI.Server.Setup_x64_1.0.0.2002.exe

http://downloads.finalbuilder.com/downloads/continua/1.0/ContinuaCI.Server.Setup_1.0.0.2002.exe

30

[code]Exception: NullReferenceException Message: Object reference not set to an instance of an object. Stack Trace:at Continua.Modules.Builds.Common.Services.DataTransferObjects.UnitTestDTO…ctor(Guid configurationId, Int32 buildId, Guid stageId, String assemblyName, String testNamespace, String testFixture, String testName, Boolean testSuccess, String testResult, String messages, String stackTrace, Boolean testExecuted, Decimal testTime, Boolean shelved) at Continua.Modules.Builds.Agent.AgentStageSteppingContext.CreateUnitTest(String assemblyName, String namespace, String fixture, String test, Boolean success, String testResult, Boolean executed, Decimal time, String messages, String stackTrace)at Continua.Modules.Builds.UnitTests.ParserHelpers.MSTestParserHelper.CreateTests(TestRun testResults, IBuildSteppingContext context)at Continua.Modules.Builds.UnitTests.ParserHelpers.MSTestParserHelper.ParseReport(IBuildSteppingContext context, String reportFilename)at Continua.Modules.Builds.Stepping.Actions.MSTestAction.OnInterpretResult(IBuildSteppingContext context, Int32 exitCode) [/code]

31

I think I may have found the problem but don’t know for sure since I wasn’t able to reproduce the problem here :

http://downloads.finalbuilder.com/downloads/continua/1.0/ContinuaCI.Server.Setup_x64_1.0.0.2005.exe

http://downloads.finalbuilder.com/downloads/continua/1.0/ContinuaCI.Server.Setup_1.0.0.2005.exe

32

yeah, everything is green now. whatever you did, it works thank you.