Skip to content

mock sonic_py_common.logger for integration testing#25870

Draft
yijingyan2 wants to merge 1 commit intosonic-net:masterfrom
yijingyan2:sonic-platform-modules-nextop-test
Draft

mock sonic_py_common.logger for integration testing#25870
yijingyan2 wants to merge 1 commit intosonic-net:masterfrom
yijingyan2:sonic-platform-modules-nextop-test

Conversation

@yijingyan2
Copy link
Contributor

Why I did it

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Yijing Yan <yijingyan@microsoft.com>
@yijingyan2 yijingyan2 requested a review from lguohan as a code owner March 4, 2026 02:58
Copilot AI review requested due to automatic review settings March 4, 2026 02:58
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@yijingyan2 yijingyan2 marked this pull request as draft March 4, 2026 02:58
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Nexthop platform integration test harness to mock sonic_py_common.logger so the sonic_platform module tree can be imported in integration tests without requiring the real sonic_py_common package/modules to be present.

Changes:

  • Add sonic_py_common.logger to the sys.modules patch set used by the integration test fixture.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants