Skip to content

Conversation

@michael-o
Copy link
Member

This should depict the failing behavior.

@slawekjaranowski
Copy link
Member

Unit tests still fail for me ....
But executing ITs are ok ...

@michael-o
Copy link
Member Author

Unit tests still fail for me .... But executing ITs are ok ...

Yes, that's the point. A misconfig or dep is missing.

@slawekjaranowski
Copy link
Member

@michael-o @cstamas please look at my commit ... I fix one test by adding mock MavenProject and MavenSession as component in test

@slawekjaranowski
Copy link
Member

Unit test pass.

Site build due to parameters list can be null 😄

@michael-o
Copy link
Member Author

@slawekjaranowski This is fantastic work, thank you. Your change must go with the upgrade of MPLUGIN with every modified plugin as well as this one.

@michael-o michael-o marked this pull request as draft December 29, 2023 18:13
@slawekjaranowski slawekjaranowski changed the title Use @Component only MDEP-894 Use @Component only Mar 22, 2024
@slawekjaranowski
Copy link
Member

rebased and use latest released versions of m-plugin-tools

@slawekjaranowski slawekjaranowski marked this pull request as ready for review March 22, 2024 18:54
@slawekjaranowski slawekjaranowski changed the title MDEP-894 Use @Component only [MDEP-894] Use @Component only - fix tests Mar 22, 2024
@slawekjaranowski slawekjaranowski merged commit c9e488b into master Mar 22, 2024
@slawekjaranowski slawekjaranowski deleted the MPLUGIN-495 branch March 22, 2024 19:00
@slawekjaranowski slawekjaranowski added the enhancement New feature or request label Apr 21, 2024
@jira-importer
Copy link

Resolve #1399

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants