Skip to content

Commit

Permalink
fix: tests
Browse files Browse the repository at this point in the history
  • Loading branch information
ralphrass committed Jun 7, 2024
1 parent e297615 commit cc4fb54
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion butterfree/pipelines/feature_set_pipeline.py
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ def __init__(
source: Source,
feature_set: FeatureSet,
sink: Sink,
spark_client: Optional[SparkClient],
spark_client: Optional[SparkClient] = None,
):
self.source = source
self.feature_set = feature_set
Expand Down
2 changes: 1 addition & 1 deletion butterfree/transform/utils/function.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ def func(self, value: Callable) -> None:
"""Definitions to be used in the transformation."""
if value is None:
raise ValueError("Function must not be empty.")
if callable(value) is None:
if callable(value) is False:
raise TypeError("Function must be callable.")

self._func = value
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ def test_blank_aggregation(self, feature_set_dataframe):
name="feature1",
description="unit test",
transformation=AggregatedTransform(
functions=[Function(func="", data_type="")]
functions=[Function(func=None, data_type="")]
),
)

Expand Down

0 comments on commit cc4fb54

Please sign in to comment.