From 98e45b9b2e1d7a4b0576d1944b1769c21b52b11e Mon Sep 17 00:00:00 2001 From: Nick Schonning Date: Tue, 23 Nov 2021 18:49:46 -0500 Subject: [PATCH] fix: clenaup non-functional typos (#727) --- README.md | 2 +- docs/user-guide/rules/input-requires-label.md | 2 +- docs/user-guide/rules/src-not-empty.md | 2 +- src/cli/htmlhint.ts | 6 ++-- src/core/rules/tags-check.ts | 2 +- test/core.spec.js | 2 +- test/rules/alt-require.spec.js | 20 +++++------ test/rules/attr-lowercase.spec.js | 20 +++++------ test/rules/attr-no-duplication.spec.js | 8 ++--- .../attr-no-unnecessary-whitespace.spec.js | 8 ++--- test/rules/attr-sort.spec.js | 2 +- test/rules/attr-unsafe-chars.spec.js | 8 ++--- test/rules/attr-value-double-quotes.spec.js | 12 +++---- test/rules/attr-value-not-empty.spec.js | 8 ++--- test/rules/attr-value-single-quotes.spec.js | 12 +++---- test/rules/attr-whitespace.spec.js | 12 +++---- test/rules/doctype-first.spec.js | 8 ++--- test/rules/doctype-html5.spec.js | 8 ++--- test/rules/empty-tag-not-self-closed.spec.js | 10 +++--- test/rules/head-require.spec.js | 10 +++--- test/rules/head-script-disabled.spec.js | 10 +++--- test/rules/href-abs-or-rel.spec.js | 20 +++++------ test/rules/html-lang-require.spec.js | 10 +++--- test/rules/id-class-ad-disabled.spec.js | 34 +++++++++---------- test/rules/id-class-value.spec.js | 12 +++---- test/rules/id-unique.spec.js | 8 ++--- test/rules/inline-script-disabled.spec.js | 12 +++---- test/rules/inline-style-disabled.spec.js | 8 ++--- test/rules/script-disabled.spec.js | 10 +++--- test/rules/space-tab-mixed-disabled.spec.js | 20 +++++------ test/rules/spec-char-escape.spec.js | 14 ++++---- test/rules/src-not-empty.spec.js | 8 ++--- test/rules/style-disabled.spec.js | 8 ++--- test/rules/tag-pair.spec.js | 14 ++++---- test/rules/tag-self-close.spec.js | 10 +++--- test/rules/tagname-lowercase.spec.js | 14 ++++---- test/rules/tagname-specialchars.spec.js | 10 +++--- test/rules/tags-check.spec.js | 30 ++++++++-------- test/rules/title-require.spec.js | 6 ++-- 39 files changed, 210 insertions(+), 210 deletions(-) diff --git a/README.md b/README.md index 02afd0269..50a771bd0 100644 --- a/README.md +++ b/README.md @@ -18,7 +18,7 @@ NPM count - MIT Licence + MIT License Chat diff --git a/docs/user-guide/rules/input-requires-label.md b/docs/user-guide/rules/input-requires-label.md index 8848740e5..63c5ecf1e 100644 --- a/docs/user-guide/rules/input-requires-label.md +++ b/docs/user-guide/rules/input-requires-label.md @@ -4,7 +4,7 @@ title: input-requires-label keywords: - input - label - - accessiblity + - accessibility --- All [ input ] tags must have a corresponding [ label ] tag. diff --git a/docs/user-guide/rules/src-not-empty.md b/docs/user-guide/rules/src-not-empty.md index 87956d206..38482c9af 100644 --- a/docs/user-guide/rules/src-not-empty.md +++ b/docs/user-guide/rules/src-not-empty.md @@ -5,7 +5,7 @@ title: src-not-empty Src of img(script, link) must set value. -Emtpy of src will visit current page twice. +Empty of src will visit current page twice. Level: `error` diff --git a/src/cli/htmlhint.ts b/src/cli/htmlhint.ts index 03c5327e6..e490604d7 100644 --- a/src/cli/htmlhint.ts +++ b/src/cli/htmlhint.ts @@ -174,7 +174,7 @@ function hintTargets( }) } -// load custom rles +// load custom rules function loadCustomRules(rulesdir: string) { rulesdir = rulesdir.replace(/\\/g, '/') if (existsSync(rulesdir)) { @@ -215,7 +215,7 @@ function hintAllFiles( formatter: Formatter ruleset?: Ruleset }, - onFinised: (result: { + onFinished: (result: { targetFileCount: number targetHintFileCount: number targetHintCount: number @@ -292,7 +292,7 @@ function hintAllFiles( function checkAllHinted() { if (isWalkDone && isHintDone) { - onFinised({ + onFinished({ targetFileCount: targetFileCount, targetHintFileCount: targetHintFileCount, targetHintCount: targetHintCount, diff --git a/src/core/rules/tags-check.ts b/src/core/rules/tags-check.ts index 272ecb42f..3f98206ef 100644 --- a/src/core/rules/tags-check.ts +++ b/src/core/rules/tags-check.ts @@ -146,7 +146,7 @@ export default { redundantAttrs.forEach((attrName) => { if (attrs.some((attr) => attr.name === attrName)) { reporter.error( - `The attr '${attrName}' is redundant for <${tagName}> and should be ommited.`, + `The attr '${attrName}' is redundant for <${tagName}> and should be omitted.`, event.line, col, this, diff --git a/test/core.spec.js b/test/core.spec.js index 3ee43ee35..2428136cc 100644 --- a/test/core.spec.js +++ b/test/core.spec.js @@ -62,7 +62,7 @@ describe('Core', () => { expect(messages.length).to.be(0) }) - it('Show formated result should not result in an error', () => { + it('Show formatted result should not result in an error', () => { const code = 'tttttttttttttttttttttttttttttttttttt
中文tttttttttttttttttttttttttttttttttttttttttttttt' const messages = HTMLHint.verify(code, { diff --git a/test/rules/alt-require.spec.js b/test/rules/alt-require.spec.js index 2af623258..177671ed1 100644 --- a/test/rules/alt-require.spec.js +++ b/test/rules/alt-require.spec.js @@ -2,12 +2,12 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'alt-require' +const ruleId = 'alt-require' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Img tag have empty alt attribute should not result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) @@ -24,7 +24,7 @@ describe(`Rules: ${ruldId}`, () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) expect(messages[0].type).to.be('warning') @@ -47,7 +47,7 @@ describe(`Rules: ${ruldId}`, () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(6) expect(messages[0].type).to.be('warning') @@ -57,13 +57,13 @@ describe(`Rules: ${ruldId}`, () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(6) expect(messages[0].type).to.be('warning') }) - it('Area[href] tag have non emtpy alt attribute should not result in an error', () => { + it('Area[href] tag have non empty alt attribute should not result in an error', () => { const code = 'test' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) @@ -85,7 +85,7 @@ describe(`Rules: ${ruldId}`, () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) expect(messages[0].type).to.be('warning') @@ -95,13 +95,13 @@ describe(`Rules: ${ruldId}`, () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) expect(messages[0].type).to.be('warning') }) - it('Input[type="image"] tag have non emtpy alt attribute should not result in an error', () => { + it('Input[type="image"] tag have non empty alt attribute should not result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) diff --git a/test/rules/attr-lowercase.spec.js b/test/rules/attr-lowercase.spec.js index d610529b2..318dee70d 100644 --- a/test/rules/attr-lowercase.spec.js +++ b/test/rules/attr-lowercase.spec.js @@ -2,27 +2,27 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-lowercase' +const ruleId = 'attr-lowercase' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Not all lowercase attr should result in an error', () => { let code = '

' let messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(3) code = '

' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(2) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(2) expect(messages[0].col).to.be(1) - expect(messages[1].rule.id).to.be(ruldId) + expect(messages[1].rule.id).to.be(ruleId) expect(messages[1].line).to.be(2) expect(messages[1].col).to.be(13) }) @@ -35,28 +35,28 @@ describe(`Rules: ${ruldId}`, () => { it('Set is false should not result in an error', () => { const code = '

' - ruleOptions[ruldId] = false + ruleOptions[ruleId] = false const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) }) it('Set to array list should not result in an error', () => { const code = '

' - ruleOptions[ruldId] = ['testBox', 'tttAAA'] + ruleOptions[ruleId] = ['testBox', 'tttAAA'] const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) }) it('Set to array list with RegExp should not result in an error', () => { const code = '

' - ruleOptions[ruldId] = ['testBox', /bind:.*/] + ruleOptions[ruleId] = ['testBox', /bind:.*/] const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) }) it('Set to array list with regex string should not result in an error', () => { const code = '

' - ruleOptions[ruldId] = ['testBox', '/\\[.*\\]/'] + ruleOptions[ruleId] = ['testBox', '/\\[.*\\]/'] const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) }) diff --git a/test/rules/attr-no-duplication.spec.js b/test/rules/attr-no-duplication.spec.js index eb5e2a614..7d80694aa 100644 --- a/test/rules/attr-no-duplication.spec.js +++ b/test/rules/attr-no-duplication.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-no-duplication' +const ruleId = 'attr-no-duplication' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute name been duplication should result in an error', () => { const code = 'bbb' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(12) }) diff --git a/test/rules/attr-no-unnecessary-whitespace.spec.js b/test/rules/attr-no-unnecessary-whitespace.spec.js index 39ad33585..8316121ac 100644 --- a/test/rules/attr-no-unnecessary-whitespace.spec.js +++ b/test/rules/attr-no-unnecessary-whitespace.spec.js @@ -2,12 +2,12 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-no-unnecessary-whitespace' +const ruleId = 'attr-no-unnecessary-whitespace' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute with spaces should result in an error', () => { const codes = [ '

', @@ -17,7 +17,7 @@ describe(`Rules: ${ruldId}`, () => { for (let i = 0; i < codes.length; i++) { const messages = HTMLHint.verify(codes[i], ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) } diff --git a/test/rules/attr-sort.spec.js b/test/rules/attr-sort.spec.js index 1bfcd68a9..f25cdad4f 100644 --- a/test/rules/attr-sort.spec.js +++ b/test/rules/attr-sort.spec.js @@ -9,7 +9,7 @@ ruleOptions[ruleId] = true describe(`Rules: ${ruleId}`, () => { it('Attribute unsorted tags must result in an error', () => { - const code = '
' + const code = '
' const messages = HTMLHint.verify(code, ruleOptions) diff --git a/test/rules/attr-unsafe-chars.spec.js b/test/rules/attr-unsafe-chars.spec.js index 60b5b0ffd..8c5285da1 100644 --- a/test/rules/attr-unsafe-chars.spec.js +++ b/test/rules/attr-unsafe-chars.spec.js @@ -2,18 +2,18 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-unsafe-chars' +const ruleId = 'attr-unsafe-chars' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute value have unsafe chars should result in an error', () => { const code = 'Sud Web 2012' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(3) expect(messages[0].type).to.be('warning') diff --git a/test/rules/attr-value-double-quotes.spec.js b/test/rules/attr-value-double-quotes.spec.js index 3ae786fe1..0a35190a4 100644 --- a/test/rules/attr-value-double-quotes.spec.js +++ b/test/rules/attr-value-double-quotes.spec.js @@ -2,23 +2,23 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-value-double-quotes' +const ruleId = 'attr-value-double-quotes' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute value closed by single quotes should result in an error', () => { const code = "" const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(3) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(3) - expect(messages[1].rule.id).to.be(ruldId) + expect(messages[1].rule.id).to.be(ruleId) expect(messages[1].line).to.be(1) expect(messages[1].col).to.be(14) - expect(messages[2].rule.id).to.be(ruldId) + expect(messages[2].rule.id).to.be(ruleId) expect(messages[2].line).to.be(1) expect(messages[2].col).to.be(24) }) diff --git a/test/rules/attr-value-not-empty.spec.js b/test/rules/attr-value-not-empty.spec.js index 57f083d23..d2fd6e840 100644 --- a/test/rules/attr-value-not-empty.spec.js +++ b/test/rules/attr-value-not-empty.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-value-not-empty' +const ruleId = 'attr-value-not-empty' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute value have no value should result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) expect(messages[0].type).to.be('warning') diff --git a/test/rules/attr-value-single-quotes.spec.js b/test/rules/attr-value-single-quotes.spec.js index e0da6cb05..e05203b41 100644 --- a/test/rules/attr-value-single-quotes.spec.js +++ b/test/rules/attr-value-single-quotes.spec.js @@ -2,23 +2,23 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-value-single-quotes' +const ruleId = 'attr-value-single-quotes' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute value closed by double quotes should result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(3) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(3) - expect(messages[1].rule.id).to.be(ruldId) + expect(messages[1].rule.id).to.be(ruleId) expect(messages[1].line).to.be(1) expect(messages[1].col).to.be(14) - expect(messages[2].rule.id).to.be(ruldId) + expect(messages[2].rule.id).to.be(ruleId) expect(messages[2].line).to.be(1) expect(messages[2].col).to.be(24) }) diff --git a/test/rules/attr-whitespace.spec.js b/test/rules/attr-whitespace.spec.js index e093b74ee..67edfc5a2 100644 --- a/test/rules/attr-whitespace.spec.js +++ b/test/rules/attr-whitespace.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-whitespace' +const ruleId = 'attr-whitespace' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Double spaces in attributes should result in an error', () => { const code = '

' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) }) @@ -20,7 +20,7 @@ describe(`Rules: ${ruldId}`, () => { const code = '

' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) }) @@ -28,7 +28,7 @@ describe(`Rules: ${ruldId}`, () => { const code = '

' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(2) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) }) }) diff --git a/test/rules/doctype-first.spec.js b/test/rules/doctype-first.spec.js index 5e381a401..993c2d86e 100644 --- a/test/rules/doctype-first.spec.js +++ b/test/rules/doctype-first.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'doctype-first' +const ruleId = 'doctype-first' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Doctype not be first should result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(1) }) diff --git a/test/rules/doctype-html5.spec.js b/test/rules/doctype-html5.spec.js index cafdcd550..a90f51614 100644 --- a/test/rules/doctype-html5.spec.js +++ b/test/rules/doctype-html5.spec.js @@ -2,18 +2,18 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'doctype-html5' +const ruleId = 'doctype-html5' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Doctype not html5 should result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(1) expect(messages[0].type).to.be('warning') diff --git a/test/rules/empty-tag-not-self-closed.spec.js b/test/rules/empty-tag-not-self-closed.spec.js index 9c2b69e00..418933826 100644 --- a/test/rules/empty-tag-not-self-closed.spec.js +++ b/test/rules/empty-tag-not-self-closed.spec.js @@ -2,12 +2,12 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'empty-tag-not-self-closed' +const ruleId = 'empty-tag-not-self-closed' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('The empty tag no closed should not result in an error', () => { const code = '
' const messages = HTMLHint.verify(code, ruleOptions) @@ -18,11 +18,11 @@ describe(`Rules: ${ruldId}`, () => { const code = '
' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(2) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(1) expect(messages[0].type).to.be('error') - expect(messages[1].rule.id).to.be(ruldId) + expect(messages[1].rule.id).to.be(ruleId) expect(messages[1].line).to.be(1) expect(messages[1].col).to.be(7) expect(messages[1].type).to.be('error') diff --git a/test/rules/head-require.spec.js b/test/rules/head-require.spec.js index eddd646c4..10fe84d61 100644 --- a/test/rules/head-require.spec.js +++ b/test/rules/head-require.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'head-script-disabled' +const ruleId = 'head-script-disabled' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('External script in head should result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) expect(messages[0].type).to.be('warning') @@ -22,7 +22,7 @@ describe(`Rules: ${ruldId}`, () => { let code = '' let messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) code = '' diff --git a/test/rules/head-script-disabled.spec.js b/test/rules/head-script-disabled.spec.js index eddd646c4..10fe84d61 100644 --- a/test/rules/head-script-disabled.spec.js +++ b/test/rules/head-script-disabled.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'head-script-disabled' +const ruleId = 'head-script-disabled' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('External script in head should result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) expect(messages[0].type).to.be('warning') @@ -22,7 +22,7 @@ describe(`Rules: ${ruldId}`, () => { let code = '' let messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) code = '' diff --git a/test/rules/href-abs-or-rel.spec.js b/test/rules/href-abs-or-rel.spec.js index e8c408abb..d3829acb5 100644 --- a/test/rules/href-abs-or-rel.spec.js +++ b/test/rules/href-abs-or-rel.spec.js @@ -2,20 +2,20 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'href-abs-or-rel' +const ruleId = 'href-abs-or-rel' const ruleOptions = {} -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Href value is not absolute with abs mode should result in an error', () => { const code = '
aaabbbcccddd' - ruleOptions[ruldId] = 'abs' + ruleOptions[ruleId] = 'abs' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(2) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(3) - expect(messages[1].rule.id).to.be(ruldId) + expect(messages[1].rule.id).to.be(ruleId) expect(messages[1].line).to.be(1) expect(messages[1].col).to.be(27) }) @@ -23,7 +23,7 @@ describe(`Rules: ${ruldId}`, () => { it('Href value is absolute with abs mode should not result in an error', () => { const code = 'aaabbbcccddd' - ruleOptions[ruldId] = 'abs' + ruleOptions[ruleId] = 'abs' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) }) @@ -31,13 +31,13 @@ describe(`Rules: ${ruldId}`, () => { it('Href value is not relative with rel mode should result in an error', () => { const code = 'aaabbbcccddd' - ruleOptions[ruldId] = 'rel' + ruleOptions[ruleId] = 'rel' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(2) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(3) - expect(messages[1].rule.id).to.be(ruldId) + expect(messages[1].rule.id).to.be(ruleId) expect(messages[1].line).to.be(1) expect(messages[1].col).to.be(44) }) @@ -45,7 +45,7 @@ describe(`Rules: ${ruldId}`, () => { it('Href value is relative with rel mode should not result in an error', () => { const code = 'aaabbbcccddd' - ruleOptions[ruldId] = 'rel' + ruleOptions[ruleId] = 'rel' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) }) diff --git a/test/rules/html-lang-require.spec.js b/test/rules/html-lang-require.spec.js index 3123d0aab..1b3363a85 100644 --- a/test/rules/html-lang-require.spec.js +++ b/test/rules/html-lang-require.spec.js @@ -2,12 +2,12 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'html-lang-require' +const ruleId = 'html-lang-require' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('All the rest(non HTML) tags should not result in an error', () => { const code = '

' const messages = HTMLHint.verify(code, ruleOptions) @@ -28,12 +28,12 @@ describe(`Rules: ${ruldId}`, () => { const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) }) - it('HTML tag have an non emtpy and valid(en-EN) lang attribute should not result in an error', () => { + it('HTML tag have an non empty and valid(en-EN) lang attribute should not result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) }) - it('HTML tag have an non emtpy and valid(en) lang attribute should not result in an error', () => { + it('HTML tag have an non empty and valid(en) lang attribute should not result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) diff --git a/test/rules/id-class-ad-disabled.spec.js b/test/rules/id-class-ad-disabled.spec.js index 13dc68155..27068765b 100644 --- a/test/rules/id-class-ad-disabled.spec.js +++ b/test/rules/id-class-ad-disabled.spec.js @@ -2,60 +2,60 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'id-class-ad-disabled' +const ruleId = 'id-class-ad-disabled' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Id use ad keyword should result in an error', () => { let code = '' let messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) expect(messages[0].type).to.be('warning') - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '
test
' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '
test
' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '
test
' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '
test
' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '
test
' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) }) @@ -64,49 +64,49 @@ describe(`Rules: ${ruldId}`, () => { let code = '
test
' let messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '
test
' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '
test
' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '
test
' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '
test
' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) code = '
test
' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) }) diff --git a/test/rules/id-class-value.spec.js b/test/rules/id-class-value.spec.js index 44ed5abbd..edb0414ae 100644 --- a/test/rules/id-class-value.spec.js +++ b/test/rules/id-class-value.spec.js @@ -2,21 +2,21 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'id-class-value' +const ruleId = 'id-class-value' const ruleOptionsUnderline = {} const ruleOptionsDash = {} const ruleOptionsHump = {} const ruleOptionsReg = {} -ruleOptionsUnderline[ruldId] = 'underline' -ruleOptionsDash[ruldId] = 'dash' -ruleOptionsHump[ruldId] = 'hump' -ruleOptionsReg[ruldId] = { +ruleOptionsUnderline[ruleId] = 'underline' +ruleOptionsDash[ruleId] = 'dash' +ruleOptionsHump[ruleId] = 'hump' +ruleOptionsReg[ruleId] = { regId: /^_[a-z\d]+(-[a-z\d]+)*$/, message: 'Id and class value must meet regexp', } -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Id and class value be not lower case and split by underline should result in an error', () => { const code = '
' const messages = HTMLHint.verify(code, ruleOptionsUnderline) diff --git a/test/rules/id-unique.spec.js b/test/rules/id-unique.spec.js index 2a4309c82..51e03814d 100644 --- a/test/rules/id-unique.spec.js +++ b/test/rules/id-unique.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'id-unique' +const ruleId = 'id-unique' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Id redefine should result in an error', () => { const code = '
' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(26) expect(messages[0].type).to.be('error') diff --git a/test/rules/inline-script-disabled.spec.js b/test/rules/inline-script-disabled.spec.js index 8e62699cf..c702e1957 100644 --- a/test/rules/inline-script-disabled.spec.js +++ b/test/rules/inline-script-disabled.spec.js @@ -2,18 +2,18 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'inline-script-disabled' +const ruleId = 'inline-script-disabled' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Inline on event should result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(2) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(26) expect(messages[0].type).to.be('warning') @@ -31,7 +31,7 @@ describe(`Rules: ${ruldId}`, () => { '' let messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(2) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(11) expect(messages[0].type).to.be('warning') @@ -41,7 +41,7 @@ describe(`Rules: ${ruldId}`, () => { 'test1test2' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(2) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(9) expect(messages[0].type).to.be('warning') diff --git a/test/rules/inline-style-disabled.spec.js b/test/rules/inline-style-disabled.spec.js index 95a211686..deb775e69 100644 --- a/test/rules/inline-style-disabled.spec.js +++ b/test/rules/inline-style-disabled.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'inline-style-disabled' +const ruleId = 'inline-style-disabled' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Inline style should result in an error', () => { let code = '
' let messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(11) expect(messages[0].type).to.be('warning') diff --git a/test/rules/script-disabled.spec.js b/test/rules/script-disabled.spec.js index 55233a40f..2705ee013 100755 --- a/test/rules/script-disabled.spec.js +++ b/test/rules/script-disabled.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'script-disabled' +const ruleId = 'script-disabled' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Add external script file should result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) expect(messages[0].type).to.be('error') @@ -21,7 +21,7 @@ describe(`Rules: ${ruldId}`, () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) expect(messages[0].type).to.be('error') diff --git a/test/rules/space-tab-mixed-disabled.spec.js b/test/rules/space-tab-mixed-disabled.spec.js index 436744162..7ce87b828 100644 --- a/test/rules/space-tab-mixed-disabled.spec.js +++ b/test/rules/space-tab-mixed-disabled.spec.js @@ -2,40 +2,40 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'space-tab-mixed-disabled' +const ruleId = 'space-tab-mixed-disabled' const ruleMixOptions = {} const ruleSpaceOptions = {} const ruleSpace4Options = {} const ruleSpace5Options = {} const ruleTabOptions = {} -ruleMixOptions[ruldId] = true -ruleSpaceOptions[ruldId] = 'space' -ruleSpace4Options[ruldId] = 'space4' -ruleSpace5Options[ruldId] = 'space5' -ruleTabOptions[ruldId] = 'tab' +ruleMixOptions[ruleId] = true +ruleSpaceOptions[ruleId] = 'space' +ruleSpace4Options[ruleId] = 'space4' +ruleSpace5Options[ruleId] = 'space5' +ruleTabOptions[ruleId] = 'tab' -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Spaces and tabs mixed in front of line should result in an error', () => { // space before tab let code = ' bbb' let messages = HTMLHint.verify(code, ruleMixOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(1) // tab before space code = ' bbb' messages = HTMLHint.verify(code, ruleMixOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(1) // multi line code = '
\r\n bbb' messages = HTMLHint.verify(code, ruleMixOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(2) expect(messages[0].col).to.be(1) }) diff --git a/test/rules/spec-char-escape.spec.js b/test/rules/spec-char-escape.spec.js index 147d8a9bb..96ca24e00 100644 --- a/test/rules/spec-char-escape.spec.js +++ b/test/rules/spec-char-escape.spec.js @@ -2,23 +2,23 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'spec-char-escape' +const ruleId = 'spec-char-escape' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Special characters: <> should result in an error', () => { const code = '

aaa>bbb< ccc

ddd\r\neee<' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(3) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) - expect(messages[1].rule.id).to.be(ruldId) + expect(messages[1].rule.id).to.be(ruleId) expect(messages[1].line).to.be(1) expect(messages[1].col).to.be(11) - expect(messages[2].rule.id).to.be(ruldId) + expect(messages[2].rule.id).to.be(ruleId) expect(messages[2].line).to.be(2) expect(messages[2].col).to.be(4) }) @@ -27,7 +27,7 @@ describe(`Rules: ${ruldId}`, () => { const code = '

Steinway & Sons

' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(12) }) diff --git a/test/rules/src-not-empty.spec.js b/test/rules/src-not-empty.spec.js index 8e6a6a740..1fee585bc 100644 --- a/test/rules/src-not-empty.spec.js +++ b/test/rules/src-not-empty.spec.js @@ -2,13 +2,13 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'src-not-empty' +const ruleId = 'src-not-empty' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { - it('Src be emtpy should result in an error', () => { +describe(`Rules: ${ruleId}`, () => { + it('Src be empty should result in an error', () => { const code = '