From 6430605e30bac50a88b941abf594bd7c75a5e6c2 Mon Sep 17 00:00:00 2001 From: Peifan Li Date: Tue, 16 Dec 2025 14:43:57 -0500 Subject: [PATCH] refactor: Update quotation marks to use double quotes consistently --- .../subscriptionController.test.ts | 75 ++++++++++--------- 1 file changed, 39 insertions(+), 36 deletions(-) diff --git a/backend/src/__tests__/controllers/subscriptionController.test.ts b/backend/src/__tests__/controllers/subscriptionController.test.ts index ec5c4a5..cd2d04d 100644 --- a/backend/src/__tests__/controllers/subscriptionController.test.ts +++ b/backend/src/__tests__/controllers/subscriptionController.test.ts @@ -1,22 +1,22 @@ -import { Request, Response } from 'express'; -import { beforeEach, describe, expect, it, vi } from 'vitest'; +import { Request, Response } from "express"; +import { beforeEach, describe, expect, it, vi } from "vitest"; import { createSubscription, deleteSubscription, getSubscriptions, -} from '../../controllers/subscriptionController'; -import { ValidationError } from '../../errors/DownloadErrors'; -import { subscriptionService } from '../../services/subscriptionService'; -import { logger } from '../../utils/logger'; +} from "../../controllers/subscriptionController"; +import { ValidationError } from "../../errors/DownloadErrors"; +import { subscriptionService } from "../../services/subscriptionService"; +import { logger } from "../../utils/logger"; -vi.mock('../../services/subscriptionService'); -vi.mock('../../utils/logger', () => ({ +vi.mock("../../services/subscriptionService"); +vi.mock("../../utils/logger", () => ({ logger: { info: vi.fn(), }, })); -describe('SubscriptionController', () => { +describe("SubscriptionController", () => { let req: Partial; let res: Partial; let json: any; @@ -36,33 +36,37 @@ describe('SubscriptionController', () => { }; }); - describe('createSubscription', () => { - it('should create a subscription', async () => { - req.body = { url: 'https://www.youtube.com/@testuser', interval: 60 }; + describe("createSubscription", () => { + it("should create a subscription", async () => { + req.body = { url: "https://www.youtube.com/@testuser", interval: 60 }; const mockSubscription = { - id: 'sub-123', - url: 'https://www.youtube.com/@testuser', + id: "sub-123", + url: "https://www.youtube.com/@testuser", interval: 60, - author: '@testuser', - platform: 'YouTube', + author: "@testuser", + platform: "YouTube", }; - (subscriptionService.subscribe as any).mockResolvedValue(mockSubscription); + (subscriptionService.subscribe as any).mockResolvedValue( + mockSubscription + ); await createSubscription(req as Request, res as Response); - expect(logger.info).toHaveBeenCalledWith('Creating subscription:', { - url: 'https://www.youtube.com/@testuser', + expect(logger.info).toHaveBeenCalledWith("Creating subscription:", { + url: "https://www.youtube.com/@testuser", interval: 60, + authorName: undefined, }); expect(subscriptionService.subscribe).toHaveBeenCalledWith( - 'https://www.youtube.com/@testuser', - 60 + "https://www.youtube.com/@testuser", + 60, + undefined ); expect(status).toHaveBeenCalledWith(201); expect(json).toHaveBeenCalledWith(mockSubscription); }); - it('should throw ValidationError when URL is missing', async () => { + it("should throw ValidationError when URL is missing", async () => { req.body = { interval: 60 }; await expect( @@ -72,8 +76,8 @@ describe('SubscriptionController', () => { expect(subscriptionService.subscribe).not.toHaveBeenCalled(); }); - it('should throw ValidationError when interval is missing', async () => { - req.body = { url: 'https://www.youtube.com/@testuser' }; + it("should throw ValidationError when interval is missing", async () => { + req.body = { url: "https://www.youtube.com/@testuser" }; await expect( createSubscription(req as Request, res as Response) @@ -82,7 +86,7 @@ describe('SubscriptionController', () => { expect(subscriptionService.subscribe).not.toHaveBeenCalled(); }); - it('should throw ValidationError when both URL and interval are missing', async () => { + it("should throw ValidationError when both URL and interval are missing", async () => { req.body = {}; await expect( @@ -91,11 +95,11 @@ describe('SubscriptionController', () => { }); }); - describe('getSubscriptions', () => { - it('should return all subscriptions', async () => { + describe("getSubscriptions", () => { + it("should return all subscriptions", async () => { const mockSubscriptions = [ - { id: 'sub-1', url: 'https://www.youtube.com/@test1', interval: 60 }, - { id: 'sub-2', url: 'https://space.bilibili.com/123', interval: 120 }, + { id: "sub-1", url: "https://www.youtube.com/@test1", interval: 60 }, + { id: "sub-2", url: "https://space.bilibili.com/123", interval: 120 }, ]; (subscriptionService.listSubscriptions as any).mockResolvedValue( mockSubscriptions @@ -108,7 +112,7 @@ describe('SubscriptionController', () => { expect(status).not.toHaveBeenCalled(); // Default status is 200 }); - it('should return empty array when no subscriptions', async () => { + it("should return empty array when no subscriptions", async () => { (subscriptionService.listSubscriptions as any).mockResolvedValue([]); await getSubscriptions(req as Request, res as Response); @@ -117,20 +121,19 @@ describe('SubscriptionController', () => { }); }); - describe('deleteSubscription', () => { - it('should delete a subscription', async () => { - req.params = { id: 'sub-123' }; + describe("deleteSubscription", () => { + it("should delete a subscription", async () => { + req.params = { id: "sub-123" }; (subscriptionService.unsubscribe as any).mockResolvedValue(undefined); await deleteSubscription(req as Request, res as Response); - expect(subscriptionService.unsubscribe).toHaveBeenCalledWith('sub-123'); + expect(subscriptionService.unsubscribe).toHaveBeenCalledWith("sub-123"); expect(status).toHaveBeenCalledWith(200); expect(json).toHaveBeenCalledWith({ success: true, - message: 'Subscription deleted', + message: "Subscription deleted", }); }); }); }); -