Skip to content

Commit

Permalink
Added test with parameters
Browse files Browse the repository at this point in the history
Signed-off-by: Elena Khaustova <ymax70rus@gmail.com>
  • Loading branch information
ElenaKhaustova committed Nov 18, 2024
1 parent b2ebfe2 commit a07107a
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
5 changes: 5 additions & 0 deletions tests/io/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,11 @@ def correct_config_versioned(filepath):
},
"copy_mode": None,
},
"parameters": {
"type": "kedro.io.memory_dataset.MemoryDataset",
"data": [4, 5, 6],
"copy_mode": None,
},
},
"credentials": {
"cars_credentials": {"key": "FAKE_ACCESS_KEY", "secret": "FAKE_SECRET_KEY"},
Expand Down
2 changes: 2 additions & 0 deletions tests/io/test_kedro_data_catalog.py
Original file line number Diff line number Diff line change
Expand Up @@ -303,6 +303,7 @@ def test_to_config(self, correct_config_versioned, dataset, filepath):
catalog = KedroDataCatalog.from_config(config, credentials)
catalog["resolved_ds"] = dataset
catalog["memory_ds"] = [1, 2, 3]
catalog["params:a.b"] = {"abc": "def"}
# Materialize cached_ds
_ = catalog["cached_ds"]

Expand Down Expand Up @@ -348,6 +349,7 @@ def test_to_config(self, correct_config_versioned, dataset, filepath):
},
}
expected_config.update(config)
expected_config.pop("parameters", None)

# TODO: Add expected load/save versions when #4327 resolved

Expand Down

0 comments on commit a07107a

Please sign in to comment.