Fix #827

* Add a public method for testing and transparency
* Add tests

SmartDeviceLink/SDLFileManager.h

@@ -31,6 +31,11 @@ typedef void (^SDLFileManagerStartupCompletionHandler)(BOOL success, NSError *__
  */
 @property (copy, nonatomic, readonly) NSSet<SDLFileName *> *remoteFileNames;
 
+/**
+ A set of all names of files uploaded this session (and this session only)
+ */
+@property (copy, nonatomic, readonly) NSSet<SDLFileName *> *uploadedEphemeralFileNames;
+
 /**
  *  The number of bytes still available for files for this app.
  */

SmartDeviceLink/SDLFileManager.m

@@ -40,13 +40,13 @@ @interface SDLFileManager ()
 @property (weak, nonatomic) id<SDLConnectionManagerType> connectionManager;
 
 // Remote state
-@property (strong, nonatomic, readwrite) NSMutableSet<SDLFileName *> *mutableRemoteFileNames;
+@property (strong, nonatomic) NSMutableSet<SDLFileName *> *mutableRemoteFileNames;
 @property (assign, nonatomic, readwrite) NSUInteger bytesAvailable;
 
 // Local state
 @property (strong, nonatomic) NSOperationQueue *transactionQueue;
 @property (strong, nonatomic) NSMutableDictionary<SDLFileName *, NSOperation *> *uploadsInProgress;
-@property (strong, nonatomic) NSMutableSet<SDLFileName *> *localUploadedFileNames;
+@property (strong, nonatomic) NSMutableSet<SDLFileName *> *mutableUploadedEphemeralFileNames;
 @property (strong, nonatomic) SDLStateMachine *stateMachine;
 @property (copy, nonatomic, nullable) SDLFileManagerStartupCompletionHandler startupCompletionHandler;
 
@@ -72,7 +72,7 @@ - (instancetype)initWithConnectionManager:(id<SDLConnectionManagerType>)manager
     _transactionQueue.name = @"SDLFileManager Transaction Queue";
     _transactionQueue.maxConcurrentOperationCount = 1;
     _uploadsInProgress = [[NSMutableDictionary alloc] init];
-    _localUploadedFileNames = [[NSMutableSet<SDLFileName *> alloc] init];
+    _mutableUploadedEphemeralFileNames = [[NSMutableSet<SDLFileName *> alloc] init];
 
     _stateMachine = [[SDLStateMachine alloc] initWithTarget:self initialState:SDLFileManagerStateShutdown states:[self.class sdl_stateTransitionDictionary]];
 
@@ -103,6 +103,10 @@ - (void)stop {
     return [NSSet setWithSet:self.mutableRemoteFileNames];
 }
 
+- (NSSet<SDLFileName *> *)uploadedEphemeralFileNames {
+    return [NSSet setWithSet:self.mutableUploadedEphemeralFileNames];
+}
+
 - (NSString *)currentState {
     return self.stateMachine.currentState;
 }
@@ -275,7 +279,7 @@ - (void)uploadFiles:(NSArray<SDLFile *> *)files progressHandler:(nullable SDLFil
         dispatch_group_enter(uploadFilesTask);
 
         // HAX: [#827](https://github.com/smartdevicelink/sdl_ios/issues/827) Older versions of Core had a bug where list files would cache incorrectly. This led to attempted uploads failing due to the system thinking they were already there when they were not.
-        if (!file.persistent && [self.remoteFileNames containsObject:file.name] && ![self.localUploadedFileNames containsObject:file.name]) {
+        if (!file.persistent && [self.remoteFileNames containsObject:file.name] && ![self.mutableUploadedEphemeralFileNames containsObject:file.name]) {
             file.overwrite = true;
         }
 
@@ -390,7 +394,7 @@ - (void)sdl_uploadFile:(SDLFile *)file completionHandler:(nullable SDLFileManage
         }
         if (success) {
             [weakSelf.mutableRemoteFileNames addObject:fileName];
-            [weakSelf.localUploadedFileNames addObject:fileName];
+            [weakSelf.mutableUploadedEphemeralFileNames addObject:fileName];
         }
         if (uploadCompletion != nil) {
             uploadCompletion(success, bytesAvailable, error);

SmartDeviceLinkTests/DevAPISpecs/SDLFileManagerSpec.m

@@ -29,7 +29,7 @@ @interface SDLFileManager ()
 
 QuickSpecBegin(SDLFileManagerSpec)
 
-describe(@"SDLFileManager", ^{
+fdescribe(@"SDLFileManager", ^{
     __block TestConnectionManager *testConnectionManager = nil;
     __block SDLFileManager *testFileManager = nil;
     __block NSUInteger initialSpaceAvailable = 250;
@@ -225,6 +225,7 @@ @interface SDLFileManager ()
 
                         it(@"should set the file manager state to be waiting", ^{
                             expect(testFileManager.currentState).to(match(SDLFileManagerStateReady));
+                            expect(testFileManager.uploadedEphemeralFileNames).to(beEmpty());
                         });
 
                         it(@"should create a putfile with the correct data", ^{
@@ -253,6 +254,7 @@ @interface SDLFileManager ()
                                 expect(@(testFileManager.bytesAvailable)).toEventually(equal(testResponseBytesAvailable));
                                 expect(testFileManager.currentState).toEventually(match(SDLFileManagerStateReady));
                                 expect(testFileManager.remoteFileNames).toEventually(contain(testFileName));
+                                expect(testFileManager.uploadedEphemeralFileNames).toEventually(contain(testFileName));
                             });
 
                             it(@"should call the completion handler with the correct data", ^{
@@ -282,6 +284,7 @@ @interface SDLFileManager ()
                                 expect(@(testFileManager.bytesAvailable)).toEventually(equal(@(initialSpaceAvailable)));
                                 expect(testFileManager.remoteFileNames).toEventually(contain(testFileName));
                                 expect(testFileManager.currentState).toEventually(match(SDLFileManagerStateReady));
+                                expect(testFileManager.uploadedEphemeralFileNames).to(beEmpty());
                             });
 
                             it(@"should call the completion handler with the correct data", ^{
@@ -307,7 +310,7 @@ @interface SDLFileManager ()
                         });
                     });
 
-                    context(@"when allow overwrite is false", ^{
+                    context(@"when allow overwrite is NO", ^{
                         __block SDLRPCRequest *lastRequest = nil;
 
                         beforeEach(^{
@@ -373,6 +376,7 @@ @interface SDLFileManager ()
                         it(@"should set the file manager state correctly", ^{
                             expect(@(testFileManager.bytesAvailable)).toEventually(equal(testResponseBytesAvailable));
                             expect(testFileManager.remoteFileNames).toEventually(contain(testFileName));
+                            expect(testFileManager.uploadedEphemeralFileNames).toEventually(contain(testUploadFile.name));
                             expect(testFileManager.currentState).toEventually(match(SDLFileManagerStateReady));
                         });
 
@@ -404,6 +408,7 @@ @interface SDLFileManager ()
                         it(@"should set the file manager state correctly", ^{
                             expect(@(testFileManager.bytesAvailable)).toEventually(equal(@(initialSpaceAvailable)));
                             expect(testFileManager.remoteFileNames).toEventuallyNot(contain(testFileName));
+                            expect(testFileManager.uploadedEphemeralFileNames).toEventuallyNot(contain(testUploadFile.name));
                             expect(testFileManager.currentState).toEventually(match(SDLFileManagerStateReady));
                         });