Skip to content

Fix automatic rename #774

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

Merged
tony merged 3 commits into from
May 21, 2022
Merged

Fix automatic rename #774

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
f973b05
tests(automatic rename): Fix timeout check, use config's shell comman…
tony Apr 19, 2022
709dc43
tests: Use "man ls" instead of "sh"
tony Apr 19, 2022
23face2
tests(automatic_rename_option): Use retry_until()
tony May 21, 2022
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
Prev Previous commit
tests(automatic_rename_option): Use retry_until()
  • Loading branch information
@tony
tony committed May 21, 2022
commit 23face22e44ca3c174177598f881076b23deff77
28 changes: 8 additions & 20 deletions tests/test_workspacebuilder.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
import libtmux
from libtmux import Window
from libtmux.common import has_gte_version
from libtmux.test import retry, temp_session
from libtmux.test import retry, retry_until, temp_session
from tmuxp import config, exc
from tmuxp.cli.load import load_plugins
from tmuxp.workspacebuilder import WorkspaceBuilder
Expand Down Expand Up @@ -386,36 +386,24 @@ def test_automatic_rename_option(session):
assert s.name != "tmuxp"
w = s.windows[0]

t = time.process_time()
while (time.process_time() - t) * 1000 < 20:
def check_window_name_mismatch() -> bool:
session.server._update_windows()
if w.name != portable_command:
break
time.sleep(0.2)
return w.name != portable_command

assert w.name != portable_command
assert retry_until(check_window_name_mismatch, 2, interval=0.25)

pane_base_index = w.show_window_option("pane-base-index", g=True)
w.select_pane(pane_base_index)

t = time.process_time()
while (time.process_time() - t) * 1000 < 20:
def check_window_name_match() -> bool:
session.server._update_windows()
if w.name == portable_command:
break
time.sleep(0.2)
return w.name == portable_command

assert w.name == portable_command
assert retry_until(check_window_name_match, 2, interval=0.25)

w.select_pane("-D")

t = time.process_time()
while (time.process_time() - t) * 1000 < 20:
session.server._update_windows()
if w["window_name"] != portable_command:
break
time.sleep(0.2)
assert w.name != portable_command
assert retry_until(check_window_name_mismatch, 2, interval=0.25)


def test_blank_pane_count(session):
Expand Down