[issue21820] unittest: unhelpful truncating of long strings.

Ella Sharakanski added the comment: Any news on this? I also consider this a bug. -- nosy: +Ella Sharakanski ___ Python tracker ___ ___

[issue21820] unittest: unhelpful truncating of long strings.

Change by Chris Withers : -- versions: +Python 3.7, Python 3.8, Python 3.9 -Python 3.4 ___ Python tracker ___ ___ Python-bugs-list m

[issue21820] unittest: unhelpful truncating of long strings.

Larry Hastings added the comment: This bug is marked only for 3.4, and 3.4 is now EOL. Either it should be relocated to an active version, or it should be marked wontfix. -- nosy: +larry ___ Python tracker ___

[issue21820] unittest: unhelpful truncating of long strings.

Changes by Robert Collins : -- nosy: +rbcollins ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.py

[issue21820] unittest: unhelpful truncating of long strings.

R. David Murray added the comment: For strings, all of the string was not displayed before, either, it was just truncated at the end only. I'm surprised that that didn't apply to non-strings as well. It's the diff output that is supposed to be the full text (if maxDiff is None), to my unders

[issue21820] unittest: unhelpful truncating of long strings.

Chris Withers added the comment: No, it would not be enough. Please see the report above, the repr of non-strings may be large, but it's often intended that way and all of the string should be displayed. It really is a shame that no API was provided to control this behaviour. I actually consi

[issue21820] unittest: unhelpful truncating of long strings.

Serhiy Storchaka added the comment: Will be enough just to increase default maximal string length (_MAX_LENGTH) to say 200? I think we can do this even in a bug fix release for 3.4. We can also increase the number of tail characters (_MIN_BEGIN_LEN and _MIN_END_LEN). I'm against the "don't tri

[issue21820] unittest: unhelpful truncating of long strings.

Michael Foord added the comment: I agree that it looks like a bug that this behaviour is triggering for non-strings. There is separate code (which uses maxDiff) for comparing collections. Switching off the behaviour for 3.4 / 2.7 and a new class attribute for 3.5 is a good approach. ---

[issue21820] unittest: unhelpful truncating of long strings.

Nick Coghlan added the comment: Oh, one point - the "don't trigger on non-strings" could likely go in a bug fix release for 3.4, but the flag to turn it off entirely would be a new feature for 3.5. -- ___ Python tracker

[issue21820] unittest: unhelpful truncating of long strings.

Nick Coghlan added the comment: It's ultimately up to Michael as the module maintainer, but the class attribute approach would match the way maxDiff works. -- ___ Python tracker

[issue21820] unittest: unhelpful truncating of long strings.

Chris Withers added the comment: If I worked up a patch that: - made sure this truncation wasn't used for non-strings - added a class-attribute control for the truncation Would it be well received? -- ___ Python tracker

[issue21820] unittest: unhelpful truncating of long strings.

Changes by R. David Murray : -- nosy: +r.david.murray, serhiy.storchaka ___ Python tracker ___ ___ Python-bugs-list mailing list Unsub

[issue21820] unittest: unhelpful truncating of long strings.

Changes by Ned Deily : -- nosy: +ezio.melotti, michael.foord ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: ht

[issue21820] unittest: unhelpful truncating of long strings.

Chris Withers added the comment: Agreed, but even for strings, there really should be an API to control this... -- ___ Python tracker ___

[issue21820] unittest: unhelpful truncating of long strings.

Nick Coghlan added the comment: I'd say it's actually a bug that the new behaviour is triggering for inputs that aren't strings. -- nosy: +ncoghlan ___ Python tracker ___ __

[issue21820] unittest: unhelpful truncating of long strings.

New submission from Chris Withers: This code, prior to 3.4: from testfixtures import Comparison as C class AClass: def __init__(self,x,y=None): self.x = x if y: self.y = y def __repr__(self): return '<'+self.__class__.__name__+'>' ... self.assertEqu