diff --git a/FBSnapshotTestCase.xcodeproj/project.pbxproj b/FBSnapshotTestCase.xcodeproj/project.pbxproj index c87bf0b..2cad090 100644 --- a/FBSnapshotTestCase.xcodeproj/project.pbxproj +++ b/FBSnapshotTestCase.xcodeproj/project.pbxproj @@ -50,6 +50,8 @@ BC45D5211C2AEFCE007C72F3 /* UIApplication+StrictKeyWindow.h in Headers */ = {isa = PBXBuildFile; fileRef = BC45D51F1C2AEFCE007C72F3 /* UIApplication+StrictKeyWindow.h */; }; BC45D5221C2AEFCE007C72F3 /* UIApplication+StrictKeyWindow.m in Sources */ = {isa = PBXBuildFile; fileRef = BC45D5201C2AEFCE007C72F3 /* UIApplication+StrictKeyWindow.m */; }; E5C2CD621B1F399A00669887 /* square_with_pixel.png in Resources */ = {isa = PBXBuildFile; fileRef = E5C2CD611B1F399A00669887 /* square_with_pixel.png */; }; + E704DBE4202B898800A48406 /* UIControl+SendActions.h in Headers */ = {isa = PBXBuildFile; fileRef = E704DBE2202B898800A48406 /* UIControl+SendActions.h */; }; + E704DBE5202B898800A48406 /* UIControl+SendActions.m in Sources */ = {isa = PBXBuildFile; fileRef = E704DBE3202B898800A48406 /* UIControl+SendActions.m */; }; F0D698F51B204E120005CAC9 /* SwiftSupport.swift in Sources */ = {isa = PBXBuildFile; fileRef = F0D698F41B204E120005CAC9 /* SwiftSupport.swift */; }; /* End PBXBuildFile section */ @@ -97,6 +99,8 @@ BC45D51F1C2AEFCE007C72F3 /* UIApplication+StrictKeyWindow.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = "UIApplication+StrictKeyWindow.h"; sourceTree = ""; }; BC45D5201C2AEFCE007C72F3 /* UIApplication+StrictKeyWindow.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path = "UIApplication+StrictKeyWindow.m"; sourceTree = ""; }; E5C2CD611B1F399A00669887 /* square_with_pixel.png */ = {isa = PBXFileReference; lastKnownFileType = image.png; path = square_with_pixel.png; sourceTree = ""; }; + E704DBE2202B898800A48406 /* UIControl+SendActions.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = "UIControl+SendActions.h"; sourceTree = ""; }; + E704DBE3202B898800A48406 /* UIControl+SendActions.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = "UIControl+SendActions.m"; sourceTree = ""; }; F0D698F41B204E120005CAC9 /* SwiftSupport.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = SwiftSupport.swift; sourceTree = ""; }; /* End PBXFileReference section */ @@ -145,6 +149,8 @@ 133564151B59C3F500A4E4BF /* UIImage+Snapshot.m */, BC45D51F1C2AEFCE007C72F3 /* UIApplication+StrictKeyWindow.h */, BC45D5201C2AEFCE007C72F3 /* UIApplication+StrictKeyWindow.m */, + E704DBE2202B898800A48406 /* UIControl+SendActions.h */, + E704DBE3202B898800A48406 /* UIControl+SendActions.m */, ); path = Categories; sourceTree = ""; @@ -242,6 +248,7 @@ files = ( BC45D5211C2AEFCE007C72F3 /* UIApplication+StrictKeyWindow.h in Headers */, B31988281AB7849400B0A900 /* FBSnapshotTestCase.h in Headers */, + E704DBE4202B898800A48406 /* UIControl+SendActions.h in Headers */, 13CBB39D1AEE013900B6ADBA /* FBSnapshotTestCasePlatform.h in Headers */, B319882A1AB7849400B0A900 /* FBSnapshotTestController.h in Headers */, 133564181B59C3F500A4E4BF /* UIImage+Diff.h in Headers */, @@ -443,6 +450,7 @@ files = ( 133564171B59C3F500A4E4BF /* UIImage+Compare.m in Sources */, BC45D5221C2AEFCE007C72F3 /* UIApplication+StrictKeyWindow.m in Sources */, + E704DBE5202B898800A48406 /* UIControl+SendActions.m in Sources */, B31988291AB7849400B0A900 /* FBSnapshotTestCase.m in Sources */, 133564191B59C3F500A4E4BF /* UIImage+Diff.m in Sources */, 1335641B1B59C3F500A4E4BF /* UIImage+Snapshot.m in Sources */, diff --git a/FBSnapshotTestCase/Categories/UIApplication+StrictKeyWindow.m b/FBSnapshotTestCase/Categories/UIApplication+StrictKeyWindow.m index 0f7a0c2..d935639 100644 --- a/FBSnapshotTestCase/Categories/UIApplication+StrictKeyWindow.m +++ b/FBSnapshotTestCase/Categories/UIApplication+StrictKeyWindow.m @@ -15,12 +15,8 @@ @implementation UIApplication (StrictKeyWindow) - (UIWindow *)fb_strictKeyWindow { UIWindow *keyWindow = [UIApplication sharedApplication].keyWindow; - if (!keyWindow) { - [NSException raise:@"FBSnapshotTestCaseNilKeyWindowException" - format:@"Snapshot tests must be hosted by an application with a key window. Please ensure your test" - " host sets up a key window at launch (either via storyboards or programmatically) and doesn't" - " do anything to remove it while snapshot tests are running."]; - } + NSString *message = @"Snapshot tests should be hosted by an application with a key window. Please ensure your test host sets up a key window at launch (either via storyboards or programmatically) and doesn't do anything to remove it while snapshot tests are running."; + NSLog(@"[iOS Snapshot Test Case] : %@", message); return keyWindow; } diff --git a/FBSnapshotTestCase/Categories/UIControl+SendActions.h b/FBSnapshotTestCase/Categories/UIControl+SendActions.h new file mode 100644 index 0000000..8f7846a --- /dev/null +++ b/FBSnapshotTestCase/Categories/UIControl+SendActions.h @@ -0,0 +1,36 @@ +/** + Copyright (c) 2018 Uber Technologies, Inc. + + Permission is hereby granted, free of charge, to any person obtaining a copy + of this software and associated documentation files (the "Software"), to deal + in the Software without restriction, including without limitation the rights + to use, copy, modify, merge, publish, distribute, sublicense, and/or sell + copies of the Software, and to permit persons to whom the Software is + furnished to do so, subject to the following conditions: + + The above copyright notice and this permission notice shall be included in + all copies or substantial portions of the Software. + + THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN + THE SOFTWARE. + */ + +#import + +@interface UIControl (SendActions) + +/** + In library test bundles with no test host, the default sendActionsForControlEvents: does not work. + + This replacement mimics the same idea of that method by finding all the targets associated with the control, finding all the actions on that target for the given control event, and invoking those actions on those targets. + + @param controlEvents A bitmask whose set flags specify the control events for which action messages are sent. + */ +- (void)ub_sendActionsForControlEvents:(UIControlEvents)controlEvents; + +@end diff --git a/FBSnapshotTestCase/Categories/UIControl+SendActions.m b/FBSnapshotTestCase/Categories/UIControl+SendActions.m new file mode 100644 index 0000000..6573610 --- /dev/null +++ b/FBSnapshotTestCase/Categories/UIControl+SendActions.m @@ -0,0 +1,51 @@ +/** + Copyright (c) 2018 Uber Technologies, Inc. + + Permission is hereby granted, free of charge, to any person obtaining a copy + of this software and associated documentation files (the "Software"), to deal + in the Software without restriction, including without limitation the rights + to use, copy, modify, merge, publish, distribute, sublicense, and/or sell + copies of the Software, and to permit persons to whom the Software is + furnished to do so, subject to the following conditions: + + The above copyright notice and this permission notice shall be included in + all copies or substantial portions of the Software. + + THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN + THE SOFTWARE. + */ + +#import "UIControl+SendActions.h" + +/** + UIControlEvents has options in the range 0-8, 12-13, 16-19. 9-11 are reserved for future UIControlEventTouch* options. 14-15 are reserved for other options. If new options are added after 19, this const will need to be updated. + */ +static NSUInteger const UIControlEventsMaxOffset = 19; + + +@implementation UIControl (UberTesting) + +- (void)ub_sendActionsForControlEvents:(UIControlEvents)controlEvents +{ + for (NSUInteger i = 0; i < UIControlEventsMaxOffset; i++) { + UIControlEvents controlEvent = 1 << i; + if (controlEvents & controlEvent) { + for (id target in self.allTargets) { + NSArray *targetActions = [self actionsForTarget:target forControlEvent:controlEvent]; + for (NSString *action in targetActions) { + SEL selector = NSSelectorFromString(action); + IMP imp = [target methodForSelector:selector]; + void (*func)(id, SEL, id) = (void *)imp; + func(target, selector, self); + } + } + } + } +} + +@end diff --git a/README.md b/README.md index 32bf3b8..1db9062 100644 --- a/README.md +++ b/README.md @@ -1,7 +1,7 @@ iOSSnapshotTestCase (previously named FBSnapshotTestCase) ====================== -[![Build Status](https://travis-ci.org/uber/ios-snapshot-test-case.svg)](https://travis-ci.org/uber/ios-snapshot-test-case) +[![Build Status](https://travis-ci.org/uber/ios-snapshot-test-case.svg)](https://travis-ci.org/uber/ios-snapshot-test-case) [![CocoaPods Compatible](https://img.shields.io/cocoapods/v/iOSSnapshotTestCase.svg)](https://img.shields.io/cocoapods/v/iOSSnapshotTestCase.svg) What it does @@ -82,11 +82,17 @@ Features Notes ----- -Your unit test must be an "application test", not a "logic test." (That is, it -must be run within the Simulator so that it has access to UIKit.) In Xcode 5 +Your unit tests _should_ be inside an "application" bundle, not a "logic/library" test bundle. (That is, it +should be run within the Simulator so that it has access to UIKit.) + +In Xcode 5 and later new projects only offer application tests, but older projects will have separate targets for the two types. +However, if you are writing snapshot tests inside a library/framework, you might want to keep your test bundle as a library test bundle without a Test Host or Simulator. + +Read more on this [here](docs/LibraryVsApplicationTestBundles.md). + Authors ------- diff --git a/docs/LibraryVsApplicationTestBundles.md b/docs/LibraryVsApplicationTestBundles.md new file mode 100644 index 0000000..65c6682 --- /dev/null +++ b/docs/LibraryVsApplicationTestBundles.md @@ -0,0 +1,22 @@ +# Library vs Application Test Bundles + +* Authors: [Alan Zeino](https://github.com/alanzeino) + +## Introduction + +Developers writing unit tests for iOS using XCTest typically use Application Test Bundles without much consideration as to _why_ their tests need to be inside an Application Test Bundle, as opposed to a Library Test Bundle. This document describes what you can and can't do with both, and why you might want to use a Library Test Bundle. + +Library Test Bundles were once called _Logic_ Test Bundles in Apple's nomenclature. In the context of this document, both Library and Logic are interchangeable. + +### Application tests + +Unit tests that test parts of an application (such as UIViewControllers, UIWindows, UIViews) should typically be part of an Application test bundle. An Application test bundle requires a Test Host and at test run time, a Simulator too. + +There are some APIs that only work inside Application test bundles. In our testing, we've seen a few. Here are some: + +* `-[UIControl sendActionsForControlEvents:]` — This API is commonly used to trigger actions at runtime and sometimes you might want to use it inside a test to trigger a particular code path which is ordinarily run when a user performs an action. While it does not work inside a Library test bundle, [we've written our own version for unit tests](FBSnapshotTestCase/Categories/UIControl+SendActions.h) that works well for this need. If you decide to use that category, make sure it can only be seen inside unit tests and not all of your code. +* `UIAppearance` — Most `UIAppearance` APIs break when there is no test host present. +* `UIWindow` — You cannot make a `UIWindow` you created during your test the 'key window' because `makeKeyAndVisible` crashes at test run time. One workaround is to instead set `hidden` to `false` on the `UIWindow` instance you created. However there still won't be a 'key window' so if you have code that adds a `UIView` as a subview of the `keyWindow` then that will break. + +### Library tests +Unit tests that test parts of a framework or libary should be part of a Library test bundle. This does not require a Test Host or a Simulator (though in Xcode 9, Apple still launches a Simulator for these tests). \ No newline at end of file