From 0bde29e19792679bfad9bf890e8b2e7c2e649987 Mon Sep 17 00:00:00 2001 From: Peter Argany Date: Thu, 7 Feb 2019 10:40:14 -0800 Subject: [PATCH] Consider SSTs in Platform.isTesting Summary: A common util from RN to gate on testing code is `Platform.isTesting()` Unfortunately, this util does not account for ServerSnapshotTests, since they don't use apple's XCTest infra. Reviewed By: sahrens Differential Revision: D13981728 fbshipit-source-id: bf902a04f5d7fcb98a06816f5c2c9b082e7d14b8 --- React/Base/RCTUtils.m | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/React/Base/RCTUtils.m b/React/Base/RCTUtils.m index 13067cc544a..44e1e7fe5c1 100644 --- a/React/Base/RCTUtils.m +++ b/React/Base/RCTUtils.m @@ -458,8 +458,10 @@ BOOL RCTRunningInTestEnvironment(void) static dispatch_once_t onceToken; dispatch_once(&onceToken, ^{ NSDictionary *environment = [[NSProcessInfo processInfo] environment]; - isTestEnvironment = objc_lookUpClass("SenTestCase") || objc_lookUpClass("XCTest") || - [environment[@"IS_TESTING"] boolValue]; + isTestEnvironment = objc_lookUpClass("SenTestCase") || + objc_lookUpClass("XCTest") || + objc_lookUpClass("SnapshotTestAppDelegate") || + [environment[@"IS_TESTING"] boolValue]; }); return isTestEnvironment; }