Skip to content

Pytest/utils #1269

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

Merged
PGijsbers merged 6 commits into from
Sep 20, 2023
Merged

Pytest/utils #1269

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
b61566a
Extract mocked_perform_api_call because its independent of object
PGijsbers Jul 8, 2023
3231f66
Remove _multiprocess_can_split_ as it is a nose directive
PGijsbers Jul 8, 2023
817e150
Convert test list all
PGijsbers Jul 8, 2023
8326b7f
Add markers and refactor test_list_all_for_tasks for pytest
PGijsbers Jul 9, 2023
cbc93dc
Add cache marker
PGijsbers Jul 9, 2023
bd7db3a
Converted remainder of tests to pytest
PGijsbers Jul 9, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove _multiprocess_can_split_ as it is a nose directive 
and we use pytest
  • Loading branch information
@PGijsbers
PGijsbers committed Jul 8, 2023
commit 3231f667834cd22e30db64dc0ac1e40690e1030b
2 changes: 0 additions & 2 deletions tests/test_utils/test_utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,6 @@ def mocked_perform_api_call(call, request_method):


class OpenMLTaskTest(TestBase):
_multiprocess_can_split_ = True
Comment thread
PGijsbers marked this conversation as resolved.

def test_list_all(self):
openml.utils._list_all(listing_call=openml.tasks.functions._list_tasks)
openml.utils._list_all(
Expand Down