switch poe lint and poe build-develop order in CI, revert test move

Author: yuandrew

.github/workflows/ci.yml

@@ -58,8 +58,8 @@ jobs:
       - run: uv sync --all-extras
       - run: poe bridge-lint
         if: ${{ matrix.clippyLinter }}
-      - run: poe lint
       - run: poe build-develop
+      - run: poe lint
       - run: mkdir junit-xml
       - run: poe test ${{matrix.pytestExtraArgs}} -s --junit-xml=junit-xml/${{ matrix.python }}--${{ matrix.os }}.xml
         timeout-minutes: 15

tests/bridge/test_plugins.py

@@ -6,6 +6,7 @@
 
 import pytest
 
+import temporalio.bridge.temporal_sdk_bridge
 import temporalio.client
 import temporalio.converter
 import temporalio.worker
@@ -177,6 +178,35 @@ async def test_worker_plugin_basic_config(client: Client) -> None:
     ]
 
 
+async def test_worker_plugin_names_forwarded_to_core(
+    client: Client, monkeypatch: pytest.MonkeyPatch
+) -> None:
+    captured_plugins: list[str] = []
+
+    original_new_worker = temporalio.bridge.temporal_sdk_bridge.new_worker
+
+    def new_worker_wrapper(runtime_ref, client_ref, config):
+        nonlocal captured_plugins
+        captured_plugins = list(config.plugins)
+        return original_new_worker(runtime_ref, client_ref, config)
+
+    monkeypatch.setattr(
+        temporalio.bridge.temporal_sdk_bridge,
+        "new_worker",
+        new_worker_wrapper,
+    )
+
+    plugin1 = SimplePlugin("test-worker-plugin1")
+    plugin2 = SimplePlugin("test-worker-plugin2")
+    worker = Worker(
+        client,
+        task_queue="queue",
+        activities=[never_run_activity],
+        plugins=[plugin1, plugin2],
+    )
+    assert captured_plugins == [plugin1.name(), plugin2.name()]
+
+
 async def test_worker_duplicated_plugin(client: Client) -> None:
     new_config = client.config()
     new_config["plugins"] = [MyCombinedPlugin()]