Mocha after钩子报deleted未定义ReferenceError但功能正常

这个Uncaught ReferenceError: deleted is not defined报错和你的after钩子删除测试账号的逻辑没有直接关联，属于测试收尾阶段的异步残余任务触发的业务代码错误：

• 测试运行时，Supertest发起的带登录态请求会触发passport、express-session的异步回调逻辑，这类逻辑会在接口响应返回后，继续留在Node.js事件循环里等待执行
• 你的after钩子删除数据库测试账号的逻辑先执行完成，所以实际数据清理效果符合预期
• 排在事件循环后面的残余异步任务后续执行到src/routes/profile.js第30行时，代码直接引用了未声明的deleted变量抛出错误，此时Mocha正处于after钩子的执行周期内，就会把错误标记为钩子抛出的未捕获异常，其余测试用例因为在错误抛出前已经执行完成，所以全部正常通过。

1. 修复业务代码的语法问题 #

直接定位到src/routes/profile.js第30行，修正未声明变量的问题：

• 如果是要判断用户的删除状态，应该引用对应上下文对象上的属性，比如req.user.deleted，不要直接使用裸的deleted变量
• 如果是业务逻辑里的自定义标记变量，检查是否漏了const/let声明，或者变量作用域覆盖问题。
  这是触发报错的直接原因。

2. 处理测试流程的异步残余问题 #

要从根源避免测试收尾阶段飘着未处理的异步任务，需要补全测试的收尾逻辑：

• 所有测试用例里的Supertest请求必须等待完整响应返回后再结束用例，不要发起请求后不等待返回就直接进入下一个执行阶段，所有请求都要加await等待执行完成
• 在after钩子里补充资源释放逻辑，等所有资源断开连接后再结束钩子执行，参考写法：

let server
// 测试启动时先保存服务实例
before(function(done) {
  server = app.listen(3000, done)
})

after(async function() {
  // 原有删除测试账号的逻辑
  await User.deleteMany({ testTag: 'mocha-test' })
  // 关闭HTTP服务，阻止新请求进入
  await new Promise(resolve => server.close(resolve))
  // 关闭数据库连接
  await mongoose.disconnect()
  // 如果用了独立的session存储，也在这里主动清空连接/存储
})

你观察到的「钩子报错但删除操作实际成功」是Node.js事件循环的执行顺序导致的：删除测试账号的异步任务排在事件循环前列先执行完成，触发报错的残余路由逻辑排在任务队列后面执行，错误抛出时删除逻辑早就跑完了，所以不会影响实际的数据清理效果，只是Mocha会把执行周期内抛出的未捕获错误都算到当前正在跑的钩子头上。

内容的提问来源于stack exchange，提问作者h5law