[iOS] Use correct suspension value for invalid language metrics answer (#30136)
This change ensures we use `Int32.max` instead of `Int.max` so that we match with desktop which uses `INT_MAX` to suspend the language metric
This commit is contained in:
@@ -206,7 +206,7 @@ public class LanguageMetrics {
|
||||
]
|
||||
|
||||
static func answerForLangaugeCode(_ languageCode: String?) -> Int {
|
||||
let suspendedMetricValue = Int.max - 1
|
||||
let suspendedMetricValue = Int(Int32.max - 1)
|
||||
guard let languageCode = languageCode,
|
||||
case let primaryLanguage = languageSynonyms[languageCode, default: languageCode],
|
||||
let answer = acceptedLanguages.firstIndex(of: primaryLanguage)
|
||||
|
||||
@@ -10,7 +10,7 @@ import XCTest
|
||||
|
||||
class LanguageMetricsTests: XCTestCase {
|
||||
func testInvalidLanguageCodes() {
|
||||
let expectedValue = Int.max - 1
|
||||
let expectedValue = Int(Int32.max - 1)
|
||||
XCTAssertEqual(LanguageMetrics.answerForLangaugeCode(nil), expectedValue)
|
||||
XCTAssertEqual(LanguageMetrics.answerForLangaugeCode("notalangugage"), expectedValue)
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user