Skip to content

Commit 37487e0

Browse files
committed
add OneSignal-Install-Id test to HttpClientTest
1 parent 4a989a4 commit 37487e0

File tree

1 file changed

+4
-2
lines changed
  • OneSignalSDK/onesignal/core/src/test/java/com/onesignal/core/internal/http

1 file changed

+4
-2
lines changed

OneSignalSDK/onesignal/core/src/test/java/com/onesignal/core/internal/http/HttpClientTests.kt

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,9 @@ class Mocks {
2121
internal val mockConfigModel = MockHelper.configModelStore()
2222
internal val response = MockHttpConnectionFactory.MockResponse()
2323
internal val factory = MockHttpConnectionFactory(response)
24+
internal val installIdService = InstallIdService(MockPreferencesService())
2425
internal val httpClient by lazy {
25-
HttpClient(factory, MockPreferencesService(), mockConfigModel, Time(), InstallIdService(MockPreferencesService()))
26+
HttpClient(factory, MockPreferencesService(), mockConfigModel, Time(), installIdService)
2627
}
2728
}
2829

@@ -51,7 +52,7 @@ class HttpClientTests : FunSpec({
5152
response.throwable should beInstanceOf<TimeoutCancellationException>()
5253
}
5354

54-
test("SDKHeader is included in all requests") {
55+
test("SDK Headers are included in all requests") {
5556
// Given
5657
val mocks = Mocks()
5758
val httpClient = mocks.httpClient
@@ -66,6 +67,7 @@ class HttpClientTests : FunSpec({
6667
// Then
6768
for (connection in mocks.factory.connections) {
6869
connection.getRequestProperty("SDK-Version") shouldBe "onesignal/android/${OneSignalUtils.SDK_VERSION}"
70+
connection.getRequestProperty("OneSignal-Install-Id") shouldBe mocks.installIdService.getId().toString()
6971
}
7072
}
7173

0 commit comments

Comments
 (0)