Skip to content

Commit f654290

Browse files
authored
Merge pull request #1 from Broich/master
updates to unified past tense variable naming - installed, notChanged…
2 parents 2510159 + 67ffe2b commit f654290

File tree

5 files changed

+13
-8
lines changed

5 files changed

+13
-8
lines changed

Example/Tests/VersionTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -199,7 +199,7 @@ class VersionTests: QuickSpec {
199199
it("detects updates") {
200200
let prevVersion = Version("1.0", buildString: "19", installDate: Date())
201201
let curVersion = Version("1.0", buildString: "20", installDate: Date())
202-
expect(Version.changeStateForFromVersion(prevVersion, toVersion: curVersion)).to(equal(Version.ChangeState.update(previousVersion: prevVersion)))
202+
expect(Version.changeStateForFromVersion(prevVersion, toVersion: curVersion)).to(equal(Version.ChangeState.updated(previousVersion: prevVersion)))
203203
}
204204
it("detects upgrades") {
205205
let prevVersion = Version("1.0", buildString: "19", installDate: Date())

Example/Tests/VersionTrackerTests.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ class VersionTrackerTests: QuickSpec {
5959

6060
it("it will notice build Updates") {
6161
let versionTracker = VersionTracker(currentVersion: versions[1], inScope: scope, userDefaults: userDefaults)
62-
expect(versionTracker.changeState).to(equal(Version.ChangeState.update(previousVersion: Version("1.0", buildString: "1"))));
62+
expect(versionTracker.changeState).to(equal(Version.ChangeState.updated(previousVersion: Version("1.0", buildString: "1"))));
6363
}
6464

6565
it("it will notice markting version upgrades") {
@@ -82,4 +82,4 @@ class VersionTrackerTests: QuickSpec {
8282
}
8383
}
8484
}
85-
}
85+
}

Example/VersionsTracker/AppDelegate.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ func printVersionChange(_ versionTracker: VersionTracker) {
103103
print("🆕 Congratulations, the app is launched for the very first time")
104104
case .notChanged:
105105
print("🔄 Welcome back, nothing as changed since the last time")
106-
case .update(let previousVersion):
106+
case .updated(let previousVersion):
107107
print("🆙 The app was updated making small changes: \(previousVersion) -> \(versionTracker.currentVersion)")
108108
case .upgraded(let previousVersion):
109109
print("⬆️ Cool, its a new version: \(previousVersion) -> \(versionTracker.currentVersion)")

Pod/Classes/Version.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ extension Version {
162162
public enum ChangeState {
163163
case installed
164164
case notChanged
165-
case update(previousVersion: Version)
165+
case updated(previousVersion: Version)
166166
case upgraded(previousVersion: Version)
167167
case downgraded(previousVersion: Version)
168168
}
@@ -182,7 +182,7 @@ extension Version {
182182
return .downgraded(previousVersion: olderVersion)
183183
}
184184
else if olderVersion != newerVersion {
185-
return .update(previousVersion: olderVersion)
185+
return .updated(previousVersion: olderVersion)
186186
}
187187
return .notChanged
188188
}
@@ -199,7 +199,7 @@ public func ==(lhs: Version.ChangeState, rhs: Version.ChangeState) -> Bool {
199199
return true
200200
case (.notChanged, .notChanged):
201201
return true
202-
case (let .update(previousVersionLHS), let .update(previousVersionRHS)):
202+
case (let .updated(previousVersionLHS), let .updated(previousVersionRHS)):
203203
return previousVersionLHS == previousVersionRHS
204204
case (let .upgraded(previousVersionLHS), let .upgraded(previousVersionRHS)):
205205
return previousVersionLHS == previousVersionRHS

README.md

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,22 +88,27 @@ let versionsTracker = iDontMindSingletons ? VersionsTracker.sharedInstance : Ver
8888

8989
switch versionsTracker.appVersion.changeState {
9090
case .installed:
91+
break
9192
// 🎉 Sweet, a new user just installed your app
9293
// ... start tutorial / intro
9394

9495
case .notChanged:
96+
break
9597
// 😴 nothing as changed
9698
// ... nothing to do
9799

98-
case .update(let previousVersion: Version):
100+
case .updated(let previousVersion: Version):
101+
break
99102
// 🙃 new build of the same version
100103
// ... hopefully it fixed those bugs the QA guy reported
101104

102105
case .upgraded(let previousVersion: Version)
106+
break
103107
// 😄 marketing version increased
104108
// ... migrate old app data
105109

106110
case .downgraded(let previousVersion: Version)
111+
break
107112
// 😵 marketing version decreased (hopefully we are not on production)
108113
// ... purge app data and start over
109114

0 commit comments

Comments
 (0)