Skip to content

Commit 8e0fb1e

Browse files
Shinigami92thedaviddias
authored andcommitted
style(htmlhint): formatted test files (#276)
1 parent 84e422e commit 8e0fb1e

30 files changed

+799
-728
lines changed

.jshintrc-browser

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414

1515
"browser": true,
1616
"-W099": true,
17-
"-W100": true,
17+
"-W100": true,
1818

1919
"predef": [
2020
"HTMLHint",
@@ -26,4 +26,4 @@
2626
"CSSLint",
2727
"escape"
2828
]
29-
}
29+
}

.jshintrc-node

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,9 @@
1414

1515
"node": true,
1616
"-W099": true,
17-
"-W100": true,
17+
"-W100": true,
1818

19-
"predef": [
20-
"describe",
21-
"it"
22-
]
23-
}
19+
"predef": ["describe", "it"],
20+
21+
"esversion": 6
22+
}

test/core-spec.js

Lines changed: 21 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,32 +1,29 @@
1-
var expect = require("expect.js");
1+
const expect = require('expect.js');
22

3-
var HTMLHint = require("../index").HTMLHint;
3+
const HTMLHint = require('../index').HTMLHint;
44

5-
describe('Core', function(){
6-
7-
it('Set false to rule no effected should result in an error', function(){
8-
var code = '<img src="test.gif" />';
9-
var messages = HTMLHint.verify(code, {
10-
'alt-require': false
11-
});
5+
describe('Core', function() {
6+
it('Set false to rule no effected should result in an error', function() {
7+
const code = '<img src="test.gif" />';
8+
const messages = HTMLHint.verify(code, { 'alt-require': false });
129
expect(messages.length).to.be(0);
1310
});
1411

15-
it('Not load default ruleset when use undefined ruleset should result in an error', function(){
16-
var code = '<P ATTR=\'1\' id="a">><div id="a"><img src="" a="1" a="2"/></div>';
17-
var messages = HTMLHint.verify(code);
12+
it('Not load default ruleset when use undefined ruleset should result in an error', function() {
13+
const code = '<P ATTR=\'1\' id="a">><div id="a"><img src="" a="1" a="2"/></div>';
14+
const messages = HTMLHint.verify(code);
1815
expect(messages.length).to.be(9);
1916
});
2017

21-
it('Not load default ruleset when use empty ruleset should result in an error', function(){
22-
var code = '<P ATTR=\'1\' id="a">><div id="a"><img src="" a="1" a="2"/></div>';
23-
var messages = HTMLHint.verify(code, {});
18+
it('Not load default ruleset when use empty ruleset should result in an error', function() {
19+
const code = '<P ATTR=\'1\' id="a">><div id="a"><img src="" a="1" a="2"/></div>';
20+
const messages = HTMLHint.verify(code, {});
2421
expect(messages.length).to.be(9);
2522
});
2623

27-
it('Inline ruleset not worked should result in an error', function(){
28-
var code = '<!-- htmlhint alt-require:true-->\r\n<img src="test.gif" />';
29-
var messages = HTMLHint.verify(code, {
24+
it('Inline ruleset not worked should result in an error', function() {
25+
let code = '<!-- htmlhint alt-require:true-->\r\n<img src="test.gif" />';
26+
let messages = HTMLHint.verify(code, {
3027
'alt-require': false
3128
});
3229

@@ -42,20 +39,21 @@ describe('Core', function(){
4239
expect(messages.length).to.be(0);
4340
});
4441

45-
it('Show formated result should not result in an error', function(){
46-
var code = 'tttttttttttttttttttttttttttttttttttt<div>中文<img src="test.gif" />tttttttttttttttttttttttttttttttttttttttttttttt';
47-
var messages = HTMLHint.verify(code, {
42+
it('Show formated result should not result in an error', function() {
43+
const code =
44+
'tttttttttttttttttttttttttttttttttttt<div>中文<img src="test.gif" />tttttttttttttttttttttttttttttttttttttttttttttt';
45+
const messages = HTMLHint.verify(code, {
4846
'tag-pair': true,
4947
'alt-require': true
5048
});
51-
var arrLogs = HTMLHint.format(messages);
49+
let arrLogs = HTMLHint.format(messages);
5250
expect(arrLogs.length).to.be(4);
5351

5452
arrLogs = HTMLHint.format(messages, {
5553
colors: true,
5654
indent: 4
5755
});
58-
var log = arrLogs[0];
56+
const log = arrLogs[0];
5957
expect(/\[37m/.test(log)).to.be(true);
6058
expect(/ L1 /.test(log)).to.be(true);
6159
expect(/|\.\.\./.test(log)).to.be(true);

test/executable.spec.js

Lines changed: 17 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,19 @@
1-
var expect = require('expect.js');
1+
const expect = require('expect.js');
22

3-
var ChildProcess = require('child_process');
4-
var path = require('path');
3+
const ChildProcess = require('child_process');
4+
const path = require('path');
55

6-
describe('Executable', function () {
7-
it('should close stream before exit', function (done) {
8-
var c = ChildProcess.spawn('node', [path.resolve(__dirname,'../bin/htmlhint'), '--format', 'json', path.resolve(__dirname,'./html/executable.html')]);
9-
var stdoutEnd = false;
10-
var processEnd = false;
11-
var isDone = 0;
6+
describe('Executable', function() {
7+
it('should close stream before exit', function(done) {
8+
const c = ChildProcess.spawn('node', [
9+
path.resolve(__dirname, '../bin/htmlhint'),
10+
'--format',
11+
'json',
12+
path.resolve(__dirname, './html/executable.html')
13+
]);
14+
let stdoutEnd = false;
15+
let processEnd = false;
16+
let isDone = 0;
1217

1318
function checkDone() {
1419
isDone++;
@@ -17,17 +22,17 @@ describe('Executable', function () {
1722
}
1823
}
1924

20-
c.stdout.on('close', function() {
25+
c.stdout.on('close', () => {
2126
stdoutEnd = true;
2227
checkDone();
2328
});
2429

25-
c.on('exit', function() {
30+
c.on('exit', () => {
2631
processEnd = true;
2732
checkDone();
2833
});
2934

30-
c.stdout.on('data', function() {
35+
c.stdout.on('data', () => {
3136
expect(stdoutEnd || processEnd).to.be(false);
3237
});
3338
});

0 commit comments

Comments
 (0)