[inspector] Fix exception thrown when inspecting short Eduction

[alexander-yakushev]

Because Eduction type implements clojure.lang.Sequential but not clojure.lang.Counted, the inspector threw an exception when trying to inspect it. A minor but annoying bug. Besides, there could be other similar classes out there, so it's worth fixing.

Before submitting a PR make sure the following things have been done:

• [x] The commits are consistent with our contribution guidelines
• [x] You've added tests to cover your change(s)
• [x] All tests are passing
• [x] The new code is not generating reflection warnings
• [x] You've updated the changelog (if adding/changing user-visible functionality)

Thanks!

[alexander-yakushev]

This is a band-aid to prevent an exception thrown (and overall a fix that makes the underlying code more correct). Whether an Eduction should be rendered in the current way or differently can be discussed separately.

[vemv]

Tests are failing on Clojure master

Besides from that, all lgtm, if you want to expand the tests you are on time since I'll create an unrelated PR tomorrow, so it will get all in the same release train.

[alexander-yakushev]

Hmm, looks like a flaky test to me, I don't think I changed anything that could trigger that case.

[alexander-yakushev]

See, passed without changes:)

[vemv]

🍻!

With a bit of luck we'll have new orchard/cider-nrepl today/tomorrow

[alexander-yakushev]

🌳 🌳 🌳 🌳 🌳 🌳 🍎 🍎 🍎 🌳 🌳 🍺 🍺 🍺 🌳 🌳 🥴 🥴 🥴 🌳 🌳 🌳 🌳 🌳 🌳

[vemv]

https://clojars.org/cider/cider-nrepl/versions/0.34.0