feat(attr-sort): sort unknown attributes alphabetically (#668)

* feat(attr-sort): sort unknown attributes alphabetically

Previously unknown attributes were just ignored. Now they get sorted alphabetically.

BREAKING CHANGE: HTMLHint will now throw an error if unkown attributes are not sorted.

fix #661

* Fix failing test

I used GitHub Copilot to help find the issue and fix...

> The error occurs because the test is using .to.be() and .to.contain() methods, which are not part of the Jest assertion library. These methods are typically used in Chai, another assertion library. In Jest, the equivalent methods are .toBe() and .toContain().

---------

Co-authored-by: coliff <[email protected]>

Author: chrisguttandin

dist/core/rules/attr-sorted.js

@@ -484,7 +484,7 @@
 	            const originalAttrs = JSON.stringify(listOfAttributes);
 	            listOfAttributes.sort((a, b) => {
 	                if (orderMap[a] == undefined && orderMap[b] == undefined) {
-	                    return 0;
+	                    return a.localeCompare(b);
 	                }
 	                if (orderMap[a] == undefined) {
 	                    return 1;

dist/htmlhint.js

@@ -34,7 +34,7 @@ export default {
       const originalAttrs = JSON.stringify(listOfAttributes)
       listOfAttributes.sort((a, b) => {
         if (orderMap[a] == undefined && orderMap[b] == undefined) {
-          return 0
+          return a.localeCompare(b)
         }
         if (orderMap[a] == undefined) {
           return 1

dist/htmlhint.min.js

@@ -16,14 +16,24 @@ describe(`Rules: ${ruleId}`, () => {
     expect(messages[0].message).toContain('["id","class","title"]')
   })
 
-  it('Attribute unsorted tags that are unrecognizable should not throw error', () => {
-    const code = '<div img="image" meta="meta" font="font"></div>'
+  it('Attribute sorted tags that are unrecognizable should not throw error', () => {
+    const code = '<div font="font" img="image" meta="meta"></div>'
 
     const messages = HTMLHint.verify(code, ruleOptions)
 
     expect(messages.length).toBe(0)
   })
 
+  it('Attribute unsorted tags that are unrecognizable should throw error', () => {
+    const code = '<div img="image" meta="meta" font="font"></div>'
+
+    const messages = HTMLHint.verify(code, ruleOptions)
+
+    expect(messages.length).toBe(1)
+    expect(messages[0].rule.id).toBe(ruleId)
+    expect(messages[0].message).toContain('["img","meta","font"]')
+  })
+
   it('Attribute unsorted of tags of various types should throw error', () => {
     const code = '<div type="type" img="image" id="id" font="font"></div>'