Fix CI due to lazy-loading

[noklam]

Description

kedro-viz/package/kedro_viz/integrations/kedro/data_loader.py

Lines 15 to 32 in 6e1d127

	try:
	from kedro_datasets import ( # isort:skip
	json,
	matplotlib,
	plotly,
	tracking,
	)
	except ImportError:
	from kedro.extras.datasets import ( # Safe since ImportErrors are suppressed within kedro.
	json,
	matplotlib,
	plotly,
	tracking,
	)
	from kedro.io import DataCatalog
	from kedro.io.core import get_filepath_str
	from kedro.pipeline import Pipeline
	from semver import VersionInfo

This is no longer valid because kedro-datasets is lazily load and ImportError is not produce immediately.

The original code is slightly confusing, because ImportError can happens in 2 situations

• When a specific dependency is missing
• When kedro-datasets is not installed (This is what the original code intend to catch and have nothing to do with whether dependencies (i.e. pandas, matplotlib) is installed or not.

This is the comments in the code base

# The dataset type is available as an attribute if and only if the import from kedro
# did not suppress an ImportError. i.e. hasattr(matplotlib, "MatplotlibWriter") is True
# when matplotlib dependencies are installed.

This is no longer True. Previously, if the dependencies is not installed, MatplotlibWriter won't be available due to the suppression of ImportError.

Development notes

• Catch ImportError for lazy-evaluation (kedro_datasets), catch AttributeError for eager evaluation (kedro.extras)

QA notes

Checklist

• Read the contributing guidelines
• Opened this PR as a 'Draft Pull Request' if it is work-in-progress
• Updated the documentation to reflect the code changes
• Added new entries to the RELEASE.md file
• Added tests to cover my changes

[noklam]

Attempt to fix it in kedro-org/kedro-plugins#298

[tynandebold]

Looks great. All checks are passing. Thanks so much.

[noklam]

@tynandebold @ravi-kumar-pilla The test start failing after I merge from main, would you have a look at this? Feel free to merge it whenever it passed.