fix(comment): fix comments

Author: hewenguang

src/namespaces/namespaces.e2e-spec.ts

@@ -411,7 +411,10 @@ describe('NamespacesController (e2e)', () => {
         .send(specialNameWorkspace)
         .expect(HttpStatus.CREATED);
 
-      expect(response.body.name).toBe(specialNameWorkspace.name);
+      // Emojis should be filtered out
+      expect(response.body.name).toBe(
+        'Test Workspace with 特殊字符 and émojis',
+      );
 
       // Clean up
       await client

src/user/user.controller.ts

@@ -39,8 +39,8 @@ export class UserController {
   }
 
   @Post('email/validate')
-  async validateEmail(@Req() req, @Body('email') email: string) {
-    return await this.userService.validateEmail(req.user.id, email);
+  async validateEmail(@UserId() userId: string, @Body('email') email: string) {
+    return await this.userService.validateEmail(userId, email);
   }
 
   @Patch(':id')
@@ -54,29 +54,32 @@ export class UserController {
   }
 
   @Post('option')
-  async createOption(@Req() req, @Body() createOptionDto: CreateUserOptionDto) {
+  async createOption(
+    @UserId() userId: string,
+    @Body() createOptionDto: CreateUserOptionDto,
+  ) {
     const option = await this.userService.getOption(
-      req.user.id,
+      userId,
       createOptionDto.name,
     );
     if (option && option.name) {
       return await this.userService.updateOption(
-        req.user.id,
+        userId,
         option.name,
         createOptionDto.value,
       );
     }
-    return await this.userService.createOption(req.user.id, createOptionDto);
+    return await this.userService.createOption(userId, createOptionDto);
   }
 
   @Get('option/list')
-  async listOption(@Req() req) {
-    return await this.userService.listOption(req.user.id);
+  async listOption(@UserId() userId: string) {
+    return await this.userService.listOption(userId);
   }
 
   @Get('option/:name')
-  async getOption(@Req() req, @Param('name') name: string) {
-    return await this.userService.getOption(req.user.id, name);
+  async getOption(@UserId() userId: string, @Param('name') name: string) {
+    return await this.userService.getOption(userId, name);
   }
 
   @Get('binding/list')

src/user/user.e2e-spec.ts

@@ -125,9 +125,10 @@ describe('UserController (e2e)', () => {
     });
 
     it('should fail with existing email', async () => {
+      // Use secondClient's email to test email already in use by another user
       await client
         .post('/api/v1/user/email/validate')
-        .send({ email: client.user.email })
+        .send({ email: secondClient.user.email })
         .expect(HttpStatus.BAD_REQUEST);
     });
 

src/utils/emoji.ts

@@ -26,5 +26,6 @@ export function isEmoji(char: string) {
 export function filterEmoji(data: string) {
   return Array.from(data)
     .filter((char) => !isEmoji(char))
-    .join('');
+    .join('')
+    .trim();
 }