swiftlint

Author: tom-ludwig

CodeEdit/Features/About/Views/AboutDefaultView.swift

@@ -26,6 +26,7 @@ struct AboutDefaultView: View {
     @Environment(\.colorScheme)
     var colorScheme
 
+    // swiftlint:disable:next force_unwrapping
     private static var licenseURL = URL(string: "https://github.com/CodeEditApp/CodeEdit/blob/main/LICENSE.md")!
 
     let smallTitlebarHeight: CGFloat = 28

CodeEdit/Features/Documents/Indexer/AsyncFileIterator.swift

@@ -39,6 +39,8 @@ struct AsyncFileIterator: AsyncSequence, AsyncIteratorProtocol {
                 currentIdx += 1
             }
         }
+
+        // swiftlint:disable:next force_unwrapping
         return (foundContent!, currentIdx)
     }
 

CodeEdit/Features/Documents/Indexer/SearchIndexer+AsyncController.swift

@@ -165,12 +165,12 @@ extension SearchIndexer {
             let dispatchGroup = DispatchGroup()
 
             let fileManager = FileManager.default
-            let enumerator = fileManager.enumerator(
+            guard let enumerator = fileManager.enumerator(
                 at: url,
                 includingPropertiesForKeys: [.isRegularFileKey],
                 options: [.skipsHiddenFiles],
                 errorHandler: nil
-            )!
+            ) else { return }
 
             for case let fileURL as URL in enumerator {
                 dispatchGroup.enter()

CodeEdit/Features/Documents/WorkspaceDocument+Find.swift

@@ -32,8 +32,10 @@ extension WorkspaceDocument.SearchState {
     }
 
     func stripSpecialCharacters(from string: String) -> String {
-        let regex = try? NSRegularExpression(pattern: "[^a-zA-Z0-9]+", options: .caseInsensitive)
-        return regex!.stringByReplacingMatches(
+        guard let regex = try? NSRegularExpression(pattern: "[^a-zA-Z0-9]+", options: .caseInsensitive) else {
+            return string
+        }
+        return regex.stringByReplacingMatches(
             in: string,
             options: [],
             range: NSRange(location: 0, length: string.utf16.count),

CodeEditTests/Features/Documents/Indexer/MemoryIndexingTests.swift

@@ -130,13 +130,16 @@ final class MemoryIndexingTests: XCTestCase {
 
         // Save the current index.
         let savedIndex = index.getAsData()
-        XCTAssertNotNil(savedIndex, "Failed to save the index.")
+        guard let savedIndex = savedIndex else {
+            XCTFail("Failed to save the index.")
+            return
+        }
 
         // Close the index, i.e. the index gets deallocated form memory.
         index.close()
 
         // Load the saved index
-        guard let loadedIndex = SearchIndexer.Memory(data: savedIndex!) else {
+        guard let loadedIndex = SearchIndexer.Memory(data: savedIndex) else {
             XCTFail("Failed to create an index")
             return
         }

CodeEditTests/Features/Documents/Indexer/TemporaryFile.swift

@@ -24,6 +24,7 @@ class TemporaryFile {
         let folder = NSTemporaryDirectory()
         let name = UUID().uuidString
 
+        // swiftlint:disable:next force_unwrapping
         return NSURL.fileURL(withPathComponents: [folder, name])! as URL
     }()
 

CodeEditTests/Features/Search/FuzzySearch/FuzzySearchTests.swift

@@ -39,10 +39,11 @@ final class FuzzySearchTests: XCTestCase {
 
     func testFuzzySearch() async {
         let urls = [
-            URL(string: "FuzzySearchable.swift")!,
-            URL(string: "ContentView.swift")!,
-            URL(string: "FuzzyMatch.swift")!
-        ]
+            URL(string: "FuzzySearchable.swift"),
+            URL(string: "ContentView.swift"),
+            URL(string: "FuzzyMatch.swift")
+        ].compactMap { $0 }
+
         let fuzzyMatchResult = await urls.fuzzySearch(query: "mch").map {
             $0.item
         }

CodeEditTests/Utils/CEWorkspaceFileManager/CEWorkspaceFileManagerTests.swift

@@ -46,10 +46,10 @@ final class CEWorkspaceFileManagerUnitTests: XCTestCase {
         let randomCount = Int.random(in: 1 ... 100)
         let files = generateRandomFiles(amount: randomCount)
         try files.forEach {
-            let fakeData = "fake string".data(using: .utf8)
+            let fakeData = Data("fake string".utf8)
             let fileUrl = directory
                 .appendingPathComponent($0)
-            try fakeData!.write(to: fileUrl)
+            try fakeData.write(to: fileUrl)
         }
         let client = CEWorkspaceFileManager(
             folderUrl: directory,
@@ -83,10 +83,10 @@ final class CEWorkspaceFileManagerUnitTests: XCTestCase {
         var files = client.flattenedFileItems.map { $0.value.name }
         files.append(newFile)
         try files.forEach {
-            let fakeData = "fake string".data(using: .utf8)
+            let fakeData = Data("fake string".utf8)
             let fileUrl = directory
                 .appendingPathComponent($0)
-            try fakeData!.write(to: fileUrl)
+            try fakeData.write(to: fileUrl)
         }
 
         wait(for: [expectation])
@@ -106,7 +106,7 @@ final class CEWorkspaceFileManagerUnitTests: XCTestCase {
 
     func randomString(length: Int) -> String {
         let letters = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789"
-        return String((0 ..< length).map { _ in letters.randomElement()! })
+        return String((0 ..< length).compactMap { _ in letters.randomElement() })
     }
 
     func testGetFile() throws {