Skip to content

Commit 40df252

Browse files
committed
updated tests
Signed-off-by: David Y Liu <davidyliuliu@gmail.com>
1 parent dad9fda commit 40df252

File tree

3 files changed

+15
-9
lines changed

3 files changed

+15
-9
lines changed

sdk/python/tests/conftest.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ def simple_dataset_1() -> pd.DataFrame:
9999
now = datetime.utcnow()
100100
ts = pd.Timestamp(now).round("ms")
101101
data = {
102-
"id": [1, 2, 1, 3, 3],
102+
"id_join_key": [1, 2, 1, 3, 3],
103103
"float_col": [0.1, 0.2, 0.3, 4, 5],
104104
"int64_col": [1, 2, 3, 4, 5],
105105
"string_col": ["a", "b", "c", "d", "e"],
@@ -119,7 +119,7 @@ def simple_dataset_2() -> pd.DataFrame:
119119
now = datetime.utcnow()
120120
ts = pd.Timestamp(now).round("ms")
121121
data = {
122-
"id": ["a", "b", "c", "d", "e"],
122+
"id_join_key": ["a", "b", "c", "d", "e"],
123123
"float_col": [0.1, 0.2, 0.3, 4, 5],
124124
"int64_col": [1, 2, 3, 4, 5],
125125
"string_col": ["a", "b", "c", "d", "e"],

sdk/python/tests/integration/registration/test_feature_store.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -218,6 +218,8 @@ def test_feature_view_inference_success(test_feature_store, dataframe_source):
218218
with prep_file_source(
219219
df=dataframe_source, event_timestamp_column="ts_1"
220220
) as file_source:
221+
entity = Entity(name="id", join_key="id_join_key", value_type=ValueType.INT64)
222+
221223
fv1 = FeatureView(
222224
name="fv1",
223225
entities=["id"],
@@ -245,7 +247,7 @@ def test_feature_view_inference_success(test_feature_store, dataframe_source):
245247
tags={},
246248
)
247249

248-
test_feature_store.apply([fv1, fv2, fv3]) # Register Feature Views
250+
test_feature_store.apply([entity, fv1, fv2, fv3]) # Register Feature Views
249251
feature_view_1 = test_feature_store.list_feature_views()[0]
250252
feature_view_2 = test_feature_store.list_feature_views()[1]
251253
feature_view_3 = test_feature_store.list_feature_views()[2]
@@ -433,7 +435,7 @@ def test_reapply_feature_view_success(test_feature_store, dataframe_source):
433435
df=dataframe_source, event_timestamp_column="ts_1"
434436
) as file_source:
435437

436-
e = Entity(name="id", value_type=ValueType.STRING)
438+
e = Entity(name="id", join_key="id_join_key", value_type=ValueType.STRING)
437439

438440
# Create Feature View
439441
fv1 = FeatureView(

sdk/python/tests/integration/registration/test_inference.py

+9-5
Original file line numberDiff line numberDiff line change
@@ -30,22 +30,26 @@ def test_update_entities_with_inferred_types_from_feature_views(
3030
name="fv2", entities=["id"], batch_source=file_source_2, ttl=None,
3131
)
3232

33-
actual_1 = Entity(name="id")
34-
actual_2 = Entity(name="id")
33+
actual_1 = Entity(name="id", join_key="id_join_key")
34+
actual_2 = Entity(name="id", join_key="id_join_key")
3535

3636
update_entities_with_inferred_types_from_feature_views(
3737
[actual_1], [fv1], RepoConfig(provider="local", project="test")
3838
)
3939
update_entities_with_inferred_types_from_feature_views(
4040
[actual_2], [fv2], RepoConfig(provider="local", project="test")
4141
)
42-
assert actual_1 == Entity(name="id", value_type=ValueType.INT64)
43-
assert actual_2 == Entity(name="id", value_type=ValueType.STRING)
42+
assert actual_1 == Entity(
43+
name="id", join_key="id_join_key", value_type=ValueType.INT64
44+
)
45+
assert actual_2 == Entity(
46+
name="id", join_key="id_join_key", value_type=ValueType.STRING
47+
)
4448

4549
with pytest.raises(RegistryInferenceFailure):
4650
# two viable data types
4751
update_entities_with_inferred_types_from_feature_views(
48-
[Entity(name="id")],
52+
[Entity(name="id", join_key="id_join_key")],
4953
[fv1, fv2],
5054
RepoConfig(provider="local", project="test"),
5155
)

0 commit comments

Comments
 (0)