fix(strings): fix unintentional, IDE-induced Unicode encoding issue

- use unicode Kelvin sign as `\u212a` to avoid/fix bug in `encodeEntitiesNum()` - potential culprits for this issue: VSCode or Prettier - add tests
thi-ng · Mar 22, 2024 · a54a69b · a54a69b
1 parent 918c3b8
commit a54a69b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/packages/strings/src/entities.ts b/packages/strings/src/entities.ts
@@ -30,7 +30,7 @@ export const ENTITIES: Record<string, string> = {
 	"™": "&trade;",
 	"℃": "&#8451;",
 	"℉": "&#8457;",
-	K: "&#8490;",
+	"\u212a": "&#8490;", // Kelvin
 	"◂": "&ltrif;",
 	"▸": "&rtrif;",
 	"▴": "&utrif;",

diff --git a/packages/strings/test/escape.test.ts b/packages/strings/test/escape.test.ts
@@ -1,5 +1,10 @@
 import { expect, test } from "bun:test";
-import { escape, unescape } from "../src/index.js";
+import {
+	escape,
+	escapeEntitiesNum,
+	unescape,
+	unescapeEntities,
+} from "../src/index.js";
 
 const SRC = "\ta\nb😎c£\\\x00";
 
@@ -10,3 +15,8 @@ test("escape", () => {
 test("roundtrip", () => {
 	expect(unescape(escape(SRC))).toBe(SRC);
 });
+
+test("encodeEntitiesNum", () => {
+	expect(escapeEntitiesNum("KK℉Ü&")).toBe("K&#x212a;&#x2109;&#xdc;&amp;");
+	expect(unescapeEntities("K&#x212a;&#x2109;&#xdc;&amp;")).toBe("KK℉Ü&");
+});