Skip to content

Add xUnit test suite for Services project #243

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
Open
anna-dieu wants to merge 466 commits into
base: main
Choose a base branch
from
Open

Add xUnit test suite for Services project #243

anna-dieu wants to merge 466 commits into from

Conversation

@anna-dieu
Copy link

@anna-dieu anna-dieu commented Nov 26, 2025

Summary

Added comprehensive unit test suite for GoodBooks Services project using xUnit framework and Moq for mocking.

Changes

  • Created GoodBooks.ServicesTests xUnit project with 31 passing tests
  • Tests cover repository operations, base DTO validation, and Company DTO patterns
  • Added reusable MockRepositoryFixtures for consistent mock setup
  • Simplified README with quick-start commands

Test Coverage

  • BaseServiceTests: 8 tests for repository operations (SequenceNumber, GeneralLedgerSetting, Bank)
  • BaseDtoTests: 7 tests for base DTO class properties and inheritance
  • CompanyDtoTests: 10 tests demonstrating complete DTO testing pattern
  • Total: 31 passing tests, 0 failures

Testing

Run tests with:

dotnet test test/GoodBooks.ServicesTests/

ydongchoi and others added 30 commits August 3, 2024 22:27
@ydongchoi
[fix] modified mapper parameter.
2c99408
@ydongchoi
[feature] Added Validation Filter for AddNewTax action method.
13677e2
@ydongchoi
[feature] Added deletetax action method in TaxController.
0f702a3
@ydongchoi
[feature] Added deletetax and edittax action method in TaxController.
d2512cb
@ydongchoi
[chore] Removed MappingProfile class.
48ef003
@ydongchoi
[fix] refactored Edit Tax Action method and added validation filter.
ef0b1f2
@ydongchoi
[fix] CreateMap for mapping TaxDto to TaxEntity,
91a65a2
@ydongchoi
[chore] Removed Required Data Annotation.
b15d57b
@ydongchoi
[feature] Created EditTax Method and fixed CreateTax Method in Admini…
add2ae1 
…stration Service.
@ydongchoi
[fix] Added Include to add multiple entities.
43f83bf
@ydongchoi
[feature] Created Edit Tax Page, Action Method, View Model.
d363625
@ydongchoi
[chore] Modified accountgo.sln.
86c2545
@ydongchoi
[feature] Created check logic wheter Tax data already exists or not.
62346aa
@ydongchoi
[feature] Added the feature creating TaxGroup, ItemTaxGroup in Accoun…
434efcd 
…tGoWeb.
@ydongchoi
[feature] Added the feature creating TaxGroup and ItemTaxGroup in Api…
5cff546 
… and Dto and Service.
@ydongchoi
[fix] added ; to fix error.
e4dd23e
@ydongchoi
[featue] Added ViewModel for Tax System.
1d35f8b
@ydongchoi
[feature] Added AutoMapper for mapping between Dto and View Model.
fb74d07
@ydongchoi
[feature] Added span tag for value Validation in EditTax Page.
f08fb3e
@ydongchoi
[feature] Applied Mapper and Map Between Dto and ViewModel.
b0e1447
@ydongchoi
[feature] Added ModelState Validation Logic in AddNewTax action method.
83a3ea9
@ydongchoi
[fix] Changed class in span tag for validation.
b204246
@ydongchoi
[fix] Changed TaxName, TaxCode MaxLength.
081f346
@ydongchoi
[feature] Added DeleteTaxGroup and ItemTaxGroup feature in AccountGoW…
55fa6bb 
…eb and Api.
@medhatelmasry
updated nullable in projects dto & api
e1d05be
@medhatelmasry
fixed nullables
208261e
@ydongchoi
[fix] Removed the condition that finds the tax that has same rate.
09d1094
@medhatelmasry
edits on Tax System
ffb1ab8
@ydongchoi
[feature] Applied Result Pattern to AddNewTax Service.
951c019
@ydongchoi
[feature] Applied Result Pattern to Delete Tax/TaxGroup/Item TaxGroup…
5df00ea 
… feature.
Bdiep6 and others added 30 commits November 26, 2025 23:10
@Bdiep6
Made SalesReceipt's view page.
a0fa139
@jefferymj
Fix SQL Server version to 2022-latest across environments
9b7b446
@jefferymj
Added BlazorGDB Blazor project to Aspire so that it shows up on the d…
0464058 
…ashboard.
@jefferymj
Deleted require-HTTP-connection option
5fa7706
@medhatelmasry
Merge pull request #178 from medhatelmasry/146BrianDiep
8adb585 
146 - Brian Diep - Accounts Receivable >> Sales Orders
@medhatelmasry
Merge branch '4976fall2025' into 142JaskunwarHunjan
b9a10ec
@medhatelmasry
Merge pull request #176 from medhatelmasry/142JaskunwarHunjan
4c4eca4 
Implemented editable subaccounts and delete protection logic in Chart…
@shayannikpour
Hidden items are visible now
8fe5512
@Gems2019
added full information of contact section in customer cards
a61c204
@Gems2019
adjusted the foreground color to light make it visible
d9185aa
@jefferymj
Bug fix. Updated Aspire sdk to v13.0.1.
122ad92
@shayannikpour
currency issue fixed
25b7957
@medhatelmasry
Merge pull request #172 from medhatelmasry/149GemSha
06a2408 
149 GemSha - Convert Customer and Contact Pages to Blazor
@medhatelmasry
Merge pull request #162 from medhatelmasry/136ShayanNikpour
354ba50 
136 shayan nikpour
@Bdiep6
Fix receipt allocation workflow with invoice updates and Changed Allo…
b927f49 
…cation title Receipt Allocation to Sales Receipt Allocation
@Bdiep6
Add 'Sales Receipt' title to SalesReceipt detail page
e6bec8f
resorted back to AppHost Program.cs without registry because the Jeff…
2c73f3c 
…rey solution did not work on Medhat mac
@medhatelmasry
Merge pull request #184 from medhatelmasry/159JefferyMJ
038430b 
Fix the version of SQL Server database image in Aspire
@Bdiep6
Implement receipt editing with UpdateReceipt endpoint and debit accou…
702ad0f 
…nt display fix
removal of empty line in BlazorGDB/Program.cs
eb95055
@medhatelmasry
Merge branch '4976fall2025' into 147BrianDiep
98b5895
@medhatelmasry
Merge pull request #177 from medhatelmasry/147BrianDiep
fa17d32 
147 - Brian Diep - Accounts Receivable >> Sales Receipts
@amalallahham
fix checkbox
9f23bed
@medhatelmasry
Merge pull request #163 from medhatelmasry/155AmalAllaham
a99c2fa 
#155 Amal Allaham
@jefferymj
Merge branch '4976fall2025' of https://github.com/medhatelmasry/GoodB…
acad832 
…ooks into 165JefferyMJ
@jefferymj
Bug fix: Update BlazorGDB to use MapStaticAssets() for wasm support
29c84c7
@jefferymj
Git merge
139a623
@medhatelmasry
Merge pull request #193 from medhatelmasry/165JefferyMJ
7f1e8a1 
#165, #191: Make the BlazorGDB participate in Aspire
updated GoodBooks.ServicesTests to .net 10
3c021ff
@medhatelmasry
Merge branch '4976fall2025' into 158AnnaDao
c95ead7
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.