Fix #33083 (part 2) (#33453)

* Fix #33083 (part 2)

* Equality nonsense

* Test fix

* Remove leftover test

* Cljs fix

* Appease Kondo

* Appease Kondo more

* PR feedback

* `index-of-matching-metadata`

* Update some other things to use `index-of-closest-matching-metadata`

* PR feedback

* Don't use metadata provider for ref <=> metadata comparisons

* mark-selected-columns doesn't need metadata provider

* Update some join stuff to match with metadata

* Update more stuff to use `closest-matching-metadata`

* Test fix 

* Test fix

* Take advantage of the QP now using MLv2 metadata providers to simplify test

* Well ok maybe we do need the 3-arity after all =(

* Test fix

* Remove stray NOCOMMIT

* Some cleanup and extra tests

* Hopefully the last few test fixes

* Ok NOW I think I fixed stuff.

* Add a test for binning display name (degrees)

* Fix indentation

* Give up on the Card nonsense for now

* Ok I give up

* Revert changes to testing