NodeJS控制器方法Jest测试规范咨询：直接调用还是走路由？

我的背景与代码 #

我是JavaScript单元测试新手，正在使用NodeJS和Mongoose开发API，经调研后选用Jest和Supertest进行单元测试。目前需测试一个从数据库获取所有记录的控制器方法，代码如下：

const Gender = require("../models/gender");
exports.getMany = async (req, res) => {
  try {
    const genders = await Gender.find();
    return res.status(200).json({ data: genders });
  } catch (error) {
    console.error(error);
    res.status(400).end();
  }
};

我已编写两个可正常运行的测试用例，但不确定是否规范，测试代码如下：

it("Get an empty array if there are no genders in the database", async (done) => {
  const response = await request.get("/api/v1/genders");
  expect(response.status).toBe(200);
  expect(response.body.data.length).toBe(0);
  done();
});
it("Get an array with existing genders in the database", async (done) => {
  const male = await request.post("/api/v1/genders").send({ name: "Male" });
  const female = await request.post("/api/v1/genders").send({ name: "Female" });
  const response = await request.get("/api/v1/genders");
  expect(response.status).toBe(200);
  expect(response.body.data.length).toBe(2);
  expect(male.body.data.name).toBe("Male");
  expect(female.body.data.name).toBe("Female");
  done();
});

请问我的测试方式是否正确？直接调用getMany()方法是否比访问路由更合适？

你的测试方式分析与建议 #

1. 当前测试的合理性 #

首先，你的测试用例是合理且有效的，属于集成测试范畴——它模拟了真实用户的请求流程，从路由层到控制器再到数据库，能验证整个链路是否正常工作，这对于确保API的实际可用性非常有价值。

不过有几个小细节可以优化：

• 因为你用了async/await，其实不需要再用done()回调，Jest会自动等待异步操作完成，去掉done()可以避免潜在的回调地狱或超时问题。
• 测试之间应该保持独立，第二个测试用例里创建的Male和Female数据可能会影响其他测试（比如第一个测试如果在第二个之后运行，就会得到非空数组）。建议在每个测试前/后清理数据库，比如用beforeEach或afterEach调用Gender.deleteMany({})。

优化后的测试示例：

beforeEach(async () => {
  await Gender.deleteMany({}); // 每次测试前清空数据，保证测试独立性
});

it("Get an empty array if there are no genders in the database", async () => {
  const response = await request.get("/api/v1/genders");
  expect(response.status).toBe(200);
  expect(response.body.data.length).toBe(0);
});

it("Get an array with existing genders in the database", async () => {
  // 直接用Mongoose模型创建数据，不用调用POST接口，让测试更聚焦GET逻辑
  await Gender.create([{ name: "Male" }, { name: "Female" }]);
  
  const response = await request.get("/api/v1/genders");
  expect(response.status).toBe(200);
  expect(response.body.data.length).toBe(2);
  expect(response.body.data.map(g => g.name)).toEqual(expect.arrayContaining(["Male", "Female"]));
});

2. 直接调用控制器vs访问路由：该选哪种？ #

这两种方式对应不同的测试层级，没有绝对的“更合适”，要看你的测试目标：

• 访问路由（集成测试）：适合验证整个API链路（路由匹配、中间件执行、控制器逻辑、数据库交互）是否正常，能发现比如路由配置错误、中间件拦截等问题，更贴近真实用户的使用场景。
• 直接调用控制器（单元测试）：适合单独测试控制器的业务逻辑，此时需要模拟req和res对象，隔离数据库依赖（比如用jest.mock模拟Gender.find()方法）。这种测试速度更快，能精准定位控制器内部的逻辑问题（比如错误分支的处理）。

举个单元测试的示例：

const { getMany } = require("../controllers/genderController");
const Gender = require("../models/gender");

// 模拟Mongoose模型，隔离数据库依赖
jest.mock("../models/gender");

describe("getMany controller", () => {
  it("should return 200 with empty array when no genders exist", async () => {
    // 模拟find方法返回空数组
    Gender.find.mockResolvedValue([]);
    
    // 模拟req和res对象
    const req = {};
    const res = {
      status: jest.fn().mockReturnThis(),
      json: jest.fn(),
      end: jest.fn()
    };
    
    await getMany(req, res);
    
    expect(res.status).toHaveBeenCalledWith(200);
    expect(res.json).toHaveBeenCalledWith({ data: [] });
  });
  
  it("should return 400 when database query fails", async () => {
    const mockError = new Error("Database connection failed");
    Gender.find.mockRejectedValue(mockError);
    
    const req = {};
    const res = {
      status: jest.fn().mockReturnThis(),
      json: jest.fn(),
      end: jest.fn()
    };
    
    await getMany(req, res);
    
    expect(res.status).toHaveBeenCalledWith(400);
    expect(res.end).toHaveBeenCalled();
  });
});

总结 #

• 你的现有测试是有效的集成测试，调整细节后会更规范可靠。
• 建议同时覆盖集成测试和单元测试：集成测试保障API整体链路可用，单元测试聚焦控制器的逻辑边界（比如错误处理分支），两者结合能让你的测试体系更完整。

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