Skip to content

[MNG-7432] Proposed UT for maven-3.9.x and 3.8.x - #706

Closed
cstamas wants to merge 1 commit into
maven-3.9.xfrom
maven-3.9.x-mng-7432-ut
Closed

[MNG-7432] Proposed UT for maven-3.9.x and 3.8.x#706
cstamas wants to merge 1 commit into
maven-3.9.xfrom
maven-3.9.x-mng-7432-ut

Conversation

@cstamas

@cstamas cstamas commented Apr 2, 2022

Copy link
Copy Markdown
Member

This UT makes sure that DefaultMaven sets up proper type
of WorkspaceReader in RepositorySystemSession.

For example, this UT fails with current maven-3.8.x and
maven-3.9.x branches due MNG-7432

This UT makes sure that DefaultMaven sets up proper type
of WorkspaceReader in RepositorySystemSession.

For example, this UT fails with current maven-3.8.x and
maven-3.9.x branches.
@laeubi

laeubi commented Apr 4, 2022

Copy link
Copy Markdown
Contributor

@michael-o whats the process here, should the test be included (with @Ignore) and then the fix with enabling it?

@kwin

kwin commented Apr 4, 2022

Copy link
Copy Markdown
Member

Just take over the UT to your branch and then this PR can be closed. That makes it easier to commit to 3.9/3.8.x.

@cstamas

cstamas commented Apr 4, 2022

Copy link
Copy Markdown
Member Author

Yes please, just lift it. Am currently a bit overwhelmed with $other work, otherwise would help with all of this. No need to retain author or anything (is few liner), just lift it/squash all right.

@laeubi

laeubi commented Apr 4, 2022

Copy link
Copy Markdown
Contributor

All right I'll try that! Thanks for the Test it looks quite straight forward but I'm not sure I would have been able to come up with such one :-)

@cstamas cstamas closed this Apr 7, 2022
@cstamas
cstamas deleted the maven-3.9.x-mng-7432-ut branch April 7, 2022 06:53
@jira-importer

Copy link
Copy Markdown

Resolve #9148

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.

4 participants