stable

Clone or download

Read-only

chore: fix Psalm errors in Stubs & Builders

part of request #17148 Tracker move to PSR4 😱 No functional change. Unit tests and Psalm should pass. Notes: Psalm was reporting a type mismatch error for a call to ChangesetTestBuilder::aChangeset(), so I chose to change the type everywhere to int (as it should be). When I wrote the builder, I used string because I had been burned by the DAO returning a string id. We should instead set-up our builders so they build correct data, the id should be int. Change-Id: Iaa39365bdac0b37fc0bfb1785ca3559d821bad3c

Modified Files

Name
M plugins/agiledashboard/tests/unit/AgileDashboard/ExplicitBacklog/DirectArtifactLinkCleanerTest.php +3 −3 Go to diff View file
M plugins/agiledashboard/tests/unit/AgileDashboard/FormElement/Burnup/CountElementsCalculatorTest.php +16 −16 Go to diff View file
M plugins/agiledashboard/tests/unit/AgileDashboard/FormElement/BurnupCalculatorTest.php +4 −4 Go to diff View file
M plugins/agiledashboard/tests/unit/AgileDashboard/Milestone/Sidebar/PromotedMilestoneBuilderTest.php +1 −1 Go to diff View file
M plugins/agiledashboard/tests/unit/AgileDashboard/Planning/MilestoneFactoryGetAllMilestonesTest.php +4 −4 Go to diff View file
M plugins/agiledashboard/tests/unit/AgileDashboard/Planning/MilestoneFactoryGetBareMilestoneByArtifactIdTest.php +1 −1 Go to diff View file
M plugins/agiledashboard/tests/unit/AgileDashboard/Planning/MilestoneFactoryGetMilestoneTest.php +5 −5 Go to diff View file
M plugins/agiledashboard/tests/unit/AgileDashboard/Planning/MilestoneFactoryGetPaginatedMilestonesTest.php +1 −1 Go to diff View file
M plugins/agiledashboard/tests/unit/AgileDashboard/Planning/MilestoneFactoryGetTopMilestonesTest.php +3 −3 Go to diff View file
M plugins/agiledashboard/tests/unit/AgileDashboard/Planning/MilestoneFactoryTest.php +1 −1 Go to diff View file
M plugins/agiledashboard/tests/unit/REST/v1/Scrum/BacklogItem/InitialEffortSemanticUpdaterTest.php +1 −1 Go to diff View file
M plugins/cardwall/tests/unit/CardwallConfigXmlImportTest.php +4 −4 Go to diff View file
M plugins/gitlab/tests/unit/Repository/Webhook/PostPush/PostPushWebhookCloseArtifactHandlerTest.php +2 −2 Go to diff View file
M plugins/program_management/tests/unit/Adapter/Events/ArtifactCreatedProxyTest.php +1 −1 Go to diff View file
M plugins/program_management/tests/unit/Adapter/Events/ArtifactUpdatedProxyTest.php +2 −2 Go to diff View file
M plugins/program_management/tests/unit/Adapter/Program/Backlog/AsynchronousCreation/LastChangesetRetrieverTest.php +1 −1 Go to diff View file
M plugins/program_management/tests/unit/Adapter/Program/Backlog/ProgramIncrement/Source/Changeset/SubmissionDateRetrieverTest.php +1 −1 Go to diff View file
M plugins/program_management/tests/unit/Adapter/Program/Backlog/ProgramIncrement/Source/Changeset/Values/DateValueRetrieverTest.php +1 −1 Go to diff View file
M plugins/program_management/tests/unit/Adapter/Program/Backlog/ProgramIncrement/Source/Changeset/Values/FieldValuesGathererTest.php +1 −1 Go to diff View file
M plugins/program_management/tests/unit/Adapter/Program/Backlog/Timebox/TimeframeValueRetrieverTest.php +1 −1 Go to diff View file
M plugins/program_management/tests/unit/Adapter/Program/Backlog/TopBacklog/Workflow/AddToTopBacklogPostActionTest.php +2 −2 Go to diff View file
M plugins/roadmap/tests/unit/Roadmap/REST/v1/RoadmapTasksRetrieverTest.php +2 −2 Go to diff View file
M plugins/roadmap/tests/unit/Roadmap/REST/v1/TaskRepresentationBuilderForTrackerTest.php +1 −1 Go to diff View file
M plugins/taskboard/tests/unit/Column/CardColumnFinderTest.php +1 −1 Go to diff View file
M plugins/taskboard/tests/unit/Column/FieldValuesToColumnMapping/ArtifactMappedFieldValueRetrieverTest.php +3 −3 Go to diff View file
M plugins/taskboard/tests/unit/REST/v1/Cell/CardMappedFieldUpdaterTest.php +2 −2 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Changeset/AfterNewChangesetHandlerTest.php +2 −2 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Changeset/CommentOnlyChangesetCreatorTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Changeset/NewChangesetPostProcessorTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Changeset/PostCreation/ActionsRunnerTest.php +4 −4 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Changeset/PostCreation/CalendarEvent/EventDatesRetrieverTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Changeset/PostCreation/CalendarEvent/EventDescriptionRetrieverTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Changeset/PostCreation/CalendarEvent/EventOrganizerRetrieverTest.php +2 −2 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Changeset/PostCreation/CalendarEvent/EventSummaryRetrieverTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Changeset/PostCreation/EmailNotificationAttachmentProviderTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/ChangesetValue/ArtifactLink/ArtifactForwardLinksRetrieverTest.php +3 −3 Go to diff View file
M plugins/tracker/tests/unit/Artifact/ChangesetValue/ArtifactLink/ReverseLinksRetrieverTest.php +3 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/ChangesetValue/ArtifactLink/ReverseLinksToNewChangesetsConverterTest.php +6 −4 Go to diff View file
M plugins/tracker/tests/unit/Artifact/ChangesetValue/ArtifactLink/StoredReverseLinkTest.php +2 −0 Go to diff View file
M plugins/tracker/tests/unit/Artifact/ChangesetValue/ChangesetValueSaverIgnoringPermissionsTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/ChangesetValue/ChangesetValueSaverTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/ChangesetValue/Text/TextValueInterpreterTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Closure/ArtifactCloserTest.php +2 −2 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Closure/ArtifactClosingReferencesHandlerTest.php +2 −2 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Link/ArtifactLinkerTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/Link/ArtifactReverseLinksUpdaterTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Artifact/StatusBadgeBuilderTest.php +3 −3 Go to diff View file
M plugins/tracker/tests/unit/Masschange/MasschangeUpdaterTest.php +5 −5 Go to diff View file
M plugins/tracker/tests/unit/Notifications/Recipient/MentionedUserInCommentRetrieverTest.php +2 −2 Go to diff View file
M plugins/tracker/tests/unit/Notifications/RemoveRecipient/RemoveRecipientThatAreTechnicalUsersTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Notifications/RemoveRecipient/RemoveRecipientThatCannotReadAnythingTest.php +4 −4 Go to diff View file
M plugins/tracker/tests/unit/Notifications/RemoveRecipient/RemoveRecipientThatDoesntWantMailForTheirOwnActionsTest.php +4 −4 Go to diff View file
M plugins/tracker/tests/unit/Notifications/RemoveRecipient/RemoveRecipientWhenTheyAreInStatusUpdateOnlyModeTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/REST/Artifact/PUTHandlerTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Semantic/Timeframe/TimeframeChangesetFieldsValueRetrieverTest.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Semantic/Timeframe/TimeframeWithDurationTest.php +4 −4 Go to diff View file
M plugins/tracker/tests/unit/Semantic/Timeframe/TimeframeWithEndDateTest.php +4 −4 Go to diff View file
M plugins/tracker/tests/unit/Semantic/Tooltip/OtherSemantic/TimeframeTooltipEntryTest.php +5 −5 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/ArtifactImportedMappingBuilder.php +3 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/ArtifactTestBuilder.php +2 −2 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/ArtifactValuesRepresentationBuilder.php +4 −5 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/ChangesetCommentTestBuilder.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/ChangesetTestBuilder.php +2 −5 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/Fields/CheckboxFieldBuilder.php +3 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/Fields/DateFieldBuilder.php +3 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/Fields/ExternalFieldBuilder.php +59 −0 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/Fields/FileFieldBuilder.php +3 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/Fields/ListFieldBuilder.php +3 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/Fields/OpenListFieldBuilder.php +3 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/Fields/RadioButtonFieldBuilder.php +3 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/Fields/StringFieldBuilder.php +3 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/LinkWithDirectionRepresentationBuilder.php +19 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Builders/TrackerTestBuilder.php +14 −14 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/BindValueIdCollectionStub.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/CheckBeforeMoveStub.php +0 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/CreateNewChangesetStub.php +8 −1 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/MoveDryRunStub.php +0 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/MoveRestArtifactStub.php +4 −7 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/NewParentLinkStub.php +1 −1 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/RetrieveAllUsableTypesInProjectStub.php +7 −4 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/Tracker/Artifact/Changeset/PostCreation/CalendarEvent/RetrieveEventDatesStub.php +6 −0 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/Tracker/Artifact/Changeset/PostCreation/CalendarEvent/RetrieveEventOrganizerStub.php +3 −0 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/Tracker/Artifact/Changeset/PostCreation/CalendarEvent/RetrieveEventSummaryStub.php +3 −0 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/Tracker/Artifact/Changeset/PostCreation/PostCreationActionsQueuerStub.php +3 −1 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/Tracker/Artifact/Changeset/ProcessChangesetPostCreationStub.php +0 −3 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/Tracker/FormElement/Field/ListFields/Bind/BindVisitorStub.php +4 −4 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/Tracker/Permission/RetrieveUserPermissionOnTrackersStub.php +3 −0 Go to diff View file
M plugins/tracker/tests/unit/Test/Stub/UpdateArtifactLinkXMLStub.php +4 −12 Go to diff View file
M plugins/tracker_functions/tests/unit/TrackerFunctions/CustomCodeExecutionTaskTest.php +7 −7 Go to diff View file
M plugins/tracker_functions/tests/unit/TrackerFunctions/Notification/BuildMessagesForAdminsTest.php +1 −1 Go to diff View file
M plugins/tracker_functions/tests/unit/TrackerFunctions/Notification/SendMessagesForAdminsTest.php +1 −1 Go to diff View file
M plugins/tracker_functions/tests/unit/TrackerFunctions/Notification/SendNotificationToTrackerAdministratorTest.php +2 −2 Go to diff View file
M tests/psalm/psalm.xml +0 −11 Go to diff View file
M tests/psalm/tuleap-baseline.xml +0 −313 Go to diff View file