forked from eslint-community/eslint-plugin-promise
-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathno-callback-in-promise.js
89 lines (81 loc) · 2.4 KB
/
no-callback-in-promise.js
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
'use strict'
const rule = require('../rules/no-callback-in-promise')
const { RuleTester } = require('./rule-tester')
const ruleTester = new RuleTester({
parserOptions: {
ecmaVersion: 6,
},
})
const errorMessage = 'Avoid calling back inside of a promise.'
ruleTester.run('no-callback-in-promise', rule, {
valid: [
'function thing(cb) { cb() }',
'doSomething(function(err) { cb(err) })',
'function thing(callback) { callback() }',
'doSomething(function(err) { callback(err) })',
// TODO: support safe callbacks (see #220)
// 'whatever.then((err) => { process.nextTick(() => cb()) })',
// 'whatever.then((err) => { setImmediate(() => cb())) })',
// 'whatever.then((err) => setImmediate(() => cb()))',
// 'whatever.then((err) => process.nextTick(() => cb()))',
// 'whatever.then((err) => process.nextTick(cb))',
// 'whatever.then((err) => setImmediate(cb))',
// arrow functions and other things
'let thing = (cb) => cb()',
'doSomething(err => cb(err))',
// exceptions test
{
code: 'a.then(() => next())',
options: [{ exceptions: ['next'] }],
},
],
invalid: [
{
code: 'a.then(cb)',
errors: [{ message: errorMessage, column: 8 }],
},
{
code: 'a.then(() => cb())',
errors: [{ message: errorMessage }],
},
{
code: 'a.then(function(err) { cb(err) })',
errors: [{ message: errorMessage, column: 24 }],
},
{
code: 'a.then(function(data) { cb(data) }, function(err) { cb(err) })',
errors: [
{ column: 25, message: errorMessage },
{ column: 53, message: errorMessage },
],
},
{
code: 'a.catch(function(err) { cb(err) })',
errors: [{ message: errorMessage }],
},
// callback should also complain
{
code: 'a.then(callback)',
errors: [{ message: errorMessage, column: 8 }],
},
{
code: 'a.then(() => callback())',
errors: [{ message: errorMessage }],
},
{
code: 'a.then(function(err) { callback(err) })',
errors: [{ message: errorMessage, column: 24 }],
},
{
code: 'a.then(function(data) { callback(data) }, function(err) { callback(err) })',
errors: [
{ message: errorMessage },
{ column: 59, message: errorMessage },
],
},
{
code: 'a.catch(function(err) { callback(err) })',
errors: [{ message: errorMessage }],
},
],
})