Заметки с синхронизацией — тестовое задание

Что не так. Ответ сервера разбирается через try!. Любая ошибка — нет сети, таймаут, неожиданный JSON — превращается в падение приложения.

func load() {
    api.fetchNotes { result in
        self.notes = try! result.get()
    }
}

Как исправить. Перейдите на async/await и показывайте ошибку с кнопкой «Повторить»:

@MainActor
final class NotesViewModel: ObservableObject {
    @Published private(set) var notes: [Note] = []
    @Published var error: Error?
    private let api: NotesAPI

    init(api: NotesAPI = LiveNotesAPI()) { self.api = api }

    func load() async {
        do {
            notes = try await api.notes()
            error = nil
        } catch {
            self.error = error
        }
    }
}

В экране загрузка запускается через .task { await model.load() } — задача отменится сама, когда экран закроют.

Обработчик URLSession вызывается в фоновой очереди, а из него меняется notes. Xcode показывает предупреждение «Publishing changes from background threads is not allowed», интерфейс обновляется с задержкой или не обновляется. Пометка модели @MainActor из P1-1 решает проблему целиком: компилятор сам проследит, чтобы состояние менялось в главном потоке.

final class NotesViewModel: ObservableObject {
    @Published var notes: [Note] = []

    func load() {
        api.fetchNotes { result in
            // обработчик URLSession вызывается в фоновой очереди
            self.notes = try! result.get()
        }
    }
}

@MainActor
final class NotesViewModel: ObservableObject {
    @Published private(set) var notes: [Note] = []
    // load() — async, как в P1-1: после await код продолжается в главном потоке
}

Что не так. UserDefaults — обычный plist в папке приложения. Он попадает в незашифрованные бэкапы и читается на взломанном устройстве.

UserDefaults.standard.set(token, forKey: "authToken")

Как исправить. Храните токен в Keychain. Небольшой обёртки достаточно:

enum TokenStore {
    private static let query: [String: Any] = [
        kSecClass as String: kSecClassGenericPassword,
        kSecAttrAccount as String: "auth-token",
    ]

    static func save(_ token: String) throws {
        SecItemDelete(query as CFDictionary)
        var attributes = query
        attributes[kSecValueData as String] = Data(token.utf8)
        attributes[kSecAttrAccessible as String] = kSecAttrAccessibleAfterFirstUnlockThisDeviceOnly
        let status = SecItemAdd(attributes as CFDictionary, nil)
        guard status == errSecSuccess else { throw KeychainError(status: status) }
    }
}

Модель создаётся прямо в объявлении @ObservedObject var model = NotesViewModel(). Когда родитель перерисовывается, SwiftUI пересоздаёт структуру экрана — и вместе с ней новую пустую модель.

struct NotesListView: View {
    @ObservedObject var model = NotesViewModel()

@StateObject private var model = NotesViewModel()

DateFormatter() создаётся внутри body. Это дорогой объект, а body вызывается часто — при прокрутке длинного списка это заметно. Используйте встроенное форматирование:

var body: some View {
    let formatter = DateFormatter()
    formatter.dateFormat = "d MMM, HH:mm"
    return Text(formatter.string(from: note.updatedAt))
        .font(.caption)
}

Text(note.updatedAt, format: .dateTime.day().month(.abbreviated).hour().minute())
    .font(.caption)
    .foregroundStyle(.secondary)

Таймер держит замыкание, замыкание держит self — модель редактора никогда не освобождается, и автосохранение продолжает работать после закрытия экрана. Захватывайте self слабо и останавливайте таймер:

timer = Timer.scheduledTimer(withTimeInterval: 5, repeats: true) { _ in
    self.saveIfChanged()
}

timer = Timer.scheduledTimer(withTimeInterval: 5, repeats: true) { [weak self] _ in
    Task { @MainActor in self?.saveIfChanged() }
}

deinit { timer?.invalidate() }

Тесты обращаются к настоящему серверу и ждут ответа через wait(for:timeout: 5). Без сети они падают, с сетью — медленные. Протокол NotesAPI уже есть — подмените его фейком:

func testLoadsNotes() {
    let model = NotesViewModel()
    let loaded = expectation(description: "notes loaded")
    model.load()
    DispatchQueue.main.asyncAfter(deadline: .now() + 3) { loaded.fulfill() }
    wait(for: [loaded], timeout: 5)
    XCTAssertFalse(model.notes.isEmpty)
}

@MainActor
final class NotesViewModelTests: XCTestCase {
    func testShowsErrorWhenOffline() async {
        let model = NotesViewModel(api: FakeNotesAPI(result: .failure(URLError(.notConnectedToInternet))))
        await model.load()
        XCTAssertTrue(model.notes.isEmpty)
        XCTAssertNotNil(model.error)
    }
}

Подключите SwiftLint с базовыми правилами — он ловит force unwrap и try!, как в P1-1. Конфиг в репозитории показывает, что вы следите за качеством кода.

Тексты интерфейса записаны прямо в коде. Вынесите их в Localizable.xcstrings — даже без перевода это показывает, что приложение готово к локализации.