Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces the new dataconnect module, including the ConnectorConfig and DataConnect classes, along with a comprehensive test suite. However, the implementation is currently incomplete and will fail the tests. Specifically, the DataConnect class is missing the public app and config properties, the client function is left as a stub, and the _DataConnectService class is completely missing. Additionally, the validation in ConnectorConfig needs to be updated to explicitly check for string types to prevent invalid types from bypassing the checks.
|
it's important that PR/CL titles and descriptions are detailed enough so that reviewers (or future viewers) have the context they need to understand your changes will you check out go/cl-descriptions for some guidance and examples, and then add an updated description and title for the PR? |
mk2023
changed the title
mk2023
changed the title
…d docstrings/synatx Refactored test suite to use module-level BASE_CONFIG and added parameterized client caching tests.
…ervice architecture Implemented ConnectorConfig dataclass, DataConnect client instance, _DataConnectService caching layer, and public client() factory function. Fixed mock recursion in test_client_successful using lambda delegation in test_data_connect.py.
…00/10 linter score
mk2023
changed the title
3 participants
Feat: Implemented the foundational Firebase Data Connect Python SDK architecture (for Milestone 2) and their respective unit/integration tests:
This PR introduces a comprehensive test suite for the
firebase_admin.dataconnectmodule. The tests cover:ConnectorConfigclass behavior and validation.DataConnectclass initialization.dataconnect.client()factory function, including caching logic._DataConnectServiceclient management.This PR also includes the implementation of the core Data Connect module (
firebase_admin/dataconnect.py), designed to maintain architectural parity with the Node.js Admin SDK while leveraging idiomatic Python patterns:ConnectorConfig: Implemented as an immutable frozen dataclass (@dataclass(frozen=True)) with strict input validation that raises idiomaticValueErrorexceptions. Because it is frozen, Python automatically generates value-based hashing (__hash__), enabling clean object lookup.DataConnect: Implemented the primary client class with clean property accessors (.appand.config)._DataConnectService&client(): Built instance caching integrated directly into Python's FirebaseAppservice lifecycle (_utils.get_app_service), ensuring duplicate calls return cached instances.