-
Notifications
You must be signed in to change notification settings - Fork 361
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
[SW-2146] Cleanup in tests -> move to right ai.h2o.sparkling packages #2012
Conversation
@@ -1,73 +0,0 @@ | |||
/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unused class
@@ -1,69 +0,0 @@ | |||
/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved to ai.h2o.sparkling.TestUtils
@@ -1,114 +0,0 @@ | |||
/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved to ai.h2o.sparkling.TestUtils
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Last big shuffle with classes, this time with tests classes only:
-> Moving all remaining classes to ai.h2o.sparkling package
-> Putting TestUtils to a single class test utils
-> Extract ml integ tests from core and put them into ml integ tests
-> Few other extractions -> just moving tests to packages where they belong
Wanted to do this change now as we are getting closer to major release as we will switch rel branch to current master soon and avoid possible cherry-pick issues later.
No tests were removed ( edit: except 2 in SupportedRDDTestSuite -> those tests are useless now)