Skip to content

Commit

Permalink
Fix tests
Browse files Browse the repository at this point in the history
  • Loading branch information
fraimondo committed Apr 24, 2024
1 parent 7719120 commit c187b80
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -104,3 +104,12 @@ def test_merger_errors() -> None:
match="must have the same named steps.",
):
merge_pipelines(pipe1, pipe4, search_params=None)

search_params = {"kind": "grid"}
pipe5 = creator2.to_pipeline(search_params={"kind": "bayes"})

with pytest.raises(
ValueError,
match="one of the pipelines to merge is a BayesSearchCV",
):
merge_pipelines(pipe1, pipe5, search_params=search_params)

Check failure on line 115 in julearn/pipeline/tests/test_merger.py

View workflow job for this annotation

GitHub Actions / lint

Ruff (W292)

julearn/pipeline/tests/test_merger.py:115:67: W292 No newline at end of file

Check failure on line 115 in julearn/pipeline/tests/test_merger.py

View workflow job for this annotation

GitHub Actions / lint

Ruff (W292)

julearn/pipeline/tests/test_merger.py:115:67: W292 No newline at end of file

0 comments on commit c187b80

Please sign in to comment.