diff --git a/firebase-dataconnect/CHANGELOG.md b/firebase-dataconnect/CHANGELOG.md index b6eb119fb90..9470b414112 100644 --- a/firebase-dataconnect/CHANGELOG.md +++ b/firebase-dataconnect/CHANGELOG.md @@ -9,6 +9,8 @@ ([#7957](https://github.com/firebase/firebase-android-sdk/pull/7957)) - [changed] Internal refactor for calculating debug logging strings. ([#8024](https://github.com/firebase/firebase-android-sdk/pull/8024)) +- [changed] Internal refactor to use more descriptive variable names. + ([#8025](https://github.com/firebase/firebase-android-sdk/pull/8025)) # 17.2.0 diff --git a/firebase-dataconnect/src/main/kotlin/com/google/firebase/dataconnect/core/DataConnectGrpcClient.kt b/firebase-dataconnect/src/main/kotlin/com/google/firebase/dataconnect/core/DataConnectGrpcClient.kt index ecb783e163f..6e8c59761c5 100644 --- a/firebase-dataconnect/src/main/kotlin/com/google/firebase/dataconnect/core/DataConnectGrpcClient.kt +++ b/firebase-dataconnect/src/main/kotlin/com/google/firebase/dataconnect/core/DataConnectGrpcClient.kt @@ -47,7 +47,7 @@ internal class DataConnectGrpcClient( val instanceId: String get() = logger.nameWithId - private val requestName = + private val connectorResourceName = "projects/$projectId/" + "locations/${connector.location}" + "/services/${connector.serviceId}" + @@ -67,7 +67,7 @@ internal class DataConnectGrpcClient( fetchPolicy: FetchPolicy, ): OperationResult { val request = executeQueryRequest { - this.name = requestName + this.name = connectorResourceName this.operationName = operationName this.variables = variables } @@ -87,7 +87,7 @@ internal class DataConnectGrpcClient( callerSdkType: FirebaseDataConnect.CallerSdkType, ): OperationResult { val request = executeMutationRequest { - this.name = requestName + this.name = connectorResourceName this.operationName = operationName this.variables = variables } diff --git a/firebase-dataconnect/src/test/kotlin/com/google/firebase/dataconnect/core/DataConnectGrpcClientUnitTest.kt b/firebase-dataconnect/src/test/kotlin/com/google/firebase/dataconnect/core/DataConnectGrpcClientUnitTest.kt index 8b0bd30c92f..873a28697d2 100644 --- a/firebase-dataconnect/src/test/kotlin/com/google/firebase/dataconnect/core/DataConnectGrpcClientUnitTest.kt +++ b/firebase-dataconnect/src/test/kotlin/com/google/firebase/dataconnect/core/DataConnectGrpcClientUnitTest.kt @@ -170,14 +170,14 @@ class DataConnectGrpcClientUnitTest { fetchPolicy ) - val expectedName = + val expectedConnectorResourceName = "projects/${projectId}" + "/locations/${connectorConfig.location}" + "/services/${connectorConfig.serviceId}" + "/connectors/${connectorConfig.connector}" val expectedRequest = ExecuteQueryRequest.newBuilder() - .setName(expectedName) + .setName(expectedConnectorResourceName) .setOperationName(operationName) .setVariables(variables) .build() @@ -200,14 +200,14 @@ class DataConnectGrpcClientUnitTest { fun `executeMutation() should send the right ExecuteMutationRequest`() = runTest { dataConnectGrpcClient.executeMutation(requestId, operationName, variables, callerSdkType) - val expectedName = + val expectedConnectorResourceName = "projects/${projectId}" + "/locations/${connectorConfig.location}" + "/services/${connectorConfig.serviceId}" + "/connectors/${connectorConfig.connector}" val expectedRequest = ExecuteMutationRequest.newBuilder() - .setName(expectedName) + .setName(expectedConnectorResourceName) .setOperationName(operationName) .setVariables(variables) .build()