[MCC-1501368] Python Lib - remove client validation for Fetch Data ()

[afieraru-mdsol]

feature/bug/enhance/refactor/tech_debt/hotfix

• Replace this line with Project title.
• MCC-1501368

Changes Summary

This PR removes client-side input validation from DefaultDataConnectService.fetch_data(), shifting responsibility for validation to the transport/server layer and updating the unit tests accordingly.

Changes:

• Removed UUID and first_n_rows validation logic from DefaultDataConnectService.fetch_data().
• Deleted unit tests that asserted client-side ValidationError behavior for fetch_data().

[Copilot]

Pull request overview

This PR removes client-side input validation from DefaultDataConnectService.fetch_data(), shifting responsibility for validation to the transport/server layer and updating the unit tests accordingly.

Changes:

• Removed UUID and first_n_rows validation logic from DefaultDataConnectService.fetch_data().
• Deleted unit tests that asserted client-side ValidationError behavior for fetch_data().

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
dataconnect/service/default.py	Removes pre-flight validators and ErrorDetail usage from fetch_data().
tests/test_fetch_data.py	Removes fetch_data() client-validation test cases and the ValidationError import.

Comments suppressed due to low confidence (1)

dataconnect/service/default.py:93

• With client-side validation removed, there’s no longer a unit test that locks in the new behavior (e.g., that first_n_rows=0/negative values are passed through into DatasetTicket.limit rather than raising a ValidationError). Adding a focused test for this behavior will help prevent validation from being unintentionally reintroduced later.

    def fetch_data(self, dataset_uuid: UUID, first_n_rows: int | None = None) -> pd.DataFrame:
        """Fetch data for a dataset"""

        ticket = DatasetTicket(
            dataset_uuid=str(dataset_uuid),
            limit=first_n_rows,
        )

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.