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

src/core/rules/attr-sorted.ts

@@ -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

test/rules/attr-sort.spec.js

@@ -18,14 +18,24 @@ describe(`Rules: ${ruleId}`, () => {
  expect(messages[0].message).to.contain('["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).to.be(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).to.be(1)
+ expect(messages[0].rule.id).to.be(ruleId)
+ expect(messages[0].message).to.contain('["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>'