refactor TasksTest.kt to be shorter and comments to improve readability #1040
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request refactors the
TasksTest
class inTasksTest.kt
to improve code readability and maintainability by consolidating test cases, simplifying methods, and removing redundant code. The most important changes include combining related test cases, streamlining helper methods, and reducing boilerplate code.Test Case Consolidation:
editTask
,createOneTask_deleteTask
,createTwoTasks_deleteOneTask
) into more concise and reusable tests such ascreateAndEditTask
,createAndDeleteTask
, andtoggleTaskCompletion
. This reduces redundancy and improves clarity.Helper Method Simplification:
findTextField
and introducedonTask
for better readability and code reuse. These methods now use concise lambda expressions.Boilerplate Reduction:
Code Formatting:
@Inject
and@Before
into single-line declarations) to make the code more compact.These changes streamline the test suite, making it easier to maintain and extend in the future.