Skip to content

Basic functionality testing#317

Open
Joel04kayy wants to merge 25 commits intofinos:mainfrom
Joel04kayy:basic-FunctionalityTesting
Open

Basic functionality testing#317
Joel04kayy wants to merge 25 commits intofinos:mainfrom
Joel04kayy:basic-FunctionalityTesting

Conversation

@Joel04kayy
Copy link

No description provided.

Your Name and others added 24 commits November 6, 2025 12:37
…agement, Trade Service, Position Retrieval, Health Checks, Service Integration, and Error Scenarios
- Add CI workflow (ci.yml) that runs tests and builds all services
- Add test workflow (test.yml) for standalone test execution
- Update build-and-publish workflow to depend on CI completion
- Add comprehensive test coverage for Java, Node.js, and .NET services
- Add CI/CD documentation
- Fix Node.js setup to handle missing package-lock.json files
- Fix Java service workflow chmod path issue
- Fix publish test results action (comment_mode: always)
- Fix Angular test failures in trade-blotter component
- Fix trade-service integration tests (expectation order and 5xx error handling)

All services should now pass in GitHub Actions CI/CD pipeline.
- Fix Node.js setup to handle missing package-lock.json files
- Fix Java service workflow chmod path issue
- Fix publish test results action (comment_mode: always)
- Fix Angular test failures in trade-blotter component
- Fix trade-service integration tests (expectation order, 5xx error handling, mock server setup)

All services should now pass in GitHub Actions CI/CD pipeline.
- Fix Node.js setup to handle missing package-lock.json files
- Fix Java service workflow chmod path issue
- Fix publish test results action (comment_mode: always)
- Fix Angular test failures in trade-blotter component
- Fix trade-service integration tests (expectation order, 5xx error handling, mock server setup with RestTemplate injection in all tests)

All services should now pass in GitHub Actions CI/CD pipeline.
- Replaced MockRestServiceServer with Mockito mocks for better reliability
- Matches the approach used in TradeOrderControllerTest
- All 6 integration tests should now pass
@netlify
Copy link

netlify bot commented Nov 27, 2025

Deploy Preview for lucky-concha-f3599f canceled.

Name Link
🔨 Latest commit 5437ea6
🔍 Latest deploy log https://app.netlify.com/projects/lucky-concha-f3599f/deploys/692a0b36a4f1580008bd1de6

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Nov 27, 2025

CLA Missing ID CLA Not Signed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant