Skip to content

Use unittest.mock instead of mock in tests - #2775

Closed
CloudCorpRecords wants to merge 1 commit into
elastic:mainfrom
CloudCorpRecords:tree/unittest-mock-tests-2611
Closed

CloudCorpRecords wants to merge 1 commit into
elastic:mainfrom
CloudCorpRecords:tree/unittest-mock-tests-2611

Conversation

@CloudCorpRecords

Copy link
Copy Markdown

What does this pull request do?

Replaces the remaining standalone mock imports in the test suite with unittest.mock, finishing the migration started for the main package in 7241b27. Twenty-eight test files changed, import lines only, plus one log-message expectation in tests/processors/tests.py that named the old module path. No production code changes.

Related issues

Closes #2611

Finish the unittest.mock migration in the remaining test files,
matching the main package change.

Signed-off-by: Rene Turcios <95189389+CloudCorpRecords@users.noreply.github.com>
@CloudCorpRecords
CloudCorpRecords requested a review from a team as a code owner November 8, 2026 09:47
@cla-checker-service

Copy link
Copy Markdown

❌ Author of the following commits did not sign a Contributor Agreement:
012b2ab

Please, read and sign the above mentioned agreement if you want to contribute to this project

@github-actions

github-actions Bot commented Nov 8, 2026

Copy link
Copy Markdown

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@github-actions github-actions Bot added agent-python community Issues opened by the community triage Issues awaiting triage labels Nov 8, 2026
@xrmx

xrmx commented Nov 9, 2026

Copy link
Copy Markdown
Member

Thanks for the PR but you haven't signed the CLA and you didn't seem to have read the discussion on the issue you are fixing. Closing.

@xrmx xrmx closed this Nov 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-python community Issues opened by the community triage Issues awaiting triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

use unittest.mock instead of mock in tests

2 participants