Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix CI #221

Merged
merged 10 commits into from
Dec 12, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -85,12 +85,15 @@ jobs:
fail-fast: false
matrix:
os: [ubuntu-latest, windows-latest, macos-latest]
python-version: ["3.8", "3.11"]
python-version: ["3.8", "3.11", "3.12"]
# PyPy is not supported because we use the file_id_manager. See:
# https://github.com/jupyter-server/jupyter_server_fileid/issues/44
#include:
# - os: ubuntu-latest
# python-version: "pypy-3.8"
exclude:
- os: windows-latest
python-version: "3.12"
steps:
- name: Checkout
uses: actions/checkout@v3
Expand Down Expand Up @@ -172,7 +175,7 @@ jobs:

- name: Install the Python dependencies
run: |
pip install --pre -e ".[test]"
pip install -e ".[test]"
- name: List installed packages
run: |
Expand Down
4 changes: 3 additions & 1 deletion jupyter_collaboration/handlers.py
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,9 @@ async def prepare(self):
await self._websocket_server.start_room(self.room)
self._websocket_server.add_room(self._room_id, self.room)

return await super().prepare()
res = super().prepare()
if res is not None:
return await res

def initialize(
self,
Expand Down
9 changes: 4 additions & 5 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,13 @@
"jupyterlab",
"jupyterlab-extension"
],
"homepage": "https://github.com/jupyterlab/jupyter_collaboration",
"homepage": "https://github.com/jupyterlab/jupyter-collaboration",
"bugs": {
"url": "https://github.com/jupyterlab/jupyter_collaboration/issues"
"url": "https://github.com/jupyterlab/jupyter-collaboration/issues"
},
"repository": {
"type": "git",
"url": "https://github.com/jupyterlab/jupyter_collaboration.git"
"url": "https://github.com/jupyterlab/jupyter-collaboration.git"
},
"license": "BSD-3-Clause",
"author": {
Expand Down Expand Up @@ -65,6 +65,5 @@
"stylelint-prettier": "^3.0.0",
"typedoc": "~0.23.28",
"typescript": "~5.0.4"
},
"packageManager": "[email protected]"
}
}
6 changes: 3 additions & 3 deletions packages/collaboration-extension/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,13 @@
"jupyterlab",
"jupyterlab-extension"
],
"homepage": "https://github.com/jupyterlab/jupyter_collaboration",
"homepage": "https://github.com/jupyterlab/jupyter-collaboration",
"bugs": {
"url": "https://github.com/jupyterlab/jupyter_collaboration/issues"
"url": "https://github.com/jupyterlab/jupyter-collaboration/issues"
},
"repository": {
"type": "git",
"url": "https://github.com/jupyterlab/jupyter_collaboration.git"
"url": "https://github.com/jupyterlab/jupyter-collaboration.git"
},
"license": "BSD-3-Clause",
"author": "Project Jupyter",
Expand Down
6 changes: 3 additions & 3 deletions packages/collaboration/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,13 @@
"name": "@jupyter/collaboration",
"version": "1.0.1",
"description": "JupyterLab - Real-Time Collaboration Widgets",
"homepage": "https://github.com/jupyterlab/jupyter_collaboration",
"homepage": "https://github.com/jupyterlab/jupyter-collaboration",
"bugs": {
"url": "https://github.com/jupyterlab/jupyter_collaboration/issues"
"url": "https://github.com/jupyterlab/jupyter-collaboration/issues"
},
"repository": {
"type": "git",
"url": "https://github.com/jupyterlab/jupyter_collaboration.git"
"url": "https://github.com/jupyterlab/jupyter-collaboration.git"
},
"license": "BSD-3-Clause",
"author": "Project Jupyter",
Expand Down
6 changes: 3 additions & 3 deletions packages/docprovider/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,13 @@
"name": "@jupyter/docprovider",
"version": "1.0.1",
"description": "JupyterLab - Document Provider",
"homepage": "https://github.com/jupyterlab/jupyter_collaboration",
"homepage": "https://github.com/jupyterlab/jupyter-collaboration",
"bugs": {
"url": "https://github.com/jupyterlab/jupyter_collaboration/issues"
"url": "https://github.com/jupyterlab/jupyter-collaboration/issues"
},
"repository": {
"type": "git",
"url": "https://github.com/jupyterlab/jupyter_collaboration.git"
"url": "https://github.com/jupyterlab/jupyter-collaboration.git"
},
"license": "BSD-3-Clause",
"author": "Project Jupyter",
Expand Down
3 changes: 2 additions & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@ test = [
"jupyter_server_fileid[test]",
"pytest>=7.0",
"pytest-cov",
"pytest-asyncio",
"websockets"
]
docs = [
Expand Down Expand Up @@ -141,6 +140,8 @@ filterwarnings = [
"module:Jupyter is migrating its paths to use standard platformdirs:DeprecationWarning",
# In PyPy/Cython: see https://github.com/yaml/pyyaml/issues/688
"ignore:can't resolve package from __spec__ or __package__, falling back on __name__ and __path__:ImportWarning",
# see https://github.com/dateutil/dateutil/issues/1314
"ignore:.*datetime.utcfromtimestamp\\(\\) is deprecated.*:DeprecationWarning:",
]

[tool.mypy]
Expand Down
5 changes: 0 additions & 5 deletions tests/test_loaders.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,14 +6,11 @@
import asyncio
from datetime import datetime

import pytest

from jupyter_collaboration.loaders import FileLoader, FileLoaderMapping

from .utils import FakeContentsManager, FakeFileIDManager


@pytest.mark.asyncio
async def test_FileLoader_with_watcher():
id = "file-4567"
path = "myfile.txt"
Expand Down Expand Up @@ -46,7 +43,6 @@ async def trigger(*args):
await loader.clean()


@pytest.mark.asyncio
async def test_FileLoader_without_watcher():
id = "file-4567"
path = "myfile.txt"
Expand Down Expand Up @@ -78,7 +74,6 @@ async def trigger(*args):
await loader.clean()


@pytest.mark.asyncio
async def test_FileLoaderMapping_with_watcher():
id = "file-4567"
path = "myfile.txt"
Expand Down
8 changes: 0 additions & 8 deletions tests/test_rooms.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,11 @@
import asyncio
from datetime import datetime

import pytest
from jupyter_ydoc import YUnicode

from .utils import overite_msg, reload_msg


@pytest.mark.asyncio
async def test_should_initialize_document_room_without_store(rtc_create_mock_document_room):
content = "test"
_, _, room = rtc_create_mock_document_room("test-id", "test.txt", content)
Expand All @@ -21,7 +19,6 @@ async def test_should_initialize_document_room_without_store(rtc_create_mock_doc
assert room._document.source == content


@pytest.mark.asyncio
async def test_should_initialize_document_room_from_store(
rtc_create_SQLite_store, rtc_create_mock_document_room
):
Expand All @@ -38,7 +35,6 @@ async def test_should_initialize_document_room_from_store(
assert room._document.source == content


@pytest.mark.asyncio
async def test_should_overwrite_the_store(rtc_create_SQLite_store, rtc_create_mock_document_room):
id = "test-id"
content = "test"
Expand All @@ -54,7 +50,6 @@ async def test_should_overwrite_the_store(rtc_create_SQLite_store, rtc_create_mo
assert doc.source == content


@pytest.mark.asyncio
async def test_defined_save_delay_should_save_content_after_document_change(
rtc_create_mock_document_room,
):
Expand All @@ -70,7 +65,6 @@ async def test_defined_save_delay_should_save_content_after_document_change(
assert "save" in cm.actions


@pytest.mark.asyncio
async def test_undefined_save_delay_should_not_save_content_after_document_change(
rtc_create_mock_document_room,
):
Expand All @@ -86,7 +80,6 @@ async def test_undefined_save_delay_should_not_save_content_after_document_chang
assert "save" not in cm.actions


@pytest.mark.asyncio
async def test_should_reload_content_from_disk(rtc_create_mock_document_room):
content = "test"
last_modified = datetime.now()
Expand All @@ -109,7 +102,6 @@ async def test_should_reload_content_from_disk(rtc_create_mock_document_room):
assert room._document.source == content


@pytest.mark.asyncio
async def test_should_not_reload_content_from_disk(rtc_create_mock_document_room):
content = "test"
last_modified = datetime.now()
Expand Down
Loading