| Type |
Task
|
| Priority |
Normal
|
| Assignee | |
| Version |
1.0
|
| Sprints |
n/a
|
| Customer |
n/a
|
Issue Votes (0)
| Type |
Task
|
| Priority |
Normal
|
| Assignee | |
| Version |
1.0
|
| Sprints |
n/a
|
| Customer |
n/a
|
I just fixed a defect in the service layer correctly support updating a project's owner and linked Git repository. Due to lack of time, I am deferring augmenting test coverage to cover this code change.
What changed
The
ProjectServiceImpl.updateProject()method was enhanced so that it now:ownerIdis provided,gitRepoIdis provided,This behavior is important for the UI, which allows project owners and project managers to modify both fields.
What needs to be done
Add focused unit tests that verify:
ownerIdand/orgitRepoIdleaves the existing associations unchanged.These tests should exercise only the service layer. Controller-level behavior is already covered elsewhere.
Priority
Low. The logic is working correctly and is safe for the UI, but tests should be added to avoid regressions later.