Skip to content

fix: fixed copilot suggestions#794

Merged
Roopan-Microsoft merged 1 commit into
dev-v4from
psl-codefix
Feb 3, 2026
Merged

fix: fixed copilot suggestions#794
Roopan-Microsoft merged 1 commit into
dev-v4from
psl-codefix

Conversation

@Kanchan-Microsoft

@Kanchan-Microsoft Kanchan-Microsoft commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

Purpose

This pull request introduces unit tests for the auth_utils.py module, improves test and coverage configuration, and makes minor code cleanups across the backend and frontend. The most significant changes are the addition of comprehensive authentication utility tests, configuration of code coverage, and some workflow and code simplifications.

Authentication Testing and Coverage Improvements:

  • Added new unit tests for get_authenticated_user_details and get_tenantid in src/tests/backend/auth/test_auth_utils.py, covering a variety of normal and edge cases, including error handling and integration scenarios.
  • Created supporting test fixtures in src/tests/backend/auth/conftest.py for reusable test data and mocks.
  • Added __init__.py to src/tests/backend/auth to ensure it is recognized as a package.

Test and Coverage Configuration:

  • Added a .coveragerc file to configure code coverage, specifying source directories and omit patterns for irrelevant files and folders.
  • Updated the GitHub Actions workflow (.github/workflows/test.yml) to use the new coverage configuration, enforce a minimum 80% coverage threshold, and adjust branch and path filters. [1] [2] [3]

Backend and Frontend Code Cleanups:

  • Added missing __init__.py files to src and src/backend for package recognition. [1] [2]
  • Removed commented-out or unused code in backend service and agent lifecycle files to improve readability. [1] [2]
  • Minor frontend code cleanups, such as removing unnecessary imports and simplifying logic in components. [1] [2] [3] [4] [5]
  • Fixed missing return statements in backend API endpoints for consistency. [1] [2]

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft Roopan-Microsoft merged commit b92a822 into dev-v4 Feb 3, 2026
3 checks passed
@github-actions

github-actions Bot commented Mar 2, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 4.0.7 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Roopan-Microsoft Roopan-Microsoft Roopan-Microsoft approved these changes

@Avijit-Microsoft Avijit-Microsoft Awaiting requested review from Avijit-Microsoft Avijit-Microsoft is a code owner

@Prajwal-Microsoft Prajwal-Microsoft Awaiting requested review from Prajwal-Microsoft

@marktayl1 marktayl1 Awaiting requested review from marktayl1 marktayl1 is a code owner

@Fr4nc3 Fr4nc3 Awaiting requested review from Fr4nc3 Fr4nc3 is a code owner

@Vinay-Microsoft Vinay-Microsoft Awaiting requested review from Vinay-Microsoft

@aniaroramsft aniaroramsft Awaiting requested review from aniaroramsft aniaroramsft is a code owner

@toherman-msft toherman-msft Awaiting requested review from toherman-msft toherman-msft is a code owner

@nchandhi nchandhi Awaiting requested review from nchandhi nchandhi is a code owner

@dgp10801 dgp10801 Awaiting requested review from dgp10801 dgp10801 is a code owner

Assignees

No one assigned

Labels

released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Kanchan-Microsoft @Roopan-Microsoft