fix: missing test cases (#327)

**Fixes**: #325 

- [x] Check the commit's or even all commits' message styles matches our requested structure.
- [x] Check your code additions will fail neither code linting checks nor unit test.

#### Short description of what this resolves:
In the original pull request some of the test cases were missing. Added --recursive option for mocha to check for all individual rules.

.travis.yml

@@ -15,7 +15,7 @@ install:
   - npm install
 script:
   - 'npm run build'
-  - istanbul cover ./node_modules/mocha/bin/_mocha --reporter test -- -R spec
+  - ./node_modules/.bin/istanbul cover ./node_modules/mocha/bin/_mocha -- --recursive
   - codecov
 branches:
   only:

package.json

@@ -24,7 +24,7 @@
   "scripts": {
     "commit": "npx git-cz",
     "prettier": "prettier --write ./**/*.{js,json,md} --ignore-path ./.prettierignore",
-    "test": "mocha",
+    "test": "mocha --recursive",
     "build": "webpack",
     "travis-deploy-once": "travis-deploy-once",
     "semantic-release": "semantic-release"

src/reporter.js

@@ -7,11 +7,9 @@ class Reporter {
     this.ruleset = ruleset;
     this.messages = [];
 
-    ['error', 'warn', 'info'].forEach(type => {
-      this.error = this.report.bind(this, 'error');
-      this.warn = this.report.bind(this, 'warn');
-      this.info = this.report.bind(this, 'info');
-    });
+    this.error = this.report.bind(this, 'error');
+    this.warn = this.report.bind(this, 'warning');
+    this.info = this.report.bind(this, 'info');
   }
   report(type, message, line, col, rule, raw) {
             var self = this;

test/rules/alt-require.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'alt-require';
 const ruleOptions = {};

test/rules/attr-lowercase.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'attr-lowercase';
 const ruleOptions = {};

test/rules/attr-no-duplication.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'attr-no-duplication';
 const ruleOptions = {};

test/rules/attr-unsafe-chars.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'attr-unsafe-chars';
 const ruleOptions = {};

test/rules/attr-value-double-quotes.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'attr-value-double-quotes';
 const ruleOptions = {};

test/rules/attr-value-not-empty.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'attr-value-not-empty';
 const ruleOptions = {};

test/rules/csslint.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'csslint';
 const ruleOptions = {};

test/rules/default.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 describe('Rules: default', function() {
   it('should result 3 errors', function() {

test/rules/doctype-first.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'doctype-first';
 const ruleOptions = {};

test/rules/doctype-html5.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'doctype-html5';
 const ruleOptions = {};

test/rules/head-require.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'head-script-disabled';
 const ruleOptions = {};

test/rules/head-script-disabled.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'head-script-disabled';
 const ruleOptions = {};

test/rules/href-abs-or-rel.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'href-abs-or-rel';
 const ruleOptions = {};

test/rules/id-class-ad-disabled.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'id-class-ad-disabled';
 const ruleOptions = {};

test/rules/id-class-value.spec.js

@@ -1,6 +1,6 @@
 var expect  = require("expect.js");
 
-var HTMLHint  = require("../../index").HTMLHint;
+var HTMLHint  = require('../../dist/htmlhint.js').default;
 
 var ruldId = 'id-class-value',
     ruleOptionsUnderline = {}, ruleOptionsDash = {}, ruleOptionsHump = {}, ruleOptionsReg = {};

test/rules/id-unique.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'id-unique';
 const ruleOptions = {};

test/rules/inline-script-disabled.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'inline-script-disabled';
 const ruleOptions = {};

test/rules/inline-style-disabled.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'inline-style-disabled';
 const ruleOptions = {};

test/rules/jshint.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'jshint';
 const ruleOptions = {};

test/rules/script-disabled.spec.js

@@ -1,5 +1,5 @@
 var expect = require('expect.js'),
-  HTMLHint = require('../../index').HTMLHint;
+  HTMLHint = require('../../dist/htmlhint.js').default;
 var ruldId = 'script-disabled',
   ruleOptions = {};
 ruleOptions[ruldId] = true;

test/rules/space-tab-mixed-disabled.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'space-tab-mixed-disabled';
 const ruleMixOptions = {};

test/rules/spec-char-escape.spec.js

@@ -1,6 +1,6 @@
 var expect  = require("expect.js");
 
-var HTMLHint  = require("../../index").HTMLHint;
+var HTMLHint  = require('../../dist/htmlhint.js').default;
 
 var ruldId = 'spec-char-escape',
     ruleOptions = {};

test/rules/src-not-empty.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'src-not-empty';
 const ruleOptions = {};

test/rules/style-disabled.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'style-disabled';
 const ruleOptions = {};

test/rules/tag-pair.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'tag-pair';
 const ruleOptions = {};

test/rules/tag-self-close.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'tag-self-close';
 const ruleOptions = {};
@@ -15,6 +15,7 @@ describe(`Rules: ${ruldId}`, function() {
     expect(messages[0].rule.id).to.be(ruldId);
     expect(messages[0].line).to.be(1);
     expect(messages[0].col).to.be(1);
+    console.log('lll', messages[0].type)
     expect(messages[0].type).to.be('warning');
     expect(messages[1].rule.id).to.be(ruldId);
     expect(messages[1].line).to.be(1);

test/rules/tagname-lowercase.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'tagname-lowercase';
 const ruleOptions = {};

test/rules/title-require.spec.js

@@ -1,6 +1,6 @@
 const expect = require('expect.js');
 
-const HTMLHint = require('../../index').HTMLHint;
+const HTMLHint = require('../../dist/htmlhint.js').default;
 
 const ruldId = 'title-require';
 const ruleOptions = {};