When running a batched delete mutation, if no objects are in the filter set, the mutation should be no-op, and an empty array should be returned. This matches the behavior of a batched update and also ensures idempotence.
Current behavior: An empty set throws an error
Expected behavior: An empty set does not throw an error
Scope of change:
Remove a seemingly necessary assert statement.
Add some tests (including a breaking test)
Types of Changes
[ ] Core
[x] Bugfix
[ ] New feature
[ ] Enhancement/optimization
[ ] Documentation
Issues Fixed or Closed by This PR
n/a
Checklist
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).
Description
When running a batched delete mutation, if no objects are in the filter set, the mutation should be no-op, and an empty array should be returned. This matches the behavior of a batched update and also ensures idempotence.
Current behavior: An empty set throws an error Expected behavior: An empty set does not throw an error
Scope of change:
Types of Changes
Issues Fixed or Closed by This PR
n/a
Checklist