Fix unit test

This commit is contained in:
Carlos Monastyrski
2025-12-04 10:53:39 -03:00
parent 022c2a860e
commit 3f73f7cb05
3 changed files with 59 additions and 32 deletions
@@ -203,7 +203,7 @@ export function convertConditionToDbFilter(condition: Record<string, unknown>):
switch (operator) { switch (operator) {
case "$glob": { case "$glob": {
// Convert "*.es" glob → regex /^.*\.es$/i // Convert "*.es" glob → regex /^.*\.es$/i
const regex = new RegExp(`^${String(opValue).replace(/\*/g, ".*")}$`, "i"); const regex = new RE2(`^${String(opValue).replace(/\*/g, ".*")}$`, "i");
dbFilter[key] = { $regex: regex }; dbFilter[key] = { $regex: regex };
break; break;
} }
@@ -169,7 +169,8 @@ describe("CertificateProfileService", () => {
const mockPermissionService = { const mockPermissionService = {
getProjectPermission: vi.fn().mockResolvedValue({ getProjectPermission: vi.fn().mockResolvedValue({
permission: { permission: {
throwUnlessCan: vi.fn() throwUnlessCan: vi.fn(),
rules: []
} }
}) })
} as unknown as Pick<TPermissionServiceFactory, "getProjectPermission">; } as unknown as Pick<TPermissionServiceFactory, "getProjectPermission">;
@@ -598,13 +599,18 @@ describe("CertificateProfileService", () => {
expect(result.profiles).toEqual(mockProfiles); expect(result.profiles).toEqual(mockProfiles);
expect(result.totalCount).toBe(1); expect(result.totalCount).toBe(1);
expect(mockCertificateProfileDAL.findByProjectId).toHaveBeenCalledWith("project-123", { expect(mockCertificateProfileDAL.findByProjectId).toHaveBeenCalledWith(
offset: 0, "project-123",
limit: 20, {
search: undefined, offset: 0,
enrollmentType: undefined, limit: 20,
caId: undefined search: undefined,
}); enrollmentType: undefined,
caId: undefined,
issuerType: undefined
},
{}
);
}); });
it("should list profiles with filters", async () => { it("should list profiles with filters", async () => {
@@ -618,13 +624,18 @@ describe("CertificateProfileService", () => {
caId: "ca-123" caId: "ca-123"
}); });
expect(mockCertificateProfileDAL.findByProjectId).toHaveBeenCalledWith("project-123", { expect(mockCertificateProfileDAL.findByProjectId).toHaveBeenCalledWith(
offset: 10, "project-123",
limit: 5, {
search: "test", offset: 10,
enrollmentType: EnrollmentType.API, limit: 5,
caId: "ca-123" search: "test",
}); enrollmentType: EnrollmentType.API,
caId: "ca-123",
issuerType: undefined
},
{}
);
}); });
}); });
@@ -267,14 +267,22 @@ describe("CertificateTemplateV2Service", () => {
limit: 20 limit: 20
}); });
expect(mockCertificateTemplateV2DAL.findByProjectId).toHaveBeenCalledWith("project-123", { expect(mockCertificateTemplateV2DAL.findByProjectId).toHaveBeenCalledWith(
offset: 0, "project-123",
limit: 20, {
search: undefined offset: 0,
}); limit: 20,
expect(mockCertificateTemplateV2DAL.countByProjectId).toHaveBeenCalledWith("project-123", { search: undefined
search: undefined },
}); {}
);
expect(mockCertificateTemplateV2DAL.countByProjectId).toHaveBeenCalledWith(
"project-123",
{
search: undefined
},
{}
);
expect(result).toEqual({ templates, totalCount }); expect(result).toEqual({ templates, totalCount });
}); });
@@ -291,14 +299,22 @@ describe("CertificateTemplateV2Service", () => {
search: "web server" search: "web server"
}); });
expect(mockCertificateTemplateV2DAL.findByProjectId).toHaveBeenCalledWith("project-123", { expect(mockCertificateTemplateV2DAL.findByProjectId).toHaveBeenCalledWith(
offset: 0, "project-123",
limit: 20, {
search: "web server" offset: 0,
}); limit: 20,
expect(mockCertificateTemplateV2DAL.countByProjectId).toHaveBeenCalledWith("project-123", { search: "web server"
search: "web server" },
}); {}
);
expect(mockCertificateTemplateV2DAL.countByProjectId).toHaveBeenCalledWith(
"project-123",
{
search: "web server"
},
{}
);
}); });
}); });