Skip to content

Commit 91b480b

Browse files
committed
refactor: update imports to use centralized database module and clean up unused imports in auth and song services
1 parent e6834bc commit 91b480b

File tree

8 files changed

+36
-53
lines changed

8 files changed

+36
-53
lines changed

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

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,9 @@
1+
import type { UserDocument } from '@nbw/database';
12
import { JwtService } from '@nestjs/jwt';
23
import { Test, TestingModule } from '@nestjs/testing';
4+
import { UserService } from '@server/user/user.service';
35
import { beforeEach, describe, expect, it, jest, mock, spyOn } from 'bun:test';
46
import type { Request, Response } from 'express';
5-
6-
import type { UserDocument } from '@server/user/entity/user.entity';
7-
import { UserService } from '@server/user/user.service';
8-
97
import { AuthService } from './auth.service';
108
import { Profile } from './types/profile';
119

packages/backend/src/auth/auth.service.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,10 @@
1+
import type { UserDocument } from '@nbw/database';
2+
import { CreateUser } from '@nbw/database';
13
import { Inject, Injectable, Logger } from '@nestjs/common';
24
import { JwtService } from '@nestjs/jwt';
3-
import { CreateUser } from '@nbw/database';
5+
import { UserService } from '@server/user/user.service';
46
import axios from 'axios';
57
import type { Request, Response } from 'express';
6-
7-
import type { UserDocument } from '@server/user/entity/user.entity';
8-
import { UserService } from '@server/user/user.service';
9-
108
import { DiscordUser } from './types/discordProfile';
119
import { GithubAccessToken, GithubEmailList } from './types/githubProfile';
1210
import { GoogleProfile } from './types/googleProfile';

packages/backend/src/song/my-songs/my-songs.controller.spec.ts

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,8 @@
1+
import type { UserDocument } from '@nbw/database';
2+
import { PageQueryDTO, SongPageDto } from '@nbw/database';
13
import { HttpException } from '@nestjs/common';
24
import { AuthGuard } from '@nestjs/passport';
35
import { Test, TestingModule } from '@nestjs/testing';
4-
import { PageQueryDTO } from '@nbw/database';
5-
import { SongPageDto } from '@nbw/database';
6-
7-
import type { UserDocument } from '@server/user/entity/user.entity';
8-
96
import { SongService } from '../song.service';
107
import { MySongsController } from './my-songs.controller';
118

packages/backend/src/song/my-songs/my-songs.controller.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,9 @@
1+
import type { UserDocument } from '@nbw/database';
2+
import { PageQueryDTO, SongPageDto } from '@nbw/database';
13
import { Controller, Get, Query, UseGuards } from '@nestjs/common';
24
import { AuthGuard } from '@nestjs/passport';
35
import { ApiBearerAuth, ApiOperation, ApiTags } from '@nestjs/swagger';
4-
import { PageQueryDTO, SongPageDto } from '@nbw/database';
5-
66
import { GetRequestToken, validateUser } from '@server/lib/GetRequestUser';
7-
import type { UserDocument } from '@server/user/entity/user.entity';
87

98
import { SongService } from '../song.service';
109

packages/backend/src/song/song-webhook/song-webhook.service.spec.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,11 @@
1+
import { Song as SongEntity, SongWithUser } from '@nbw/database';
12
import { ConfigModule, ConfigService } from '@nestjs/config';
23
import { getModelToken } from '@nestjs/mongoose';
34
import { Test, TestingModule } from '@nestjs/testing';
45
import { beforeEach, describe, expect, it, jest, mock, spyOn } from 'bun:test';
56
import { Model } from 'mongoose';
6-
7-
import { SongWebhookService } from './song-webhook.service';
8-
import { Song as SongEntity, SongWithUser } from '../entity/song.entity';
97
import { getUploadDiscordEmbed } from '../song.util';
8+
import { SongWebhookService } from './song-webhook.service';
109

1110
mock.module('../song.util', () => ({
1211
getUploadDiscordEmbed: jest.fn(),

packages/backend/src/song/song.controller.spec.ts

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,16 @@
1-
import { HttpStatus, UnauthorizedException } from '@nestjs/common';
2-
import { AuthGuard } from '@nestjs/passport';
3-
import { Test, TestingModule } from '@nestjs/testing';
1+
import type { UserDocument } from '@nbw/database';
42
import {
53
PageQueryDTO,
64
SongPreviewDto,
75
SongViewDto,
86
UploadSongDto,
97
UploadSongResponseDto,
108
} from '@nbw/database';
11-
import { Response } from 'express';
12-
9+
import { HttpStatus, UnauthorizedException } from '@nestjs/common';
10+
import { AuthGuard } from '@nestjs/passport';
11+
import { Test, TestingModule } from '@nestjs/testing';
1312
import { FileService } from '@server/file/file.service';
14-
import type { UserDocument } from '@server/user/entity/user.entity';
15-
13+
import { Response } from 'express';
1614
import { SongController } from './song.controller';
1715
import { SongService } from './song.service';
1816

packages/backend/src/song/song.controller.ts

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,13 @@
1+
import type { UserDocument } from '@nbw/database';
2+
import {
3+
PageQueryDTO,
4+
SongPreviewDto,
5+
SongViewDto,
6+
UploadConst,
7+
UploadSongDto,
8+
UploadSongResponseDto,
9+
} from '@nbw/database';
10+
import type { RawBodyRequest } from '@nestjs/common';
111
import {
212
Body,
313
Controller,
@@ -16,7 +26,6 @@ import {
1626
UseGuards,
1727
UseInterceptors,
1828
} from '@nestjs/common';
19-
import type { RawBodyRequest } from '@nestjs/common';
2029
import { AuthGuard } from '@nestjs/passport';
2130
import { FileInterceptor } from '@nestjs/platform-express';
2231
import { MulterOptions } from '@nestjs/platform-express/multer/interfaces/multer-options.interface';
@@ -27,20 +36,9 @@ import {
2736
ApiOperation,
2837
ApiTags,
2938
} from '@nestjs/swagger';
30-
import {
31-
PageQueryDTO,
32-
SongPreviewDto,
33-
SongViewDto,
34-
UploadSongDto,
35-
UploadSongResponseDto,
36-
UploadConst,
37-
} from '@nbw/database';
38-
import type { Response } from 'express';
39-
4039
import { FileService } from '@server/file/file.service';
4140
import { GetRequestToken, validateUser } from '@server/lib/GetRequestUser';
42-
import type { UserDocument } from '@server/user/entity/user.entity';
43-
41+
import type { Response } from 'express';
4442
import { SongService } from './song.service';
4543

4644
// Handles public-facing song routes.

packages/backend/src/song/song.service.spec.ts

Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,20 @@
1-
import { HttpException } from '@nestjs/common';
2-
import { getModelToken } from '@nestjs/mongoose';
3-
import { Test, TestingModule } from '@nestjs/testing';
1+
import type { UserDocument } from '@nbw/database';
42
import {
3+
SongDocument,
4+
Song as SongEntity,
55
SongPreviewDto,
6+
SongSchema,
67
SongStats,
78
SongViewDto,
9+
SongWithUser,
810
UploadSongDto,
911
UploadSongResponseDto,
1012
} from '@nbw/database';
11-
import mongoose, { Model } from 'mongoose';
12-
13+
import { HttpException } from '@nestjs/common';
14+
import { getModelToken } from '@nestjs/mongoose';
15+
import { Test, TestingModule } from '@nestjs/testing';
1316
import { FileService } from '@server/file/file.service';
14-
import type { UserDocument } from '@server/user/entity/user.entity';
15-
16-
import {
17-
SongDocument,
18-
Song as SongEntity,
19-
SongSchema,
20-
SongWithUser,
21-
} from './entity/song.entity';
17+
import mongoose, { Model } from 'mongoose';
2218
import { SongUploadService } from './song-upload/song-upload.service';
2319
import { SongWebhookService } from './song-webhook/song-webhook.service';
2420
import { SongService } from './song.service';

0 commit comments

Comments
 (0)