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.

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

Fix 'Chassis' object has no attribute 'get_num_psu' in test_psu.py #35

Merged
merged 1 commit into from
Jan 30, 2024

Conversation

nonodark
Copy link
Contributor

Why I did it

Fix 'Chassis' object has no attribute 'get_num_psu' in test_psu.py

How I did it

Update 'platform_chassis.get_num_psu()' to 'platform_chassis.get_num_psus()'

How to verify it

root@sonic:~# pde-test -v test_psu.py
==================================================================== test session starts ====================================================================
platform linux -- Python 3.9.2, pytest-7.4.4, pluggy-1.3.0 -- /usr/bin/python3
cachedir: .pytest_cache
rootdir: /usr/local/sonic_pde_tests
collected 13 items                                                                                                                                          

test_psu.py::test_for_num_psus PASSED                                                                                                                 [  7%]
test_psu.py::test_for_psu_present PASSED                                                                                                              [ 15%]
test_psu.py::test_for_psu_notpresent PASSED                                                                                                           [ 23%]
test_psu.py::test_for_psu_status PASSED                                                                                                               [ 30%]
test_psu.py::test_for_psu_serial_num PASSED                                                                                                           [ 38%]
test_psu.py::test_for_psu_model PASSED                                                                                                                [ 46%]
test_psu.py::test_for_psu_voltage PASSED                                                                                                              [ 53%]
test_psu.py::test_for_psu_current PASSED                                                                                                              [ 61%]
test_psu.py::test_for_psu_power PASSED                                                                                                                [ 69%]
test_psu.py::test_for_psu_input_voltage PASSED                                                                                                        [ 76%]
test_psu.py::test_for_psu_input_current PASSED                                                                                                        [ 84%]
test_psu.py::test_for_psu_capacity PASSED                                                                                                             [ 92%]
test_psu.py::test_for_psu_type PASSED                                                                                                                 [100%]

===================================================================== warnings summary ======================================================================
test_psu.py:3
  /usr/local/sonic_pde_tests/test_psu.py:3: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
    import imp

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=============================================================== 13 passed, 1 warning in 1.37s ===============================================================

@nonodark
Copy link
Contributor Author

@geans-pin pls help review, thanks

@geans-pin
Copy link
Contributor

Approved

@nonodark
Copy link
Contributor Author

@geans-pin @lizhuhuams @qiluo-msft please help merge, thanks.

@geans-pin geans-pin merged commit f2cc748 into sonic-net:master Jan 30, 2024
3 checks passed
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.

2 participants