Merge pull request #183 from cashapp/krueger/inverted-colors-identifier

Include the identifier in the call to FBSnapshotVerifyView for SnapshotVerifyWithInvertedColors

Author: kruegermj

Example/AccessibilitySnapshot/InvertColorsViewController.swift

@@ -33,6 +33,14 @@ final class InvertColorsViewController: AccessibilityViewController {
         view = View()
     }
 
+    override func viewDidLoad() {
+        super.viewDidLoad()
+
+        // This additional call to updateStatusLabel() is needed because
+        // of a view life cycle issue with the snapshot test framework
+        updateStatusLabel()
+    }
+
     override func viewWillAppear(_ animated: Bool) {
         super.viewWillAppear(animated)
 

Example/SnapshotTests/InvertColorsTests.swift

@@ -25,7 +25,19 @@ final class InvertColorsTests: SnapshotTestCase {
         let viewController = InvertColorsViewController()
         viewController.view.frame = UIScreen.main.bounds
 
+        FBSnapshotVerifyView(viewController.view, identifier: "disabled")
+
         SnapshotVerifyWithInvertedColors(viewController.view)
+
+        // Run the plain snapshot a second time to ensure that the view was restored to its original state.
+        FBSnapshotVerifyView(viewController.view, identifier: "disabled")
+    }
+
+    func testInvertColorsWithIdentifier() {
+        let viewController = InvertColorsViewController()
+        viewController.view.frame = UIScreen.main.bounds
+
+        SnapshotVerifyWithInvertedColors(viewController.view, identifier: "someIdentifier")
     }
 
 }

Example/SnapshotTests/ReferenceImages/_64/SnapshotTests.InvertColorsTests/testInvertColorsWithIdentifier_someIdentifier_13_7_375x812@3x.png

@@ -137,7 +137,7 @@ extension FBSnapshotTestCase {
         }
 
         let imageView = UIImageView(image: image)
-        FBSnapshotVerifyView(imageView, suffixes: suffixes, file: file, line: line)
+        FBSnapshotVerifyView(imageView, identifier: identifier, suffixes: suffixes, file: file, line: line)
 
         statusUtility.unmockStatuses()
         postNotification()