thjs-124: * fix typescript build

Author: v0ldemar01

apps/backend/src/libs/modules/logger/logger.module.ts

@@ -1,13 +1,12 @@
 import { type Logger as LibraryLogger, pino } from 'pino';
-import pretty from 'pino-pretty';
 
 import { type LoggerModule } from './libs/types/types.js';
 
 class Logger implements LoggerModule {
   private logger: LibraryLogger;
 
   public constructor() {
-    this.logger = pino(pretty.default());
+    this.logger = pino({ transport: { target: 'pino-pretty' } });
 
     this.logger.info('Logger is created…');
   }

apps/backend/src/modules/user/libs/types/user-repository.type.ts

@@ -4,8 +4,6 @@ import { type User } from './types.js';
 
 type UserRepository = {
   getByEmail(_email: string): Promise<User | null>;
-
-  getByUsername(_username: string): Promise<User | null>;
 } & Pick<Repository<User>, 'create'>;
 
 export { type UserRepository };

apps/backend/src/modules/user/user.repository.ts

@@ -21,12 +21,6 @@ class User
 
     return user ?? null;
   }
-
-  public async getByUsername(username: string): Promise<TUser | null> {
-    const user = await this.model.query().select().findOne({ username });
-
-    return user ?? null;
-  }
 }
 
 export { User };

apps/backend/tests/modules/auth/auth.api.spec.ts

@@ -42,53 +42,6 @@ describe(`${authApiPath} routes`, () => {
   describe(`${registerEndpoint} (${HTTPMethod.POST}) endpoint`, () => {
     const app = getApp();
 
-    it(`should return ${HTTPCode.UNPROCESSED_ENTITY} of empty ${UserPayloadKey.USERNAME} validation error`, async () => {
-      const response = await app.inject().post(registerEndpoint).body({});
-
-      expect(response.statusCode).toBe(HTTPCode.UNPROCESSED_ENTITY);
-      expect(response.json<Record<'message', string>>().message).toBe(
-        `${UserValidationMessage.EMAIL_REQUIRE}. ${UserValidationMessage.PASSWORD_REQUIRE}. ${UserValidationMessage.USERNAME_REQUIRE}`
-      );
-    });
-
-    it(`should return ${HTTPCode.UNPROCESSED_ENTITY} of too short ${UserPayloadKey.USERNAME} validation error`, async () => {
-      const [validTestUser] = TEST_USERS_CREDENTIALS;
-
-      const response = await app
-        .inject()
-        .post(registerEndpoint)
-        .body({
-          ...validTestUser,
-          [UserPayloadKey.USERNAME]: faker.string.alpha(
-            UserValidationRule.USERNAME_MIN_LENGTH - VALIDATION_RULE_DELTA
-          )
-        });
-
-      expect(response.statusCode).toBe(HTTPCode.UNPROCESSED_ENTITY);
-      expect(response.json<Record<'message', string>>().message).toBe(
-        UserValidationMessage.USERNAME_MIN_LENGTH
-      );
-    });
-
-    it(`should return ${HTTPCode.UNPROCESSED_ENTITY} of too long ${UserPayloadKey.USERNAME} validation error`, async () => {
-      const [validTestUser] = TEST_USERS_CREDENTIALS;
-
-      const response = await app
-        .inject()
-        .post(registerEndpoint)
-        .body({
-          ...validTestUser,
-          [UserPayloadKey.USERNAME]: faker.string.alpha(
-            UserValidationRule.USERNAME_MAX_LENGTH + VALIDATION_RULE_DELTA
-          )
-        });
-
-      expect(response.statusCode).toBe(HTTPCode.UNPROCESSED_ENTITY);
-      expect(response.json<Record<'message', string>>().message).toBe(
-        UserValidationMessage.USERNAME_MAX_LENGTH
-      );
-    });
-
     it(`should return ${HTTPCode.UNPROCESSED_ENTITY} of empty ${UserPayloadKey.EMAIL} validation error`, async () => {
       const [validTestUser] = TEST_USERS_CREDENTIALS;
       const { [UserPayloadKey.EMAIL]: _email, ...user } =
@@ -183,8 +136,7 @@ describe(`${authApiPath} routes`, () => {
       expect(response.statusCode).toBe(HTTPCode.CREATED);
       expect(response.json()).toEqual(
         expect.objectContaining({
-          [UserPayloadKey.EMAIL]: validTestUser[UserPayloadKey.EMAIL],
-          [UserPayloadKey.USERNAME]: validTestUser[UserPayloadKey.USERNAME]
+          [UserPayloadKey.EMAIL]: validTestUser[UserPayloadKey.EMAIL]
         })
       );
 
@@ -196,8 +148,7 @@ describe(`${authApiPath} routes`, () => {
 
       expect(savedDatabaseUser).toEqual(
         expect.objectContaining({
-          [UserPayloadKey.EMAIL]: validTestUser[UserPayloadKey.EMAIL],
-          [UserPayloadKey.USERNAME]: validTestUser[UserPayloadKey.USERNAME]
+          [UserPayloadKey.EMAIL]: validTestUser[UserPayloadKey.EMAIL]
         })
       );
     });

apps/backend/tests/modules/user/libs/constants/test-user-credentials.constant.ts

@@ -10,8 +10,7 @@ const TEST_USERS_CREDENTIALS = Array.from(
   (): UserSignUpRequestDto => {
     return {
       [UserPayloadKey.EMAIL]: faker.internet.email(),
-      [UserPayloadKey.PASSWORD]: faker.internet.password(),
-      [UserPayloadKey.USERNAME]: faker.person.firstName()
+      [UserPayloadKey.PASSWORD]: faker.internet.password()
     };
   }
 );

apps/frontend/src/pages/auth/components/sign-up-form/libs/common/constants.ts

@@ -2,8 +2,7 @@ import { UserPayloadKey } from '~/modules/user/enums/enums.js';
 
 const DEFAULT_REGISTRATION_PAYLOAD = {
   [UserPayloadKey.EMAIL]: '',
-  [UserPayloadKey.PASSWORD]: '',
-  [UserPayloadKey.USERNAME]: ''
+  [UserPayloadKey.PASSWORD]: ''
 };
 
 export { DEFAULT_REGISTRATION_PAYLOAD };

apps/frontend/src/pages/auth/components/sign-up-form/sign-up-form.tsx

@@ -34,13 +34,6 @@ const SignUpForm: React.FC<Properties> = ({ onSubmit }) => {
       <h2 className={styles['title']}>Register for free account</h2>
       <form name="registrationForm" onSubmit={handleSubmit(handleFormSubmit)}>
         <fieldset className={styles['fieldset']} disabled={isLoading}>
-          <Input
-            control={control}
-            errors={errors}
-            name={UserPayloadKey.USERNAME}
-            placeholder="Username"
-            type="text"
-          />
           <Input
             control={control}
             errors={errors}

knip.config.ts

@@ -10,7 +10,7 @@ const config: KnipConfig = {
     'apps/backend': {
       entry: ['src/index.ts', 'src/db/**/*.ts', 'knexfile.ts'],
       ignoreBinaries: ['ts-paths-esm-loader'],
-      ignoreDependencies: ['ts-paths-esm-loader']
+      ignoreDependencies: ['ts-paths-esm-loader', 'pino-pretty']
     },
     'apps/frontend': {
       entry: ['src/index.tsx']

packages/shared/src/modules/auth/libs/validation-schemas/sign-up.validation-schema.ts

@@ -26,17 +26,6 @@ const signUp = Joi.object({
       'string.empty': UserValidationMessage.PASSWORD_REQUIRE,
       'string.max': UserValidationMessage.PASSWORD_MAX_LENGTH,
       'string.min': UserValidationMessage.PASSWORD_MIN_LENGTH
-    }),
-  [UserPayloadKey.USERNAME]: Joi.string()
-    .trim()
-    .min(UserValidationRule.USERNAME_MIN_LENGTH)
-    .max(UserValidationRule.USERNAME_MAX_LENGTH)
-    .required()
-    .messages({
-      'any.required': UserValidationMessage.USERNAME_REQUIRE,
-      'string.empty': UserValidationMessage.USERNAME_REQUIRE,
-      'string.max': UserValidationMessage.USERNAME_MAX_LENGTH,
-      'string.min': UserValidationMessage.USERNAME_MIN_LENGTH
     })
 });
 

packages/shared/src/modules/user/libs/enums/user-payload-key.enum.ts

@@ -2,8 +2,7 @@ const UserPayloadKey = {
   EMAIL: 'email',
   PASSWORD: 'password',
   STATUS: 'status',
-  TOKEN: 'token',
-  USERNAME: 'username'
+  TOKEN: 'token'
 } as const;
 
 export { UserPayloadKey };