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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor tests #297
Refactor tests #297
Conversation
Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually? |
c1c77f5
to
340666e
Compare
f5247e5
to
96f0e2a
Compare
@sethmlarson . Can you please review this and merge into master? because i was writing tets for
|
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.
Looks good! Instead of test_eland/
let's do tests/...
, also leave all the folder structure under tests/
the same (ie don't rename tests/dataframe
to tests/test_dataframe
)
Yep refactored them |
@sethmlarson Right now builds are failing in master. The pytest issue looks similar to this https://stackoverflow.com/questions/10253826/path-issue-with-pytest-importerror-no-module-named-yadayadayada So, made changes, |
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! Will run CI to make sure everything is good.
jenkins test this please |
@sethmlarson , can you re-ask jenkins to test this please. I made a small path mistake. |
jenkins test this please |
Not sure why it failed for 7.6,7.7. Looks like ES issue. |
It's an unrelated failure due to Elasticsearch changing the routes for ML trained models. I'll fix the issue. |
It looks like there's a merge conflict after I merged the progress bar PR, could you fix that up? |
Yes on it |
8d5c416
to
5ffff94
Compare
jenkins test this please |
Good bye merge conflicts :) Done! |
Finally, this looks good |
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
Changes made are:
eland/tests/*
totest_eland/*
and imports used in tests.noxfile.py
contributing.rst
@sethmlarson Please review, Happy to make any changes required 馃槃