Add rename-folder network call

BookPlayer/Library/ItemDetails Screen/ItemDetailsViewModel.swift

@@ -61,10 +61,18 @@ class ItemDetailsViewModel: BaseViewModel<ItemDetailsCoordinator> {
     var cacheKey = item.relativePath
 
     if item.title != formViewModel.title {
-      if item.type == .book {
+      switch item.type {
+      case .book:
         libraryService.renameBook(at: item.relativePath, with: formViewModel.title)
-      } else if let updatedCacheKey = try? libraryService.renameFolder(at: item.relativePath, with: formViewModel.title) {
-        cacheKey = updatedCacheKey
+      case .bound, .folder:
+        do {
+          let newRelativePath = try libraryService.renameFolder(at: item.relativePath, with: formViewModel.title)
+          cacheKey = newRelativePath
+          syncService.scheduleRenameFolder(at: item.relativePath, name: formViewModel.title)
+        } catch {
+          sendEvent(.showAlert(content: BPAlertContent.errorAlert(message: error.localizedDescription)))
+          return
+        }
       }
     }
 

BookPlayer/Profile/Profile Screen/QueuedSyncTasksView.swift

@@ -21,6 +21,8 @@ struct QueuedSyncTasksView<Model: QueuedSyncTasksViewModelProtocol>: View {
       return "arrow.2.circlepath"
     case .move:
       return "arrow.forward"
+    case .renameFolder:
+      return "square.and.pencil"
     case .delete:
       return "xmark.bin.fill"
     case .shallowDelete:

BookPlayerTests/Services/LibraryServiceTests.swift

@@ -615,7 +615,7 @@ class LibraryServiceTests: XCTestCase {
       duration: 100
     )
 
-    _ = try self.sut.renameBook(at: book.relativePath, with: "rename-test")
+    sut.renameBook(at: book.relativePath, with: "rename-test")
     XCTAssert(book.title == "rename-test")
   }
 

Shared/Services/LibraryService.swift

@@ -1237,8 +1237,6 @@ extension LibraryService {
       items.forEach({ rebuildRelativePaths(for: $0, parentFolder: folder.relativePath) })
     }
 
-    // TODO: Sync folder rename
-
     self.dataManager.saveContext()
 
     return newRelativePath

Shared/Services/Sync/LibraryAPI.swift

@@ -13,6 +13,7 @@ public enum LibraryAPI {
   case upload(params: [String: Any])
   case update(params: [String: Any])
   case move(origin: String, destination: String)
+  case renameFolder(path: String, name: String)
   case remoteFileURL(path: String)
   case remoteContentsURL(path: String)
   case delete(path: String)
@@ -33,6 +34,8 @@ extension LibraryAPI: Endpoint {
       return "/v1/library"
     case .move:
       return "/v1/library/move"
+    case .renameFolder:
+      return "/v1/library/rename"
     case .remoteFileURL:
       return "/v1/library"
     case .remoteContentsURL:
@@ -60,6 +63,8 @@ extension LibraryAPI: Endpoint {
       return .post
     case .move:
       return .post
+    case .renameFolder:
+      return .post
     case .remoteFileURL:
       return .get
     case .remoteContentsURL:
@@ -93,6 +98,11 @@ extension LibraryAPI: Endpoint {
         "origin": origin,
         "destination": destination
       ]
+    case .renameFolder(let path, let name):
+      return [
+        "relativePath": path,
+        "name": name
+      ]
     case .remoteFileURL(let path):
       return [
         "relativePath": path,

Shared/Services/Sync/LibraryItemSyncJob.swift

@@ -13,6 +13,7 @@ public enum JobType: String {
   case upload
   case update
   case move
+  case renameFolder
   case delete
   case shallowDelete
   case setBookmark
@@ -49,6 +50,7 @@ class LibraryItemSyncJob: Job, BPLogger {
     self.parameters = parameters
   }
 
+  // swiftlint:disable:next function_body_length
   func onRun(callback: SwiftQueue.JobResult) {
     Task { [weak self, callback] in
       guard let self = self else {
@@ -79,6 +81,13 @@ class LibraryItemSyncJob: Job, BPLogger {
           }
           let _: Empty = try await self.provider.request(.move(origin: origin, destination: destination))
           callback.done(.success)
+        case .renameFolder:
+          guard let name = parameters["name"] as? String else {
+            throw BookPlayerError.runtimeError("Missing parameters for renaming")
+          }
+
+          let _: Empty = try await provider.request(.renameFolder(path: self.relativePath, name: name))
+          callback.done(.success)
         case .delete:
           let _: Empty = try await provider.request(.delete(path: self.relativePath))
           callback.done(.success)

Shared/Services/Sync/SyncJobScheduler.swift

@@ -29,6 +29,8 @@ public protocol JobSchedulerProtocol {
   )
   /// Delete a bookmark
   func scheduleDeleteBookmarkJob(with relativePath: String, time: Double)
+  /// Rename a folder
+  func scheduleRenameFolderJob(with relativePath: String, name: String)
   /// Get all queued jobs
   func getAllQueuedJobs() -> [QueuedJobInfo]
   /// Cancel all stored and ongoing jobs
@@ -216,6 +218,22 @@ public class SyncJobScheduler: JobSchedulerProtocol, BPLogger {
       .schedule(manager: libraryQueueManager)
   }
 
+  public func scheduleRenameFolderJob(with relativePath: String, name: String) {
+    let params: [String: Any] = [
+      "relativePath": relativePath,
+      "name": name,
+      "jobType": JobType.renameFolder.rawValue
+    ]
+
+    JobBuilder(type: LibraryItemSyncJob.type)
+      .singleInstance(forId: "\(JobType.renameFolder.identifier)/\(relativePath)")
+      .persist()
+      .retry(limit: .unlimited)
+      .internet(atLeast: .cellular)
+      .with(params: params)
+      .schedule(manager: libraryQueueManager)
+  }
+
   public func getAllQueuedJobs() -> [QueuedJobInfo] {
     return pendingOperations.compactMap({ job -> QueuedJobInfo? in
       guard

Shared/Services/Sync/SyncService.swift

@@ -46,6 +46,8 @@ public protocol SyncServiceProtocol {
 
   func scheduleMove(items: [String], to parentFolder: String?)
 
+  func scheduleRenameFolder(at relativePath: String, name: String)
+
   func scheduleSetBookmark(
     relativePath: String,
     time: Double,
@@ -440,4 +442,10 @@ extension SyncService {
       time: bookmark.time
     )
   }
+
+  public func scheduleRenameFolder(at relativePath: String, name: String) {
+    guard isActive else { return }
+
+    jobManager.scheduleRenameFolderJob(with: relativePath, name: name)
+  }
 }