UUID fixes (#31)

* implement CustomDebugStringConvertible

* add UUID PostgresDataConvertible conformance

* add Optional PostgresDataConvertible conformance

* add missing return statement

* don't run linux-perf test by default

Author: tanner0101

Sources/PostgresNIO/Data/PostgresData+Optional.swift

@@ -0,0 +1,18 @@
+extension Optional: PostgresDataConvertible where Wrapped: PostgresDataConvertible {
+    public static var postgresDataType: PostgresDataType {
+        return Wrapped.postgresDataType
+    }
+
+    public init?(postgresData: PostgresData) {
+        self = Wrapped.init(postgresData: postgresData)
+    }
+
+    public var postgresData: PostgresData? {
+        switch self {
+        case .some(let wrapped):
+            return wrapped.postgresData
+        case .none:
+            return PostgresData.null
+        }
+    }
+}

Sources/PostgresNIO/Data/PostgresData+UUID.swift

@@ -20,3 +20,20 @@ extension PostgresData {
         return value.readUUID()
     }
 }
+
+extension UUID: PostgresDataConvertible {
+    public static var postgresDataType: PostgresDataType {
+        return .uuid
+    }
+    
+    public init?(postgresData: PostgresData) {
+        guard let uuid = postgresData.uuid else {
+            return nil
+        }
+        self = uuid
+    }
+
+    public var postgresData: PostgresData? {
+        return .init(uuid: self)
+    }
+}

Sources/PostgresNIO/Data/PostgresData.swift

@@ -1,6 +1,6 @@
 import NIO
 
-public struct PostgresData: CustomStringConvertible {
+public struct PostgresData: CustomStringConvertible, CustomDebugStringConvertible {
     public static var null: PostgresData {
         return .init(type: .null)
     }
@@ -30,23 +30,29 @@ public struct PostgresData: CustomStringConvertible {
         if let string = self.string {
             return string
         } else {
-            return self.debugDescription
+            let string: String
+            if var value = self.value {
+                switch self.formatCode {
+                case .text:
+                    let raw = value.readString(length: value.readableBytes) ?? ""
+                    string = "\"\(raw)\""
+                case .binary:
+                    string = "0x" + value.readableBytesView.hexdigest()
+                }
+            } else {
+                string = "<null>"
+            }
+            return string + " (\(self.type))"
         }
     }
-    
-    var debugDescription: String {
-        let string: String
-        if var value = self.value {
-            switch self.formatCode {
-            case .text:
-                let raw = value.readString(length: value.readableBytes) ?? ""
-                string = "\"\(raw)\""
-            case .binary:
-                string = "0x" + value.readableBytesView.hexdigest()
-            }
+
+    public var debugDescription: String {
+        let valueDescription: String
+        if let value = self.value {
+            valueDescription = "\(value.readableBytes.description) bytes"
         } else {
-            string = "<null>"
+            valueDescription = "nil"
         }
-        return string + " (\(self.type))"
+        return "PostgresData(type: \(self.type), value: \(valueDescription))"
     }
 }

circle.yml

@@ -66,4 +66,4 @@ workflows:
       - 10-bionic
       - 9-bionic
       - linux-release
-      - linux-performance
+      # - linux-performance