Fix and clean up the tests implemented in python #35

Merged
darthmama merged 9 commits from htgoebel/pEpEngine:fix-python-tests into master 2 years ago

Most relevant change is:

88686b8cef tests: python: Make test_handshake actually work.

Most relevant change is: 88686b8cef tests: python: Make test_handshake actually work.
htgoebel added 9 commits 2 years ago
09143dce40 tests: Simplify setup.
From gtest-parallel only a single file is used, which is not even required to
be installed in any way. Thus instead of asking the users to clone the repo,
simply download the one file as part of the Makefile.
6925e36839 tests: Fix running python tests.
This was hard-coded to only seach "test_handshake", thus inhibiting running
other test suites.
Also remove options specific to pytest-xdist, to be able to move to pytest-mp.

fixup! tests: Fix running python tests.
88686b8cef tests: python: Make test_handshake actually work.
This test misused parallel test-processing of pytest-xdist to run two
processes in parallel - while these are actually interactig and form one
test-case. Fixed by forming one test-case which spawns two processes.
fe2f9c0179 tests: python: Don't link paths from real home into dummy home.
It is completely unclear, why this is done. There is not even a comment
explaining this. But it has unwanted side-effects like making pytest search
all tests from ~/.local/lib/python*/site-packages.
So: no reason, but harmful -> remove.
htgoebel changed title from This fixes and cleans up the tests implemented in python to Fix and clean up the tests implemented in python 2 years ago
darthmama merged commit 2173bba0a8 into master 2 years ago
The pull request has been merged as 2173bba0a8.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b htgoebel-fix-python-tests master
git pull fix-python-tests

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff htgoebel-fix-python-tests
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: pEp.foundation/pEpEngine#35
Loading…
There is no content yet.