From 17a8eedb4ef1898f6c86c6765e7ea2d9bc7d6317 Mon Sep 17 00:00:00 2001 From: Mark Adams Date: Wed, 16 Dec 2015 13:50:01 -0800 Subject: [PATCH 1/2] Fix will never be executed warning --- DVR/Session.swift | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/DVR/Session.swift b/DVR/Session.swift index 82bc206..4d95076 100644 --- a/DVR/Session.swift +++ b/DVR/Session.swift @@ -143,6 +143,10 @@ public class Session: NSURLSession { } private func persist(interactions: [Interaction]) { + defer { + abort() + } + // Create directory let outputDirectory = (self.outputDirectory as NSString).stringByExpandingTildeInPath let fileManager = NSFileManager.defaultManager() @@ -151,13 +155,14 @@ public class Session: NSURLSession { try fileManager.createDirectoryAtPath(outputDirectory, withIntermediateDirectories: true, attributes: nil) } catch { print("[DVR] Failed to create cassettes directory.") - abort() } } let cassette = Cassette(name: cassetteName, interactions: interactions) // Persist + + do { let outputPath = ((outputDirectory as NSString).stringByAppendingPathComponent(cassetteName) as NSString).stringByAppendingPathExtension("json")! let data = try NSJSONSerialization.dataWithJSONObject(cassette.dictionary, options: [.PrettyPrinted]) @@ -165,21 +170,18 @@ public class Session: NSURLSession { // Add trailing new line guard var string = NSString(data: data, encoding: NSUTF8StringEncoding) else { print("[DVR] Failed to persist cassette.") - abort() + return } string = string.stringByAppendingString("\n") if let data = string.dataUsingEncoding(NSUTF8StringEncoding) { data.writeToFile(outputPath, atomically: true) print("[DVR] Persisted cassette at \(outputPath). Please add this file to your test target") - abort() } print("[DVR] Failed to persist cassette.") - abort() } catch { print("[DVR] Failed to persist cassette.") - abort() } } From 987c405b9fb6a349c76da6716fdc4ab50ca402b2 Mon Sep 17 00:00:00 2001 From: Mark Adams Date: Wed, 16 Dec 2015 13:58:35 -0800 Subject: [PATCH 2/2] Don't require app extension safe API on test targets --- DVR.xcodeproj/project.pbxproj | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/DVR.xcodeproj/project.pbxproj b/DVR.xcodeproj/project.pbxproj index 6174223..46db810 100644 --- a/DVR.xcodeproj/project.pbxproj +++ b/DVR.xcodeproj/project.pbxproj @@ -551,6 +551,7 @@ 3647AFB31B335D5500EF10D4 /* Debug */ = { isa = XCBuildConfiguration; buildSettings = { + APPLICATION_EXTENSION_API_ONLY = NO; INFOPLIST_FILE = DVR/Tests/Info.plist; LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks @loader_path/Frameworks"; PRODUCT_BUNDLE_IDENTIFIER = com.venmo.DVR.iostests; @@ -561,6 +562,7 @@ 3647AFB41B335D5500EF10D4 /* Release */ = { isa = XCBuildConfiguration; buildSettings = { + APPLICATION_EXTENSION_API_ONLY = NO; INFOPLIST_FILE = DVR/Tests/Info.plist; LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks @loader_path/Frameworks"; PRODUCT_BUNDLE_IDENTIFIER = com.venmo.DVR.iostests; @@ -609,6 +611,7 @@ 3690A0901B33AA3C00731222 /* Debug */ = { isa = XCBuildConfiguration; buildSettings = { + APPLICATION_EXTENSION_API_ONLY = NO; COMBINE_HIDPI_IMAGES = YES; INFOPLIST_FILE = DVR/Tests/Info.plist; LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/../Frameworks @loader_path/../Frameworks"; @@ -621,6 +624,7 @@ 3690A0911B33AA3C00731222 /* Release */ = { isa = XCBuildConfiguration; buildSettings = { + APPLICATION_EXTENSION_API_ONLY = NO; COMBINE_HIDPI_IMAGES = YES; INFOPLIST_FILE = DVR/Tests/Info.plist; LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/../Frameworks @loader_path/../Frameworks";