Skip to content

Commit febb150

Browse files
committed
Add comments and rename a variable for readability
1 parent af0a722 commit febb150

File tree

2 files changed

+11
-5
lines changed

2 files changed

+11
-5
lines changed

.github/workflows/test.yaml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,12 @@ jobs:
103103
jupyter lab build --minimize=False --debug
104104
105105
- name: Check the lab extension
106-
if: ${{ !(matrix.jupyterlab-version == '2' && matrix.python-version == '3.11') }}
106+
# We test the labextension thoroughly in the acceptance tests below, so
107+
# we have conditionally disabled this basic check is to avoid issues in
108+
# jupyterlab.browser_check with jupyterlab 2 and a modern version of
109+
# python (3.11+).
110+
#
111+
if: ${{ !(matrix.jupyterlab-version == '2' && startsWith(matrix.python-version, '3.1')) }}
107112
run: |
108113
jupyter labextension list
109114
jupyter labextension list 2>&1 | grep -ie '@jupyterlab/server-proxy.*OK.*'

tests/acceptance/test_acceptance.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,11 @@
77

88
HERE = Path(__file__).parent
99
OUTPUT = HERE.parent.parent / "build/robot"
10-
SERVER_INFO = None
11-
LAB_INFO = None
10+
JUPYTER_SERVER_INFO = None
1211

1312
try:
1413
import jupyter_server
15-
SERVER_INFO = jupyter_server.version_info
14+
JUPYTER_SERVER_INFO = jupyter_server.version_info
1615
except ImportError:
1716
pass
1817

@@ -23,7 +22,9 @@ def test_robot():
2322

2423
env = dict(**os.environ)
2524

26-
if SERVER_INFO is None:
25+
# JUPYTER_LIBRARY_* env vars documentation:
26+
# https://robotframework-jupyterlibrary.readthedocs.io/en/stable/LIMITS.html#notebookapp-vs-serverapp
27+
if JUPYTER_SERVER_INFO is None:
2728
env.update(
2829
JUPYTER_LIBRARY_APP_COMMAND="jupyter-notebook",
2930
JUPYTER_LIBRARY_APP="NotebookApp",

0 commit comments

Comments
 (0)