deps: Consolidate dependency updates#102
Conversation
--- updated-dependencies: - dependency-name: Testcontainers.PostgreSql dependency-version: 4.10.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
--- updated-dependencies: - dependency-name: Testcontainers.FirebirdSql dependency-version: 4.10.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
--- updated-dependencies: - dependency-name: Testcontainers.MySql dependency-version: 4.10.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
* Initial plan * Plan to fix obsolete Testcontainers API calls Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com> * Fix obsolete Testcontainers API calls Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com>
--- updated-dependencies: - dependency-name: Testcontainers.MsSql dependency-version: 4.10.0 dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: Testcontainers.MsSql dependency-version: 4.10.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
* deps: Bump Testcontainers.MsSql from 4.4.0 to 4.10.0 --- updated-dependencies: - dependency-name: Testcontainers.MsSql dependency-version: 4.10.0 dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: Testcontainers.MsSql dependency-version: 4.10.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * Initial plan * Fix Testcontainers obsolete constructors to use new API Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com> * Update Testcontainers.MsSql to 4.10.0 in Sdk.IntegrationTests Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com> Co-authored-by: JD Davis <mxjerrett@gmail.com>
--- updated-dependencies: - dependency-name: Testcontainers.Oracle dependency-version: 4.10.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
…Build.Tests/Testcontainers.FirebirdSql-4.10.0' into deps/consolidated-update
…Build.Tests/multi-cb2d0bd432' into deps/consolidated-update
- Add 'partial' keyword to all test classes (required by TinyBDD 0.19.2 source generator) - Restore TinyBDD analyzer disabling target (source generator produces broken code) - Fix obsolete parameterless Testcontainers constructors (MySql, Oracle, PostgreSql) - All 858 unit tests passing
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #102 +/- ##
==========================================
+ Coverage 80.87% 88.61% +7.74%
==========================================
Files 61 61
Lines 3848 3848
Branches 558 470 -88
==========================================
+ Hits 3112 3410 +298
+ Misses 511 438 -73
+ Partials 225 0 -225
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Consolidates multiple dependency-update PRs into a single update across the SDK/build tooling and test suites, including required code tweaks for the newer Testcontainers + TinyBDD versions.
Changes:
- Updated Testcontainers packages to 4.10.0 and adjusted container setup code to use the newer “explicit image” constructor pattern.
- Updated TinyBDD.Xunit (and other NuGet deps) and modified test classes to be
partialto satisfy the updated toolchain. - Refreshed NuGet lock files for affected projects.
Reviewed changes
Copilot reviewed 69 out of 69 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/JD.Efcpt.Sdk.IntegrationTests/packages.lock.json | Updates locked versions for integration test dependencies (incl. Testcontainers). |
| tests/JD.Efcpt.Sdk.IntegrationTests/SqlGenerationIntegrationTests.cs | Updates MsSql container creation to new Testcontainers builder constructor style. |
| tests/JD.Efcpt.Sdk.IntegrationTests/JD.Efcpt.Sdk.IntegrationTests.csproj | Bumps Testcontainers.MsSql package version. |
| tests/JD.Efcpt.Build.Tests/packages.lock.json | Updates locked versions for build test suite dependencies. |
| tests/JD.Efcpt.Build.Tests/StringExtensionsTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/StageEfcptInputsTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/SqlProjectTargetGenerationTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/SqlProjectDetectorTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/SplitOutputsTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/SerializeConfigPropertiesTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Schema/SnowflakeSchemaReaderTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Schema/SchemaFingerprinterTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Schema/OracleSchemaReaderTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Schema/FirebirdSchemaReaderTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Schema/DatabaseProviderFactoryTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/RunSqlPackageTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/RunEfcptTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/ResolveSqlProjAndInputsTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/ResolveDbContextNameTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/ResolutionChainTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/RenameGeneratedFilesTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Profiling/ProfilingSecurityTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Profiling/ProfilingHelperTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Profiling/JsonTimeSpanConverterTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Profiling/InitializeBuildProfilingTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Profiling/FinalizeBuildProfilingTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Profiling/BuildRunOutputTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Profiling/BuildProfilerTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Profiling/BuildProfilerManagerTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Profiling/BuildProfilerAdditionalTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/PipelineTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/PathUtilsTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/MsBuildPropertyHelpersTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/MessageLevelHelpersTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/JD.Efcpt.Build.Tests.csproj | Updates Testcontainers/TinyBDD versions and restores analyzer-removal target. |
| tests/JD.Efcpt.Build.Tests/Integration/SqliteSchemaIntegrationTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Integration/SqlServerSchemaIntegrationTests.cs | Updates MsSql builder construction; marks class partial. |
| tests/JD.Efcpt.Build.Tests/Integration/SnowflakeSchemaIntegrationTests.cs | Updates ContainerBuilder construction; marks class partial. |
| tests/JD.Efcpt.Build.Tests/Integration/QuerySchemaMetadataIntegrationTests.cs | Updates MsSql builder construction; marks class partial. |
| tests/JD.Efcpt.Build.Tests/Integration/PostgreSqlSchemaIntegrationTests.cs | Updates PostgreSql builder construction; marks class partial. |
| tests/JD.Efcpt.Build.Tests/Integration/OracleSchemaIntegrationTests.cs | Updates Oracle builder construction; marks class partial. |
| tests/JD.Efcpt.Build.Tests/Integration/MySqlSchemaIntegrationTests.cs | Updates MySql builder construction; marks class partial. |
| tests/JD.Efcpt.Build.Tests/Integration/FirebirdSchemaIntegrationTests.cs | Updates Firebird builder construction; marks class partial. |
| tests/JD.Efcpt.Build.Tests/Integration/EndToEndReverseEngineeringTests.cs | Updates MsSql builder construction; marks class partial. |
| tests/JD.Efcpt.Build.Tests/FileSystemHelpersTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/FileHashTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/EnumerableExtensionsTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/EnsureDacpacBuiltTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/DotNetToolUtilitiesTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/DirectDacpacTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/DetectSqlProjectTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Decorators/ProfileAttributeTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/DbContextNameGeneratorTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/DataRowExtensionsTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/DacpacFingerprintTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/ConnectionStrings/ConfigurationFileTypeValidatorTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/ConnectionStrings/AppSettingsConnectionStringParserTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/ConnectionStrings/AppConfigConnectionStringParserTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/Config/EfcptConfigGeneratorTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/ComputeFingerprintTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/CleanTargetTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/CheckSdkVersionTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/BuildLogTests.cs | Marks test classes as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/ApplyConfigOverridesTests.cs | Marks test class as partial for TinyBDD update. |
| tests/JD.Efcpt.Build.Tests/AddSqlFileWarningsTests.cs | Marks test class as partial for TinyBDD update. |
| src/JD.Efcpt.Build/packages.lock.json | Updates locked dependencies across net8/net9/net10 TFMs. |
| src/JD.Efcpt.Build.Tasks/JD.Efcpt.Build.Tasks.csproj | Bumps core build-task dependencies (MSBuild, DB drivers, hashing, etc.). |
| outdated.txt | Adds dotnet outdated output snapshot. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
tests/JD.Efcpt.Build.Tests/Integration/PostgreSqlSchemaIntegrationTests.cs
Outdated
Show resolved
Hide resolved
tests/JD.Efcpt.Build.Tests/Integration/OracleSchemaIntegrationTests.cs
Outdated
Show resolved
Hide resolved
tests/JD.Efcpt.Build.Tests/Integration/MySqlSchemaIntegrationTests.cs
Outdated
Show resolved
Hide resolved
|
@copilot open a new pull request to apply changes based on the comments in this thread |
|
@JerrettDavis I've opened a new pull request, #103, to work on those changes. Once the pull request is ready, I'll request review from you. |
…103) * Initial plan * refactor: remove redundant WithImage() calls and outdated.txt Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com>
Code Coverage |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 68 out of 69 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Consolidates all 6 open dependency update PRs into a single clean PR:
Changes
Build & Test
Closes #86, closes #87, closes #88, closes #89, closes #90, closes #93