Background
PR #69 added multi-status filtering and includeAllStatuses support to the motion_tasks list operation. The new behavior currently has no test coverage because no test framework exists in the codebase.
What needs testing
Once a test framework is set up, the following scenarios should be covered:
status as array serializes to repeated status= query params in getTasks()
includeAllStatuses: true appends includeAllStatuses=true to query params
- Combining
status (non-empty) and includeAllStatuses returns a validation error
status: [] (empty array) + includeAllStatuses: true does not error (empty array = no filter)
status array with empty string elements is rejected with validation error
- Single
status string still works (backward compatibility)
status array rejected with clear error message for create and update operations
Related
Background
PR #69 added multi-status filtering and
includeAllStatusessupport to themotion_taskslist operation. The new behavior currently has no test coverage because no test framework exists in the codebase.What needs testing
Once a test framework is set up, the following scenarios should be covered:
statusas array serializes to repeatedstatus=query params ingetTasks()includeAllStatuses: trueappendsincludeAllStatuses=trueto query paramsstatus(non-empty) andincludeAllStatusesreturns a validation errorstatus: [](empty array) +includeAllStatuses: truedoes not error (empty array = no filter)statusarray with empty string elements is rejected with validation errorstatusstring still works (backward compatibility)statusarray rejected with clear error message forcreateandupdateoperationsRelated