Skip to content

Commit 3d7cb70

Browse files
committed
More'
'
1 parent 7638190 commit 3d7cb70

File tree

5 files changed

+21
-4
lines changed

5 files changed

+21
-4
lines changed

firebase-sessions/src/main/kotlin/com/google/firebase/sessions/ApplicationInfo.kt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,9 @@ internal data class AndroidApplicationInfo(
4242

4343
/** The build version of the application. */
4444
val buildVersion: String,
45+
46+
/** The manufacturer of the device that runs the application. */
47+
val deviceManufacturer: String,
4548
)
4649

4750
internal data class ApplicationInfo(

firebase-sessions/src/main/kotlin/com/google/firebase/sessions/SessionEvents.kt

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,8 @@ internal object SessionEvents {
9393
run {
9494
ctx.add(FieldDescriptor.of("package_name"), androidAppInfo.packageName)
9595
ctx.add(FieldDescriptor.of("version_name"), androidAppInfo.versionName)
96+
ctx.add(FieldDescriptor.of("app_build_version"), androidAppInfo.buildVersion)
97+
ctx.add(FieldDescriptor.of("device_manufacturer"), androidAppInfo.deviceManufacturer)
9698
}
9799
}
98100
}
@@ -137,7 +139,8 @@ internal object SessionEvents {
137139
AndroidApplicationInfo(
138140
packageName = packageName,
139141
versionName = packageInfo.versionName,
140-
buildVersion = packageInfo.versionCode.toString()
142+
buildVersion = packageInfo.versionCode.toString(),
143+
deviceManufacturer = Build.MANUFACTURER,
141144
)
142145
)
143146
}

firebase-sessions/src/test/kotlin/com/google/firebase/sessions/ApplicationInfoTest.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ class ApplicationInfoTest {
4545
packageName = ApplicationProvider.getApplicationContext<Context>().packageName,
4646
versionName = FakeFirebaseApp.MOCK_APP_VERSION,
4747
buildVersion = FakeFirebaseApp.MOCK_APP_BUILD_VERSION,
48+
deviceManufacturer = Build.MANUFACTURER,
4849
)
4950
)
5051
)

firebase-sessions/src/test/kotlin/com/google/firebase/sessions/SessionEventEncoderTest.kt

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,9 @@ class SessionEventEncoderTest {
7575
"log_environment":3,
7676
"android_app_info":{
7777
"package_name":"com.google.firebase.sessions.test",
78-
"version_name":"1.0.0"
78+
"version_name":"1.0.0",
79+
"app_build_version":"0",
80+
"device_manufacturer":"${Build.MANUFACTURER}"
7981
}
8082
}
8183
}
@@ -104,7 +106,12 @@ class SessionEventEncoderTest {
104106
sessionSdkVersion = "",
105107
osDisplayVersion = "",
106108
logEnvironment = LogEnvironment.LOG_ENVIRONMENT_PROD,
107-
AndroidApplicationInfo(packageName = "", versionName = "", buildVersion = ""),
109+
AndroidApplicationInfo(
110+
packageName = "",
111+
versionName = "",
112+
buildVersion = "",
113+
deviceManufacturer = "",
114+
),
108115
)
109116
)
110117

@@ -135,7 +142,9 @@ class SessionEventEncoderTest {
135142
"log_environment":3,
136143
"android_app_info":{
137144
"package_name":"",
138-
"version_name":""
145+
"version_name":"",
146+
"app_build_version":"",
147+
"device_manufacturer":""
139148
}
140149
}
141150
}

firebase-sessions/src/test/kotlin/com/google/firebase/sessions/testing/TestSessionEventData.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,7 @@ internal object TestSessionEventData {
6363
packageName = ApplicationProvider.getApplicationContext<Context>().packageName,
6464
versionName = FakeFirebaseApp.MOCK_APP_VERSION,
6565
buildVersion = FakeFirebaseApp.MOCK_APP_BUILD_VERSION,
66+
deviceManufacturer = Build.MANUFACTURER,
6667
),
6768
)
6869
)

0 commit comments

Comments
 (0)