Skip to content

Commit 4f88d75

Browse files
novemberbornnovemberborn
authored
Bump dependencies, actually run XO
1 parent 022a729 commit 4f88d75

31 files changed

+331
-331
lines changed

‎package.json

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
"node": ">=6"
99
},
1010
"scripts": {
11-
"test": "nyc ava"
11+
"test": "xo && nyc ava"
1212
},
1313
"files": [
1414
"index.js",
@@ -32,24 +32,24 @@
3232
"arrify": "^1.0.1",
3333
"deep-strict-equal": "^0.2.0",
3434
"enhance-visitors": "^1.0.0",
35-
"esm": "^3.0.71",
35+
"esm": "^3.0.82",
3636
"espree": "^4.0.0",
37-
"espurify": "^1.5.0",
37+
"espurify": "^1.8.1",
3838
"import-modules": "^1.1.0",
3939
"is-plain-object": "^2.0.4",
4040
"multimatch": "^2.1.0",
4141
"pkg-up": "^2.0.0"
4242
},
4343
"devDependencies": {
4444
"ava": "*",
45-
"babel-eslint": "^8.0.2",
46-
"coveralls": "^3.0.0",
47-
"eslint": "^5.0.0",
48-
"eslint-ava-rule-tester": "^2.0.0",
49-
"js-combinatorics": "^0.5.0",
50-
"nyc": "^12.0.2",
51-
"pify": "^3.0.0",
52-
"xo": "*"
45+
"babel-eslint": "^9.0.0",
46+
"coveralls": "^3.0.2",
47+
"eslint": "^5.5.0",
48+
"eslint-ava-rule-tester": "^2.0.2",
49+
"js-combinatorics": "^0.5.3",
50+
"nyc": "^13.0.1",
51+
"pify": "^4.0.0",
52+
"xo": "^0.23.0"
5353
},
5454
"peerDependencies": {
5555
"eslint": ">=5.0.0"

‎rules/assertion-arguments.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ const create = context => {
105105

106106
if (members[0] === 'end') {
107107
if (gottenArgs > 1) {
108-
report(node, `Too many arguments. Expected at most 1.`);
108+
report(node, 'Too many arguments. Expected at most 1.');
109109
}
110110

111111
return;

‎rules/no-ignored-test-files.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ const schema = [{
8383
type: 'object',
8484
properties: {
8585
files: {
86-
type: 'array',
86+
type: 'array'
8787
}
8888
}
8989
}];

‎rules/no-import-test-files.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ function createImportValidator(context, files, projectInfo, filename) {
2828
if (isImportingTestFile) {
2929
context.report({
3030
node,
31-
message: `Test files should not be imported`
31+
message: 'Test files should not be imported'
3232
});
3333
}
3434
};

‎test/assertion-arguments.js

Lines changed: 193 additions & 193 deletions
Large diffs are not rendered by default.

‎test/assertion-message.js

Lines changed: 76 additions & 76 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ const foundError = [{
1818
message: '(DEPRECATED) Expected no assertion message, but found one.'
1919
}];
2020

21-
const header = `const test = require('ava');`;
21+
const header = 'const test = require(\'ava\');';
2222

2323
function testCase(option, content, errors, useHeader) {
2424
const testFn = `
@@ -36,86 +36,86 @@ function testCase(option, content, errors, useHeader) {
3636

3737
ruleTester.run('assertion-message', rule, {
3838
valid: [
39-
testCase('always', `t.pass('message');`),
40-
testCase('always', `t.fail('message');`),
41-
testCase('always', `t.truthy('unicorn', 'message');`),
42-
testCase('always', `t.falsy('unicorn', 'message');`),
43-
testCase('always', `t.true(true, 'message');`),
44-
testCase('always', `t.false(false, 'message');`),
45-
testCase('always', `t.is('same', 'same', 'message');`),
46-
testCase('always', `t.not('not', 'same', 'message');`),
47-
testCase('always', `t.deepEqual({}, {}, 'message');`),
48-
testCase('always', `t.notDeepEqual({}, {a: true}, 'message');`),
49-
testCase('always', `t.throws(Promise.reject(), 'message');`),
50-
testCase('always', `t.notThrows(Promise.resolve(), 'message');`),
51-
testCase('always', `t.regex(a, /a/, 'message');`),
52-
testCase('always', `t.notRegex(a, /a/, 'message');`),
53-
testCase('always', `t.ifError(new Error(), 'message');`),
54-
testCase('always', `t.skip.is('same', 'same', 'message');`),
55-
testCase('always', `t.is.skip('same', 'same', 'message');`),
56-
testCase('always', `t.plan('a', 'b', 'c', 'd', 'message');`),
57-
testCase('always', `t.end('a', 'b', 'c', 'd', 'message');`),
39+
testCase('always', 't.pass(\'message\');'),
40+
testCase('always', 't.fail(\'message\');'),
41+
testCase('always', 't.truthy(\'unicorn\', \'message\');'),
42+
testCase('always', 't.falsy(\'unicorn\', \'message\');'),
43+
testCase('always', 't.true(true, \'message\');'),
44+
testCase('always', 't.false(false, \'message\');'),
45+
testCase('always', 't.is(\'same\', \'same\', \'message\');'),
46+
testCase('always', 't.not(\'not\', \'same\', \'message\');'),
47+
testCase('always', 't.deepEqual({}, {}, \'message\');'),
48+
testCase('always', 't.notDeepEqual({}, {a: true}, \'message\');'),
49+
testCase('always', 't.throws(Promise.reject(), \'message\');'),
50+
testCase('always', 't.notThrows(Promise.resolve(), \'message\');'),
51+
testCase('always', 't.regex(a, /a/, \'message\');'),
52+
testCase('always', 't.notRegex(a, /a/, \'message\');'),
53+
testCase('always', 't.ifError(new Error(), \'message\');'),
54+
testCase('always', 't.skip.is(\'same\', \'same\', \'message\');'),
55+
testCase('always', 't.is.skip(\'same\', \'same\', \'message\');'),
56+
testCase('always', 't.plan(\'a\', \'b\', \'c\', \'d\', \'message\');'),
57+
testCase('always', 't.end(\'a\', \'b\', \'c\', \'d\', \'message\');'),
5858
// Shouldn't be triggered since it's not a test file
59-
testCase('always', `t.true(true);`, [], false),
59+
testCase('always', 't.true(true);', [], false),
6060

61-
testCase('never', `t.pass();`),
62-
testCase('never', `t.fail();`),
63-
testCase('never', `t.truthy('unicorn');`),
64-
testCase('never', `t.falsy('unicorn');`),
65-
testCase('never', `t.true(true);`),
66-
testCase('never', `t.false(false);`),
67-
testCase('never', `t.is('same', 'same');`),
68-
testCase('never', `t.not('not', 'same');`),
69-
testCase('never', `t.deepEqual({}, {});`),
70-
testCase('never', `t.notDeepEqual({}, {a: true});`),
71-
testCase('never', `t.throws(Promise.reject());`),
72-
testCase('never', `t.notThrows(Promise.resolve());`),
73-
testCase('never', `t.regex(a, /a/);`),
74-
testCase('never', `t.notRegex(a, /a/);`),
75-
testCase('never', `t.ifError(new Error());`),
76-
testCase('never', `t.skip.is('same', 'same');`),
77-
testCase('never', `t.is.skip('same', 'same');`),
78-
testCase('never', `t.plan('a', 'b', 'c', 'd');`),
79-
testCase('never', `t.end('a', 'b', 'c', 'd');`),
61+
testCase('never', 't.pass();'),
62+
testCase('never', 't.fail();'),
63+
testCase('never', 't.truthy(\'unicorn\');'),
64+
testCase('never', 't.falsy(\'unicorn\');'),
65+
testCase('never', 't.true(true);'),
66+
testCase('never', 't.false(false);'),
67+
testCase('never', 't.is(\'same\', \'same\');'),
68+
testCase('never', 't.not(\'not\', \'same\');'),
69+
testCase('never', 't.deepEqual({}, {});'),
70+
testCase('never', 't.notDeepEqual({}, {a: true});'),
71+
testCase('never', 't.throws(Promise.reject());'),
72+
testCase('never', 't.notThrows(Promise.resolve());'),
73+
testCase('never', 't.regex(a, /a/);'),
74+
testCase('never', 't.notRegex(a, /a/);'),
75+
testCase('never', 't.ifError(new Error());'),
76+
testCase('never', 't.skip.is(\'same\', \'same\');'),
77+
testCase('never', 't.is.skip(\'same\', \'same\');'),
78+
testCase('never', 't.plan(\'a\', \'b\', \'c\', \'d\');'),
79+
testCase('never', 't.end(\'a\', \'b\', \'c\', \'d\');'),
8080
// Shouldn't be triggered since it's not a test file
81-
testCase('never', `t.true(true, 'message');`, [], false)
81+
testCase('never', 't.true(true, \'message\');', [], false)
8282
],
8383
invalid: [
84-
testCase('always', `t.pass();`, missingError),
85-
testCase('always', `t.fail();`, missingError),
86-
testCase('always', `t.truthy('unicorn');`, missingError),
87-
testCase('always', `t.falsy('unicorn');`, missingError),
88-
testCase('always', `t.true(true);`, missingError),
89-
testCase('always', `t.false(false);`, missingError),
90-
testCase('always', `t.is('same', 'same');`, missingError),
91-
testCase('always', `t.not('not', 'same');`, missingError),
92-
testCase('always', `t.deepEqual({}, {});`, missingError),
93-
testCase('always', `t.notDeepEqual({}, {a: true});`, missingError),
94-
testCase('always', `t.throws(Promise.reject());`, missingError),
95-
testCase('always', `t.notThrows(Promise.resolve());`, missingError),
96-
testCase('always', `t.regex(a, /a/);`, missingError),
97-
testCase('always', `t.notRegex(a, /a/);`, missingError),
98-
testCase('always', `t.ifError(new Error());`, missingError),
99-
testCase('always', `t.skip.is('same', 'same');`, missingError),
100-
testCase('always', `t.is.skip('same', 'same');`, missingError),
101-
testCase('always', `t.pass();`, missingError),
84+
testCase('always', 't.pass();', missingError),
85+
testCase('always', 't.fail();', missingError),
86+
testCase('always', 't.truthy(\'unicorn\');', missingError),
87+
testCase('always', 't.falsy(\'unicorn\');', missingError),
88+
testCase('always', 't.true(true);', missingError),
89+
testCase('always', 't.false(false);', missingError),
90+
testCase('always', 't.is(\'same\', \'same\');', missingError),
91+
testCase('always', 't.not(\'not\', \'same\');', missingError),
92+
testCase('always', 't.deepEqual({}, {});', missingError),
93+
testCase('always', 't.notDeepEqual({}, {a: true});', missingError),
94+
testCase('always', 't.throws(Promise.reject());', missingError),
95+
testCase('always', 't.notThrows(Promise.resolve());', missingError),
96+
testCase('always', 't.regex(a, /a/);', missingError),
97+
testCase('always', 't.notRegex(a, /a/);', missingError),
98+
testCase('always', 't.ifError(new Error());', missingError),
99+
testCase('always', 't.skip.is(\'same\', \'same\');', missingError),
100+
testCase('always', 't.is.skip(\'same\', \'same\');', missingError),
101+
testCase('always', 't.pass();', missingError),
102102

103-
testCase('never', `t.pass('message');`, foundError),
104-
testCase('never', `t.fail('message');`, foundError),
105-
testCase('never', `t.truthy('unicorn', 'message');`, foundError),
106-
testCase('never', `t.falsy('unicorn', 'message');`, foundError),
107-
testCase('never', `t.true(true, 'message');`, foundError),
108-
testCase('never', `t.false(false, 'message');`, foundError),
109-
testCase('never', `t.is('same', 'same', 'message');`, foundError),
110-
testCase('never', `t.not('not', 'same', 'message');`, foundError),
111-
testCase('never', `t.deepEqual({}, {}, 'message');`, foundError),
112-
testCase('never', `t.notDeepEqual({}, {a: true}, 'message');`, foundError),
113-
testCase('never', `t.throws(Promise.reject(), 'message');`, foundError),
114-
testCase('never', `t.notThrows(Promise.resolve(), 'message');`, foundError),
115-
testCase('never', `t.regex(a, /a/, 'message');`, foundError),
116-
testCase('never', `t.notRegex(a, /a/, 'message');`, foundError),
117-
testCase('never', `t.ifError(new Error(), 'message');`, foundError),
118-
testCase('never', `t.skip.is('same', 'same', 'message');`, foundError),
119-
testCase('never', `t.is.skip('same', 'same', 'message');`, foundError)
103+
testCase('never', 't.pass(\'message\');', foundError),
104+
testCase('never', 't.fail(\'message\');', foundError),
105+
testCase('never', 't.truthy(\'unicorn\', \'message\');', foundError),
106+
testCase('never', 't.falsy(\'unicorn\', \'message\');', foundError),
107+
testCase('never', 't.true(true, \'message\');', foundError),
108+
testCase('never', 't.false(false, \'message\');', foundError),
109+
testCase('never', 't.is(\'same\', \'same\', \'message\');', foundError),
110+
testCase('never', 't.not(\'not\', \'same\', \'message\');', foundError),
111+
testCase('never', 't.deepEqual({}, {}, \'message\');', foundError),
112+
testCase('never', 't.notDeepEqual({}, {a: true}, \'message\');', foundError),
113+
testCase('never', 't.throws(Promise.reject(), \'message\');', foundError),
114+
testCase('never', 't.notThrows(Promise.resolve(), \'message\');', foundError),
115+
testCase('never', 't.regex(a, /a/, \'message\');', foundError),
116+
testCase('never', 't.notRegex(a, /a/, \'message\');', foundError),
117+
testCase('never', 't.ifError(new Error(), \'message\');', foundError),
118+
testCase('never', 't.skip.is(\'same\', \'same\', \'message\');', foundError),
119+
testCase('never', 't.is.skip(\'same\', \'same\', \'message\');', foundError)
120120
]
121121
});

‎test/max-asserts.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ const ruleTester = avaRuleTester(test, {
99
});
1010

1111
const errors = [{ruleId: 'max-asserts'}];
12-
const header = `const test = require('ava');\n`;
12+
const header = 'const test = require(\'ava\');\n';
1313

1414
function nbAssertions(n) {
1515
return Array.from({length: n}).map(() => 't.is(1, 1);').join('\n');

‎test/no-async-fn-without-await.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import rule from '../rules/no-async-fn-without-await';
44

55
const ruleId = 'no-async-fn-without-await';
66
const message = 'Function was declared as `async` but doesn\'t use `await`';
7-
const header = `const test = require('ava');\n`;
7+
const header = 'const test = require(\'ava\');\n';
88

99
const ruleTesterOptions = [
1010
{
@@ -23,7 +23,7 @@ const ruleTesterOptions = [
2323
ruleTesterOptions.forEach(options => {
2424
const ruleTester = avaRuleTester(test, options);
2525

26-
ruleTester.run(`no-async-fn-without-await`, rule, {
26+
ruleTester.run('no-async-fn-without-await', rule, {
2727
valid: [
2828
`${header}test(fn);`,
2929
`${header}test(t => {});`,

‎test/no-cb-test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ const ruleTester = avaRuleTester(test, {
1010

1111
const ruleId = 'no-cb-test';
1212
const message = '`test.cb()` should not be used.';
13-
const header = `const test = require('ava');\n`;
13+
const header = 'const test = require(\'ava\');\n';
1414

1515
ruleTester.run('no-cb-test', rule, {
1616
valid: [

‎test/no-duplicate-modifiers.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ const ruleTester = avaRuleTester(test, {
99
});
1010

1111
const ruleId = 'no-duplicate-modifiers';
12-
const header = `const test = require('ava');\n`;
12+
const header = 'const test = require(\'ava\');\n';
1313

1414
const modifiers = [
1515
'after',
@@ -47,7 +47,7 @@ ruleTester.run('no-duplicate-modifiers', rule, {
4747
`${header}test.afterEach.always(t => {});`,
4848
`${header}test.failing.cb(t => {});`,
4949
// Shouldn't be triggered since it's not a test file
50-
`test.serial.serial(t => {});`
50+
'test.serial.serial(t => {});'
5151
]),
5252
invalid: invalid.concat([
5353
{

0 commit comments

Comments
 (0)