Skip to content

Add tests#125

Open
jjasloot wants to merge 1 commit into
masterfrom
add-tests
Open

Add tests#125
jjasloot wants to merge 1 commit into
masterfrom
add-tests

Conversation

@jjasloot
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings September 30, 2025 22:34
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds comprehensive test coverage to the OVDB project, including both unit tests and integration tests for the ASP.NET Core web application.

  • Adds unit tests for controllers, services, and helper classes
  • Implements integration tests with real database using TestContainers
  • Sets up CI pipeline to run tests automatically

Reviewed Changes

Copilot reviewed 53 out of 65 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
docker-compose.integration-tests.yml Docker Compose configuration for integration test database
OV_DB/Startup.cs Adds conditional SPA configuration to support testing environment
OV_DB/Services/TimezoneService.cs Adds null/invalid LineString validation
OV_DB.Tests/* Comprehensive unit tests for all major components
OV_DB.IntegrationTests/* Integration test infrastructure and API endpoint tests
.github/workflows/ci.yml Extended CI pipeline with test execution
Comments suppressed due to low confidence (2)

OV_DB.Tests/Services/RouteRegionsServiceTests.cs:1

  • Property name contains a spelling error. It should be 'Latitude' not 'Lattitude' (single 't').
using Microsoft.EntityFrameworkCore;

OV_DB.IntegrationTests/Infrastructure/TestDataGenerator.cs:1

  • Property name contains a spelling error. It should be 'Latitude' not 'Lattitude' (single 't').
using Bogus;

Comment thread OV_DB.Tests/Services/TrawellingServiceTests.cs
Comment thread OV_DB.Tests/Services/StationRegionsServiceTests.cs
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.

2 participants