Clean up ActiveRecord code in TodosFinder and TodoService
What does this MR do?
This refactors the TodosFinder Finder and the TodoService Service classes according to our new code reuse guidelines.
TODO
-
Tests for PendingTodosFinder
-
Tests for UsersWithPendingTodosFinder
-
Tests for TodosFinder#any_for_target?
-
Tests for Todo.for_target
-
Tests for Todo.for_commit
-
Tests for Todo.any_for_target?
-
Tests for Todo.update_state
Does this MR meet the acceptance criteria?
-
Tests added for this feature/bug -
Conforms to the code review guidelines -
Conforms to the merge request performance guidelines -
Conforms to the style guides -
Conforms to the database guides
Edited by Dmytro Zaporozhets (DZ)