test: add comprehensive integration tests for ModelRegistry module

[ooples]

Summary

• Adds 54 comprehensive integration tests for the ModelRegistry module
• Tests run on both .NET 10.0 and .NET Framework 4.7.1
• All tests pass

Test Coverage

Category	Tests	Description
Constructor	3	Directory creation, default path, initialization
RegisterModel	6	Basic registration, tags, validation, versioning
CreateModelVersion	3	New versions, non-existent model, descriptions
GetModel	4	Existing model, specific version, error cases
GetLatestModel	1	Returns highest version
GetModelByStage	3	By stage, no model in stage, non-existent
TransitionModelStage	3	Stage updates, auto-archive, no-archive
ListModels	3	All models, filter, tags
ListModelVersions	2	All versions, non-existent
SearchModels	4	By name, stage, version range, null criteria
UpdateModelMetadata	2	Updates, null validation
UpdateModelTags	2	Add tags, null validation
DeleteModelVersion	3	Specific version, last version, non-existent
DeleteModel	2	All versions, non-existent
CompareModels	2	Different metadata, same metadata
GetModelLineage	1	Returns lineage info
ArchiveModel	1	Sets archived stage
GetModelStoragePath	1	Returns valid path
ModelCard	6	Attach, null validation, get, generate, save
Persistence	2	Cross-instance, version loading

Test Results

.NET 10.0:     Passed: 54, Failed: 0
.NET 4.7.1:    Passed: 54, Failed: 0

Key Test Patterns

• Uses temp directories for isolated file system tests
• Proper cleanup with IDisposable
• Tests for thread-safe operations
• Validation of error conditions
• Mock model implementation for testing

Closes #664

🤖 Generated with Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Review	Updated (UTC)
aidotnet-playground-api	Ready	Preview, Comment	Jan 24, 2026 2:30am

[coderabbitai]

Summary by CodeRabbit

• Tests
  • Added comprehensive integration test suite for model registry functionality, validating model registration, version management, retrieval, lifecycle transitions, metadata updates, error handling, and data persistence.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

This pull request introduces a comprehensive integration test suite for the ModelRegistry module, covering model registration, version management, metadata handling, model retrieval, lifecycle transitions, persistence, and error handling scenarios across 1025 lines of test code.

Changes

Cohort / File(s)

Summary

ModelRegistry Integration Tests tests/AiDotNet.Tests/IntegrationTests/ModelRegistry/ModelRegistryIntegrationTests.cs

Adds ModelRegistryIntegrationTests class with IDisposable implementation for test lifecycle management (temp directory setup/cleanup). Includes test regions for constructor behavior, model registration with tagging and null/empty validation, version creation/management, model retrieval by name/version/stage, lifecycle transitions (production/archived states), persistence verification across instances, error handling for invalid inputs, model card operations, and metadata/tag updates. Introduces MockModel helper class (implementing IModel<double[], double, object>) and CreateTestMetadata() utility method for test fixtures.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested labels

feature

Poem

🐰 A registry of models, now tested with care,
Integration tests woven with versions to spare,
From registration to lifecycle, each scenario's clear,
With mocks and with metadata, persisting right here! 🌟

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately and concisely describes the main change: adding comprehensive integration tests for the ModelRegistry module.
Description check	✅ Passed	The description is clearly related to the changeset, providing detailed test coverage breakdown, test results, and confirmation that all 54 tests pass on both .NET versions.
Linked Issues check	✅ Passed	The PR addresses all coding objectives from #664: provides integration tests for ModelRegistry covering model registration, version management, model retrieval, and metadata handling through 54 comprehensive test cases.
Out of Scope Changes check	✅ Passed	All changes are directly within scope—the PR adds only the ModelRegistryIntegrationTests class with supporting test methods and helpers, all focused on testing the ModelRegistry module as specified in #664.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch test/modelregistry-integration-tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud