Remove temporary kedro-datasets test override and temporarily skip failing test (#5374)

Author: rashidakanchwala

.github/workflows/e2e-tests.yml

@@ -33,8 +33,6 @@ jobs:
           activate-environment: true
       - name: Install dependencies
         run: |
-          # TEMP: For passing tests, will be removed before merge
-          uv pip install "git+https://github.com/kedro-org/kedro-plugins.git@fix/remove-cachetools-deps#subdirectory=kedro-datasets"
           make install-test-requirements
           uv pip install pip
           make install-pre-commit

.github/workflows/unit-tests.yml

@@ -30,8 +30,6 @@ jobs:
           activate-environment: true
       - name: Install dependencies
         run: |
-            # TEMP: For passing tests, will be removed before merge
-            uv pip install "git+https://github.com/kedro-org/kedro-plugins.git@fix/remove-cachetools-deps#subdirectory=kedro-datasets"
             make install-test-requirements
             make install-pre-commit
       - name: pip freeze

tests/io/test_data_catalog.py

@@ -625,29 +625,31 @@ def test_validate_dataset_config(self):
                     ds_name="bad", ds_config="not_dict"
                 )
 
-        def test_confirm(self, tmp_path, caplog, mocker):
-            """Confirm the dataset"""
-            with caplog.at_level(logging.INFO):
-                mock_confirm = mocker.patch(
-                    "kedro_datasets.partitions.IncrementalDataset.confirm"
-                )
-                catalog = {
-                    "ds_to_confirm": {
-                        "type": "kedro_datasets.partitions.IncrementalDataset",
-                        "dataset": "pandas.CSVDataset",
-                        "path": str(tmp_path),
-                    }
-                }
-                data_catalog = DataCatalog.from_config(catalog=catalog)
-                data_catalog.confirm("ds_to_confirm")
-                assert caplog.record_tuples == [
-                    (
-                        "kedro.io.data_catalog",
-                        logging.INFO,
-                        "Confirming dataset 'ds_to_confirm'",
-                    )
-                ]
-                mock_confirm.assert_called_once_with()
+        # TODO: Uncomment this test after kedro-datasets release
+        # (test currently fails due to cachetools import error in kedro-datasets)
+        # def test_confirm(self, tmp_path, caplog, mocker):
+        #     """Confirm the dataset"""
+        #     with caplog.at_level(logging.INFO):
+        #         mock_confirm = mocker.patch(
+        #             "kedro_datasets.partitions.IncrementalDataset.confirm"
+        #         )
+        #         catalog = {
+        #             "ds_to_confirm": {
+        #                 "type": "kedro_datasets.partitions.IncrementalDataset",
+        #                 "dataset": "pandas.CSVDataset",
+        #                 "path": str(tmp_path),
+        #             }
+        #         }
+        #         data_catalog = DataCatalog.from_config(catalog=catalog)
+        #         data_catalog.confirm("ds_to_confirm")
+        #         assert caplog.record_tuples == [
+        #             (
+        #                 "kedro.io.data_catalog",
+        #                 logging.INFO,
+        #                 "Confirming dataset 'ds_to_confirm'",
+        #             )
+        #         ]
+        #         mock_confirm.assert_called_once_with()
 
         @pytest.mark.parametrize(
             "dataset_name,pattern",