Implemented Comparable on NSOperatingSystemVersion with tests

Author: rsmoz

Foundation/NSProcessInfo.swift

@@ -20,6 +20,7 @@ public struct NSOperatingSystemVersion {
     public var majorVersion: Int
     public var minorVersion: Int
     public var patchVersion: Int
+	
     public init() {
         self.init(majorVersion: 0, minorVersion: 0, patchVersion: 0)
     }
@@ -31,6 +32,21 @@ public struct NSOperatingSystemVersion {
     }
 }
 
+extension NSOperatingSystemVersion : Comparable {}
+
+public func ==(lhs: NSOperatingSystemVersion, rhs: NSOperatingSystemVersion) -> Bool {
+    let lhsTuple = (lhs.majorVersion, lhs.minorVersion, lhs.patchVersion)
+    let rhsTuple = (rhs.majorVersion, rhs.minorVersion, rhs.patchVersion)
+    
+    return lhsTuple == rhsTuple
+}
+
+public func <(lhs: NSOperatingSystemVersion, rhs: NSOperatingSystemVersion) -> Bool {
+    let lhsTuple = (lhs.majorVersion, lhs.minorVersion, lhs.patchVersion)
+    let rhsTuple = (rhs.majorVersion, rhs.minorVersion, rhs.patchVersion)
+    
+    return lhsTuple < rhsTuple
+}
 
 
 public class NSProcessInfo : NSObject {
@@ -123,26 +139,7 @@ public class NSProcessInfo : NSObject {
     }
 
     public func isOperatingSystemAtLeastVersion(version: NSOperatingSystemVersion) -> Bool {
-        let ourVersion = operatingSystemVersion
-        if ourVersion.majorVersion < version.majorVersion {
-            return false
-        }
-        if ourVersion.majorVersion > version.majorVersion {
-            return true
-        }
-        if ourVersion.minorVersion < version.minorVersion {
-            return false
-        }
-        if ourVersion.minorVersion > version.minorVersion {
-            return true
-        }
-        if ourVersion.patchVersion < version.patchVersion {
-            return false
-        }
-        if ourVersion.patchVersion > version.patchVersion {
-            return true
-        }
-        return true
+        return operatingSystemVersion >= version
     }
 
     public var systemUptime: NSTimeInterval {

TestFoundation/TestNSProcessInfo.swift

@@ -34,6 +34,11 @@ class TestNSProcessInfo : XCTestCase {
 
         let version = processInfo.operatingSystemVersion
         XCTAssertNotNil(version.majorVersion != 0)
+        
+        XCTAssertEqual(NSOperatingSystemVersion(), NSOperatingSystemVersion())
+        XCTAssertEqual(NSOperatingSystemVersion(majorVersion: 0, minorVersion: 0, patchVersion: 0), NSOperatingSystemVersion())
+        XCTAssertLessThan(NSOperatingSystemVersion(majorVersion: 0, minorVersion: 0, patchVersion: 1), NSOperatingSystemVersion(majorVersion: 0, minorVersion: 0, patchVersion: 2))
+        XCTAssertGreaterThan(NSOperatingSystemVersion(majorVersion: 0, minorVersion: 0, patchVersion: 2), NSOperatingSystemVersion(majorVersion: 0, minorVersion: 0, patchVersion: 1)) //Guards against false positives in LT test
     }
 
     func test_processName() {