Skip to content

Commit 8187c18

Browse files
authored
Remove Swift flag from user agent (#8215)
1 parent 446b452 commit 8187c18

File tree

3 files changed

+9
-13
lines changed

3 files changed

+9
-13
lines changed

FirebaseCore/Sources/FIRFirebaseUserAgent.m

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,6 @@ - (void)reset {
9191
NSDictionary<NSString *, id> *info = [[NSBundle mainBundle] infoDictionary];
9292
NSString *xcodeVersion = info[@"DTXcodeBuild"];
9393
NSString *appleSdkVersion = info[@"DTSDKBuild"];
94-
95-
NSString *swiftFlagValue = [GULAppEnvironmentUtil hasSwiftRuntime] ? @"true" : @"false";
9694
NSString *isFromAppstoreFlagValue = [GULAppEnvironmentUtil isFromAppStore] ? @"true" : @"false";
9795

9896
components[@"apple-platform"] = [GULAppEnvironmentUtil applePlatform];
@@ -101,7 +99,6 @@ - (void)reset {
10199
components[@"deploy"] = [GULAppEnvironmentUtil deploymentType];
102100
components[@"device"] = [GULAppEnvironmentUtil deviceModel];
103101
components[@"os-version"] = [GULAppEnvironmentUtil systemVersion];
104-
components[@"swift"] = swiftFlagValue;
105102
components[@"xcode"] = xcodeVersion;
106103

107104
return [components copy];

FirebaseCore/Tests/SwiftUnit/FirebaseAppTests.swift

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,6 @@ class FirebaseAppTests: XCTestCase {
3131
FirebaseApp.resetApps()
3232
}
3333

34-
func testSwiftFlagWithSwift() {
35-
XCTAssertTrue(FirebaseApp.firebaseUserAgent().contains("swift"))
36-
}
37-
3834
func testConfigure() throws {
3935
expectAppConfigurationNotification(appName: Constants.App.defaultName, isDefaultApp: true)
4036

@@ -331,8 +327,15 @@ class FirebaseAppTests: XCTestCase {
331327

332328
// MARK: - Firebase User Agent
333329

334-
func testFirebaseUserAgent_SwiftRuntime() {
335-
XCTAssertTrue(FirebaseApp.firebaseUserAgent().contains("swift/true"))
330+
func testUserAgent() {
331+
let agent = FirebaseApp.firebaseUserAgent()
332+
XCTAssertTrue(agent.contains("apple-platform"))
333+
XCTAssertTrue(agent.contains("apple-sdk"))
334+
XCTAssertTrue(agent.contains("appstore"))
335+
XCTAssertTrue(agent.contains("deploy"))
336+
XCTAssertTrue(agent.contains("device"))
337+
XCTAssertTrue(agent.contains("os-version"))
338+
XCTAssertTrue(agent.contains("xcode"))
336339
}
337340

338341
// MARK: - Helpers

FirebaseCore/Tests/Unit/FIRAppTest.m

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -841,10 +841,6 @@ - (void)testRegisterLibrary_RegisteringNonConformingLibrary {
841841
XCTAssertFalse([[FIRApp firebaseUserAgent] containsString:@"InvalidLibrary`/1.0.0"]);
842842
}
843843

844-
- (void)testFirebaseUserAgent_SwiftFlagWithNoSwift {
845-
XCTAssertTrue([[FIRApp firebaseUserAgent] containsString:@"swift/false"]);
846-
}
847-
848844
- (void)testFirebaseUserAgent_ApplePlatformFlag {
849845
// When a Catalyst app is run on macOS then both `TARGET_OS_MACCATALYST` and `TARGET_OS_IOS` are
850846
// `true`.

0 commit comments

Comments
 (0)