Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MDEP-894] Use @Component only - fix tests #360

Merged
merged 4 commits into from
Mar 22, 2024
Merged

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
49 checks passed
@slawekjaranowski slawekjaranowski deleted the MPLUGIN-495 branch March 22, 2024 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants