Repository navigation
Prettify the error message of TestObserver #6352
Description
Version: 2.2.5
It may be better if the Expected message and Actual one are printed on its OWN line instead of contacting them. It's a hard time to find out what the difference is.
java.lang.AssertionError: Values at position 0 differ; Expected: MobileNumberState(isErrorMsgVisible=false, isNextButtonVisible=false, isNextButtonEnabled=false, selectedCountry=Country(name=China, code=86), mobileNumber=, navigation=null) (class: MobileNumberState), Actual: MobileNumberState(isErrorMsgVisible=false, isNextButtonVisible=false, isNextButtonEnabled=false, selectedCountry=Country(name=HK, code=852), mobileNumber=, navigation=null) (class: MobileNumberState) (latch = 1, values = 1, errors = 0, completions = 0)What I expect is
java.lang.AssertionError: Values at position 0 differ;
Expected: MobileNumberState(isErrorMsgVisible=false....
Actual: MobileNumberState(isErrorMsgVisible=false....
Activity
akarnokd commentedon Dec 31, 2018
I don't know how IntelliJ Idea handles multiline exceptions with its diff viewer. If you are not on IntelliJ, I recommend using Notepad++ or alike to disect such long messages. Or you could always have your own test comparisons that print your objects formatted.
jaychang0917 commentedon Dec 31, 2018
I'm using IntelliJ, the above error message is shown on the console when my test case failed. It seems doesn't relate to diff viewer.
jaychang0917 commentedon Dec 31, 2018
The source code of
BaseTestConsumer.assertValues():Changing the format of that string with newline operator will solve the long message issue, doesn't it? Am I missing something?
akarnokd commentedon Dec 31, 2018
See #6262. Also you could include a newline in your
toStringrepresentation.