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.

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

Centralize tests in Modin #3622

Closed
devin-petersohn opened this issue Oct 29, 2021 · 3 comments · Fixed by #7137
Closed

Centralize tests in Modin #3622

devin-petersohn opened this issue Oct 29, 2021 · 3 comments · Fixed by #7137
Assignees
@anmyachev
Labels
Code Quality 💯 Improvements or issues to improve quality of codebase P3 Very minor bugs, or features we can hopefully add some day. Sprint 🏃

Comments

@devin-petersohn
Copy link
Collaborator

Right now, Modin tests are split between the top-level modin/test and in the directories (e.g. modin/pandas/test). I would like to centralize the test location to make it easier to run the test suite with pytest.
@devin-petersohn devin-petersohn added Code Quality 💯 Improvements or issues to improve quality of codebase Sprint 🏃 labels Oct 29, 2021
@teaVeloper
Copy link

Hi,
I will look at this issue in participating at the Sprint.

This was referenced Nov 1, 2021
Closed
Closed
Closed
@vnlitvinov
Copy link
Collaborator

Should we also move stress_tests and asv_bench (cc @anmyachev) under this location to clean up the root?

@anmyachev
Copy link
Collaborator

asv_bench has a separate benchmark infrastructure, I think it should not be mixed with other tests.

@RehanSD RehanSD added the P3 Very minor bugs, or features we can hopefully add some day. label Oct 12, 2022
@anmyachev anmyachev self-assigned this Feb 23, 2024
anmyachev added a commit to anmyachev/modin that referenced this issue Mar 27, 2024
@anmyachev
TEST-modin-project#3622: Centralize tests in Modin
86686e7 
Signed-off-by: Anatoly Myachev <anatoly.myachev@intel.com>
anmyachev added a commit to anmyachev/modin that referenced this issue Mar 28, 2024
@anmyachev
TEST-modin-project#3622: Centralize tests in Modin
4f09b8b 
Signed-off-by: Anatoly Myachev <anatoly.myachev@intel.com>
@anmyachev anmyachev mentioned this issue Mar 28, 2024
Merged
7 tasks
anmyachev added a commit to anmyachev/modin that referenced this issue Mar 28, 2024
@anmyachev
TEST-modin-project#3622: Centralize tests in Modin
9db8878 
Signed-off-by: Anatoly Myachev <anatoly.myachev@intel.com>
anmyachev added a commit to anmyachev/modin that referenced this issue Mar 28, 2024
@anmyachev
TEST-modin-project#3622: Centralize tests in Modin
5244e3e 
Signed-off-by: Anatoly Myachev <anatoly.myachev@intel.com>
anmyachev added a commit to anmyachev/modin that referenced this issue Mar 28, 2024
@anmyachev
TEST-modin-project#3622: Centralize tests in Modin
06ebd15 
Signed-off-by: Anatoly Myachev <anatoly.myachev@intel.com>
YarShev pushed a commit that referenced this issue Apr 9, 2024
@anmyachev
TEST-#3622: Centralize tests in Modin (#7137)
d8cd9ce 
Signed-off-by: Anatoly Myachev <anatoly.myachev@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@anmyachev anmyachev

Labels
Code Quality 💯 Improvements or issues to improve quality of codebase P3 Very minor bugs, or features we can hopefully add some day. Sprint 🏃
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

TEST-#3622: Centralize tests in Modin anmyachev/modin
Refactor#3622
5 participants
@devin-petersohn @RehanSD @vnlitvinov @anmyachev @teaVeloper