From 7729d612a594dfbdf09c22c79fec1c8837a30154 Mon Sep 17 00:00:00 2001
From: minorcell <minorcell6789@gmail.com>
Date: Sat, 26 Apr 2025 23:05:25 +0800
Subject: [PATCH] Remove test files and enhance upload service with error
 handling and validation

---
 .../resume-template.controller.spec.ts        | 20 -------------------
 .../resume-template.service.spec.ts           | 18 -----------------
 src/resume/resume.controller.spec.ts          | 20 -------------------
 src/resume/resume.service.spec.ts             | 18 -----------------
 src/upload/upload.service.ts                  | 10 ++++++++++
 5 files changed, 10 insertions(+), 76 deletions(-)
 delete mode 100644 src/resume-template/resume-template.controller.spec.ts
 delete mode 100644 src/resume-template/resume-template.service.spec.ts
 delete mode 100644 src/resume/resume.controller.spec.ts
 delete mode 100644 src/resume/resume.service.spec.ts

diff --git a/src/resume-template/resume-template.controller.spec.ts b/src/resume-template/resume-template.controller.spec.ts
deleted file mode 100644
index 84b3b4b..0000000
--- a/src/resume-template/resume-template.controller.spec.ts
+++ /dev/null
@@ -1,20 +0,0 @@
-import { Test, TestingModule } from '@nestjs/testing';
-import { ResumeTemplateController } from './resume-template.controller';
-import { ResumeTemplateService } from './resume-template.service';
-
-describe('ResumeTemplateController', () => {
-  let controller: ResumeTemplateController;
-
-  beforeEach(async () => {
-    const module: TestingModule = await Test.createTestingModule({
-      controllers: [ResumeTemplateController],
-      providers: [ResumeTemplateService],
-    }).compile();
-
-    controller = module.get<ResumeTemplateController>(ResumeTemplateController);
-  });
-
-  it('should be defined', () => {
-    expect(controller).toBeDefined();
-  });
-});
diff --git a/src/resume-template/resume-template.service.spec.ts b/src/resume-template/resume-template.service.spec.ts
deleted file mode 100644
index dbba412..0000000
--- a/src/resume-template/resume-template.service.spec.ts
+++ /dev/null
@@ -1,18 +0,0 @@
-import { Test, TestingModule } from '@nestjs/testing';
-import { ResumeTemplateService } from './resume-template.service';
-
-describe('ResumeTemplateService', () => {
-  let service: ResumeTemplateService;
-
-  beforeEach(async () => {
-    const module: TestingModule = await Test.createTestingModule({
-      providers: [ResumeTemplateService],
-    }).compile();
-
-    service = module.get<ResumeTemplateService>(ResumeTemplateService);
-  });
-
-  it('should be defined', () => {
-    expect(service).toBeDefined();
-  });
-});
diff --git a/src/resume/resume.controller.spec.ts b/src/resume/resume.controller.spec.ts
deleted file mode 100644
index 33b0a4b..0000000
--- a/src/resume/resume.controller.spec.ts
+++ /dev/null
@@ -1,20 +0,0 @@
-import { Test, TestingModule } from '@nestjs/testing';
-import { ResumeController } from './resume.controller';
-import { ResumeService } from './resume.service';
-
-describe('ResumeController', () => {
-  let controller: ResumeController;
-
-  beforeEach(async () => {
-    const module: TestingModule = await Test.createTestingModule({
-      controllers: [ResumeController],
-      providers: [ResumeService],
-    }).compile();
-
-    controller = module.get<ResumeController>(ResumeController);
-  });
-
-  it('should be defined', () => {
-    expect(controller).toBeDefined();
-  });
-});
diff --git a/src/resume/resume.service.spec.ts b/src/resume/resume.service.spec.ts
deleted file mode 100644
index 97b4c3f..0000000
--- a/src/resume/resume.service.spec.ts
+++ /dev/null
@@ -1,18 +0,0 @@
-import { Test, TestingModule } from '@nestjs/testing';
-import { ResumeService } from './resume.service';
-
-describe('ResumeService', () => {
-  let service: ResumeService;
-
-  beforeEach(async () => {
-    const module: TestingModule = await Test.createTestingModule({
-      providers: [ResumeService],
-    }).compile();
-
-    service = module.get<ResumeService>(ResumeService);
-  });
-
-  it('should be defined', () => {
-    expect(service).toBeDefined();
-  });
-});
diff --git a/src/upload/upload.service.ts b/src/upload/upload.service.ts
index 6d08d97..517b6c2 100644
--- a/src/upload/upload.service.ts
+++ b/src/upload/upload.service.ts
@@ -33,7 +33,17 @@ export class UploadService {
   async uploadFile(file: Express.Multer.File) {
     const formUploader = new qiniu.form_up.FormUploader(this.config);
     const putExtra = new qiniu.form_up.PutExtra();
+    
+    // 验证配置是否存在
+    const accessKey = this.configService.get('kodo.ACCESS_KEY');
+    const secretKey = this.configService.get('kodo.SECRET_KEY');
+    const bucket = this.configService.get('kodo.BUCKET');
+    const baseUrl = this.configService.get('kodo.BASE_URL');
 
+    if (!accessKey || !secretKey || !bucket || !baseUrl) {
+      throw new Error('Missing required Qiniu configuration. Please check your configuration file.');
+    }
+    
     return new Promise<Upload>((resolve, reject) => {
       formUploader.put(
         this.uploadToken,