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

reorganize tests directory #3196

Merged
merged 14 commits into from
Jan 29, 2024
Merged

reorganize tests directory #3196

merged 14 commits into from
Jan 29, 2024

Conversation

njzjz
Copy link
Member

@njzjz njzjz commented Jan 29, 2024 

- source
  - tests
     - common
     - tf
     - pt

njzjz added 7 commits January 28, 2024 18:36
@njzjz
move tests
e3c576b 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
fix infer_path
bc65fa1 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
fix infer_path
0215e85 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
move from tests/tf to tests/common
53fcbe8 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
fix paths
d1e0132 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
fix tests_path
2e86c39 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
systems should be a list
ca3b1c9 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz requested a review from wanghan-iapcm January 29, 2024 00:28
@codecov Codecov
Copy link

codecov bot commented Jan 29, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a8168b5) 74.27% compared to head (8d8f564) 74.27%.
Report is 2 commits behind head on devel.

Additional details and impacted files 
@@            Coverage Diff             @@
##            devel    #3196      +/-   ##
==========================================
- Coverage   74.27%   74.27%   -0.01%     
==========================================
  Files         343      343              
  Lines       31636    31630       -6     
  Branches     1592     1592              
==========================================
- Hits        23499    23493       -6     
  Misses       7212     7212              
  Partials      925      925

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

njzjz added 7 commits January 28, 2024 19:48
@njzjz
insert sys.path
5fd6824 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
fix common
1e843bc 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
fix
1342e6b 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
add __init__.py
cd72f9d 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
fix the path of test_lammps.py
219b78a 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
fix infer_path in test_lammps
ca582fd 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
fix import
8d8f564 
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz merged commit 8900561 into deepmodeling:devel Jan 29, 2024
45 checks passed
@njzjz njzjz deleted the migrate-tests branch January 29, 2024 02:13
@njzjz njzjz mentioned this pull request Apr 2, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wanghan-iapcm wanghan-iapcm wanghan-iapcm approved these changes

Assignees
No one assigned
Labels
Python
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@njzjz @wanghan-iapcm