FIX: ServiceManager.start_service ignores configured AEDT path in PYAEDT_SERVER_AEDT_PATH env var

[alexciurana]

Description

Please provide a brief description of the changes made in this pull request.

Modified if/else condition when checking the Ansys EM path retrieved from the PYAEDT_SERVER_AEDT_PATH env var to avoid the case where it exists but it's overridden with the latest one installed in the system.

Issue linked

Please mention the issue number or describe the problem this pull request addresses.

#6866

Checklist

• I have tested my changes locally.
• I have added necessary documentation or updated existing documentation.
• I have followed the coding style guidelines of this project.
• I have added appropriate tests (unit, integration, system).
• I have reviewed my changes before submitting this pull request.
• I have linked the issue or issues that are solved by the PR if any.
• I have agreed with the Contributor License Agreement (CLA).

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.48%. Comparing base (a2e6778) to head (0404ae1).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6867      +/-   ##
==========================================
- Coverage   83.51%   83.48%   -0.03%     
==========================================
  Files         246      246              
  Lines       77864    77864              
==========================================
- Hits        65027    65006      -21     
- Misses      12837    12858      +21     

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Samuelopez-ansys]

Thank you for your contribution! Good catch!

[Samuelopez-ansys]

@SMoraisAnsys @ecoussoux-ansys Do you know why smoke tests are failing? It is related to wsproto module... we could skip it...

[SMoraisAnsys]

@SMoraisAnsys @ecoussoux-ansys Do you know why smoke tests are failing? It is related to wsproto module... we could skip it...

This is now handled :)