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.

Sign up for GitHub

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

Enable launch test discovery in pytest #312

Merged
merged 13 commits into from
Aug 23, 2019
Merged

Enable launch test discovery in pytest #312

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
c177332
Enable launch test discovery in pytest.
hidmic Aug 13, 2019
67d53fe
Test examples using pytest.
hidmic Aug 16, 2019
94922bc
Enable downstream customization of launch tests execution.
hidmic Aug 16, 2019
8014b4c
Move launch testing examples README content to the root README.
hidmic Aug 21, 2019
4491759
Mark launch tests explicitly.
hidmic Aug 21, 2019
5be3345
Refactor parameterization support to play nice with pytest markers.
hidmic Aug 21, 2019
e7cc2b9
Improve pytest output on launch test failure.
hidmic Aug 21, 2019
b658591
Please flake8
hidmic Aug 21, 2019
d15a47c
Fix failing launch_testing tests.
hidmic Aug 21, 2019
b4b56a2
Prevent pytest from performing any assertion rewriting outside the la…
hidmic Aug 21, 2019
0705777
Change explicit setattr for regular assignment.
hidmic Aug 22, 2019
c7e6f80
Better document PYTEST_DONT_REWRITE docstring.
hidmic Aug 22, 2019
dfa47c4
Please flake8
hidmic Aug 22, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions launch_testing/launch_testing/asserts/assert_exit_codes.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,13 @@
# See the License for the specific language governing permissions and
# limitations under the License.

"""
A module providing exit code assertions.

PYTEST_DONT_REWRITE
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe, add a comment/link explaining why this is needed (e.g.: https://docs.pytest.org/en/latest/assert.html#assertion-introspection-details).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, done in c7e6f80.

"""


import os

from ..util import resolveProcesses
Expand Down
7 changes: 7 additions & 0 deletions launch_testing/launch_testing/asserts/assert_output.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,13 @@
# See the License for the specific language governing permissions and
# limitations under the License.

"""
A module providing process output assertions.

PYTEST_DONT_REWRITE
"""


import os

from osrf_pycommon.terminal_color import remove_ansi_escape_senquences
Expand Down
7 changes: 7 additions & 0 deletions launch_testing/launch_testing/asserts/assert_sequential_output.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,13 @@
# See the License for the specific language governing permissions and
# limitations under the License.

"""
A module providing process output sequence assertions.

PYTEST_DONT_REWRITE
"""


from contextlib import contextmanager

from ..util import resolveProcesses
Expand Down
7 changes: 7 additions & 0 deletions launch_testing/launch_testing/io_handler.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,13 @@
# See the License for the specific language governing permissions and
# limitations under the License.

"""
A module providing process IO capturing classes.

PYTEST_DONT_REWRITE
"""


import threading

from .asserts.assert_output import assertInStdout
Expand Down
7 changes: 7 additions & 0 deletions launch_testing/launch_testing/proc_info_handler.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,13 @@
# See the License for the specific language governing permissions and
# limitations under the License.

"""
A module providing process info capturing classes.

PYTEST_DONT_REWRITE
"""


import threading
from launch.actions import ExecuteProcess # noqa
from launch.events.process import ProcessExited
Expand Down