fix(cli): check access process on windows
Description
When running on windows, the process username() can contain the DOMAIN which makes the comparison fail.
Issue linked
None
Checklist
- [ ] I have visited and read Develop PyMAPDL.
- [x] I have tested my changes locally
- [ ] I have added necessary documentation or updated existing documentation.
- [ ] I have followed the PyMAPDL coding style guidelines.
- [ ] I have added appropriate unit tests that also ensure the minimal coverage criteria.
- [x] I have reviewed my changes before submitting this pull request.
- [x] I have linked the issue or issues that are solved to the PR if any.
- [x] I have assigned this PR to myself.
- [ ] I have marked this PR as
draftif it is not ready to be reviewed yet. - [x] I have made sure that the title of my PR follows Commit naming conventions (e.g.
feat: adding new MAPDL command)
Codecov Report
:x: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 91.34%. Comparing base (be80897) to head (9a30857).
:warning: Report is 3 commits behind head on main.
:x: Your patch status has failed because the patch coverage (66.66%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.
Additional details and impacted files
@@ Coverage Diff @@
## main #4293 +/- ##
==========================================
+ Coverage 91.28% 91.34% +0.06%
==========================================
Files 193 193
Lines 15742 15745 +3
==========================================
+ Hits 14370 14383 +13
+ Misses 1372 1362 -10
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
@germa89 The current implementation of the tests do not allow me to test without having mapdl installed (or that's my understand from the need to provide a path for ansysxxx). Would you like me to update the current test so that it patches a few pymapdl commands and runs without requiring any local install ?