diff options
| author | Florian Bruhin <me@the-compiler.org> | 2019-07-15 09:52:44 +0200 |
|---|---|---|
| committer | Florian Bruhin <me@the-compiler.org> | 2019-07-15 09:53:50 +0200 |
| commit | 1738e2768bbdfea860aba72ac21410ee364df9c8 (patch) | |
| tree | 367437fdac353c24612666a66c6d7643f61235b9 | |
| parent | 4ac50139f7084cd5044af9cf25a841b113aeda44 (diff) | |
| download | qutebrowser-1738e2768bbdfea860aba72ac21410ee364df9c8.tar.gz qutebrowser-1738e2768bbdfea860aba72ac21410ee364df9c8.zip | |
Make sure the process is cleaned up before tmpdir is gone
When using both the short_tmpdir and the quteproc_new fixtures, the order of
teardown seems to be undefined, so it could happen that short_tmpdir is torn
down *before* quteproc_new. That causes short_tmpdir to be unable to remove the
temporary directory at teardown time.
| -rw-r--r-- | tests/end2end/test_invocations.py | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/tests/end2end/test_invocations.py b/tests/end2end/test_invocations.py index 5f4056356..ed01c91a3 100644 --- a/tests/end2end/test_invocations.py +++ b/tests/end2end/test_invocations.py @@ -387,3 +387,6 @@ def test_qute_settings_persistence(short_tmpdir, request, quteproc_new): quteproc_new.start(args) assert quteproc_new.get_setting('search.ignore_case') == 'always' + + quteproc_new.send_cmd(':quit') + quteproc_new.wait_for_quit() |