Skip to content

Conversation

Jmohak2004
Copy link

@Jmohak2004 Jmohak2004 commented Jul 12, 2025

1)Cleared messages and updatedTasks in @before to prevent cross-test contamination.
2)Renamed test methods to meaningful names (shouldUpdateTaskByReferenceName, etc.).
3)Replaced anonymous inner classes with cleaner lambda expressions.
4)Extracted task and workflow creation into helper methods for reuse (createTask, createWorkflow).

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • WHOSUSING.md
  • Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

Describe the new behavior from this PR, and why it's needed
Issue #

Alternatives considered

Describe alternative implementation you have considered

1)Cleared messages and updatedTasks in @before to prevent cross-test contamination.
2)Renamed test methods to meaningful names (shouldUpdateTaskByReferenceName, etc.).
3)Replaced anonymous inner classes with cleaner lambda expressions.
4)Extracted task and workflow creation into helper methods for reuse (createTask, createWorkflow).
Copy link
Author

@Jmohak2004 Jmohak2004 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

optimised code without changing any functionality

@v1r3n
Copy link
Collaborator

v1r3n commented Aug 9, 2025

Thanks for the PR, can you fix the build with the spotless?

@Jmohak2004
Copy link
Author

Thanks for the PR, can you fix the build with the spotless?

sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants