Skip to content

Conversation

Javier-Tan
Copy link
Contributor

Description of PR

Summary:
Fixes #20994

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

Fix syntax/bracket error causing test failures. See details in issue.

How did you do it?

Move bracket to correct place and format the code to make it readable.

How did you verify/test it?

Ran the test to make sure the error no longer occured

Any platform specific information?

N/A

Supported testbed topology if it's a new test case?

N/A

Documentation

N/A

* Move bracket to the correct place so .format(...) applies on string rather than pytest_assert

Signed-off-by: Javier Tan [email protected]
@Javier-Tan Javier-Tan requested a review from prgeor as a code owner October 14, 2025 00:23
@mssonicbld
Copy link
Collaborator

/azp run

@Javier-Tan Javier-Tan requested a review from arlakshm October 14, 2025 00:23
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Javier-Tan Javier-Tan changed the title [test_show_chassis_module.py] Fix syntax / bracket issue for format [test_show_chassis_module.py] Fix syntax / bracket issue for .format(...) Oct 14, 2025
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.

Bug: .format(...) called on pytest_assert rather than string within in test_show_chassis_module

2 participants