Skip to content

Add Unit Tests for Backend Services and Controllers #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 74 commits into from
Dec 1, 2024
Merged

Add Unit Tests for Backend Services and Controllers #13

merged 74 commits into from
Dec 1, 2024

Conversation

tomast1337
Copy link
Member

@tomast1337 tomast1337 commented Nov 17, 2024 •
edited
Loading

This pull request introduces unit testing for all .services and .controllers in the backend NestJS application.

@tomast1337
test: add unit tests for various services and controllers
5377b8d
@tomast1337
chore: remove jest configuration from package.json
7f5fad0
@tomast1337
fix: update coverage directory path in jest configuration
7f32cff
@tomast1337
fix: add module name mapping for server imports in jest configuration
be1dcd9
@tomast1337
refactor: update file module to use dynamic configuration and remove …
e829c7f 
…unused dependencies
@tomast1337
test: enhance unit tests by adding service providers for dependency i…
bd7fb02 
…njection
@tomast1337
refactor: update AuthModule and FileModule to use asynchronous initia…
fddc98f 
…lization
@tomast1337
test: enhance FileService unit tests with S3 client mocks and error h…
ae12a74 
…andling
@tomast1337
test: enhance AuthService unit tests with axios mocks and token verif…
9cf840d 
…ication logic
@tomast1337
test: enhance AuthController unit tests with mock AuthService and exc…
06fb75d 
…eption handling
@tomast1337
test: enhance MySongsController unit tests with mock SongService and …
2de8b3b 
…exception handling
@tomast1337
test: enhance SongUploadService unit tests with mock FileService and …
1dd4c87 
…UserService
@tomast1337
test: add TODO for additional tests in SongUploadService unit tests
0daee3d
@tomast1337
test: enhance SongController unit tests with mock services and error …
94a7539 
…handling
@tomast1337
test: enhance PlaylistController unit tests with mock PlaylistService…
d380f0b 
… and error handling
@tomast1337
test: rename controller variable for consistency in SongController un…
42205cd 
…it tests
@tomast1337
test: enhance SongBrowserController unit tests with mock service meth…
84731bb 
…ods and additional test cases
@tomast1337
test: enhance UserController unit tests with mock UserService and add…
37a78c0 
…itional test cases
@tomast1337
Merge branch 'main' of https://github.com/OpenNBS/NoteBlockWorld into…
7c759f1 
… unit-testing
@tomast1337
refactor: replace console.error with Logger in FileService for improv…
b9f2ea8 
…ed logging
@tomast1337
test: mock thumbnail and song pack features in SongUploadService tests
9b185f9
@tomast1337
refactor: update method signatures in SongUploadService for improved …
df8bda5 
…type safety
@tomast1337
Merge https://github.com/OpenNBS/NoteBlockWorld into unit-testing
b67f438
@tomast1337
test: add exception handling for null userDocument in MySongsController
489fe16
@tomast1337
fix: remove unnecessary null check for user in getSelfUserData method
5d7f3c3
@tomast1337
test: update user controller test to handle null user with HttpException
1240741
@tomast1337
feat: song service tests
3c61013
@tomast1337 tomast1337 changed the base branch from develop to main November 17, 2024 21:49
@tomast1337 tomast1337 changed the base branch from main to develop November 17, 2024 21:49
tomast1337 and others added 29 commits November 19, 2024 12:55
@tomast1337
test: update getFeaturedSongs test to remove unnecessary result assig…
a9729d6 
…nment
@tomast1337
test: add unit tests for processUploadedSong and processSongPatch met…
72968de 
…hods in SongUploadService
@tomast1337
test: enhance SongUploadService tests with instrument layers and note…
874ea6e 
…s setup
@tomast1337
test: update jest configuration to ignore specific paths for test and…
36a909e 
… coverage
@tomast1337
test: rename userService variable to _userService for consistency in …
c8005c1 
…SongUploadService tests
@tomast1337
test: add unit tests for JWT, GitHub, and Google strategies
6fa169e
@tomast1337
test: update tests for DiscordStrategy to replace GithubStrategy refe…
c4716bb 
…rences
@tomast1337
test: add unit tests for GetRequestUser, initializeSwagger, and Parse…
e97fdfb 
…TokenPipe
@tomast1337
test: remove unused DocumentBuilder import in initializeSwagger tests
385e1d5
@tomast1337
refactor: simplify configuration retrieval in auth strategies
a255315
@tomast1337
refactor: update FileModule initialization to use forRootAsync
7fb1e68
@tomast1337
refactor: simplify MongoDB connection configuration by using getOrThrow
2fe046b
@tomast1337
refactor: update JwtStrategy tests to use getOrThrow for configuratio…
4167c5a 
…n retrieval
@tomast1337
refactor: update authentication strategy tests to use getOrThrow for …
de7ac73 
…configuration retrieval
@Bentroen
chore: add vscode-jest to workspace recommended extensions
c10e46b
@tomast1337
feat: add Jest configuration and initial test suite for song statistics
30cf453
@tomast1337
chore: add workspace configuration for project structure and Jest set…
eb6fb17 
…tings
@tomast1337
refactor: improve test descriptions for song statistics calculations
1398e8f
@tomast1337
Merge branch 'develop' of https://github.com/OpenNBS/NoteBlockWorld i…
37533cf 
…nto unit-testing
@tomast1337
test: update expected username in UserService tests
3af6a7d
@tomast1337
test: implement tests for googleLogin, githubLogin, and discordLogin …
807be74 
…in AuthService
@tomast1337
test: add unit tests for JWT token creation and user verification in …
ca4a44d 
…AuthService
@tomast1337
chore: remove commented-out configuration code in AuthService constru…
f05438e 
…ctor
@tomast1337
test: add error handling tests for thumbnail and file uploads in Song…
7286ab0 
…UploadService
@Bentroen
Merge branch 'develop' into unit-testing
7b379cf
@tomast1337
chore: remove unused dependencies from pnpm-lock.yaml
114266d
@tomast1337
feat: add SongWebhookService mock and formatDuration utility function
c64a7ac
@tomast1337
refactor: mark formatDuration for future relocation to shared utilities
da3713b
@tomast1337
refactor: add TODO to move formatDuration to shared/util
9fcf4ae
@tomast1337 tomast1337 merged commit 39af96f into develop Dec 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tomast1337 @Bentroen