Skip to content

[MNG-8403] Collapse IT utils and helpers #1949

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Dec 5, 2024
Merged

[MNG-8403] Collapse IT utils and helpers #1949

merged 7 commits into from
Dec 5, 2024

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Dec 5, 2024 •
edited
Loading

Changes:

  • move all into same package
  • stop direct use of ResourceExtractor, redirect all ITs to superclass method
  • drop unneeded ResourceExtractor, it never "extracts" anything
  • un-fork ITs where applicable (ie. due new CLI param or so that was not supported before)

https://issues.apache.org/jira/browse/MNG-8403

@cstamas
Collaps IT utils
a90c9b6 
Just move things in new place, next to
abstract IT class.
@cstamas
Collapse IT utils
bb3001d
@cstamas cstamas requested a review from gnodet December 5, 2024 08:07
@cstamas cstamas self-assigned this Dec 5, 2024
@cstamas cstamas added this to the 4.0.0-rc-2 milestone Dec 5, 2024
@cstamas cstamas marked this pull request as ready for review December 5, 2024 12:22
@cstamas cstamas changed the title Collapse IT utils and helper [MNG-8403] Collapse IT utils and helpers Dec 5, 2024
@cstamas cstamas merged commit 8cd6f83 into apache:master Dec 5, 2024
13 checks passed
@cstamas cstamas deleted the collapse-it-helper branch December 5, 2024 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gnodet gnodet gnodet approved these changes

Assignees

@cstamas cstamas

Labels
None yet
Projects
None yet
Milestone
4.0.0-rc-2
Development

Successfully merging this pull request may close these issues.
2 participants
@cstamas @gnodet