Mathis/memegoat
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Activity

chore: update lint scripts and improve formatting consistency

Browse Source 
- Added `lint:fix` scripts for backend, frontend, and documentation.
- Enabled `biome check --write` for unsafe fixes in backend scripts.
- Fixed imports, formatting, and logging for improved code clarity.
- Adjusted service unit tests for better readability and maintainability.
This commit is contained in:
Mathis HERRIOT
2026-01-21 11:38:25 +01:00
parent a90aba2748
commit 951b38db67
10 changed files with 62 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
backend/src/auth/bootstrap.service.spec.ts
View File

@@ -1,14 +1,14 @@
import { Test, TestingModule } from "@nestjs/testing";
import { ConfigService } from "@nestjs/config";
import { UnauthorizedException } from "@nestjs/common";
import { ConfigService } from "@nestjs/config";
import { Test, TestingModule } from "@nestjs/testing";
import { UsersService } from "../users/users.service";
import { BootstrapService } from "./bootstrap.service";
import { RbacService } from "./rbac.service";
import { UsersService } from "../users/users.service";
describe("BootstrapService", () => {
let service: BootstrapService;
let rbacService: RbacService;
let usersService: UsersService;
let _usersService: UsersService;
const mockRbacService = {
countAdmins: jest.fn(),
@@ -36,7 +36,7 @@ describe("BootstrapService", () => {
service = module.get<BootstrapService>(BootstrapService);
rbacService = module.get<RbacService>(RbacService);
usersService = module.get<UsersService>(UsersService);
_usersService = module.get<UsersService>(UsersService);
});
it("should be defined", () => {
@@ -46,7 +46,10 @@ describe("BootstrapService", () => {
describe("onApplicationBootstrap", () => {
it("should generate a token if no admin exists", async () => {
mockRbacService.countAdmins.mockResolvedValue(0);
const generateTokenSpy = jest.spyOn(service as any, "generateBootstrapToken");
const generateTokenSpy = jest.spyOn(
service as any,
"generateBootstrapToken",
);
await service.onApplicationBootstrap();
@@ -56,7 +59,10 @@ describe("BootstrapService", () => {
it("should not generate a token if admin exists", async () => {
mockRbacService.countAdmins.mockResolvedValue(1);
const generateTokenSpy = jest.spyOn(service as any, "generateBootstrapToken");
const generateTokenSpy = jest.spyOn(
service as any,
"generateBootstrapToken",
);
await service.onApplicationBootstrap();
@@ -70,9 +76,9 @@ describe("BootstrapService", () => {
mockRbacService.countAdmins.mockResolvedValue(0);
await service.onApplicationBootstrap();
await expect(
service.consumeToken("wrong-token", "user1"),
).rejects.toThrow(UnauthorizedException);
await expect(service.consumeToken("wrong-token", "user1")).rejects.toThrow(
UnauthorizedException,
);
});
it("should throw UnauthorizedException if user not found", async () => {
@@ -97,7 +103,10 @@ describe("BootstrapService", () => {
const result = await service.consumeToken(token, "user1");
expect(rbacService.assignRoleToUser).toHaveBeenCalledWith("user-uuid", "admin");
expect(rbacService.assignRoleToUser).toHaveBeenCalledWith(
"user-uuid",
"admin",
);
expect((service as any).bootstrapToken).toBeNull();
expect(result.message).toContain("user1 is now an administrator");
});