Migrating to jupyter-server 2, jupyter_client 7.x (voila-dashboar…

…ds#1308) * Get changes from voila-dashboards#1283 Co-authored-by: Mike <[email protected]> * Update tree handler * Add token flag * Update error template * Fix types/node version issue * Update binder build --------- Co-authored-by: Mike <[email protected]> (cherry picked from commit e30f6aa)
martinRenou · Jul 26, 2023 · 60a1863 · 60a1863
1 parent d5d422b
commit 60a1863
Show file tree

Hide file tree

Showing 15 changed files with 4,075 additions and 3,993 deletions.
diff --git a/.binder/environment.yml b/.binder/environment.yml
@@ -2,7 +2,7 @@ name: voila
 channels:
   - conda-forge
 dependencies:
-  - jupyterlab=3
+  - jupyterlab=3.6
   - ipywidgets=8
   - ipyvolume
   - bqplot
@@ -11,3 +11,4 @@ dependencies:
   - ipympl
   - xleaflet=0.16.0
   - xeus-cling=0.13.0
+  - python=3.10
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -19,7 +19,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest]
-        python_version: ['3.7', '3.8', '3.9', '3.10']
+        python_version: ['3.8', '3.9', '3.10']
 
     steps:
     - uses: actions/checkout@v2
@@ -30,9 +30,15 @@ jobs:
         miniforge-variant: Mambaforge
         channels: conda-forge
 
+<<<<<<< HEAD
     - name: Create the conda environment
       shell: bash -l {0}
       run: mamba install -q python=${{ matrix.python_version }} pip jupyterlab_pygments==0.1.0 pytest-cov pytest-rerunfailures nodejs yarn=1 flake8 ipywidgets matplotlib xeus-cling "traitlets>=5.0.3,<6"
+=======
+      - name: Create the conda environment
+        shell: bash -l {0}
+        run: mamba install -q python=${{ matrix.python_version }} pip jupyterlab_pygments==0.1.0 pytest-cov pytest-rerunfailures nodejs=18 yarn=1 ipywidgets matplotlib xeus-cling  "traitlets>=5.0.3,<6" ipykernel
+>>>>>>> e30f6aa (Migrating to `jupyter-server` 2, `jupyter_client` 7.x (#1308))
 
     - name: Install dependencies
       shell: bash -l {0}
@@ -69,7 +75,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [macos-10.15]
-        python_version: ['3.7', '3.8', '3.9', '3.10']
+        python_version: ['3.8', '3.9', '3.10']
 
     steps:
     - uses: actions/checkout@v2
@@ -82,7 +88,7 @@ jobs:
 
     - name: Create the conda environment
       shell: bash -l {0}
-      run: mamba install -q python=${{ matrix.python_version }} pip jupyterlab_pygments==0.1.0 pytest-cov pytest-rerunfailures nodejs yarn=1 flake8 ipywidgets matplotlib xeus-cling "traitlets>=5.0.3,<6"
+      run: mamba install -q python=${{ matrix.python_version }} pip jupyterlab_pygments==0.1.0 pytest-cov pytest-rerunfailures nodejs=18 yarn=1 ipywidgets matplotlib xeus-cling "traitlets>=5.0.3,<6" ipykernel
 
     - name: Install dependencies
       shell: bash -l {0}
@@ -113,7 +119,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [windows-latest]
-        python-version: ['3.7', '3.8', '3.9', '3.10']
+        python-version: ['3.8', '3.9', '3.10']
     steps:
       - uses: actions/checkout@v2
 
@@ -124,11 +130,11 @@ jobs:
 
       - uses: actions/setup-node@v2
         with:
-          node-version: '16'
+          node-version: '18'
 
       - name: Install dependencies
         run: |
-          python -m pip install jupyterlab_pygments==0.1.0 pytest-cov pytest-rereunfailures flake8 ipywidgets matplotlib traitlets
+          python -m pip install jupyterlab_pygments==0.1.0 pytest-cov pytest-rerunfailures ipywidgets matplotlib traitlets ipykernel
           yarn install --network-timeout 100000
           python -m pip install ".[test]"
           cd tests/test_template

diff --git a/.github/workflows/packaging.yml b/.github/workflows/packaging.yml
@@ -58,9 +58,9 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu, macos, windows]
-        python: ['3.7', '3.10']
+        python: ['3.8', '3.10']
         include:
-          - python: '3.7'
+          - python: '3.8'
             dist: 'voila*.tar.gz'
           - python: '3.10'
             dist: 'voila*.whl'

diff --git a/package.json b/package.json
@@ -22,8 +22,8 @@
     "build:prod": "lerna run build:prod",
     "build:test": "lerna run build:test",
     "clean": "lerna run clean",
-    "eslint": "eslint . --ext .ts,.tsx,.js,.jsx --fix",
-    "eslint:check": "eslint . --ext .ts,.tsx,.js,.jsx",
+    "eslint": "eslint . --ext .ts,.tsx --fix",
+    "eslint:check": "eslint . --ext .ts,.tsx",
     "install": "lerna bootstrap",
     "prettier": "prettier --write \"**/*{.ts,.tsx,.js,.jsx,.css,.json,.md}\"",
     "prettier:check": "prettier --list-different \"**/*{.ts,.tsx,.js,.jsx,.css,.json,.md}\"",

diff --git a/pyproject.toml b/pyproject.toml
@@ -10,7 +10,7 @@ build-backend = "hatchling.build"
 name = "voila"
 description = "Voilà turns Jupyter notebooks into standalone web applications"
 readme = "README.md"
-requires-python = ">=3.7"
+requires-python = ">=3.8"
 authors = [
     { name = "Voila Development Team" },
 ]
@@ -34,9 +34,9 @@ classifiers = [
     "Programming Language :: Python :: 3.10",
 ]
 dependencies = [
-    "jupyter_client>=6.1.3,<=7.4.1",
+    "jupyter_client>=7.4.4,<9",
     "jupyter_core>=4.11.0",
-    "jupyter_server>=1.18,<2.0.0",
+    "jupyter_server>=2.0.0,<3",
     "jupyterlab_server>=2.3.0,<3",
     "nbclient>=0.4.0,<0.8",
     "nbconvert>=6.4.5,<8",
@@ -64,7 +64,6 @@ test = [
     "pandas",
     "papermill",
     "pytest",
-    "pytest-rerunfailures",
     "pytest-tornasync",
 ]
 

diff --git a/tests/app/shutdown_kernel_test.py b/tests/app/shutdown_kernel_test.py
@@ -1,4 +1,13 @@
 import re
+import pytest
+
+
+@pytest.fixture
+def voila_config():
+    def config(voila_app):
+        voila_app.tornado_settings["disable_check_xsrf"] = True
+
+    return config
 
 
 async def test_shutdown_handler(http_server_client, base_url):

diff --git a/tests/execute_output_test.py b/tests/execute_output_test.py
@@ -39,7 +39,7 @@ def test_execute_output():
     path = os.path.join(BASE_DIR, 'notebooks/output.ipynb')
     nb = read(path, NO_CONVERT)
     nb_voila = deepcopy(nb)
-    executenb(nb_voila)
+    executenb(nb_voila, timeout=30)
 
     widget_states = nb.metadata.widgets[WIDGET_MIME_TYPE_STATE]['state']
     widget_states_voila = nb_voila.metadata.widgets[WIDGET_MIME_TYPE_STATE][

diff --git a/tests/notebooks/output.ipynb b/tests/notebooks/output.ipynb
@@ -23,7 +23,6 @@
    ],
    "source": [
     "import ipywidgets as widgets\n",
-    "from IPython.display import clear_output\n",
     "output1 = widgets.Output()\n",
     "output1"
    ]
@@ -79,7 +78,6 @@
     }
    ],
    "source": [
-    "import ipywidgets as widgets\n",
     "output2 = widgets.Output()\n",
     "output2"
    ]
@@ -100,8 +98,7 @@
    "source": [
     "print(\"hi2\")\n",
     "with output2:\n",
-    "    print(\"in output2\")\n",
-    "    clear_output(wait=True)"
+    "    print(\"in output2\")\n"
    ]
   },
   {
@@ -126,7 +123,6 @@
     }
    ],
    "source": [
-    "import ipywidgets as widgets\n",
     "output3 = widgets.Output()\n",
     "output3"
    ]
@@ -148,7 +144,6 @@
     "print(\"hi3\")\n",
     "with output3:\n",
     "    print(\"hello\")\n",
-    "    clear_output(wait=True)\n",
     "    print(\"world\")"
    ]
   },
@@ -174,7 +169,6 @@
     }
    ],
    "source": [
-    "import ipywidgets as widgets\n",
     "output4 = widgets.Output()\n",
     "output4"
    ]
@@ -195,8 +189,7 @@
    "source": [
     "print(\"hi4\")\n",
     "with output4:\n",
-    "    print(\"hello world\")\n",
-    "    clear_output()"
+    "    print(\"hello world\")\n"
    ]
   },
   {
@@ -221,7 +214,6 @@
     }
    ],
    "source": [
-    "import ipywidgets as widgets\n",
     "output5 = widgets.Output()\n",
     "output5"
    ]
@@ -234,8 +226,7 @@
    "source": [
     "print(\"hi5\")\n",
     "with output5:\n",
-    "    display(\"hello world\") # this is not a stream but plain text\n",
-    "clear_output()"
+    "    display(\"hello world\") # this is not a stream but plain text\n"
    ]
   },
   {
@@ -260,7 +251,6 @@
     }
    ],
    "source": [
-    "import ipywidgets as widgets\n",
     "output_outer = widgets.Output()\n",
     "output_inner = widgets.Output()\n",
     "output_inner"
@@ -296,13 +286,7 @@
    "execution_count": 14,
    "metadata": {},
    "outputs": [],
-   "source": [
-    "with output_inner:\n",
-    "    print('in inner')\n",
-    "    with output_outer:\n",
-    "        print('in outer')\n",
-    "    print('also in inner')"
-   ]
+   "source": []
   },
   {
    "cell_type": "code",