Consider using arrow functions for callbacks JS-0241
Anti-pattern
Minor
a year ago
—
a year old
Unexpected function expression.
10 verificationKeys: 'LS0tLS1CRUdJTiBQR1AgUFVCTElDIEtFWSBCTE9DSy0tLS0tCgp4c0ZOQkdLRGpXSUJFQUNiWnRqMklVTEFGdndDVnZKUHdiNVp3REhHeGFFRzcwWUdYeXR1bXNuM1d3eGMKSWpJcmN6VkxwSWh3dU52Q05aZzJPbjF5elpNU1NtczM5M0t1ZTF3ak9ycXRYbXdNcUpCK2tzbHh2V1JHCjVhanJ5Z3pOUzBFZ2Nqd1huV2hoRWxFM1lOdklPNDUzdUQrZWlIMkZtMzRWdnZ5MVZyZGROczRqVmNTaApMdXBLSi9aclpBdHg4aWR6NFVmZGZwQjZmaUdNb0M1ZnR3RGJZQWphVStoVlFSM1lDbVA3WjNBRjR5WHEKMFIzRndQMFVXeXROR2x4UURPaFhLQ2Z5VXpFQWVxRjd5QkJmTEJvRVV1YXZsUVk3RUhabWJXSDZyNkRiCmhhcmJCYUU2L0wxNHR3L2VsSUNoZjl5UEFzM2FRMVFnSEd5Wnprd3dlVWdkREhCdzA1UkM0dU5mcXBidQpycy8xTnZycnk5cUxydGt6b1RMa1ZrM3V5bE80b1Z1L3NEa0RrREJEUWR6UzNYc1U1Nis4d2pTd3BpbmUKS1BCR2kycVM0elg3VlVTcDYvNnNWS3lwVkIvWk9mOENCM1oxa2RCaWhwVncrdksxR3lnVk9UZFYxekhlCjQxTUtHNHpEcDlBTU9KYWtrc3lQRTJWK0M5MFM3Z21rbXduZFRmdmo1R1ovZ3B6ZVlub041a2lTbnRsTQpkdGJGVE5UUFp2NjBpdnNTcExLdSsrZTN2b3VtM2JZNmtVaFF0ZGNwU1hTQXZvS0wvWlExSnowQ1JWMTYKQXFSOXNINktXRmtyS0RxZGZsbk1tRHNuV3FCaWxWeTZBcGVubkxFZFlNVjJLWHozN2g5QTM4OFN6ei9vCnNZdTdjUytMUTk0enhoSFc3QllpQjd4Z3hpdHc0TXdqYzBaa0J3QVJBUUFCelJkS1lXNWxJRVJ2WlNBOAphbUZ1WlVCa2IyVXVZMjl0UHNMQmlnUVFBUWdBSFFVQ1lvT05ZZ1FMQ1FjSUF4VUlDZ1FXQUFJQkFoa0IKQWhzREFoNEJBQ0VKRUJjQnhxZFo4cHJvRmlFRWlJTUlySjlmeDRQRjVaQldGd0hHcDFueW11aDRvZy8rCktDaW95Q0lVcjhzTWMyUTBDelExWEZkcDBvSmVNSDNmdDFRMCtCclFZVmljRDlFWXZ1NXAwWHFWOElzWgpjTDQyVjZsbmJzSDhsTm9KelM2cVNMM1ZTOGdEdUZuNEc4eDNqaHE3c2hsdDBrclFHR2hVQ2VoNk9VUjYKaVFrWm82L2VWL2trdDVZR0o5SFVhS09LTzJic2o2bkNyQ0IxY2ZFeElsM2JTUGlGeno2MnZKTnVabzN0Ckg0ZGlJN3ZhV0x0VWVNajZzUmVKcGlBSWNZK0FIR3RiWU5Wd3dNSlMxSlROdUkwWHlxM3h1dW1TbVhmWgpKdExlaVFmWjQvL25XTjQrV2pTc2JYYnRSc3JYSkdWeEs2UlVlaUJTaWI5Nk9jU2g4V1JvWUJVdUk4OXoKVCs3R1QxVDh5aTI5VVdQN0dJOUZxcVR2MGRxWnBQZklrQURZTFhqZ3hLV290eEk0WlNPNit1bk1zTFZDCmZ2RHpOTHUwcjhBYU1xNjIxcVRodGxyZ1N6aXVsb29LNFFGTFYrbjdUQ1pHUkJmdDd1WSs0bUFBRVM1cwpKR2pxRVR0WExvYU1PY3NQRk9CSGtVOHRHbUQxa1VjM0xlbXc4TVhpN01mMTExR0UrZHFCeEF5SUVBRWwKSUhVVTBMQUxYem5abG5FdjJiYlc5T2hRT1V6Rk0wcDh5cGFlMEtZeXc1bUlTT3JyTytaV01uQ1p3VVV4CnRKTVZydkc2bHIrN2ZPd0I3dElYQUpXeXNsV3RrcldHdnhUYVp3NkhMM2lGMkJ3aHVydW5qUTc2QzlZeQpWd253WnJoL2dMOHU2amhwTEo0VGNQQ0Q3dHFvaW42Ry9xcDVzNWFwejJBcmFvWGlmS1Jka2R2WmlHRVAKS3RQMmJmTXZlcFlUdnl0eHN5RG5Ba3pPd1UwRVlvT05ZZ0VRQUxRanpiOEVrallTU01JMVp0aEhTS0hKCkxTK3F5aGFYZGVMMkV3RmtuMUFDWGdwVmpldVpqMk9sd00zYTVPSlpMN284SlRRMGZXVjNjaXJDVVB6UQpFMWdKaTVMOVhINHB5YVd0V1IxZS9oY29yVi81ZERMQ2FneXo1Nk03SkJWclVGNTY1dVVjNkxmbjh6Y2gKM3g3U0p2WUo5NTdGeFIyNTdGNVhUNEZmbmVBNVkxcGZMaEFRMEhHSldpSS9SVm1Ccmcyc2JwaUhJai8rCjBMQ1Znckh1MEVXL1lnRlNqLzRYQlpDQVd6STJXang3YjhtVUZPWGdZaVVwWCtKdW5sdWNFOXhRQXdJYwpKeHoyRm5yaFBneVZNK0hCUGQwdk03QnFQQjg0Sm9nY0ZERDQ3RWNFTjlqYzdXYzRDNjRRZnBJRDFZYUgKQmRGWGRmeUJ1SXBHT3lBQzgrMnFQTVBwaFdpaXZDcm9tZG93QVdWblU4WTgxbnBRbXI2ZFE2MU92NjVUCjczM0dYVWdtTlBWNEV0THlxekdiVDFOMTdVY1MzbmxjL3ZNR2RkN1FwNzdlUEs4ZVMvVWQ4eDV6cFZiWQpteFd2VW5SUlAxRTVwNXZFRUkwNnNvdDBVTXIwckxZek53VjcyUi9VanhETEl4OG9SSVhzenQwV2tZc3MKR3RsdTlkR0JlZGV1REx2YmZha0dlT0E4NEVpMU0zTEVnU001eFhmeUE0Unh6U25aZXMrYlNpdGN0d3JSCmZ3SitPdmU5c2Nrb0ExcUlPWUZxTHRDM3BqVWFoTXhsWDRiVFZEdnpkbHhJZ2JiOVh3NHovV0Jyc1U1NAprK1g3T1JkOFRWMnBackVYeVRjMDlTSGRZeGRUcC9vOEZua2c4L05TbHdVbkUxWEJDZ0tSZUFIREJ1eFIKQUJFQkFBSEN3WFlFR0FFSUFBa0ZBbUtEaldJQ0d3d0FJUWtRRndIR3AxbnltdWdXSVFTSWd3aXNuMS9ICmc4WGxrRllYQWNhbldmS2E2TGZ0RUFDWTRud0tCTjMyS21Jc0hWT3YvRjFjaGRCT3BNN3p3cWJHeVppdwo4MmxkQ3J4VG5xNnNJR3NJWjd4TDJxbjViOUVacVlmays1anF0UDdOQlhzQ0d1a2xDYVhOcVZtTXA4MWMKTU5IcGViMXF4WVNzVFpPV0dWRHAyQ0FNbURFQW1MSWRoekc3djB2SXpjUEt0L2tNZUFhZjZPWVMxakc5CkRNQWNoWmszek40NjErdWRoWmpOYnIyWENLUVJ6NzRhT2l4N040S3l2Q0pFeENLemtCbFdRVWFSVGJiUgpDMEdiR0Z6cXlDOTczUDRCTXpoM0h3bVltLzhCa2VndFl3b3FJZk1nS2xNclpqRk5MbXVvcHBPTXFUZjIKazFzQTIwRVdNMlgwVGdNcDc0UGtOKzBibzMzUlB6RG1SWTZCajgwcDVUUFBlVXNzdjFjTWpiT3Z4Qk8zClV6VHo1d2hnS3Q2elNkUEdlaDV2bUFqK3NhMVhZUnAyZUE3bGFwRE15aTZwRU50Wmt4TCthdGRGY3loTwpMNmllSW1JejE5dzBuZXV1a2NsY3AwZWhTMjNxWnducTl5d3hGUDBoREoyejNOeW5SQ2UvdlY2Myt1Z24KdTZDQlVuN20wM3pIWVhWOEhMU3NYU1RoM1JjZ2FSOURZVVpEZ2c2YjhYTlBneTdEbDhUU3R6SmdyV1JnClJCU2FZd2V4TFVlMGsvZkREcHEyNE5IN0t0V1NDbDVEMHNwdStsN0R4eXQ5ZmdJWHNvUmFZTzVVdXpZTQpVdkZ5OGM4ZjZwTFR1MTBiRE1uTnE3SzEyKzBhQUMxYVd6K0pkM1RvQi81UXNUSTUvWXl4Rlp1NHZTUXEKK09HYUs0M2ZSYm1nQSsvMDRFQ1ZoVytIRkFUSWlnZXRMb1lnWmVibVNoSHByZz09Cj1zTWlhCi0tLS0tRU5EIFBHUCBQVUJMSUMgS0VZIEJMT0NLLS0tLS0=',
11}
12
13describe('verify', function () {14 it('should verify', async function () {15 const test = verify(data);16 await expect(test).to.eventually.be.fulfilled;17 });18});Unexpected function expression.
11}
12
13describe('verify', function () {
14 it('should verify', async function () {15 const test = verify(data);16 await expect(test).to.eventually.be.fulfilled;17 });18});
Unexpected function expression.
15};
16
17describe('sign', function () {
18 it('should sign a message', async function () {19 const test = sign(data);20 await expect(test).to.eventually.be.fulfilled;21 });22});
Unexpected function expression.
14 publicKey: 'LS0tLS1CRUdJTiBQR1AgUFVCTElDIEtFWSBCTE9DSy0tLS0tCgp4c0JOQkdLc2dFZ0JDQUM3ODBFR1hiamVzZ0E0YkFPMTNqYVN2REdqUUIrTHB1RGlWdGlDL0ZmejRIdjMKOWUxejV2dWNtckFwQjZJS3NtQytMazZuVmgxWUhaeXFTUk10N0NQU3Vham1SRVZNZTgzU3RsR0RuT1NnCkx1dkxJemduc0pTamlqcUJVQUNrbGZ5emhoNzJ0b3BIWXVjRUZFVDVDSG9FWEJaOGhYZkNpNjNITk9ILwpadWVYM3h5eHY3QUp1WFZqd0RoMlcyWUkyendTd01NK1Z3ZEhFNUs3TVo4eG0zSk9IVGpHRVYyS3dHa28KYTl6dHZ6TWVyNkpxWjQ5bmpqOGpoVHhidVlONFJBQkUrcWc1a1RJejFpbStkRXNncWhDelpyeGJOMEZtCjdFNWZ1YUorbGtGVmpYM3J6U3VWMzdTb0hNMThJZzJwRHdCN2JrelF1TjlWaXorUGNtZFFaM3JqQUJFQgpBQUhORjBwaGJtVWdSRzlsSUR4cVlXNWxRR1J2WlM1amIyMCt3c0NLQkJBQkNBQWRCUUppcklCSUJBc0oKQndnREZRZ0tCQllBQWdFQ0dRRUNHd01DSGdFQUlRa1EySlNGTTI0T1liOFdJUVJkaHNtRkhzSTdYeFdmCmZOYllsSVV6Ymc1aHY0ZWhDQUN2Vm0xcmdHWW9Pc1hES1NjTVVMOGFvRU1IRE5lUURGNVUzeWJreUJCMQpXM0RTUVhUaUtkTXBMM01qVTFMam05bDlCUFpWMDJnSElCWlVyNXNtUmpTeDBIZGJNNzZRcElSMGZQK3gKTXJDN0o5THZVcExMNVZ5bkFsWnArbjBKeWQxN216RUtUNkQ4Uk45NkVrYlFQMEhmb3F5b2V3QVpaZVdHCmJXcXZsalFteWZGdUgrb1ZUOHlLOThJZkV2NWppK2dQbFVTVDBOb2RML2hFRnY2UFdRWGo2VkZrNmV0YQozekxzV3U5aWZwREhkdHBXMFBXUEwwSmhTQ3E0NDN1ZllJVmg3em1QeW42MW16UnpyQTZsdS9VN1JKMWkKZE1NSUpsZUxic0dScEVNYWRXdkM2a3JjeG5uUmsvMEduQXFVS3N5d3BrZnJIYXljNitKRW1PcmswOWdhCnpzQk5CR0tzZ0VnQkNBQ3FwWDduemQ3cjVxdmZjMzRjL1h6dmVudGZEN215aGl4YTJqa1hzVlpOSWZuegpRRm1YR1lCVGdsazhZclFJblRhSzcyaTdleXVGSlBIVzBjV3dwTTFmRWpPYmYwb0swZG1jSGZtWm5aUWEKYWpibHg1Z0pUb1BVRURmYzBUUEdoUWNma1Q2eW9vYTUrRjdlYnNMVGk3SWJDeFFPNTFjdGRBRVArNjlDCkpyOFovdWY1am94MmgzOTZvU3pFd1FjOFF0UkdGVkR1ZHpJUEw4VCs2SVpvV1NLbWVUTTcvcXVmNHNydwpjY0FWb1V1YllQNHVuQ3IzQTVZYnB3cW83L25lWkd2UVFZcDhneXdqclcrclovNVlCc1BCUnZwQVduTGoKNy9pMlJRRFlSWEJJWG9kRmc2eXBXQU8vWVd1UTczcWd2UzZSTDBiT0VuMGpiU2RYbGJReWZsakpBQkVCCkFBSEN3SFlFR0FFSUFBa0ZBbUtzZ0VnQ0d3d0FJUWtRMkpTRk0yNE9ZYjhXSVFSZGhzbUZIc0k3WHhXZgpmTmJZbElVemJnNWh2NXliQi85aFRFNytHMktoRnh2ZUpMRG1RbUt6Z1BxUDdJRCtUbThHSi82WTZXM0YKeEsxT3M0TFBrNmFEUklSUXNhNU5FZjUrNGZkZVpXSE9mcWRwazFacW9VMU8wTG1rMThKNG1OTkUwOW02CkxWb3kwS09Qb2c5OTZQRThQVWZHM2RaYTRnMXJVNlZnT1dlZGc2UElxT3JDeFBiQmtGU2dzYmhmdzJ1MQpVNWp5UkRUeXQzVGw3YjZOQVR4OEFQSW5EZDNGVk45WDRDMkFFMGJmQWoyYU82L29JalJYOHFLeG0vMisKMDdFNDJoQWorNFBSZHJLZjJYS2tXUFZ3bk5iREdxMDJ1a0M5eWV0aHRYR3N4QXM3VjFOWmdKZWZLUXZKCnNsbXFJMWcyanp1dkFwVU1YQ1JuOWMwd3BQcEZ6a1k2SkxKZWtSNmVDZnQxK2xqeWxtMjZ1UTltTjBGTAo9UnQ3MAotLS0tLUVORCBQR1AgUFVCTElDIEtFWSBCTE9DSy0tLS0tCg==',
15};
16
17describe('sign', function () {18 it('should sign a message', async function () {19 const test = sign(data);20 await expect(test).to.eventually.be.fulfilled;21 });22});Unexpected function expression.
13};
14
15describe('session', function () {
16 it('should session a message', async function () {17 const test = session(data);18 await expect(test).to.eventually.be.fulfilled;19 });20});
Description
It is recommended to use arrow functions as callbacks.
Arrow functions can be an attractive alternative to function expressions for callbacks or function arguments.
For example, arrow functions are automatically bound to their surrounding scope/context. This provides an alternative to the pre-ES6 standard of explicitly binding function expressions to achieve similar behavior.
Additionally, arrow functions are:
less verbose, and easier to reason about.
bound lexically regardless of where or when they are invoked.
Bad Practice
func(function(a) { return a; }); // ERROR
// prefer: func(a => a)
func(function() { return this.a; }.bind(this)); // ERROR
// prefer: func(() => this.a)
Recommended
// arrow function callback
func(a => a); // OK
// generator as callback
func(function*() { yield; }); // OK
// function expression not used as callback or function argument
var funcVar = function func(a) { return a; }; // OK
// unbound function expression callback
func(function() { return this.a; }); // OK
// recursive named function callback
func(function recursiveFunc(n) { return n && n + recursiveFunc(n - 1); }); // OK