search

cnuernber / tmdjs

High Performance Data Processing for ClojureScript
MIT License
100 stars 1 forks source link

Missing transit handler for tech.v3.dataset.Text class #3

Closed hjrnunes closed 2 years ago

hjrnunes commented 2 years ago

Hi!

Title says it all. The following test passes and it shouldn't.

(deftest encode-text
  (let [dataset (tmd-io/->dataset [{:col "a"}] {:parser-fn :text})]
    (is (thrown-with-msg? Exception #"Not supported: class tech.v3.dataset.Text"
                          (tmd-transit/dataset->transit-str dataset)))))
cnuernber commented 2 years ago

Yep - good catch