Add xcodebuild exit status to travis

.travis.yml

@@ -23,13 +23,15 @@ jobs:
       install: skip
       env: TEST_DEVICE='platform=iOS Simulator,OS=12.2,name=iPhone SE'
       script:
-        - xcodebuild -scheme TokamakUIKit -sdk iphonesimulator | xcpretty
-        - xcodebuild -scheme TokamakAppKit -sdk macosx | xcpretty
-        - xcodebuild test -scheme TokamakCLI -sdk macosx | xcpretty
+        - swift package generate-xcodeproj --xcconfig-overrides Package.xcconfig
+        - git checkout HEAD Tokamak.xcodeproj/xcshareddata/xcschemes/TokamakCLI.xcscheme
+        - set -o pipefail && xcodebuild -scheme TokamakUIKit -sdk iphonesimulator | xcpretty
+        - set -o pipefail && xcodebuild -scheme TokamakAppKit -sdk macosx | xcpretty
+        - set -o pipefail && xcodebuild test -scheme TokamakCLI -sdk macosx | xcpretty
         # this runs the build the second time, but it's the only way to make sure
         # that `Package.swift` is in a good state
         - swift build --target Tokamak
-        - xcodebuild test -enableCodeCoverage YES -scheme Tokamak | xcpretty
+        - set -o pipefail && xcodebuild test -enableCodeCoverage YES -scheme Tokamak | xcpretty
       after_success:
         - bash <(curl -s https://codecov.io/bash)
 # before_install:
@@ -41,5 +43,5 @@ jobs:
 # - gem install cocoapods # Since Travis is not always on latest version
 # - pod install --project-directory=Example
 # - swift build
-# - set -o pipefail && xcodebuild test -enableCodeCoverage YES -workspace Example/Tokamak.xcworkspace -scheme Tokamak-Example -sdk iphonesimulator11.0 ONLY_ACTIVE_ARCH=NO | xcpretty
+# - set -o pipefail && set -o pipefail && xcodebuild test -enableCodeCoverage YES -workspace Example/Tokamak.xcworkspace -scheme Tokamak-Example -sdk iphonesimulator11.0 ONLY_ACTIVE_ARCH=NO | xcpretty
 # - pod lib lint

Sources/Tokamak/Components/Props/Color.swift

@@ -49,46 +49,18 @@ extension Color: ExpressibleByIntegerLiteral {
 
 extension Color {
   public init?(hex: String) {
-    let cString = hex.utf8CString
-
-    // - 1 for the trailing null terminator
-    let hexSize = cString.count - 1
-
-    // If the first character is a '#', skip it
-    var offset = cString.first == 0x23 ? 1 : 0
-
-    // We only support 6 hexadecimal characters
-    if hexSize - offset != 6 {
-      return nil
-    }
-
-    func nextByte() -> Int8? {
-      // Take the first byte as the high 4 bits
-      // Then the second byte as the low 4 bits
-      if
-        let high = cString[offset].hexDecoded(),
-        let low = cString[offset].hexDecoded() {
-        // In this case, unchecked is still safe as it's between 0 and 6
-        offset = offset &+ 2
-
-        // Adds the two 4-bit pairs together to form a full byte
-        return (high << 4) & low
-      }
-
-      return nil
-    }
+    let cArray = Array(hex.replacingOccurrences(of: "#", with: ""))
 
     guard
-      let red = nextByte(),
-      let green = nextByte(),
-      let blue = nextByte()
+      let red = Int(String(cArray[0...1]), radix: 16),
+      let green = Int(String(cArray[2...3]), radix: 16),
+      let blue = Int(String(cArray[4...5]), radix: 16)
     else {
       return nil
     }
-
-    self.red = Double(UInt8(bitPattern: red)) / 255
-    self.green = Double(UInt8(bitPattern: green)) / 255
-    self.blue = Double(UInt8(bitPattern: blue)) / 255
+    self.red = Double(red) / 255
+    self.green = Double(green) / 255
+    self.blue = Double(blue) / 255
     alpha = 1
     space = .sRGB
   }

Tests/TokamakTests/ColorTests.swift

@@ -17,5 +17,32 @@ final class ColorTests: XCTestCase {
       Color(hex: "FF00FF"),
       "The '#' before a hex code produced a different output than without it"
     )
+
+    guard let red = Color(hex: "#FF0000") else {
+      XCTFail("Hexadecimal decoding failed")
+      return
+    }
+
+    XCTAssertEqual(red.red, 1)
+    XCTAssertEqual(red.green, 0)
+    XCTAssertEqual(red.blue, 0)
+
+    guard let green = Color(hex: "#00FF00") else {
+      XCTFail("Hexadecimal decoding failed")
+      return
+    }
+
+    XCTAssertEqual(green.red, 0)
+    XCTAssertEqual(green.green, 1)
+    XCTAssertEqual(green.blue, 0)
+
+    guard let blue = Color(hex: "#0000FF") else {
+      XCTFail("Hexadecimal decoding failed")
+      return
+    }
+
+    XCTAssertEqual(blue.red, 0)
+    XCTAssertEqual(blue.green, 0)
+    XCTAssertEqual(blue.blue, 1)
   }
 }