test: Add unit tests for backend-api by Shreyas-Microsoft · Pull Request #211 · microsoft/Container-Migration-Solution-Accelerator

Shreyas-Microsoft · 2026-04-29T07:00:08Z

Purpose

This pull request adds comprehensive unit tests for several core backend components, focusing on application initialization, dependency injection, Azure App Configuration integration, FastAPI extensions, kernel agent logic, and model entities. These tests ensure correct instantiation, lifecycle management, and integration of services and models, greatly improving the reliability and maintainability of the codebase.

Application Initialization and Dependency Injection:

Added tests for the Application class to verify correct FastAPI setup, context assignment, core service registration, and router inclusion.
Introduced extensive tests for AppContext and related dependency injection features, covering service lifetimes (singleton, scoped, async), scope management, async resource cleanup, and shutdown behavior.

Azure App Configuration Integration:

Implemented tests for AppConfigurationHelper to ensure correct credential handling, endpoint validation, environment variable population from configuration, and delegation to the underlying Azure client.

FastAPI Extensions:

Added tests for TypedFastAPI, verifying context assignment, FastAPI compatibility, and utility functions for attaching application context.

Kernel Agent Logic:

Provided thorough tests for kernel_agent, including initialization, plugin and function management, service registration, error handling, and service type enumeration.

Model Entities:

Added tests for model classes such as Process, File, AgentStatus, ProcessStatus, and ProcessStartQueueMessage, ensuring correct default values, field assignments, and serialization logic.

Does this introduce a breaking change?

Yes
No

Golden Path Validation

I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

...

Other Information

github-actions · 2026-04-29T07:01:43Z

Coverage Report •

File	Stmts	Miss	Cover	Missing
TOTAL	3097	208	93%

report-only-changed-files is enabled. No files were changed during this commit :)

Tests	Skipped	Failures	Errors	Time
588	0 💤	0 ❌	0 🔥	22.518s ⏱️

github-actions · 2026-04-29T10:20:30Z

Processor Coverage Report •

File	Stmts	Miss	Cover	Missing
TOTAL	5726	719	87%

report-only-changed-files is enabled. No files were changed during this commit :)

Tests	Skipped	Failures	Errors	Time
812	0 💤	0 ❌	0 🔥	19.402s ⏱️

Copilot

Pull request overview

Copilot reviewed 50 out of 57 changed files in this pull request and generated 4 comments.

Generate a text-format summary with only the TOTAL line from coverage.xml. Use pytest-coverage-path instead of pytest-xml-coverage-path so the Coverage Report dropdown shows only TOTAL row without per-file listing. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Fix test_returns_false_for_none to call is_valid_uuid directly - Fix inaccurate comment about non-existent env file path - Make test_get_service_status_not_initialized deterministic with mock - Exclude test files from processor coverage (tool.coverage.run.omit) - Regenerate uv.lock with pytest-asyncio Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 51 out of 59 changed files in this pull request and generated 5 comments.

+import importlib
+from types import SimpleNamespace
+from unittest.mock import MagicMock, patch
+
+import pytest
+from pydantic import BaseModel, ValidationError
+from semantic_kernel.exceptions.service_exceptions import ServiceInitializationError
+
+# The source module imports SKBaseModel from libs.base.SKBase, but that module
+# is empty in the repository. Inject a minimal stand-in before importing
+# kernel_agent so tests can exercise the file without touching source.
+import libs.base.SKBase as _skbase_mod  # noqa: E402
+
+if not hasattr(_skbase_mod, "SKBaseModel"):
+
+    class _SKBaseModelStub(BaseModel):
+        model_config = {"arbitrary_types_allowed": True}
+
+    _skbase_mod.SKBaseModel = _SKBaseModelStub  # type: ignore[attr-defined]
+
+importlib.import_module("libs.base.kernel_agent")  # noqa: E402
+
+


+    def test_local_environment_recommendations(self):
+        cred = MagicMock()
+        cred.__class__.__name__ = "AzureCliCredential"
+        with patch.object(credential_util, "get_azure_credential", return_value=cred):
+            info = credential_util.validate_azure_authentication()
+            assert info["environment"] == "local_development"
+            assert info["status"] == "configured"


+def _run(coro):
+    return asyncio.get_event_loop().run_until_complete(coro) if False else asyncio.run(coro)
+


+          python -c "
+          import xml.etree.ElementTree as ET
+          tree = ET.parse('src/backend-api/reports/coverage.xml')
+          root = tree.getroot()
+          stmts = int(root.attrib.get('lines-valid', 0))
+          miss = stmts - int(root.attrib.get('lines-covered', 0))
+          cover = int(float(root.attrib.get('line-rate', 0)) * 100)
+          with open('src/backend-api/reports/coverage-summary.txt', 'w') as f:
+              f.write('Name     Stmts   Miss  Cover\n')
+              f.write(f'TOTAL    {stmts}    {miss}    {cover}%\n')
+          "


+          python -c "
+          import xml.etree.ElementTree as ET
+          tree = ET.parse('src/processor/reports/coverage.xml')
+          root = tree.getroot()
+          stmts = int(root.attrib.get('lines-valid', 0))
+          miss = stmts - int(root.attrib.get('lines-covered', 0))
+          cover = int(float(root.attrib.get('line-rate', 0)) * 100)
+          with open('src/processor/reports/coverage-summary.txt', 'w') as f:
+              f.write('Name     Stmts   Miss  Cover\n')
+              f.write(f'TOTAL    {stmts}    {miss}    {cover}%\n')
+          "


Use pytest-xml-coverage-path with coverage-path-prefix so the action correctly resolves XML paths (relative to subdir) against PR diff paths. Combined with report-only-changed-files: true, when no source files are in the diff this shows: badge + Coverage Report dropdown with TOTAL row + 'No files were changed' note + test summary table. Matches the format used in Multi-Agent-Custom-Automation-Engine PR #961. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 51 out of 59 changed files in this pull request and generated 4 comments.

+import importlib
+from types import SimpleNamespace
+from unittest.mock import MagicMock, patch
+
+import pytest
+from pydantic import BaseModel, ValidationError
+from semantic_kernel.exceptions.service_exceptions import ServiceInitializationError
+
+# The source module imports SKBaseModel from libs.base.SKBase, but that module
+# is empty in the repository. Inject a minimal stand-in before importing
+# kernel_agent so tests can exercise the file without touching source.
+import libs.base.SKBase as _skbase_mod  # noqa: E402
+
+if not hasattr(_skbase_mod, "SKBaseModel"):
+
+    class _SKBaseModelStub(BaseModel):
+        model_config = {"arbitrary_types_allowed": True}
+
+    _skbase_mod.SKBaseModel = _SKBaseModelStub  # type: ignore[attr-defined]
+
+importlib.import_module("libs.base.kernel_agent")  # noqa: E402
+
+


+        uses: actions/checkout@v5
+
+      - name: Set up Python
+        uses: actions/setup-python@v6


+          prefix = "src/backend-api/src/app/"
+          tree = ET.parse(path)
+          root = tree.getroot()
+          for cls in root.iter("class"):
+              fname = cls.attrib.get("filename", "")
+              if fname and not fname.startswith(prefix):
+                  cls.attrib["filename"] = prefix + fname


+          prefix = "src/processor/src/"
+          tree = ET.parse(path)
+          root = tree.getroot()
+          for cls in root.iter("class"):
+              fname = cls.attrib.get("filename", "")
+              if fname and not fname.startswith(prefix):
+                  cls.attrib["filename"] = prefix + fname


Processor: 75% -> 87.44% (+199 tests across 5 files) - queue_service.py: 35% -> 86% - groupchat_orchestrator.py: 25% -> ~70%+ (helpers fully covered) - mcp_mermaid.py: 52% -> 99% - application_base.py: 0% -> 100% - main.py: 0% -> 94% Backend-api: 82.27% -> 93.28% (+125 tests across 7 files) - SKLogicBase.py: 0% -> 93% - blob/helper.py: 68% -> 99% - blob/async_helper.py: 72% -> 97% - blob/config.py: 70% -> 100% - shared_config.py: 67% -> 100% - services/interfaces.py: 74% -> 100% - file_repository.py / process_repository.py: 67% -> 100% CI: add --cov-fail-under=82 to both backend and processor jobs in .github/workflows/test.yml so the pipeline fails when coverage drops below 82%. Backend pyproject.toml: add [tool.coverage.run] omit list for parity with processor so local pytest runs (which use pytest.ini's --cov=src) do not inflate coverage with test-file lines. No production code modified, no coverage exclusions added beyond the test directory, all 1077 prior + new tests pass on both services. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Add unit tests for backend-api

8a0ae2b

Shreyas-Microsoft requested review from Avijit-Microsoft, Dongbumlee, Prajwal-Microsoft, Roopan-Microsoft, Vinay-Microsoft, aniaroramsft, dgp10801, nchandhi, sethsteenken and toherman-msft as code owners April 29, 2026 07:00

Shreyas-Microsoft temporarily deployed to production April 29, 2026 07:00 — with GitHub Actions Inactive

Shreyas-Microsoft changed the base branch from main to dev April 29, 2026 07:00

github-code-quality Bot found potential problems Apr 29, 2026

View reviewed changes

add asyncio package

362ac1f

Shreyas-Microsoft temporarily deployed to production April 29, 2026 07:18 — with GitHub Actions Inactive

Shreyas-Microsoft had a problem deploying to production April 29, 2026 07:19 — with GitHub Actions Failure

Shreyas-Microsoft had a problem deploying to production April 29, 2026 07:54 — with GitHub Actions Failure

Add processor tests

98ee6f2

Shreyas-Microsoft temporarily deployed to production April 29, 2026 10:19 — with GitHub Actions Inactive

Shreyas-Microsoft had a problem deploying to production April 29, 2026 10:19 — with GitHub Actions Failure

update failing tests

431830e

Shreyas-Microsoft temporarily deployed to production April 29, 2026 10:25 — with GitHub Actions Inactive

Shreyas-Microsoft had a problem deploying to production April 29, 2026 10:26 — with GitHub Actions Failure

add more coverage to processor

3bbe8e8

Copilot started reviewing on behalf of Shreyas-Microsoft May 8, 2026 05:36 View session

Shreyas-Microsoft temporarily deployed to production May 8, 2026 05:36 — with GitHub Actions Inactive

Copilot AI reviewed May 8, 2026

View reviewed changes

Comment thread src/backend-api/src/tests/services/test_input_validation.py Outdated

Comment thread src/backend-api/pyproject.toml

Comment thread src/processor/src/tests/unit/libs/agent_framework/test_agent_framework_settings.py

Comment thread .github/workflows/test.yml

Shreyas-Microsoft temporarily deployed to production May 8, 2026 05:45 — with GitHub Actions Inactive

Shreyas-Microsoft temporarily deployed to production May 8, 2026 05:59 — with GitHub Actions Inactive

Shreyas-Microsoft temporarily deployed to production May 8, 2026 06:00 — with GitHub Actions Inactive

Shreyas-Microsoft temporarily deployed to production May 8, 2026 06:08 — with GitHub Actions Inactive

Copilot AI review requested due to automatic review settings May 8, 2026 06:46

Shreyas-Microsoft temporarily deployed to production May 8, 2026 06:46 — with GitHub Actions Inactive

Shreyas-Microsoft temporarily deployed to production May 8, 2026 06:47 — with GitHub Actions Inactive

Copilot started reviewing on behalf of Shreyas-Microsoft May 8, 2026 06:47 View session

Copilot AI reviewed May 8, 2026

View reviewed changes

Shreyas-Microsoft temporarily deployed to production May 8, 2026 06:55 — with GitHub Actions Inactive

Shreyas-Microsoft temporarily deployed to production May 8, 2026 07:47 — with GitHub Actions Inactive

Shreyas-Microsoft temporarily deployed to production May 8, 2026 07:56 — with GitHub Actions Inactive

pushing test.yml

d86bcdc

Copilot AI review requested due to automatic review settings May 8, 2026 10:27

Shreyas-Microsoft temporarily deployed to production May 8, 2026 10:27 — with GitHub Actions Inactive

Shreyas-Microsoft had a problem deploying to production May 8, 2026 10:28 — with GitHub Actions Failure

Copilot started reviewing on behalf of Shreyas-Microsoft May 8, 2026 10:28 View session

Copilot AI reviewed May 8, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

test: Add unit tests for backend-api#211

test: Add unit tests for backend-api#211
Shreyas-Microsoft wants to merge 16 commits intodevfrom
psl-unit-testing

Shreyas-Microsoft commented Apr 29, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Apr 29, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Apr 29, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		def _run(coro):
		return asyncio.get_event_loop().run_until_complete(coro) if False else asyncio.run(coro)

Conversation

Shreyas-Microsoft commented Apr 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Purpose

Does this introduce a breaking change?

Golden Path Validation

Deployment Validation

What to Check

Other Information

Uh oh!

github-actions Bot commented Apr 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Apr 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Shreyas-Microsoft commented Apr 29, 2026 •

edited

Loading

github-actions Bot commented Apr 29, 2026 •

edited

Loading

github-actions Bot commented Apr 29, 2026 •

edited

Loading