[PR #1] [MERGED] feat: add API unit tests and CI workflows #1

New issue

Closed

opened 2026-03-01 17:26:31 +03:00 by kerem · 0 comments

kerem commented 2026-03-01 17:26:31 +03:00

Owner

Copy link

📋 Pull Request Information

Original PR: https://github.com/adamchairly/soundfeed/pull/1
Author: @adamchairly
Created: 3/1/2026
Status: ✅ Merged
Merged: 3/1/2026
Merged by: @adamchairly

Base: main ← Head: feat/add_api_unit_tests

---

📝 Commits (4)

• fedc7b6 feat: add tests project and ci workflow
• 181962c feat: refactor test proj usings
• 24a256a feat: add react-doctor CI, fix test usings
• 76ed4d1 feat: fix workflow files and test errors

📊 Changes

19 files changed (+1198 additions, -5 deletions)

View changed files

➕ .github/workflows/backend-tests.yml (+30 -0)
➕ .github/workflows/react-doctor.yml (+103 -0)
📝 backend/Directory.Packages.props (+8 -3)
➕ backend/Soundfeed.Bll.Tests/Extensions/PaginationExtensionTests.cs (+121 -0)
➕ backend/Soundfeed.Bll.Tests/Features/Artists/GetArtistQueryHandlerTests.cs (+48 -0)
➕ backend/Soundfeed.Bll.Tests/Features/Artists/PostArtistCommandHandlerTests.cs (+203 -0)
➕ backend/Soundfeed.Bll.Tests/Features/Artists/SearchArtistQueryHandlerTests.cs (+88 -0)
➕ backend/Soundfeed.Bll.Tests/Features/Releases/DismissReleaseCommandHandlerTests.cs (+139 -0)
➕ backend/Soundfeed.Bll.Tests/Features/Subscription/DeleteSubscriptionCommandHandlerTests.cs (+47 -0)
➕ backend/Soundfeed.Bll.Tests/Features/Users/RecoverUserCommandHandlerTests.cs (+108 -0)
➕ backend/Soundfeed.Bll.Tests/GlobalUsings.cs (+1 -0)
➕ backend/Soundfeed.Bll.Tests/Helpers/TestDbContextFactory.cs (+16 -0)
➕ backend/Soundfeed.Bll.Tests/Models/PageResultTests.cs (+39 -0)
➕ backend/Soundfeed.Bll.Tests/Services/UserServiceTests.cs (+218 -0)
➕ backend/Soundfeed.Bll.Tests/Soundfeed.Bll.Tests.csproj (+23 -0)
📝 backend/Soundfeed.Bll/Soundfeed.Bll.csproj (+3 -0)
📝 backend/Soundfeed.slnx (+1 -0)
📝 frontend/package.json (+1 -1)
📝 frontend/src/pages/LandingPage.tsx (+1 -1)

📄 Description

• added unit tests
• added CI workflow to run the tests on PR

---

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/adamchairly/soundfeed/pull/1 **Author:** [@adamchairly](https://github.com/adamchairly) **Created:** 3/1/2026 **Status:** ✅ Merged **Merged:** 3/1/2026 **Merged by:** [@adamchairly](https://github.com/adamchairly) **Base:** `main` ← **Head:** `feat/add_api_unit_tests` --- ### 📝 Commits (4) - [`fedc7b6`](https://github.com/adamchairly/soundfeed/commit/fedc7b684bf7b4738054b0244ed280761ec79578) feat: add tests project and ci workflow - [`181962c`](https://github.com/adamchairly/soundfeed/commit/181962c1c0d9f404306d1ac568d88437b190b950) feat: refactor test proj usings - [`24a256a`](https://github.com/adamchairly/soundfeed/commit/24a256a023397ccc324b9954d97a2caad238508b) feat: add react-doctor CI, fix test usings - [`76ed4d1`](https://github.com/adamchairly/soundfeed/commit/76ed4d1f9eda628400bd7b98355021dc2acddf5f) feat: fix workflow files and test errors ### 📊 Changes **19 files changed** (+1198 additions, -5 deletions) <details> <summary>View changed files</summary> ➕ `.github/workflows/backend-tests.yml` (+30 -0) ➕ `.github/workflows/react-doctor.yml` (+103 -0) 📝 `backend/Directory.Packages.props` (+8 -3) ➕ `backend/Soundfeed.Bll.Tests/Extensions/PaginationExtensionTests.cs` (+121 -0) ➕ `backend/Soundfeed.Bll.Tests/Features/Artists/GetArtistQueryHandlerTests.cs` (+48 -0) ➕ `backend/Soundfeed.Bll.Tests/Features/Artists/PostArtistCommandHandlerTests.cs` (+203 -0) ➕ `backend/Soundfeed.Bll.Tests/Features/Artists/SearchArtistQueryHandlerTests.cs` (+88 -0) ➕ `backend/Soundfeed.Bll.Tests/Features/Releases/DismissReleaseCommandHandlerTests.cs` (+139 -0) ➕ `backend/Soundfeed.Bll.Tests/Features/Subscription/DeleteSubscriptionCommandHandlerTests.cs` (+47 -0) ➕ `backend/Soundfeed.Bll.Tests/Features/Users/RecoverUserCommandHandlerTests.cs` (+108 -0) ➕ `backend/Soundfeed.Bll.Tests/GlobalUsings.cs` (+1 -0) ➕ `backend/Soundfeed.Bll.Tests/Helpers/TestDbContextFactory.cs` (+16 -0) ➕ `backend/Soundfeed.Bll.Tests/Models/PageResultTests.cs` (+39 -0) ➕ `backend/Soundfeed.Bll.Tests/Services/UserServiceTests.cs` (+218 -0) ➕ `backend/Soundfeed.Bll.Tests/Soundfeed.Bll.Tests.csproj` (+23 -0) 📝 `backend/Soundfeed.Bll/Soundfeed.Bll.csproj` (+3 -0) 📝 `backend/Soundfeed.slnx` (+1 -0) 📝 `frontend/package.json` (+1 -1) 📝 `frontend/src/pages/LandingPage.tsx` (+1 -1) </details> ### 📄 Description - added unit tests - added CI workflow to run the tests on PR --- _{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

kerem 2026-03-01 17:26:31 +03:00

• closed this issue
• added the
  pull-request
  label

Sign in to join this conversation.

No Branch/Tag specified

Branches Tags

main

migrate-to-nextjs

dev

feature/theme_switching

snyk-fix-1e2b3a90641ac4f0f1c39f3fac287538

feat/add_api_unit_tests

No results found.

Labels

Clear labels   

pull-request

Mirrored from GitHub Pull Request

No labels

pull-request

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/soundfeed#1

No description provided.