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

Version/revamp #944

Draft
wants to merge 380 commits into
base: main
Choose a base branch
from
Draft

Version/revamp #944

wants to merge 380 commits into from

Conversation

fabapp2
Copy link
Contributor

@fabapp2 fabapp2 commented Sep 24, 2023

No description provided.

The order of resources as provided to the TestProjectContext is not guaranteed anymore, now that parsed resources are taken from filesystem
With the new parser .txt and foo resources in src/main/resources were represented as Quarks and allowed no (usual) access to their content
With the new parser .txt and foo resources in src/main/resources were represented as Quarks and allowed no (usual) access to their content
fabapp2 and others added 30 commits September 25, 2023 00:24
* Fix parsing duplicate pom files

Added target to default ignorePatterns
Refactor method names for test
Fix dependency download
Fix test by ignoring poms when parsing resources
Add logging
Replace List with Set to avoid duplicates

* Refactoring

* Add test proving duplicate parsing of resources

* Fix path calculcation to other modules

* Back to use shrinkwrap in tests

* Replace single-line with multi-line String

* Bumped maven-resolver.version

* Fix test allow SbmCoreConfig construct ExecutionContext

* GH Action

* Assert correct classpath

* Fix testdata

* Cleanup

* Disabled test that must be reactivated later

* Move JavaSourceUtil to sbm-support-rewrite

* Move OpenRewriteTestSupport to test-helper

* Back to use shrinkwrap in tests

* Add missing test projects

* Back to use shrinkwrap in tests

* Revamp/move open rewrite test support to test helper (#939)

* Revamp/make other modules compile (#943)

* Update license header

* Add sbm-support-boot module

* Add sbm-support-boot module

* Module sbm-support-boot compiles

* Module sbm-support-jee compiles

* Module sbm-support-weblogic compiles

* WIP sbm-recipes-jee-to-boot

* Make SwapResponseWithResponseEntity compile

* Make SwapCacheControl compile

* Make SwapStatusForHttpStatus compile

* Make fieldsMapping a member

* Remove unused member

* Make ReplaceMediaType compile

Also removes unused member

* Make ReplaceResponseEntityBuilder compile

* Make sbm-recipes-jee-to-boot compile

* Make sbm-recipes-spring-cloud compile

* Delete unused classes

* Delete unused classes

* Make sbm-recipes-boot-upgrade compile

* Using shrinkwrap depchain dependency

* Replace single line with multi line string

* Cleanup properties order
…rite (#950)

* Add method to sync recipe run
* Moved ProjectResourceSet related classes to sbm-support-rewrite
* Add temp integration test
* Use new ProjectResourceSetFactory
* Use RewriteMigrationResultMerger
* Remove @component and declare using @bean
* Add dependency to rewrite-java-17
* Add dependency to rewrite-migrate-java
* Add delombok
* Renamed HelperWithoutAGoodName
* Enhance README
* Moved Maven-related classes to the maven package
* Add javadoc
* Remove dead code
* Remove ExecutionContext parameter
* Renamed SbmMavenProject
* Moved logic into ParserContext
* Add MavenModuleParser
* Cleanup pom
Improve default ignorePatterns
- Introduce `ParserParityTestHelper` which takes a dir and parses the
given project with `RewriteMavenProjectParser` (comparing) and
`RewritePropjectParser` (system under test). The parsing result is then
compared using AssertJ's `usingRecursiveComparison` which verifies that
all markers and their fields of both parsing results are equal proving
that both parsers yield the same result.
- Introduce ´TestProjectHelper` which helps set up test projects.
- Fixes multiple bugs, e.g. calculation of the classpath. 
- Cleanups and refactorings
Add support for `~/.m2/settings.xml` and `~/.m2/settings-security.xml`.

- `Maven SettingsInitializer` reads the settings with decrypted
credentials and adds them to the `ExecutionContext` Maven when build
files get parsed.
- `PrivateArtifactRepositoryTest` tests the behavior e2e.
- Add Maven-related Autoconfiguration and move some beans there
-  `MavenExecutionRequest` now has the path to `settings.xml`
- Polishing
* Move Maven related classes into maven package
* Organize imports
* Merge ModuleParser into MavenModuleParser
- Updated IgnoredPatterns 
- Added support for more Rewrite recipes

---------

Co-authored-by: Fabian Krüger <56278322+fabapp2@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants