Clarifying something

Author: zappolowski

tests/fixtures/workspace/builder/environment_vars.yaml

@@ -2,6 +2,7 @@ session_name: test env vars
 start_directory: '~'
 environment:
   FOO: SESSION
+  BAR: SESSION
   PATH: /tmp
 options:
   default-shell: /bin/sh
@@ -25,3 +26,11 @@ windows:
   - pane
   - environment:
       FOO: PANE
+  - pane
+- window_name: partial_overrides
+  environment:
+    FOO: WINDOW
+    BAR: WINDOW
+  panes:
+  - environment:
+      FOO: PANE

tests/workspace/test_builder.py

@@ -371,6 +371,14 @@ def test_environment_variables(session):
     pane.send_keys("echo $FOO")
     assert pane.capture_pane()[1] == "PANE"
 
+    partial_overrides_win = session.windows[4]
+    pane = partial_overrides_win.panes[0]
+    pane.send_keys("echo $FOO")
+    assert pane.capture_pane()[1] == "PANE"
+    pane.send_keys("echo $BAR")
+    # TODO: Currently this is SESSION, but WINDOW could be expected as well.
+    assert pane.capture_pane()[3] == "SESSION"
+
 
 @pytest.mark.skipif(
     has_gte_version("3.0"),
@@ -396,15 +404,15 @@ def test_environment_variables_logs(session: Session, caplog: pytest.LogCaptureF
             for record in caplog.records
             if "Cannot set environment for new window." in record.msg
         )
-        == 4
+        == 5
     ), "Warning on creating windows missing"
     assert (
         sum(
             1
             for record in caplog.records
             if "Cannot set environment for new pane." in record.msg
         )
-        == 1
+        == 2
     ), "Warning on creating panes missing"