Public

Project privacy set to public. By default, its content is available to everyone (authenticated or not). Please note that more restrictive permissions might exist on some items.

stable

Clone or download

Read-only

Split checking of new Iteration tracker

Download Browse

part of request #24994 : Javascript error in Iterations planning No functional change expected. Notes: The goal is to remove TrackerIsValidChecker static method and to split checking the validity of Program Increment, Iteration and Plannable Trackers. Program Management domain is not supposed to know what a Tracker is. Therefore, we must split the code that checks trackers are valid. This way, the underlying implementation could change independently. It is in fact still checked the same way as all three interfaces will be implemented by a single adapter that runs the same code. ALso, in a next patch, we will need to enforce business rules on labels when building a NewIterationTrackerConfiguration object. Renamed objects with "New" prefix and added comments to underline the fact that those objects exist to be saved in storage. Change-Id: Ief9f437604c4f2843cacd61e6ba43dd3af43444c

+221 −89
Joris MASSON (jmasson) 2022-01-13 15:45
Joris MASSON (jmasson) 2022-01-13 16:12
da2337c588fe8b551213d41c5925025bd27e87aa
f7f6ce3ac2ec7e2e2fa3cb8806a30a9de4996ce2
git #tuleap/stable/da2337c588fe8b551213d41c5925025bd27e87aa

Modified Files

Side by side diff
Inline diff
Name
M plugins/program_management/include/Adapter/Program/Plan/TrackerConfigurationChecker.php +9 −1 Go to diff View file
A plugins/program_management/include/Domain/Program/Plan/CheckNewIterationTracker.php +39 −0 Go to diff View file
R plugins/program_management/include/Domain/Program/Plan/IterationTracker.php Go to diff View file
M plugins/program_management/include/Domain/Program/Plan/Plan.php +2 −7 Go to diff View file
M plugins/program_management/include/Domain/Program/Plan/PlanCreator.php +3 −2 Go to diff View file
M plugins/program_management/include/REST/v1/ProjectResource.php +1 −0 Go to diff View file
M plugins/program_management/include/program_managementPlugin.php +1 −0 Go to diff View file
M plugins/program_management/tests/unit/Adapter/Program/Plan/TrackerConfigurationCheckerTest.php +14 −0 Go to diff View file
D plugins/program_management/tests/unit/Domain/Program/Plan/IterationTrackerTest.php +0 −79 Go to diff View file
A plugins/program_management/tests/unit/Domain/Program/Plan/NewIterationTrackerConfigurationTest.php +86 −0 Go to diff View file
M plugins/program_management/tests/unit/Domain/Program/Plan/PlanCreatorTest.php +2 −0 Go to diff View file
A plugins/program_management/tests/unit/Stub/CheckNewIterationTrackerStub.php +64 −0 Go to diff View file