fix: Enhance robustness

[ansjhenry]

Enhance the package's robustness with the following actions:

• Resolve linter errors
• Remove or verify the usage of assertions
• Check the inputs of process calls
• Avoid using exec calls

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 87.95181% with 10 lines in your changes missing coverage. Please review.

Project coverage is 78.84%. Comparing base (b34f77b) to head (42cb7ce).

Files with missing lines	Patch %	Lines
src/ansys/scade/python_wrapper/gui.py	0.00%	8 Missing ⚠️
src/ansys/scade/python_wrapper/register.py	0.00%	1 Missing ⚠️
src/ansys/scade/python_wrapper/unregister.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #37      +/-   ##
==========================================
+ Coverage   78.12%   78.84%   +0.72%     
==========================================
  Files          14       14              
  Lines        1495     1513      +18     
  Branches      235      234       -1     
==========================================
+ Hits         1168     1193      +25     
+ Misses        267      262       -5     
+ Partials       60       58       -2     

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

🚀 New features to boost your workflow:

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

[ansfbessier]

[like] Frederic Bessiere reacted to your message:

________________________________ From: Jean Henry ***@***.***> Sent: Tuesday, July 15, 2025 1:42:21 PM To: ansys/scade-python-wrapper ***@***.***> Cc: Frederic Bessiere ***@***.***>; Review requested ***@***.***> Subject: Re: [ansys/scade-python-wrapper] fix: Enhance robustness (PR #37) [External Sender] @ansjhenry commented on this pull request.

________________________________ In src/ansys/scade/python_wrapper/swan_data_parser.py<#37 (comment)>:

@@ -62,7 +63,7 @@ class EK(Enum):

_c_id_j = {}

…

-def _get_model(model: data.Model, c_id: int, role: str = None) -> data.Entity: +def _get_model(model: data.Model, c_id: int, role: str) -> data.Entity: Yes, if we maintain the default value. The change consists in making the parameter mandatory since it is always provided. — Reply to this email directly, view it on GitHub<#37 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AFEE3PADNQ26TDZPPDIYG2L3IUAL3AVCNFSM6AAAAACBRN4OXSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTAMRQGM3TINBXHA>. You are receiving this because your review was requested.Message ID: ***@***.***>