Merge pull request #335 from sammajayi/feat/quiz-retake-limit

feat/quiz-retake-limit

Author: portableDD

backend/eslint.config.mjs

@@ -26,18 +26,16 @@ export default tseslint.config(
   },
   {
     rules: {
+      'no-unused-vars': 'off',
       '@typescript-eslint/no-explicit-any': 'off',
       '@typescript-eslint/no-floating-promises': 'warn',
+      '@typescript-eslint/no-unused-vars': 'warn',
       '@typescript-eslint/no-unsafe-argument': 'warn',
       '@typescript-eslint/no-unsafe-assignment': 'off',
-      '@typescript-eslint/no-unsafe-member-access': 'off',
       '@typescript-eslint/no-unsafe-call': 'off',
+      '@typescript-eslint/no-unsafe-member-access': 'off',
       '@typescript-eslint/no-unsafe-return': 'off',
-    },
-  },
-  {
-    files: ['**/*.spec.ts', '**/*.e2e-spec.ts'],
-    rules: {
+      '@typescript-eslint/require-await': 'off',
       '@typescript-eslint/unbound-method': 'off',
     },
   },

backend/src/admin/admin-courses.controller.ts

@@ -118,16 +118,27 @@ export class AdminCoursesController {
 
   @Patch(':id/publish')
   @ApiOperation({ summary: 'Publish a course (admin)' })
-  @ApiResponse({ status: 200, description: 'Course published successfully', type: CourseResponseDto })
-  @ApiResponse({ status: 400, description: 'Cannot publish a course with no lessons' })
+  @ApiResponse({
+    status: 200,
+    description: 'Course published successfully',
+    type: CourseResponseDto,
+  })
+  @ApiResponse({
+    status: 400,
+    description: 'Cannot publish a course with no lessons',
+  })
   @ApiResponse({ status: 404, description: 'Course not found' })
   async publish(@Param('id') id: string): Promise<CourseResponseDto> {
     return this.coursesService.publishCourse(id);
   }
 
   @Patch(':id/unpublish')
   @ApiOperation({ summary: 'Unpublish a course (admin)' })
-  @ApiResponse({ status: 200, description: 'Course unpublished successfully', type: CourseResponseDto })
+  @ApiResponse({
+    status: 200,
+    description: 'Course unpublished successfully',
+    type: CourseResponseDto,
+  })
   @ApiResponse({ status: 404, description: 'Course not found' })
   async unpublish(@Param('id') id: string): Promise<CourseResponseDto> {
     return this.coursesService.unpublishCourse(id);

backend/src/admin/admin-dao.controller.ts

@@ -19,7 +19,10 @@ import {
   ApiQuery,
 } from '@nestjs/swagger';
 import { DAOService } from '../dao/dao.service';
-import { DAOProposal, ProposalStatus } from '../dao/entities/dao-proposal.entity';
+import {
+  DAOProposal,
+  ProposalStatus,
+} from '../dao/entities/dao-proposal.entity';
 import { JwtAuthGuard } from '../common/guards/jwt-auth.guard';
 import { RolesGuard } from '../common/guards/roles.guard';
 import { Roles } from '../common/decorators/roles.decorator';
@@ -52,14 +55,16 @@ export class AdminDAOController {
   ) {}
 
   @Get('proposals')
-  @ApiOperation({ summary: 'Get all proposals with moderation details (admin)' })
+  @ApiOperation({
+    summary: 'Get all proposals with moderation details (admin)',
+  })
   @ApiQuery({ name: 'page', required: false, type: Number })
   @ApiQuery({ name: 'limit', required: false, type: Number })
-  @ApiQuery({ 
-    name: 'status', 
-    required: false, 
+  @ApiQuery({
+    name: 'status',
+    required: false,
     enum: ProposalStatus,
-    description: 'Filter by proposal status' 
+    description: 'Filter by proposal status',
   })
   @ApiResponse({ status: 200, description: 'Proposals retrieved successfully' })
   @ApiResponse({ status: 403, description: 'Admin access required' })
@@ -83,7 +88,7 @@ export class AdminDAOController {
 
     const [proposals, total] = await query.getManyAndCount();
 
-    const proposalsWithEmails = proposals.map(proposal => ({
+    const proposalsWithEmails = proposals.map((proposal) => ({
       ...proposal,
       proposerEmail: proposal.proposer?.email || 'Unknown',
       moderatorEmail: proposal.moderator?.email || undefined,

backend/src/admin/admin.module.ts

@@ -8,7 +8,12 @@ import { TypeOrmModule } from '@nestjs/typeorm';
 import { User } from '../users/entities/user.entity';
 
 @Module({
-  imports: [CoursesModule, LessonsModule, DAOModule, TypeOrmModule.forFeature([User])],
+  imports: [
+    CoursesModule,
+    LessonsModule,
+    DAOModule,
+    TypeOrmModule.forFeature([User]),
+  ],
   controllers: [AdminCoursesController, AdminDAOController],
 })
 export class AdminModule {}

backend/src/admin/dto/reorder-lessons.dto.ts

@@ -2,10 +2,12 @@ import { IsArray, IsString, ArrayNotEmpty } from 'class-validator';
 import { ApiProperty } from '@nestjs/swagger';
 
 export class ReorderLessonsDto {
-  @ApiProperty({ example: '123e4567-e89b-12d3-a456-426614174000', description: 'orderedIds field' })
+  @ApiProperty({
+    example: '123e4567-e89b-12d3-a456-426614174000',
+    description: 'orderedIds field',
+  })
   @IsArray()
   @ArrayNotEmpty()
   @IsString({ each: true })
   orderedIds: string[];
 }
-

backend/src/analytics/analytics.controller.ts

@@ -1,3 +1,10 @@
+import { Controller, Get, UseGuards } from '@nestjs/common';
+import {
+  ApiTags,
+  ApiOperation,
+  ApiResponse,
+  ApiBearerAuth,
+} from '@nestjs/swagger';
 import {
   Controller,
   Get,
@@ -33,36 +40,64 @@ export class AnalyticsController {
 
   @Get('overview')
   @ApiOperation({ summary: 'Get analytics overview' })
-  @ApiResponse({ status: 200, description: 'Analytics overview retrieved successfully', type: AnalyticsOverviewDto })
+  @ApiResponse({
+    status: 200,
+    description: 'Analytics overview retrieved successfully',
+    type: AnalyticsOverviewDto,
+  })
   @ApiResponse({ status: 401, description: 'Unauthorized' })
-  @ApiResponse({ status: 403, description: 'Forbidden - Admin access required' })
+  @ApiResponse({
+    status: 403,
+    description: 'Forbidden - Admin access required',
+  })
   async getOverview(): Promise<AnalyticsOverviewDto> {
     return this.analyticsService.getOverview();
   }
 
   @Get('course-performance')
   @ApiOperation({ summary: 'Get course performance analytics' })
-  @ApiResponse({ status: 200, description: 'Course performance data retrieved successfully', type: [CoursePerformanceDto] })
+  @ApiResponse({
+    status: 200,
+    description: 'Course performance data retrieved successfully',
+    type: [CoursePerformanceDto],
+  })
   @ApiResponse({ status: 401, description: 'Unauthorized' })
-  @ApiResponse({ status: 403, description: 'Forbidden - Admin access required' })
+  @ApiResponse({
+    status: 403,
+    description: 'Forbidden - Admin access required',
+  })
   async getCoursePerformance(): Promise<CoursePerformanceDto[]> {
     return this.analyticsService.getCoursePerformance();
   }
 
   @Get('learner-activity')
   @ApiOperation({ summary: 'Get learner activity analytics' })
-  @ApiResponse({ status: 200, description: 'Learner activity data retrieved successfully', type: [LearnerActivityPointDto] })
+  @ApiResponse({
+    status: 200,
+    description: 'Learner activity data retrieved successfully',
+    type: [LearnerActivityPointDto],
+  })
   @ApiResponse({ status: 401, description: 'Unauthorized' })
-  @ApiResponse({ status: 403, description: 'Forbidden - Admin access required' })
+  @ApiResponse({
+    status: 403,
+    description: 'Forbidden - Admin access required',
+  })
   async getLearnerActivity(): Promise<LearnerActivityPointDto[]> {
     return this.analyticsService.getLearnerActivity();
   }
 
   @Get('top-learners')
   @ApiOperation({ summary: 'Get top learners analytics' })
-  @ApiResponse({ status: 200, description: 'Top learners data retrieved successfully', type: [TopLearnerDto] })
+  @ApiResponse({
+    status: 200,
+    description: 'Top learners data retrieved successfully',
+    type: [TopLearnerDto],
+  })
   @ApiResponse({ status: 401, description: 'Unauthorized' })
-  @ApiResponse({ status: 403, description: 'Forbidden - Admin access required' })
+  @ApiResponse({
+    status: 403,
+    description: 'Forbidden - Admin access required',
+  })
   async getTopLearners(): Promise<TopLearnerDto[]> {
     return this.analyticsService.getTopLearners();
   }

backend/src/analytics/dto/analytics-response.dto.ts

@@ -15,7 +15,10 @@ export class AnalyticsOverviewDto {
 }
 
 export class CoursePerformanceDto {
-  @ApiProperty({ example: '123e4567-e89b-12d3-a456-426614174000', description: 'courseId field' })
+  @ApiProperty({
+    example: '123e4567-e89b-12d3-a456-426614174000',
+    description: 'courseId field',
+  })
   courseId: string;
   @ApiProperty({ example: 'Intro to Blockchain', description: 'title field' })
   title: string;
@@ -37,7 +40,10 @@ export class LearnerActivityPointDto {
 }
 
 export class TopLearnerDto {
-  @ApiProperty({ example: '123e4567-e89b-12d3-a456-426614174000', description: 'userId field' })
+  @ApiProperty({
+    example: '123e4567-e89b-12d3-a456-426614174000',
+    description: 'userId field',
+  })
   userId: string;
   @ApiProperty({ example: 'Jane Doe', description: 'username field' })
   username: string | null;
@@ -48,4 +54,3 @@ export class TopLearnerDto {
   @ApiProperty({ example: 1, description: 'certificatesEarned field' })
   certificatesEarned: number;
 }
-

backend/src/app.module.ts

@@ -33,7 +33,9 @@ import { WebhooksModule } from './webhooks/webhooks.module';
     ConfigModule.forRoot({
       isGlobal: true,
       validationSchema: Joi.object({
-        NODE_ENV: Joi.string().valid('development', 'production', 'test').default('development'),
+        NODE_ENV: Joi.string()
+          .valid('development', 'production', 'test')
+          .default('development'),
         PORT: Joi.number().default(3001),
 
         JWT_SECRET: Joi.string().min(32).required(),
@@ -56,7 +58,9 @@ import { WebhooksModule } from './webhooks/webhooks.module';
         SMTP_USER: Joi.string().optional().allow(''),
         SMTP_PASS: Joi.string().optional().allow(''),
         SMTP_FROM_NAME: Joi.string().default('ByteChain Academy'),
-        SMTP_FROM_EMAIL: Joi.string().email().default('noreply@bytechain.academy'),
+        SMTP_FROM_EMAIL: Joi.string()
+          .email()
+          .default('noreply@bytechain.academy'),
 
         AVATAR_UPLOAD_PATH: Joi.string().default('uploads/avatars'),
         MAX_AVATAR_SIZE_MB: Joi.number().default(2),

backend/src/auth/auth.controller.ts

@@ -1,6 +1,11 @@
 import { Controller, Post, Body, HttpCode, HttpStatus } from '@nestjs/common';
 import { Throttle } from '@nestjs/throttler';
-import { ApiTags, ApiOperation, ApiResponse, ApiBearerAuth } from '@nestjs/swagger';
+import {
+  ApiTags,
+  ApiOperation,
+  ApiResponse,
+  ApiBearerAuth,
+} from '@nestjs/swagger';
 
 import { AuthService } from './auth.service';
 import { RegisterDto } from './dto/register.dto';
@@ -29,9 +34,18 @@ export class AuthController {
   @Throttle({ default: { limit: 5, ttl: 900 } })
   @HttpCode(HttpStatus.OK)
   @ApiOperation({ summary: 'Authenticate user and return JWT token' })
-  @ApiResponse({ status: 200, description: 'Login successful, returns JWT token' })
-  @ApiResponse({ status: 400, description: 'Bad request - invalid credentials' })
-  @ApiResponse({ status: 401, description: 'Unauthorized - invalid credentials' })
+  @ApiResponse({
+    status: 200,
+    description: 'Login successful, returns JWT token',
+  })
+  @ApiResponse({
+    status: 400,
+    description: 'Bad request - invalid credentials',
+  })
+  @ApiResponse({
+    status: 401,
+    description: 'Unauthorized - invalid credentials',
+  })
   async login(@Body() loginDto: LoginDto) {
     return this.authService.login(loginDto);
   }
@@ -62,8 +76,14 @@ export class AuthController {
   @HttpCode(HttpStatus.OK)
   @ApiOperation({ summary: 'Reset user password using reset token' })
   @ApiResponse({ status: 200, description: 'Password reset successfully' })
-  @ApiResponse({ status: 400, description: 'Bad request - invalid token or password' })
-  @ApiResponse({ status: 401, description: 'Unauthorized - invalid reset token' })
+  @ApiResponse({
+    status: 400,
+    description: 'Bad request - invalid token or password',
+  })
+  @ApiResponse({
+    status: 401,
+    description: 'Unauthorized - invalid reset token',
+  })
   async resetPassword(@Body() resetPasswordDto: ResetPasswordDto) {
     return this.authService.resetPassword(resetPasswordDto);
   }

backend/src/auth/auth.service.spec.ts

@@ -1,6 +1,10 @@
 import { Test, TestingModule } from '@nestjs/testing';
 import { JwtService } from '@nestjs/jwt';
-import { ConflictException, UnauthorizedException, NotFoundException } from '@nestjs/common';
+import {
+  ConflictException,
+  UnauthorizedException,
+  NotFoundException,
+} from '@nestjs/common';
 import { ConfigService } from '@nestjs/config';
 import { AuthService } from './auth.service';
 import { UserService } from '../users/users.service';