ci: add InProcess testing module #4185

germa89 · 2025-08-19T15:00:37Z

Description

Adding framework for testing MAPDL InProcess interface

Issue linked

NA

Checklist

…n MAPDL

Copilot

Pull Request Overview

This PR adds a testing framework for the MAPDL InProcess interface, which allows testing Python code execution within MAPDL. The framework provides a test runner that executes MAPDL commands in batch mode and validates Python command functionality.

Introduces MapdlInProcessRunner class for executing MAPDL commands with Python integration
Adds pytest fixture for InProcess testing with version and environment checks
Provides basic test cases to verify Python command execution within MAPDL

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

tests/test_inprocess.py

codecov · 2025-08-19T15:22:01Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.66%. Comparing base (651042d) to head (57d685f).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4185      +/-   ##
==========================================
- Coverage   91.20%   87.66%   -3.54%     
==========================================
  Files         193      193              
  Lines       15719    15719              
==========================================
- Hits        14336    13780     -556     
- Misses       1383     1939     +556

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

germa89 · 2025-08-19T16:21:59Z

@valallansys this should gives you a nice headstart for InProcess testing. It should be noticed that you should set the env variable TEST_INPROCESS to true.

…ent variable

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…ent variable

…t assertions

valallansys · 2025-09-02T10:08:05Z

Tests are failing on windows (locally). It seems to be an issue with the arguments given to subprocess.run. test_start_python_from_pymapdl is not failing @germa89 any ideas ?

This is corrected, tests are passing on both linux and windows. @germa89 would you like to try locally on macos ?

tests/test_inprocess.py

germa89 · 2025-09-02T11:32:51Z

MacOS does not support MAPDL, so we do not have to test it.

What was the issue? I saw your last commit and I dont know if the error was comming from the MAPDL env var or from the additional switches...

for more information, see https://pre-commit.ci

valallansys · 2025-09-02T13:33:18Z

MacOS does not support MAPDL, so we do not have to test it.

What was the issue? I saw your last commit and I dont know if the error was comming from the MAPDL env var or from the additional switches...

Deleting the additional switches was a mistake, the error came from the env argument. But we didn't need it as we set the en var in the init

germa89 · 2025-09-29T16:44:36Z

@valallansys how are you doing with this? do you need any support?

valallansys · 2025-10-06T09:25:33Z

@germa89 The PR should be all good for merge

valallansys · 2025-10-06T13:55:35Z

@germa89 The PR should be all good for merge

Good for me I meant, we still need someone to review and approve

germa89 · 2025-10-08T15:32:57Z

@pyansys-ci-bot LGTM.

pyansys-ci-bot

✅ Approving this PR because germa89 said so in here 😬

* ci: add MapdlInProcessRunner and related tests for Python execution in MAPDL * chore: adding changelog file 4185.miscellaneous.md [dependabot-skip] * fix: remove redundant import of ON_LOCAL in test fixture * fix: correct condition for skipping InProcess tests based on environment variable * Update tests/test_inprocess.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fix: streamline subprocess execution in MapdlInProcessRunner * fix: correct condition for skipping InProcess tests based on environment variable * fix: add support for additional switches in subprocess execution * fix: improve command execution in MapdlInProcessRunner and update test assertions * fix: update skip message for InProcess tests to improve clarity * fix: enhance MapdlInProcessRunner with virtual environment detection and output handling * fix: enhance requirement checking and add virtual environment tests * fix: keep only tests from mapdl, from pymapdl and check python path with fix without python env * ci: auto fixes from pre-commit.com hooks. for more information, see https://pre-commit.ci * fix: update MapdlInProcessRunner to include version parameter and adjust environment handling * fix: Set venv to the one we're using for pymapdl * ci: auto fixes from pre-commit.com hooks. for more information, see https://pre-commit.ci * fix: add mapdl_inprocess parameter to test_start_python_from_pymapdl function to avoid running the function in MAPDL < 25.2 * chore: adding changelog file 4185.maintenance.md [dependabot-skip] * fix: Test correction for Windows * ci: auto fixes from pre-commit.com hooks. for more information, see https://pre-commit.ci --------- Co-authored-by: pyansys-ci-bot <92810346+pyansys-ci-bot@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: valallansys <valentin.allard@ansys.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

ci: add MapdlInProcessRunner and related tests for Python execution i…

35391ca

…n MAPDL

Copilot AI review requested due to automatic review settings August 19, 2025 15:00

germa89 requested a review from a team as a code owner August 19, 2025 15:00

germa89 requested review from clatapie and pyansys-ci-bot August 19, 2025 15:00

github-actions bot assigned germa89 Aug 19, 2025

github-actions bot added the CI/CD Related with CICD, Github Actions, etc label Aug 19, 2025

Copilot AI reviewed Aug 19, 2025

View reviewed changes

tests/test_inprocess.py Outdated Show resolved Hide resolved

tests/test_inprocess.py Outdated Show resolved Hide resolved

chore: adding changelog file 4185.miscellaneous.md [dependabot-skip]

5fbe518