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.

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

[FIX] Handle Python test execution runtime errors #145

Merged

Conversation

rquidute
Copy link
Contributor

What changed

The goal of this PR is to avoid test execution hangs if a runtime error occurs.

Related Issue

project-chip/matter-test-scripts#365

Testing

Execution does not Hangs when runtime error happens
Screenshot 2024-09-13 at 15 30 00

@rquidute rquidute changed the base branch from main to v2.11-beta3.1+fall2024 September 16, 2024 20:46
@rquidute rquidute merged commit 0bdd524 into v2.11-beta3.1+fall2024 Sep 16, 2024
8 checks passed
hiltonlima added a commit that referenced this pull request Sep 19, 2024
…ython_tests

* v2.11-beta3.1+fall2024:
  Fix data_model mapped volume
  [FIX] Force container removal at destroy (#143)
  [FIX] Handle Python test execution runtime errors (#145)

# Conflicts:
#	test_collections/matter/sdk_tests/support/python_testing/models/test_case.py
hiltonlima added a commit that referenced this pull request Sep 26, 2024
* [FIX] Handle Python test execution runtime errors (#145)

* Handle Python test execution runtime errors

* Update test_collections/matter/sdk_tests/support/python_testing/models/test_case.py

Co-authored-by: hiltonlima <[email protected]>

---------

Co-authored-by: hiltonlima <[email protected]>

* [FIX] Force container removal at destroy (#143)

* Force container removal

* Remove __wait_for_server_exit

* Surround __wait_for_server_exit with try except

* Fix mypy

* Fix data_model mapped volume

* Update test_collections/matter/sdk_tests/support/sdk_container.py

Co-authored-by: Romulo Quidute Filho <[email protected]>

---------

Co-authored-by: Romulo Quidute Filho <[email protected]>
Co-authored-by: Romulo Quidute Filho <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants