Skip to content

Conversation

@samtrion
Copy link
Member

No description provided.

@samtrion samtrion requested a review from Copilot July 29, 2025 07:57
@samtrion samtrion self-assigned this Jul 29, 2025
@samtrion samtrion requested a review from a team as a code owner July 29, 2025 07:57
@samtrion samtrion requested review from Hnogared and removed request for a team July 29, 2025 07:57
Copy link
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 disables integration tests for the Devart SQL Server health check functionality due to licensing requirements. The change prevents test failures in environments where the Devart license is not available.

  • Adds a Skip attribute to the entire test class to bypass execution
Comments suppressed due to low confidence (1)

tests/NetEvolve.HealthChecks.Tests.Integration/SqlServer.Devart/SqlServerDevartHealthCheckTests.cs:15

  • Skipping all tests in this class removes test coverage for the Devart SQL Server health check functionality. Consider implementing conditional skipping based on license availability or using a mock/stub implementation for CI/CD environments.
[Skip("Devart.SqlServer requires a license.")]

@samtrion samtrion merged commit 4185604 into main Jul 29, 2025
8 checks passed
@samtrion samtrion deleted the fix/devart-license-skip branch July 29, 2025 07:57
@codecov
Copy link

codecov bot commented Jul 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.60%. Comparing base (1e0c8e1) to head (0b7eeee).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #875   +/-   ##
=======================================
  Coverage   97.60%   97.60%           
=======================================
  Files         160      160           
  Lines        2634     2634           
  Branches      446      446           
=======================================
  Hits         2571     2571           
  Misses         63       63           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@samtrion samtrion added state:ready for merge Indicates that a pull request has been reviewed and approved, and is ready to be merged into the mai type:techdebt Indicates an issue or pull request that address technical debt. labels Jul 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state:ready for merge Indicates that a pull request has been reviewed and approved, and is ready to be merged into the mai type:techdebt Indicates an issue or pull request that address technical debt.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants