lambdaisland/deep-diff2

Print string-based diff if there's a mismatch of two strings

Open

#50 建立於 2023年9月28日

在 GitHub 查看
 (3 留言) (0 反應) (0 負責人)Clojure (18 fork)github user discovery
enhancementgood first issue

倉庫指標

Star
 (324 star)
PR 合併指標
 (PR 指標待抓取)

描述

I sometimes need to write tests for functions that output strings, in particular multi-line strings. Right now deep-diff2 will just say -<expected-string> +<actual-string>, which isn't very useful for longer strings and strings containing newlines. Right now I try to work around this by splitting the strings on newlines and letting deep-diff2 diff the arrays, which at least will highlight the lines with changes, but it's not great.

I think it'd be useful to print a string-based diff, like git-diff would, if both expressions for a mismatch are string.

For no newlines they could still be printed in place, but for newlines it likely would make sense to introduce a linebreak and then show the diff without worrying about indentation.

I found deep-diff2.printer-impl/print-mismatch, and (and (string? (:- expr)) (string? (:+ expr))) seems like an easy enough condition there, so I'm playing around with it.

Would you consider this for deep-diff2, or do you see problems with the approach? Perhaps it could be an option to not break any existing consumers.

貢獻者指南