add encodeHtmlEntities option test for element attributes

alanclarke · alanclarke · commit ad323111e56b · 2014-12-18T17:33:21.000Z
diff --git a/test/fixtures/serializer_test.js b/test/fixtures/serializer_test.js
@@ -70,7 +70,7 @@ exports['Regression - new line in <pre> tag'] = function () {
 };
 
 exports['Regression - escapeEntities option'] = function () {
-    var testCases = [
+    var testHtmlCases = [
             {
                 options: { encodeHtmlEntities: true },
                 src: '<!DOCTYPE html><html><head></head><body>&</body></html>',
@@ -85,7 +85,30 @@ exports['Regression - escapeEntities option'] = function () {
         ],
         parser = new Parser();
 
-    testCases.forEach(function (testCase) {
+    testHtmlCases.forEach(function (testCase) {
+        var serializer = new Serializer(null, testCase.options);
+        var document = parser.parse(testCase.src),
+            serializedResult = serializer.serialize(document);
+
+        assert.strictEqual(serializedResult, testCase.expected);
+    });
+
+    var testAttributeCases = [
+            {
+                options: { encodeHtmlEntities: true },
+                src: '<!DOCTYPE html><html><head></head><body><a href="http://example.com?hello=1&world=2"></a></body></html>',
+                expected: '<!DOCTYPE html><html><head></head><body><a href="http://example.com?hello=1&amp;world=2"></a></body></html>'
+            },
+
+            {
+                options: { encodeHtmlEntities: false },
+                src: '<!DOCTYPE html><html><head></head><body><a href="http://example.com?hello=1&world=2"></a></body></html>',
+                expected: '<!DOCTYPE html><html><head></head><body><a href="http://example.com?hello=1&world=2"></a></body></html>'
+            }
+        ],
+        parser = new Parser();
+
+    testAttributeCases.forEach(function (testCase) {
         var serializer = new Serializer(null, testCase.options);
         var document = parser.parse(testCase.src),
             serializedResult = serializer.serialize(document);
