Skip to content

Commit

Permalink
Merge pull request #572 from GAA-UAM/fix/pandas_tests
Browse files Browse the repository at this point in the history
Fix Pandas tests with boolean reductions.
  • Loading branch information
vnmabus committed Sep 20, 2023
2 parents dfbce35 + dd7758e commit 55a7e99
Show file tree
Hide file tree
Showing 2 changed files with 30 additions and 2 deletions.
16 changes: 15 additions & 1 deletion skfda/tests/test_pandas_fdatabasis.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
from __future__ import annotations

from typing import Any, Callable, Iterable, NoReturn
from typing import Any, Callable, Iterable, NoReturn, Sequence

import numpy as np
import pandas
Expand Down Expand Up @@ -271,6 +271,20 @@ def all_numeric_reductions(request: Any) -> Any:
"""
return request.param


_all_boolean_reductions: Sequence[str] = [
# "all",
# "any",
]


@pytest.fixture(params=_all_boolean_reductions)
def all_boolean_reductions(request: Any) -> Any:
"""
Fixture for boolean reduction names.
"""
return request.param

##############################################################################
# Tests
##############################################################################
Expand Down
16 changes: 15 additions & 1 deletion skfda/tests/test_pandas_fdatagrid.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
from __future__ import annotations

from typing import Any, Callable, Generator, NoReturn, Union
from typing import Any, Callable, Generator, NoReturn, Sequence, Union

import numpy as np
import pandas
Expand Down Expand Up @@ -290,6 +290,20 @@ def all_numeric_reductions(request: Any) -> Any:
"""
return request.param


_all_boolean_reductions: Sequence[str] = [
# "all",
# "any",
]


@pytest.fixture(params=_all_boolean_reductions)
def all_boolean_reductions(request: Any) -> Any:
"""
Fixture for boolean reduction names.
"""
return request.param

##############################################################################
# Tests
##############################################################################
Expand Down

0 comments on commit 55a7e99

Please sign in to comment.