Skip to content

Commit 21d08e8

Browse files
committed
Fixing testUnsupportedCatalyst unit test
1 parent 533af6a commit 21d08e8

File tree

3 files changed

+16
-3
lines changed

3 files changed

+16
-3
lines changed

iOS_SDK/OneSignalSDK/OneSignal.xcodeproj/project.pbxproj

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2334,6 +2334,7 @@
23342334
INFOPLIST_FILE = UnitTests/Info.plist;
23352335
IPHONEOS_DEPLOYMENT_TARGET = 10.1;
23362336
LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks @loader_path/Frameworks";
2337+
MACOSX_DEPLOYMENT_TARGET = 10.15;
23372338
MTL_ENABLE_DEBUG_INFO = NO;
23382339
OTHER_CFLAGS = "-fembed-bitcode";
23392340
PRODUCT_BUNDLE_IDENTIFIER = com.onesignal.UnitTests;
@@ -2489,6 +2490,7 @@
24892490
INFOPLIST_FILE = UnitTests/Info.plist;
24902491
IPHONEOS_DEPLOYMENT_TARGET = 10.1;
24912492
LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks @loader_path/Frameworks";
2493+
MACOSX_DEPLOYMENT_TARGET = 10.15;
24922494
MTL_ENABLE_DEBUG_INFO = NO;
24932495
OTHER_CFLAGS = "-fembed-bitcode";
24942496
PRODUCT_BUNDLE_IDENTIFIER = com.onesignal.UnitTests;
@@ -2581,12 +2583,14 @@
25812583
GCC_C_LANGUAGE_STANDARD = gnu11;
25822584
INFOPLIST_FILE = UnitTestApp/Info.plist;
25832585
IPHONEOS_DEPLOYMENT_TARGET = 9.0;
2586+
"IPHONEOS_DEPLOYMENT_TARGET[sdk=macosx*]" = 13.1;
25842587
LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks";
25852588
MTL_ENABLE_DEBUG_INFO = NO;
25862589
MTL_FAST_MATH = YES;
25872590
OTHER_CFLAGS = "-fembed-bitcode";
25882591
PRODUCT_BUNDLE_IDENTIFIER = com.onesignal.example.UnitTestApp;
25892592
PRODUCT_NAME = "$(TARGET_NAME)";
2593+
SUPPORTS_MACCATALYST = YES;
25902594
TARGETED_DEVICE_FAMILY = "1,2";
25912595
};
25922596
name = Release;
@@ -2629,13 +2633,15 @@
26292633
);
26302634
INFOPLIST_FILE = UnitTestApp/Info.plist;
26312635
IPHONEOS_DEPLOYMENT_TARGET = 9.0;
2636+
"IPHONEOS_DEPLOYMENT_TARGET[sdk=macosx*]" = 13.1;
26322637
LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks";
26332638
MTL_ENABLE_DEBUG_INFO = INCLUDE_SOURCE;
26342639
MTL_FAST_MATH = YES;
26352640
ONLY_ACTIVE_ARCH = NO;
26362641
OTHER_CFLAGS = "-fembed-bitcode";
26372642
PRODUCT_BUNDLE_IDENTIFIER = com.onesignal.example.UnitTestApp;
26382643
PRODUCT_NAME = "$(TARGET_NAME)";
2644+
SUPPORTS_MACCATALYST = YES;
26392645
TARGETED_DEVICE_FAMILY = "1,2";
26402646
};
26412647
name = Debug;

iOS_SDK/OneSignalSDK/UnitTestApp/UnitTestApp.entitlements

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,9 @@
44
<dict>
55
<key>aps-environment</key>
66
<string>development</string>
7+
<key>com.apple.security.app-sandbox</key>
8+
<true/>
9+
<key>com.apple.security.network.client</key>
10+
<true/>
711
</dict>
812
</plist>

iOS_SDK/OneSignalSDK/UnitTests/InAppMessagingTests.m

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -154,13 +154,16 @@ -(void)testOldUnsupportedIpadSimulator {
154154
XCTAssertEqualObjects(sharedInstance.class, DummyOSMessagingController.class); // sharedInstance should be dummy controller
155155
}
156156

157+
// This test needs to be run with Mac Catalyst as the build target
157158
-(void)testUnsupportedCatalyst {
158159
OneSignalHelperOverrider.mockIOSVersion = 10;
159160
[OSMessagingController removeInstance];
160-
[OneSignalHelperOverrider setSystemInfoMachine:@"x86_64"];
161-
[UIDeviceOverrider setSystemName:@"Mac OS X"]; // e.g. @"Mac OS X" @"iOS"
162161
let sharedInstance = OSMessagingController.sharedInstance;
163-
XCTAssertEqualObjects(sharedInstance.class, DummyOSMessagingController.class); // sharedInstance should be dummy controller
162+
#if TARGET_OS_MACCATALYST
163+
XCTAssertEqualObjects(sharedInstance.class, DummyOSMessagingController.class); // sharedInstance should be dummy controller
164+
#else
165+
XCTAssertEqualObjects(sharedInstance.class, OSMessagingController.class); // sharedInstance should be the true controller
166+
#endif
164167
}
165168

166169
-(void)testRealIphone {

0 commit comments

Comments
 (0)