Skip to content

fix: fixed new suggestions by copilot#796

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

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

Conversation

@Kanchan-Microsoft

Copy link
Copy Markdown
Contributor

Purpose

This pull request makes minor cleanups to the test codebase, mostly focused on import statements and class inheritance for async test cases. The changes improve code consistency and remove unused imports.

Test code consistency and cleanups:

  • Replaced direct imports of IsolatedAsyncioTestCase with unittest.IsolatedAsyncioTestCase in several test classes to standardize async test case inheritance. [1] [2] [3] [4]
  • Removed unused or redundant imports from multiple test files, including asyncio, unittest.mock, and typing-related imports. [1] [2] [3] [4] [5]
  • Ensured that unittest.main() is called correctly in test_human_approval_manager.py without redundant imports.

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

@Kanchan-Microsoft Kanchan-Microsoft changed the title fix: fixed new suggestions fix: fixed new suggestions by copilot Feb 3, 2026
@Roopan-Microsoft Roopan-Microsoft merged commit ff6e18d into dev-v4 Feb 3, 2026
4 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 📦🚀

@Roopan-Microsoft Roopan-Microsoft deleted the psl-codefix branch March 4, 2026 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants