Skip to content

Conversation

jevanlingen
Copy link
Contributor

Adds comprehensive unit tests for issue #5965 to verify Maven recipes properly handle dependencies with non-jar types like "ejb". The tests cover three main recipe scenarios:

  • UpgradeDependencyVersion: Tests upgrading versions of EJB dependencies
  • ChangeDependencyGroupIdAndArtifactId: Tests changing coordinates of EJB dependencies
  • ExcludeDependency: Tests adding exclusions to EJB dependencies

Each scenario tests both regular and managed dependencies. Currently, these tests fail as expected, demonstrating that Maven recipes don't properly handle non-jar dependency types.

Adds comprehensive unit tests for issue #5965 to verify Maven recipes
properly handle dependencies with non-jar types like "ejb". The tests
cover three main recipe scenarios:

- UpgradeDependencyVersion: Tests upgrading versions of EJB dependencies
- ChangeDependencyGroupIdAndArtifactId: Tests changing coordinates of EJB dependencies
- ExcludeDependency: Tests adding exclusions to EJB dependencies

Each scenario tests both regular and managed dependencies. Currently,
these tests fail as expected, demonstrating that Maven recipes don't
properly handle non-jar dependency types.

Fixes #5965

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@timtebeek
Copy link
Member

@timtebeek timtebeek closed this Sep 26, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in OpenRewrite Sep 26, 2025
@timtebeek timtebeek deleted the test/maven-ejb-dependency-recipes branch September 26, 2025 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Maven recipes do not work when having dependencies of type "ejb"
2 participants