Skip to content

Commit 95e5c75

Browse files
committed
checking for 404 error after loadingMessageContent
1 parent 9f632e2 commit 95e5c75

File tree

2 files changed

+26
-41
lines changed

2 files changed

+26
-41
lines changed

iOS_SDK/OneSignalSDK/Source/OSInAppMessageViewController.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -266,7 +266,7 @@ - (void)loadPreviewMessageContent {
266266

267267
- (void)encounteredErrorLoadingMessageContent:(NSError * _Nullable)error {
268268
let message = [NSString stringWithFormat:@"An error occurred while attempting to load message content: %@", error.description ?: @"Unknown Error"];
269-
if (error.code == 410 || [error.description containsString:@"In-App Message is not active"]) {
269+
if (error.code == 410 || error.code == 404) {
270270
[self.delegate messageIsNotActive:self.message];
271271
}
272272
[OneSignal onesignal_Log:ONE_S_LL_ERROR message:message];

iOS_SDK/OneSignalSDK/UnitTests/UnitTestCommonMethods.m

Lines changed: 25 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -157,28 +157,9 @@ + (void)runLongBackgroundThreads {
157157

158158
[[NSRunLoop mainRunLoop] runUntilDate:[NSDate dateWithTimeIntervalSinceNow:0.05]];
159159

160-
// the httpQueue makes sure all HTTP request mocks are sync'ed
161-
162-
dispatch_queue_t registerUserQueue, notifSettingsQueue;
163160
for(int i = 0; i < 10; i++) {
164161
[[NSRunLoop mainRunLoop] runUntilDate:[NSDate dateWithTimeIntervalSinceNow:0.1]];
165-
[OneSignalHelperOverrider runBackgroundThreads];
166-
167-
notifSettingsQueue = [OneSignalNotificationSettingsIOS10 getQueue];
168-
if (notifSettingsQueue)
169-
dispatch_sync(notifSettingsQueue, ^{});
170-
171-
registerUserQueue = [OneSignal getRegisterQueue];
172-
if (registerUserQueue)
173-
dispatch_sync(registerUserQueue, ^{});
174-
175-
[OneSignalClientOverrider runBackgroundThreads];
176-
177-
[UNUserNotificationCenterOverrider runBackgroundThreads];
178-
179-
dispatch_barrier_sync(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^{});
180-
181-
[UIApplicationOverrider runBackgroundThreads];
162+
[UnitTestCommonMethods runThreadsOnEachQueue];
182163
}
183164

184165
NSLog(@"END runLongBackgroundThreads");
@@ -192,32 +173,36 @@ + (void)runBackgroundThreads {
192173

193174
[[NSRunLoop mainRunLoop] runUntilDate:[NSDate dateWithTimeIntervalSinceNow:0.05]];
194175

195-
// the httpQueue makes sure all HTTP request mocks are sync'ed
196-
197-
dispatch_queue_t registerUserQueue, notifSettingsQueue;
198176
for(int i = 0; i < 10; i++) {
199-
[OneSignalHelperOverrider runBackgroundThreads];
200-
201-
notifSettingsQueue = [OneSignalNotificationSettingsIOS10 getQueue];
202-
if (notifSettingsQueue)
203-
dispatch_sync(notifSettingsQueue, ^{});
204-
205-
registerUserQueue = [OneSignal getRegisterQueue];
206-
if (registerUserQueue)
207-
dispatch_sync(registerUserQueue, ^{});
208-
209-
[OneSignalClientOverrider runBackgroundThreads];
210-
211-
[UNUserNotificationCenterOverrider runBackgroundThreads];
212-
213-
dispatch_barrier_sync(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^{});
214-
215-
[UIApplicationOverrider runBackgroundThreads];
177+
[UnitTestCommonMethods runThreadsOnEachQueue];
216178
}
217179

218180
NSLog(@"END runBackgroundThreads");
219181
}
220182

183+
+ (void)runThreadsOnEachQueue {
184+
// the httpQueue makes sure all HTTP request mocks are sync'ed
185+
dispatch_queue_t registerUserQueue, notifSettingsQueue;
186+
187+
[OneSignalHelperOverrider runBackgroundThreads];
188+
189+
notifSettingsQueue = [OneSignalNotificationSettingsIOS10 getQueue];
190+
if (notifSettingsQueue)
191+
dispatch_sync(notifSettingsQueue, ^{});
192+
193+
registerUserQueue = [OneSignal getRegisterQueue];
194+
if (registerUserQueue)
195+
dispatch_sync(registerUserQueue, ^{});
196+
197+
[OneSignalClientOverrider runBackgroundThreads];
198+
199+
[UNUserNotificationCenterOverrider runBackgroundThreads];
200+
201+
dispatch_barrier_sync(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^{});
202+
203+
[UIApplicationOverrider runBackgroundThreads];
204+
}
205+
221206
+ (void)clearStateForAppRestart:(XCTestCase *)testCase {
222207
NSLog(@"======= APP RESTART ======\n\n");
223208

0 commit comments

Comments
 (0)