Hi all

I was debugging multiple issues over the last day, and I've noticed that
it's very hard to debug asserts like this:

assertEquals(InterpreterResult.Code.SUCCESS, interpreterResult.code());

because it doesn't say what is the reason for it, just a difference. It
would be much simpler to debug if we add the InterpreterResult as a first
argument, so you can see exception, and stacktrace, like this:

assertEquals(interpreterResult.toString(), InterpreterResult.Code.SUCCESS,
interpreterResult.code());

P.S. I maybe can go through most of the tests and fix that - do we need to
have JIRA for such changes?

-- 
With best wishes,                    Alex Ott
http://alexott.net/
Twitter: alexott_en (English), alexott (Russian)

Reply via email to