Fix rendering of null and undefined in #html

Author: borkdude

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 [Squint](https://github.com/squint-cljs/squint): Light-weight ClojureScript dialect
 
+## v0.9.176
+
+- Fix rendering of `null` and `undefined` in `#html`
+
 ## v0.9.175
 
 - [#747](https://github.com/squint-cljs/squint/issues/747): `#html` escape fix

package.json

@@ -2,7 +2,7 @@
   "name": "squint-cljs",
   "type": "module",
   "sideEffects": false,
-  "version": "0.9.175",
+  "version": "0.9.176",
   "files": [
     "core.js",
     "src/squint/core.js",

src/squint/html.js

@@ -60,7 +60,7 @@ function attr(v) {
 }
 
 function toHTML(v, unsafe) {
-  if (v == null) return;
+  if (v == null) return '';
   if (v instanceof Html) return v;
   if (typeof(v) === 'string') return unsafe ? v : safe(v);
   if (v[Symbol.iterator]) {

test/squint/html_test.cljs

@@ -46,12 +46,12 @@
 (deftest html-nil-test
   (t/async done
     (let [js (squint.compiler/compile-string
-              "(let [p nil] #html [:div p])"
+              "(let [p nil] #html [:div p js/undefined \"foo\"])"
               {:repl true :elide-exports true :context :return})
           js (str/replace "(async function() { %s } )()" "%s" js)]
       (-> (js/eval js)
           (.then
-           #(is (html= "<div>undefined</div>" %)))
+           #(is (html= "<div>foo</div>" %)))
           (.catch #(is false "nooooo"))
           (.finally done)))))