Updated index.js to include formatFormulaHTML function and added test…

…s for html encoding
joshbtn · Apr 28, 2017 · 92350cd · 92350cd
1 parent 8c2390d
commit 92350cd
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/index.js b/index.js
diff --git a/test/ExcelFormulaUtilities.test.js b/test/ExcelFormulaUtilities.test.js
@@ -8,7 +8,10 @@ describe("ExcelFormulaUtilities", () => {
   describe("#formatFormula", () => {
 
     it("should replace < or > signs in a formula with &lt; and &gt;.  Tests fix for Issue #58.", () => {
-      assert.equals(true, false)
+      let inputFormula = '"<h1>foo</h1>"';
+      let expected = '"&lt;h1&gt;foo&lt;/h1&gt;"';
+      let actual = formula.formatFormula(inputFormula);
+      assert.equal(actual, expected);
     });
 
     it("should parse a formula that doesn't start with a function. Tests fix for Issue #46.", () => {
@@ -20,6 +23,16 @@ describe("ExcelFormulaUtilities", () => {
 
   });
 
+  describe("#formatFormulaHTML", () => {
+
+    it("should replace < or > signs in a formula with &lt; and &gt;.  Tests fix for Issue #58.", () => {
+      let inputFormula = '"<h1>foo</h1>"';
+      let expected = '=<span class="quote_mark">"</span><span class="text">&lt;h1&gt;foo&lt;/h1&gt;</span><span class="quote_mark">"</span>';
+      let actual = formula.formatFormulaHTML(inputFormula);
+      assert.equal(actual, expected);
+    });
+  });
+
   describe("#toCSharp", () => {
     it("should parse a formula that doesn't start with a function. Tests fix for Issue #46.", () => {
       let inputFormula='=A5';