Merge pull request #426 from nimblehq/feature/#425-KIF

[#425] As a developer, I would like to have KIF setup when I start a new project

.github/wiki/Project-Configurations.md

@@ -15,7 +15,7 @@ There are 3 default targets in a project:
 
 - {ProjectName}
 - {ProjectName}Tests
-- {ProjectName}UITests
+- {ProjectName}KIFUITests
 
 ### Schemes
 

.github/wiki/Standard-File-Organization.md

@@ -103,10 +103,9 @@ To keep all current and upcoming iOS projects aligned, we standardize an iOS pro
 │           ├── Data+Decode.swift
 │           ├── String+Data.swift
 │           └── TestError.swift
-└── {ProjectName}UITests
+└── {ProjectName}KIFUITests
     ├── Configurations
     │   └── Plists
-    ├── Resources
     └── Sources
         ├── AccessibilityIdentifiers
         │   ├── Login
@@ -121,9 +120,7 @@ To keep all current and upcoming iOS projects aligned, we standardize an iOS pro
         │   ├── Login
         │   └── Home
         └── Utilities
-            ├── Data+Decode.swift
-            ├── String+Data.swift
-            └── TestError.swift
+            └── KIF+Swift.swift
 ```
 
 ## README.md
@@ -146,6 +143,6 @@ This folder contains the main sources of the project. There are three sub-folder
 
 This folder contains the unit testing and integration testing of the main project.
 
-## {ProjectName}UITests
+## {ProjectName}KIFUITests
 
-This folder contains the UI testing of the main project.
+This folder contains the KIF UI testing of the main project. Use KIF instead of XCUITest for speed and reliability.

.swiftlint.yml

@@ -1,7 +1,7 @@
 included:
   - {PROJECT_NAME}
   - {PROJECT_NAME}Tests
-  - {PROJECT_NAME}UITests
+  - {PROJECT_NAME}KIFUITests
 
 excluded:
   - {PROJECT_NAME}Tests/Sources/Mocks/Sourcery/AutoMockable.generated.swift

Podfile

@@ -42,8 +42,10 @@ target '{PROJECT_NAME}' do
     testing_pods
   end
 
-  target '{PROJECT_NAME}UITests' do
+  target '{PROJECT_NAME}KIFUITests' do
     testing_pods
+    pod 'KIF', :configurations => ['Debug Staging', 'Debug Production']
+    pod 'KIF/IdentifierTests', :configurations => ['Debug Staging', 'Debug Production']
   end
 end
 

Project.swift

@@ -19,11 +19,12 @@ extension Project {
             targets: [
                 .mainTarget(name: name, bundleId: bundleId),
                 .testsTarget(name: name, bundleId: bundleId),
-                .uiTestsTarget(name: name, bundleId: bundleId)
+                .kifUITestsTarget(name: name, bundleId: bundleId),
             ],
             schemes: [
                 .productionScheme(name: name),
-                .stagingScheme(name: name)
+                .stagingScheme(name: name),
+                .kifUITestsScheme(name: name)
             ]
         )
     }

Tuist/ProjectDescriptionHelpers/Scheme+Initializing.swift

@@ -9,7 +9,10 @@ extension Scheme {
             name: name,
             shared: true,
             buildAction: .buildAction(targets: ["\(name)"]),
-            testAction: .targets(["\(name)Tests", "\(name)UITests"], configuration: debugConfigName),
+            testAction: .targets(
+                ["\(name)Tests", "\(name)KIFUITests"],
+                configuration: debugConfigName
+            ),
             runAction: .runAction(configuration: debugConfigName),
             archiveAction: .archiveAction(configuration: releaseConfigName),
             profileAction: .profileAction(configuration: debugConfigName),
@@ -24,11 +27,24 @@ extension Scheme {
             name: "\(name) Staging",
             shared: true,
             buildAction: .buildAction(targets: ["\(name)"]),
-            testAction: .targets(["\(name)Tests", "\(name)UITests"], configuration: debugConfigName),
+            testAction: .targets(
+                ["\(name)Tests", "\(name)KIFUITests"],
+                configuration: debugConfigName
+            ),
             runAction: .runAction(configuration: debugConfigName),
             archiveAction: .archiveAction(configuration: releaseConfigName),
             profileAction: .profileAction(configuration: debugConfigName),
             analyzeAction: .analyzeAction(configuration: debugConfigName)
         )
     }
+
+    public static func kifUITestsScheme(name: String) -> Scheme {
+        let debugConfigName = BuildConfiguration.debugStaging.name
+        let releaseConfigName = BuildConfiguration.releaseStaging.name
+        return Scheme(
+            name: "\(name)KIFUITests",
+            shared: false,
+            hidden: true
+        )
+    }
 }

Tuist/ProjectDescriptionHelpers/Target+Initializing.swift

@@ -48,18 +48,17 @@ extension Target {
         )
     }
 
-    public static func uiTestsTarget(name: String, bundleId: String) -> Target {
-        let targetName = "\(name)UITests"
+    public static func kifUITestsTarget(name: String, bundleId: String) -> Target {
+        let targetName = "\(name)KIFUITests"
         return Target(
             name: targetName,
             platform: .iOS,
-            product: .uiTests,
+            product: .unitTests,
             bundleId: bundleId,
             infoPlist: "\(targetName)/\(plistsPath)/Info.plist",
             sources: ["\(targetName)/**"],
             resources: [
                 "\(targetName)/**/.gitkeep", // To include empty folders
-                "\(targetName)/Resources/**/*"
             ], 
             dependencies: [.target(name: name)]
         )

fastlane/Constants/Constant.swift

@@ -71,7 +71,7 @@ enum Constant {
     // MARK: - Test
 
     static let testTarget: String = "\(projectName)Tests"
-    static let uiTestTarget: String = "\(projectName)UITests"
+    static let kifUITestTarget: String = "\(projectName)KIFUITests"
 }
 
 extension Constant {

fastlane/Fastfile.swift

@@ -137,7 +137,10 @@ class Fastfile: LaneFile {
         desc("Build and Test project")
         Test.buildAndTest(
             environment: .staging,
-            targets: [Constant.testTarget, Constant.uiTestTarget],
+            targets: [
+                Constant.testTarget,
+                Constant.kifUITestTarget
+            ],
             devices: Constant.devices
         )
     }

make.sh

@@ -123,8 +123,8 @@ rename_folder(){
 # Rename test folder structure
 rename_folder "${CONSTANT_PROJECT_NAME}Tests" "${PROJECT_NAME_NO_SPACES}Tests"
 
-# Rename UI Test folder structure
-rename_folder "${CONSTANT_PROJECT_NAME}UITests" "${PROJECT_NAME_NO_SPACES}UITests"
+# Rename KIF UI Test folder structure
+rename_folder "${CONSTANT_PROJECT_NAME}KIFUITests" "${PROJECT_NAME_NO_SPACES}KIFUITests"
 
 # Rename app folder structure
 rename_folder "${CONSTANT_PROJECT_NAME}" "${PROJECT_NAME_NO_SPACES}"

{PROJECT_NAME}KIFUITests/Sources/Specs/Application/ApplicationSpec.swift

@@ -0,0 +1,31 @@
+//
+//  ApplicationSpec.swift
+//
+
+import Foundation
+import Nimble
+import Quick
+
+final class ApplicationSpec: QuickSpec {
+
+    override func spec() {
+
+        describe("a {PROJECT_NAME} screen") {
+
+            beforeEach {
+                // Navigate to the testing screen
+            }
+
+            afterEach {
+                // Navigate to neutral state
+            }
+
+            context("when opens") {
+
+                it("shows its UI components") {
+                    self.tester().waitForView(withAccessibilityLabel: "Hello")
+                }
+            }
+        }
+    }
+}

{PROJECT_NAME}KIFUITests/Sources/Utilities/KIF+Swift.swift

@@ -0,0 +1,17 @@
+//  swiftlint:disable:this file_name
+//
+//  KIF+Swift.swift
+//
+
+import KIF
+
+extension XCTestCase {
+
+    func tester(file: String = #file, _ line: Int = #line) -> KIFUITestActor {
+        return KIFUITestActor(inFile: file, atLine: line, delegate: self)
+    }
+
+    func system(file: String = #file, _ line: Int = #line) -> KIFSystemTestActor {
+        return KIFSystemTestActor(inFile: file, atLine: line, delegate: self)
+    }
+}

{PROJECT_NAME}Tests/Sources/Specs/Supports/Extensions/Foundation/OptionalUnwrapSpec.swift

@@ -1,3 +1,7 @@
+//
+//  OptionalUnwrapSpec.swift
+//
+
 import Nimble
 import Quick
 
@@ -26,7 +30,7 @@ final class OptionalUnwrapSpec: QuickSpec {
                 }
 
                 it("returns empty string") {
-                    expect(value.string) == ""
+                    expect(value.string.isEmpty) == true
                 }
             }
         }