assignment/todos
0
0
Fork 0
Code Issues Pull Requests Activity

Added the order to the sort descriptors inside the initializer for the TodoFetchDescriptor+Inits extensions in the app target.

Browse Source
This commit is contained in:
javier
2026-02-11 15:45:45 +01:00
parent 9c68c5be32
commit 0c8589db27
2 changed files with 75 additions and 63 deletions
Download Patch File Download Diff File Expand all files Collapse all files
App/Sources/Extensions/TodoFetchDescriptor+Inits.swift
+9 -6
View File
@@ -13,9 +13,12 @@ extension FetchDescriptor where T == Todo {
/// relevance and completion status. Each group applies appropriate predicates and sort orders: /// relevance and completion status. Each group applies appropriate predicates and sort orders:
/// ///
/// - **Completed**: All tasks marked as done, sorted by completion date (most recent first) /// - **Completed**: All tasks marked as done, sorted by completion date (most recent first)
/// - **Overdue**: Incomplete tasks with due dates before today, sorted by due date (most recent first) /// - **Overdue**: Incomplete tasks with due dates before today, sorted by user-defined order, then by due date (most recent first)
/// - **Today**: Incomplete tasks due today, sorted by due date (earliest first) /// - **Today**: Incomplete tasks due today, sorted by user-defined order, then by due date (earliest first)
/// - **Upcoming**: Incomplete tasks due after today, sorted by due date (earliest first) /// - **Upcoming**: Incomplete tasks due after today, sorted by user-defined order, then by due date (earliest first)
///
/// The user-defined order allows tasks to be manually reordered within each group through drag and drop,
/// while maintaining a secondary sort by date for tasks with the same order value.
/// ///
/// - Parameters: /// - Parameters:
/// - group: The todo group category to filter by. /// - group: The todo group category to filter by.
@@ -52,11 +55,11 @@ extension FetchDescriptor where T == Todo {
case .completed: case .completed:
[.init(\Todo.completed, order: .reverse)] [.init(\Todo.completed, order: .reverse)]
case .overdue: case .overdue:
[.init(\Todo.due, order: .reverse)] [.init(\Todo.order, order: .forward), .init(\Todo.due, order: .reverse)]
case .today: case .today:
[.init(\Todo.due, order: .forward)] [.init(\Todo.order, order: .forward), .init(\Todo.due, order: .forward)]
case .upcoming: case .upcoming:
[.init(\Todo.due, order: .forward)] [.init(\Todo.order, order: .forward), .init(\Todo.due, order: .forward)]
} }
self.init( self.init(
UnitTests/Tests/Extensions/TodoFetchDescriptorTests.swift → UnitTests/Tests/Extensions/TodoFetchDescriptor+InitsTests.swift
+31 -22
View File
@@ -4,13 +4,13 @@ import Testing
@testable import Todos @testable import Todos
@Suite("FetchDescriptor<Todo> extension tests") @Suite("TodoFetchDescriptor+Inits extension tests")
struct TodoFetchDescriptorTests { struct TodoFetchDescriptor_InitsTests {
// MARK: Initialization tests // MARK: Initialization tests
@Test( @Test(
"initializes with group when empty tasks", "initialization when empty tasks",
arguments: TodoGroup.allCases arguments: TodoGroup.allCases
) )
@MainActor @MainActor
@@ -31,11 +31,11 @@ struct TodoFetchDescriptorTests {
} }
@Test( @Test(
"initializes with group and show completed disabled when filled tasks", "initialization with show completed disabled",
arguments: TodoGroup.allCases arguments: TodoGroup.allCases
) )
@MainActor @MainActor
func initializationWhenFilledTasks(for group: TodoGroup) throws { func initializationWithNoShowCompleted(for group: TodoGroup) throws {
// GIVEN // GIVEN
let container = try modelContainer let container = try modelContainer
let context = container.mainContext let context = container.mainContext
@@ -69,7 +69,7 @@ struct TodoFetchDescriptorTests {
} }
@Test( @Test(
"initializes with group and show completed enabled when filled tasks", "initialization with show completed enabled",
arguments: TodoGroup.allCases arguments: TodoGroup.allCases
) )
@MainActor @MainActor
@@ -90,7 +90,8 @@ struct TodoFetchDescriptorTests {
// THEN // THEN
if group == .overdue || group == .upcoming { if group == .overdue || group == .upcoming {
#expect(results.count == 3) #expect(results.count == 3)
} else { }
else {
#expect(results.count == 2) #expect(results.count == 2)
} }
@@ -99,16 +100,16 @@ struct TodoFetchDescriptorTests {
#expect(results[0].title == "Completed task #2") #expect(results[0].title == "Completed task #2")
#expect(results[1].title == "Completed task #1") #expect(results[1].title == "Completed task #1")
case .overdue: case .overdue:
#expect(results[0].title == "Completed task #1") #expect(results[0].title == "Overdue task #2")
#expect(results[1].title == "Overdue task #2") #expect(results[1].title == "Completed task #1")
#expect(results[2].title == "Overdue task #1") #expect(results[2].title == "Overdue task #1")
case .today: case .today:
#expect(results[0].title == "Today task #1") #expect(results[0].title == "Today task #1")
#expect(results[1].title == "Today task #2") #expect(results[1].title == "Today task #2")
case .upcoming: case .upcoming:
#expect(results[0].title == "Upcoming task #1") #expect(results[0].title == "Completed task #2")
#expect(results[1].title == "Upcoming task #2") #expect(results[1].title == "Upcoming task #1")
#expect(results[2].title == "Completed task #2") #expect(results[2].title == "Upcoming task #2")
} }
} }
@@ -116,7 +117,7 @@ struct TodoFetchDescriptorTests {
// MARK: - Helpers // MARK: - Helpers
private extension TodoFetchDescriptorTests { private extension TodoFetchDescriptor_InitsTests {
// MARK: Computed // MARK: Computed
@@ -138,38 +139,46 @@ private extension TodoFetchDescriptorTests {
let tasks: [Todo] = [ let tasks: [Todo] = [
.init( .init(
title: "Today task #1", title: "Today task #1",
due: .now due: .now,
order: 0
), ),
.init( .init(
title: "Today task #2", title: "Today task #2",
due: .now due: .now,
order: 1
), ),
.init( .init(
title: "Upcoming task #1", title: "Upcoming task #1",
due: calendar.date(byAdding: .day, value: 1, to: .now) ?? .now due: calendar.date(byAdding: .day, value: 2, to: .now) ?? .now,
order: 0
), ),
.init( .init(
title: "Upcoming task #2", title: "Upcoming task #2",
due: calendar.date(byAdding: .day, value: 1, to: .now) ?? .now due: calendar.date(byAdding: .day, value: 3, to: .now) ?? .now,
order: 1
), ),
.init( .init(
title: "Overdue task #1", title: "Overdue task #1",
due: calendar.date(byAdding: .day, value: -1, to: .now) ?? .now due: calendar.date(byAdding: .day, value: -1, to: .now) ?? .now,
order: 1
), ),
.init( .init(
title: "Overdue task #2", title: "Overdue task #2",
due: calendar.date(byAdding: .day, value: -1, to: .now) ?? .now due: calendar.date(byAdding: .day, value: -1, to: .now) ?? .now,
order: 0
), ),
.init( .init(
title: "Completed task #1", title: "Completed task #1",
due: calendar.date(byAdding: .day, value: -1, to: .now) ?? .now, due: calendar.date(byAdding: .day, value: -1, to: .now) ?? .now,
completed: .now completed: .now,
order: 1
), ),
.init( .init(
title: "Completed task #2", title: "Completed task #2",
due: calendar.date(byAdding: .day, value: 1, to: .now) ?? .now, due: calendar.date(byAdding: .day, value: 1, to: .now) ?? .now,
completed: .now completed: .now,
) order: 0
),
] ]
for task in tasks { for task in tasks {