cider.nrepl.middleware.test/print-object always calls deep-sorted-maps which assumes the object can be used with postwalk

[SohoPak]

Some objects (Datomic entity in my case) cannot be used with postwalk and will throw. A call to postwalk should be wrapped in try catch with catch implementing a fallback (perhaps returning the unsorted object if it does not break the code that uses the sorted object afterwards).

[alexander-yakushev]

Hi, thanks for the report! The peculiar thing is we do have try/catch around the postwalk, see https://github.com/clojure-emacs/cider-nrepl/blob/master/src/cider/nrepl/middleware/test.clj#L75-L91.

Is it feasible for you to concoct a small reproducer to demonstrate what you are seeing?

[SohoPak]

Please see the following repo - https://github.com/SohoPak/foo

Strange that I didn't notice the catch Exception clause, however in my case it won't work as I receive AbstractMethodError which is a Throwable, but not an Exception. Blanket catching of all throwables is a malpractice, so perhaps adding AbstractMethodError to the catch clause will suffice.