The Wayback Machine - https://web.archive.org/web/20201125012005/https://github.com/giampaolo/psutil/pull/1769
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

Sort results in test_cpu_affinity_all_combinations #1769

Merged
merged 1 commit into from May 26, 2020

Conversation

@mgorny
Copy link
Contributor

@mgorny mgorny commented May 26, 2020

Fix test_cpu_affinity_all_combinations to permit any CPU order
in results. This fixes test failure due to affinity being reported
out of order:

======================================================================
FAIL: psutil.tests.test_process.TestProcess.test_cpu_affinity_all_combinations
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/psutil/psutil/tests/test_process.py", line 940, in test_cpu_affinity_all_combinations
    self.assertEqual(p.cpu_affinity(), combo)
AssertionError: Lists differ: [8, 1] != [1, 8]

First differing element 0:
8
1

- [8, 1]
+ [1, 8]

----------------------------------------------------------------------
Fix test_cpu_affinity_all_combinations to permit any CPU order
in results.  This fixes test failure due to affinity being reported
out of order:

    ======================================================================
    FAIL: psutil.tests.test_process.TestProcess.test_cpu_affinity_all_combinations
    ----------------------------------------------------------------------
    Traceback (most recent call last):
      File "/tmp/psutil/psutil/tests/test_process.py", line 940, in test_cpu_affinity_all_combinations
        self.assertEqual(p.cpu_affinity(), combo)
    AssertionError: Lists differ: [8, 1] != [1, 8]

    First differing element 0:
    8
    1

    - [8, 1]
    + [1, 8]

    ----------------------------------------------------------------------
@giampaolo giampaolo merged commit 3c209c1 into giampaolo:master May 26, 2020
6 checks passed
6 checks passed
macos-latest
Details
ubuntu-latest
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
freebsd_11_py2 Task Summary
Details
freebsd_13_py3 Task Summary
Details
@mgorny
Copy link
Contributor Author

@mgorny mgorny commented May 26, 2020

Thank you!

@mgorny mgorny deleted the mgorny:cpu-affi-combos branch May 26, 2020
@giampaolo giampaolo added the tests label Nov 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.