Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Retry test_psu.py::TestPsuApi::test_power if power not within tolerance #14788

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Retry test_psu.py::TestPsuApi::test_power if power not within tolerance #14788

wants to merge 2 commits into from
+70 −60

Commits on Sep 27, 2024

  1. Retry test_power calculated power above 10% 

    If the test fails on `abs(power - (voltage*current)) < power*0.1` then retry the test. Test is repeated for a maximum of three times until it passes, else it is failed.
This change is introduced to account for the stringent tolerance level of 10% and the errors that might be caused due to time differences reading each of the parameters.
    @smagarwal-arista
    smagarwal-arista committed Sep 27, 2024
    Configuration menu
    Copy the full SHA
    b817d8e View commit details
    Browse the repository at this point in the history

  2. Retry test_power.py::TestPsuApi::test_power if power above tolerance 

    If the test fails on `abs(power - (voltage*current)) < power*0.1` then retry the test. The test is repeated for a maximum of three times until it passes.
This change is introduced to account for the stringent tolerance level of 10% and the errors that might be caused due to time differences in reading each of the parameters.
    @smagarwal-arista
    smagarwal-arista committed Sep 27, 2024
    Configuration menu
    Copy the full SHA
    e1aa4d5 View commit details
    Browse the repository at this point in the history