Skip to content

Commit

Permalink
Remove old legacyOauthToken code (from 2016) (#1946)
Browse files Browse the repository at this point in the history
* Remove old legacyOauthToken code (from 2016)

* Remove test for legacy OAuth token
  • Loading branch information
amy-at-kickstarter committed Feb 21, 2024
1 parent 22727b1 commit 2b36c16
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 23 deletions.
13 changes: 0 additions & 13 deletions Library/AppEnvironment.swift
Original file line number Diff line number Diff line change
Expand Up @@ -298,11 +298,6 @@ public struct AppEnvironment: AppEnvironmentType {
if let oauthToken = data["apiService.oauthToken.token"] as? String {
// If there is an oauth token stored in the defaults, then we can authenticate our api service
service = service.login(OauthToken(token: oauthToken))
removeLegacyOauthToken(fromUserDefaults: userDefaults)
} else if let oauthToken = legacyOauthToken(forUserDefaults: userDefaults) {
// Otherwise if there is a token in the legacy user defaults entry we can use that
service = service.login(OauthToken(token: oauthToken))
removeLegacyOauthToken(fromUserDefaults: userDefaults)
}

// Try restoring the client id for the api service
Expand Down Expand Up @@ -413,11 +408,3 @@ public struct AppEnvironment: AppEnvironmentType {
userDefaults.set(data, forKey: self.environmentStorageKey)
}
}

private func legacyOauthToken(forUserDefaults userDefaults: KeyValueStoreType) -> String? {
return userDefaults.object(forKey: "com.kickstarter.access_token") as? String
}

private func removeLegacyOauthToken(fromUserDefaults userDefaults: KeyValueStoreType) {
userDefaults.removeObject(forKey: "com.kickstarter.access_token")
}
10 changes: 0 additions & 10 deletions Library/AppEnvironmentTests.swift
Original file line number Diff line number Diff line change
Expand Up @@ -126,16 +126,6 @@ final class AppEnvironmentTests: XCTestCase {
XCTAssertNotNil(env.appTrackingTransparency)
}

func testFromStorage_LegacyUserDefaults() {
let userDefaults = MockKeyValueStore()
userDefaults.set("deadbeef", forKey: "com.kickstarter.access_token")
let env = AppEnvironment.fromStorage(ubiquitousStore: MockKeyValueStore(), userDefaults: userDefaults)

XCTAssertEqual("deadbeef", env.apiService.oauthToken?.token)
XCTAssertTrue(env.apiService.isAuthenticated)
XCTAssertNil(userDefaults.object(forKey: "com.kickstarter.access_token"))
}

func testSaveEnvironment() {
let apiService = MockService(
serverConfig: ServerConfig(
Expand Down

0 comments on commit 2b36c16

Please sign in to comment.