arturobernalg commented on PR #13539:
URL: https://github.com/apache/maven/pull/13539#issuecomment-6212849020

   Good catch, thanks. In befca4976e the dispatch is skipped when the project 
has no base directory, as there is no API project to report, and both 
DefaultDependencyResolutionEvent constructors now fail fast with 
Objects.requireNonNull. My tests had hidden the problem: new MavenProject() has 
no base directory, so the event was created with a null project. The helper now 
gives the project a file, the tests assert that the project is present, and a 
new test checks that nothing is dispatched for a project without a base 
directory.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to