finished invididual plugin method testing. Moved the plugin loading into the workspacebuilder and changed the reattach to take in a builder object instead of a session so that it can access those plugins.

Author: joseph-flinn

tests/fixtures/pluginsystem/plugins/tmuxp_test_plugin_awf/tmuxp_test_plugin_awf/plugin.py

@@ -5,4 +5,5 @@ def __init__(self):
         self.message = f'[+] This is the Tmuxp Test Plugin'
 
     def after_window_finished(self, window):  
-        pass
+        if window.name == 'editor': 
+            window.rename_window('plugin_test_awf')

tests/fixtures/pluginsystem/plugins/tmuxp_test_plugin_bs/tmuxp_test_plugin_bs/plugin.py

@@ -5,4 +5,5 @@ def __init__(self):
         self.message = f'[+] This is the Tmuxp Test Plugin'
 
     def before_script(self, session):
-        pass
+        print('Changing session_name to plugin_test_bs')
+        session.rename_session('plugin_test_bs')

tests/fixtures/pluginsystem/plugins/tmuxp_test_plugin_owc/tmuxp_test_plugin_owc/plugin.py

@@ -6,4 +6,4 @@ def __init__(self):
 
     def on_window_create(self, window):
         if window.name == 'editor': 
-            pass
+            window.rename_window('plugin_test_owc')

tests/fixtures/pluginsystem/plugins/tmuxp_test_plugin_r/tmuxp_test_plugin_r/plugin.py

@@ -5,4 +5,4 @@ def __init__(self):
         self.message = f'[+] This is the Tmuxp Test Plugin'
 
     def reattach(self, session):
-        pass
+        session.rename_session('plugin_test_r')

tests/fixtures/workspacebuilder/plugin_awf.yaml

@@ -1,6 +1,6 @@
 session_name: plugin-test-awf
 plugins:
-- 'tmuxp_test_plugin_awf.plugin.TestPluginAfterWindowFinished'
+- 'tmuxp_test_plugin_awf.plugin.PluginAfterWindowFinished'
 windows:
 - window_name: editor
   layout: tiled

tests/fixtures/workspacebuilder/plugin_bs.yaml

@@ -1,6 +1,6 @@
 session_name: plugin-test-bs
 plugins:
-- 'tmuxp_test_plugin_bs.plugin.TestPluginBeforeScript'
+- 'tmuxp_test_plugin_bs.plugin.PluginBeforeScript'
 windows:
 - window_name: editor
   layout: tiled

tests/fixtures/workspacebuilder/plugin_bwb.yaml

@@ -1,6 +1,6 @@
 session_name: plugin-test-bwb
 plugins:
-- 'tmuxp_test_plugin_bwb.plugin.TestPluginBeforeWorkspaceBuild'
+- 'tmuxp_test_plugin_bwb.plugin.PluginBeforeWorkspaceBuilder'
 windows:
 - window_name: editor
   layout: tiled

tests/fixtures/workspacebuilder/plugin_owc.yaml

@@ -1,6 +1,6 @@
 session_name: plugin-test-owc
 plugins:
-- 'tmuxp_test_plugin_owc.plugin.TestPluginOnWindowCreate'
+- 'tmuxp_test_plugin_owc.plugin.PluginOnWindowCreate'
 windows:
 - window_name: editor
   layout: tiled

tests/fixtures/workspacebuilder/plugin_r.yaml

@@ -1,6 +1,6 @@
 session_name: plugin-test-r
 plugins:
-- 'tmuxp_test_plugin_r.plugin.TestPluginReattach'
+- 'tmuxp_test_plugin_r.plugin.PluginReattach'
 windows:
 - window_name: editor
   layout: tiled

tests/test_cli.py

@@ -7,6 +7,8 @@
 
 import pytest
 
+import kaptan
+
 import click
 from click.testing import CliRunner
 
@@ -20,11 +22,12 @@
     is_pure_name,
     load_workspace,
     scan_config,
-    load_plugins,
+    _reattach
 )
+from tmuxp.workspacebuilder import WorkspaceBuilder
 
 from .fixtures._util import curjoin, loadfixture
-from tmuxp_test_plugin_bwb.plugin import PluginBeforeWorkspaceBuilder
+
 
 def test_creates_config_dir_not_exists(tmpdir):
     """cli.startup() creates config dir if not exists."""
@@ -268,18 +271,6 @@ def check_cmd(config_arg):
         assert 'config not found in config dir' in check_cmd('moo')
 
 
-def test_load_plugins():
-    session_config = curjoin("workspacebuildter/plugin_bwb.yaml")
-    plugins = load_plugins(session_config)
-    assert len(plugins) == 1
-
-    test_plugin_class_types = [
-        PluginBeforeWorkspaceBuilder().__class__, 
-    ]
-    for plugin in plugins: 
-        assert plugin.__class__ in test_plugin_class_types
-
-
 def test_load_workspace(server, monkeypatch):
     # this is an implementation test. Since this testsuite may be ran within
     # a tmux session by the developer himself, delete the TMUX variable
@@ -966,3 +957,25 @@ def test_ls_cli(monkeypatch, tmpdir):
     runner = CliRunner()
     cli_output = runner.invoke(command_ls).output
     assert cli_output == '\n'.join(stems) + '\n'
+
+
+def test_reattach_plugins(server):
+    config_plugins = loadfixture("workspacebuilder/plugin_r.yaml")
+
+    sconfig = kaptan.Kaptan(handler='yaml')
+    sconfig = sconfig.import_config(config_plugins).get()
+    sconfig = config.expand(sconfig)
+
+    # open it detached
+    builder = WorkspaceBuilder(sconf=sconfig, server=server)
+    builder.build()
+
+    try:
+        _reattach(builder)
+    except libtmux.exc.LibTmuxException as error:
+        pass
+
+
+    proc = builder.session.cmd('display-message', '-p', "'#S'") 
+    
+    assert proc.stdout[0] == "'plugin_test_r'"