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

Update unit tests: test_launcher retrocompatibility, test_busy_port and test_list on Linux #323

Merged
merged 8 commits into from
Jul 22, 2022

Conversation

anslpa
Copy link
Contributor

@anslpa anslpa commented Jul 21, 2022

Update with internal development changes - 21/07/22.

Fix the failing tests on Linux (test_python_plugins::test_list).
Add an rate acceptance criteria in conftest (set to 0).
Update unit test test_launcher.py that they work with CPython 3.7.13 (issue with os.unsetenv method that does not exist on Windows).

…ilure rate for Linux runs

An acceptance failure rate for Linux has been added to enable the tests to run in the pipelines. This will prevent from having troubles at package publication. I can remove that failure acceptance rate when setting the InProcess as default (fixing unit tests).

Use unitTestDataFiles from this repository updating the conftest.py file with local_test_repo = False.

Related work items: #674825
…on version 3.7.13

Update unit tests that they work with Windows CPython version 3.7.13 (no problem on Linux).

Add conditions that the unit tests run with v221 version.

Black formatting to be flake8 compliant.

Related work items: #681810
…on-core into anslpa/merge_internal_changes_2107

# Conflicts:
#	tests/testfiles/pythonPlugins/all_types/integral_types_op.py
@anslpa anslpa changed the title Update with internal development changes - 21/07/22 Update unit tests: test_launcher retrocompatibility, test_busy_port and test_list on Linux Jul 22, 2022
@anslpa anslpa merged commit c188985 into merge/0.5.dev2 Jul 22, 2022
@anslpa anslpa deleted the anslpa/merge_internal_changes_2107 branch July 22, 2022 10:18
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