Merge pull request jupyter-server#416 from afshin/tests-distribution

Add Tests to Distribution
hMED22 · Feb 18, 2021 · 4933c90 · 4933c90
2 parents 4bf307d + cbe23fb
commit 4933c90
Show file tree

Hide file tree

Showing 66 changed files with 64 additions and 37 deletions.
diff --git a/.github/workflows/python-linux.yml b/.github/workflows/python-linux.yml
@@ -43,14 +43,26 @@ jobs:
         pip freeze
         pip check
     - name: Run the tests
+      if: ${{ matrix.python-version != 'pypy3' }}
       run: |
         pytest -vv --cov jupyter_server --cov-branch --cov-report term-missing:skip-covered
+    - name: Run the tests on pypy
+      if: ${{ matrix.python-version == 'pypy3' }}
+      run: |
+        pytest -vv
     - name: Install the Python dependencies for the examples
       run: |
         cd examples/simple && pip install -e .
     - name: Run the tests for the examples
       run: |
         pytest examples/simple/tests/test_handlers.py
     - name: Coverage
+      if: ${{ matrix.python-version != 'pypy3' }}
       run: |
         codecov
+    - name: Test full install
+      run: |
+            pip uninstall -y jupyter_server
+            pip install .
+            cd /tmp
+            pytest --pyargs jupyter_server
diff --git a/.github/workflows/python-macos.yml b/.github/workflows/python-macos.yml
@@ -43,14 +43,20 @@ jobs:
         pip freeze
         pip check
     - name: Run the tests
+      if: ${{ matrix.python-version != 'pypy3' }}
       run: |
         pytest -vv --cov jupyter_server --cov-branch --cov-report term-missing:skip-covered
+    - name: Run the tests on pypy
+      if: ${{ matrix.python-version == 'pypy3' }}
+      run: |
+        pytest -vv
     - name: Install the Python dependencies for the examples
       run: |
         cd examples/simple && pip install -e .
     - name: Run the tests for the examples
       run: |
         pytest examples/simple/tests/test_handlers.py
     - name: Coverage
+      if: ${{ matrix.python-version != 'pypy3' }}
       run: |
         codecov
diff --git a/.github/workflows/python-windows.yml b/.github/workflows/python-windows.yml
@@ -37,7 +37,7 @@ jobs:
           ${{ runner.os }}-pip-
     - name: Install the Python dependencies
       run: |
-        pip install -e .[test] codecov
+        pip install -e .[test]
     - name: List installed packages
       run: |
         pip freeze

diff --git a/README.md b/README.md
@@ -14,7 +14,7 @@ To install the latest release locally, make sure you have
 
     $ pip install jupyter_server
 
-Jupyter Server currently supports Python>=3.5 on Linux, OSX and Windows.
+Jupyter Server currently supports Python>=3.6 on Linux, OSX and Windows.
 
 ### Versioning and Branches
 
@@ -33,6 +33,13 @@ Launch with:
 
     $ jupyter server
 
+### Testing
+
+To test an installed `jupyter_server`, run the following:
+
+    $ pip install jupyter_server[test]
+    $ pytest --pyargs jupyter_server
+
 ## Contributing
 
 If you are interested in contributing to the project, see [`CONTRIBUTING.rst`](CONTRIBUTING.rst).

diff --git a/tests/__init__.py → jupyter_server/tests/__init__.py b/tests/__init__.py → jupyter_server/tests/__init__.py
diff --git a/tests/auth/__init__.py → jupyter_server/tests/auth/__init__.py b/tests/auth/__init__.py → jupyter_server/tests/auth/__init__.py
diff --git a/tests/auth/test_login.py → jupyter_server/tests/auth/test_login.py b/tests/auth/test_login.py → jupyter_server/tests/auth/test_login.py
diff --git a/tests/auth/test_security.py → jupyter_server/tests/auth/test_security.py b/tests/auth/test_security.py → jupyter_server/tests/auth/test_security.py
diff --git a/tests/conftest.py → jupyter_server/tests/conftest.py b/tests/conftest.py → jupyter_server/tests/conftest.py
diff --git a/tests/extension/__init__.py → jupyter_server/tests/extension/__init__.py b/tests/extension/__init__.py → jupyter_server/tests/extension/__init__.py
diff --git a/tests/extension/conftest.py → jupyter_server/tests/extension/conftest.py b/tests/extension/conftest.py → jupyter_server/tests/extension/conftest.py
diff --git a/tests/extension/mockextensions/__init__.py → ...ests/extension/mockextensions/__init__.py b/tests/extension/mockextensions/__init__.py → ...ests/extension/mockextensions/__init__.py
@@ -9,16 +9,16 @@
 def _jupyter_server_extension_points():
     return [
         {
-            'module': 'tests.extension.mockextensions.app',
+            'module': 'jupyter_server.tests.extension.mockextensions.app',
             'app': MockExtensionApp
         },
         {
-            'module': 'tests.extension.mockextensions.mock1'
+            'module': 'jupyter_server.tests.extension.mockextensions.mock1'
         },
         {
-            'module': 'tests.extension.mockextensions.mock2'
+            'module': 'jupyter_server.tests.extension.mockextensions.mock2'
         },
         {
-            'module': 'tests.extension.mockextensions.mock3'
+            'module': 'jupyter_server.tests.extension.mockextensions.mock3'
         }
     ]
diff --git a/tests/extension/mockextensions/app.py → ...ver/tests/extension/mockextensions/app.py b/tests/extension/mockextensions/app.py → ...ver/tests/extension/mockextensions/app.py
diff --git a/tests/extension/mockextensions/mock1.py → ...r/tests/extension/mockextensions/mock1.py b/tests/extension/mockextensions/mock1.py → ...r/tests/extension/mockextensions/mock1.py
@@ -6,7 +6,7 @@
 def _jupyter_server_extension_paths():
     return [
         {
-            'module': 'tests.extension.mockextensions.mock1'
+            'module': 'jupyter_server.tests.extension.mockextensions.mock1'
         }
     ]
 

diff --git a/tests/extension/mockextensions/mock2.py → ...r/tests/extension/mockextensions/mock2.py b/tests/extension/mockextensions/mock2.py → ...r/tests/extension/mockextensions/mock2.py
@@ -6,7 +6,7 @@
 def _jupyter_server_extension_paths():
     return [
         {
-            'module': 'tests.extension.mockextensions.mock2'
+            'module': 'jupyter_server.tests.extension.mockextensions.mock2'
         }
     ]
 

diff --git a/tests/extension/mockextensions/mock3.py → ...r/tests/extension/mockextensions/mock3.py b/tests/extension/mockextensions/mock3.py → ...r/tests/extension/mockextensions/mock3.py
diff --git a/.../extension/mockextensions/mockext_both.py → .../extension/mockextensions/mockext_both.py b/.../extension/mockextensions/mockext_both.py → .../extension/mockextensions/mockext_both.py
@@ -7,7 +7,7 @@
 def _jupyter_server_extension_paths():
     return [
         {
-            'module': 'tests.extension.mockextensions.mockext_both'
+            'module': 'jupyter_server.tests.extension.mockextensions.mockext_both'
         }
     ]
 

diff --git a/tests/extension/mockextensions/mockext_py.py → ...ts/extension/mockextensions/mockext_py.py b/tests/extension/mockextensions/mockext_py.py → ...ts/extension/mockextensions/mockext_py.py
@@ -7,7 +7,7 @@
 def _jupyter_server_extension_paths():
     return [
         {
-            'module': 'tests.extension.mockextensions.mockext_py'
+            'module': 'jupyter_server.tests.extension.mockextensions.mockext_py'
         }
     ]
 

diff --git a/...s/extension/mockextensions/mockext_sys.py → ...s/extension/mockextensions/mockext_sys.py b/...s/extension/mockextensions/mockext_sys.py → ...s/extension/mockextensions/mockext_sys.py
@@ -7,7 +7,7 @@
 def _jupyter_server_extension_paths():
     return [
         {
-            'module': 'tests.extension.mockextensions.mockext_sys'
+            'module': 'jupyter_server.tests.extension.mockextensions.mockext_sys'
         }
     ]
 

diff --git a/.../extension/mockextensions/mockext_user.py → .../extension/mockextensions/mockext_user.py b/.../extension/mockextensions/mockext_user.py → .../extension/mockextensions/mockext_user.py
@@ -7,7 +7,7 @@
 def _jupyter_server_extension_paths():
     return [
         {
-            'module': 'tests.extension.mockextensions.mockext_user'
+            'module': 'jupyter_server.tests.extension.mockextensions.mockext_user'
         }
     ]
 

diff --git a/.../extension/mockextensions/static/mock.txt → .../extension/mockextensions/static/mock.txt b/.../extension/mockextensions/static/mock.txt → .../extension/mockextensions/static/mock.txt
diff --git a/tests/extension/test_app.py → jupyter_server/tests/extension/test_app.py b/tests/extension/test_app.py → jupyter_server/tests/extension/test_app.py
@@ -7,7 +7,7 @@ def jp_server_config(jp_template_dir):
     config = {
         "ServerApp": {
             "jpserver_extensions": {
-                "tests.extension.mockextensions": True
+                "jupyter_server.tests.extension.mockextensions": True
             },
         },
         "MockExtensionApp": {
@@ -22,7 +22,7 @@ def jp_server_config(jp_template_dir):
 
 @pytest.fixture
 def mock_extension(extension_manager):
-    name = "tests.extension.mockextensions"
+    name = "jupyter_server.tests.extension.mockextensions"
     pkg = extension_manager.extensions[name]
     point = pkg.extension_points["mockextension"]
     app = point.app

diff --git a/tests/extension/test_config.py → ...ter_server/tests/extension/test_config.py b/tests/extension/test_config.py → ...ter_server/tests/extension/test_config.py
diff --git a/tests/extension/test_entrypoint.py → ...server/tests/extension/test_entrypoint.py b/tests/extension/test_entrypoint.py → ...server/tests/extension/test_entrypoint.py
diff --git a/tests/extension/test_handler.py → ...er_server/tests/extension/test_handler.py b/tests/extension/test_handler.py → ...er_server/tests/extension/test_handler.py
@@ -6,7 +6,7 @@ def jp_server_config(jp_template_dir):
     return {
             "ServerApp": {
                 "jpserver_extensions": {
-                    "tests.extension.mockextensions": True
+                    "jupyter_server.tests.extension.mockextensions": True
                 }
             },
             "MockExtensionApp": {
@@ -40,7 +40,7 @@ async def test_handler_template(jp_fetch, mock_template):
         {
             "ServerApp": {
                 "jpserver_extensions": {
-                    "tests.extension.mockextensions": True
+                    "jupyter_server.tests.extension.mockextensions": True
                 }
             },
             "MockExtensionApp": {
@@ -81,7 +81,7 @@ async def test_handler_argv(jp_fetch, jp_argv):
             {
                 "ServerApp": {
                     "jpserver_extensions": {
-                        "tests.extension.mockextensions": True
+                        "jupyter_server.tests.extension.mockextensions": True
                     },
                     # Move extension handlers behind a url prefix
                     "base_url": "test_prefix"

diff --git a/tests/extension/test_launch.py → ...ter_server/tests/extension/test_launch.py b/tests/extension/test_launch.py → ...ter_server/tests/extension/test_launch.py
@@ -61,7 +61,7 @@ def _kill_extension_app():
             f'--port={port}',
             '--ip=127.0.0.1',
             '--no-browser',
-            f'--ServerApp.token={token}',
+            f'--ServerApp.token="{token}"',
             *argv,
         ], cwd=HERE)
 

diff --git a/tests/extension/test_manager.py → ...er_server/tests/extension/test_manager.py b/tests/extension/test_manager.py → ...er_server/tests/extension/test_manager.py
@@ -54,7 +54,7 @@ def test_extension_package_api():
     path1 = metadata_list[0]
     app = path1["app"]
 
-    e = ExtensionPackage(name='tests.extension.mockextensions')
+    e = ExtensionPackage(name='jupyter_server.tests.extension.mockextensions')
     e.extension_points
     assert hasattr(e, "extension_points")
     assert len(e.extension_points) == len(metadata_list)
@@ -68,16 +68,16 @@ def test_extension_package_notfound_error():
 
 def test_extension_manager_api():
     jpserver_extensions = {
-        "tests.extension.mockextensions": True
+        "jupyter_server.tests.extension.mockextensions": True
     }
     manager = ExtensionManager()
     manager.from_jpserver_extensions(jpserver_extensions)
     assert len(manager.extensions) == 1
-    assert "tests.extension.mockextensions" in manager.extensions
+    assert "jupyter_server.tests.extension.mockextensions" in manager.extensions
 
 
 def test_extension_manager_linked_extensions(jp_serverapp):
-    name = "tests.extension.mockextensions"
+    name = "jupyter_server.tests.extension.mockextensions"
     manager = ExtensionManager()
     manager.add_extension(name, enabled=True)
     manager.link_extension(name, jp_serverapp)

diff --git a/tests/extension/test_serverextension.py → ...r/tests/extension/test_serverextension.py b/tests/extension/test_serverextension.py → ...r/tests/extension/test_serverextension.py
@@ -51,12 +51,12 @@ def test_merge_config(
     # Toggle each extension module with a JSON config file
     # at the sys-prefix config dir.
     toggle_server_extension_python(
-        'tests.extension.mockextensions.mockext_sys',
+        'jupyter_server.tests.extension.mockextensions.mockext_sys',
         enabled=True,
         sys_prefix=True
     )
     toggle_server_extension_python(
-        'tests.extension.mockextensions.mockext_user',
+        'jupyter_server.tests.extension.mockextensions.mockext_user',
         enabled=True,
         user=True
     )
@@ -65,18 +65,18 @@ def test_merge_config(
     # sys-prefix supercedes users, so the extension should be disabled
     # when these two configs merge.
     toggle_server_extension_python(
-        'tests.extension.mockextensions.mockext_both',
+        'jupyter_server.tests.extension.mockextensions.mockext_both',
         enabled=True,
         sys_prefix=True
     )
     toggle_server_extension_python(
-        'tests.extension.mockextensions.mockext_both',
+        'jupyter_server.tests.extension.mockextensions.mockext_both',
         enabled=False,
         user=True
     )
 
     arg = "--ServerApp.jpserver_extensions={{'{mockext_py}': True}}".format(
-        mockext_py='tests.extension.mockextensions.mockext_py'
+        mockext_py='jupyter_server.tests.extension.mockextensions.mockext_py'
     )
 
     # Enable the last extension, mockext_py, using the CLI interface.
@@ -86,11 +86,11 @@ def test_merge_config(
     )
     # Verify that extensions are enabled and merged in proper order.
     extensions = app.jpserver_extensions
-    assert extensions['tests.extension.mockextensions.mockext_user']
-    assert extensions['tests.extension.mockextensions.mockext_sys']
-    assert extensions['tests.extension.mockextensions.mockext_py']
+    assert extensions['jupyter_server.tests.extension.mockextensions.mockext_user']
+    assert extensions['jupyter_server.tests.extension.mockextensions.mockext_sys']
+    assert extensions['jupyter_server.tests.extension.mockextensions.mockext_py']
     # Merging should causes this extension to be disabled.
-    assert not extensions['tests.extension.mockextensions.mockext_both']
+    assert not extensions['jupyter_server.tests.extension.mockextensions.mockext_both']
 
 
 @pytest.mark.parametrize(
@@ -99,8 +99,8 @@ def test_merge_config(
         {
             "ServerApp": {
                 "jpserver_extensions": OrderedDict([
-                    ('tests.extension.mockextensions.mock2', True),
-                    ('tests.extension.mockextensions.mock1', True)
+                    ('jupyter_server.tests.extension.mockextensions.mock2', True),
+                    ('jupyter_server.tests.extension.mockextensions.mock1', True)
                 ])
             }
         }

diff --git a/tests/extension/test_utils.py → jupyter_server/tests/extension/test_utils.py b/tests/extension/test_utils.py → jupyter_server/tests/extension/test_utils.py
@@ -10,10 +10,10 @@
 
 def test_validate_extension():
     # enabled at sys level
-    assert validate_extension('tests.extension.mockextensions.mockext_sys')
+    assert validate_extension('jupyter_server.tests.extension.mockextensions.mockext_sys')
     # enabled at sys, disabled at user
-    assert validate_extension('tests.extension.mockextensions.mockext_both')
+    assert validate_extension('jupyter_server.tests.extension.mockextensions.mockext_both')
     # enabled at user
-    assert validate_extension('tests.extension.mockextensions.mockext_user')
+    assert validate_extension('jupyter_server.tests.extension.mockextensions.mockext_user')
     # enabled at Python
-    assert validate_extension('tests.extension.mockextensions.mockext_py')
+    assert validate_extension('jupyter_server.tests.extension.mockextensions.mockext_py')
diff --git a/tests/nbconvert/__init__.py → jupyter_server/tests/nbconvert/__init__.py b/tests/nbconvert/__init__.py → jupyter_server/tests/nbconvert/__init__.py
diff --git a/tests/nbconvert/test_handlers.py → ...r_server/tests/nbconvert/test_handlers.py b/tests/nbconvert/test_handlers.py → ...r_server/tests/nbconvert/test_handlers.py
diff --git a/tests/services/__init__.py → jupyter_server/tests/services/__init__.py b/tests/services/__init__.py → jupyter_server/tests/services/__init__.py
diff --git a/tests/services/api/__init__.py → ...ter_server/tests/services/api/__init__.py b/tests/services/api/__init__.py → ...ter_server/tests/services/api/__init__.py
diff --git a/tests/services/api/test_api.py → ...ter_server/tests/services/api/test_api.py b/tests/services/api/test_api.py → ...ter_server/tests/services/api/test_api.py
diff --git a/tests/services/config/__init__.py → ..._server/tests/services/config/__init__.py b/tests/services/config/__init__.py → ..._server/tests/services/config/__init__.py
diff --git a/tests/services/config/test_api.py → ..._server/tests/services/config/test_api.py b/tests/services/config/test_api.py → ..._server/tests/services/config/test_api.py
diff --git a/tests/services/contents/__init__.py → ...erver/tests/services/contents/__init__.py b/tests/services/contents/__init__.py → ...erver/tests/services/contents/__init__.py
diff --git a/tests/services/contents/test_api.py → ...erver/tests/services/contents/test_api.py b/tests/services/contents/test_api.py → ...erver/tests/services/contents/test_api.py
diff --git a/tests/services/contents/test_config.py → ...er/tests/services/contents/test_config.py b/tests/services/contents/test_config.py → ...er/tests/services/contents/test_config.py
diff --git a/tests/services/contents/test_fileio.py → ...er/tests/services/contents/test_fileio.py b/tests/services/contents/test_fileio.py → ...er/tests/services/contents/test_fileio.py
diff --git a/...ervices/contents/test_largefilemanager.py → ...ervices/contents/test_largefilemanager.py b/...ervices/contents/test_largefilemanager.py → ...ervices/contents/test_largefilemanager.py
diff --git a/tests/services/contents/test_manager.py → ...r/tests/services/contents/test_manager.py b/tests/services/contents/test_manager.py → ...r/tests/services/contents/test_manager.py
diff --git a/tests/services/kernels/__init__.py → ...server/tests/services/kernels/__init__.py b/tests/services/kernels/__init__.py → ...server/tests/services/kernels/__init__.py
diff --git a/tests/services/kernels/test_api.py → ...server/tests/services/kernels/test_api.py b/tests/services/kernels/test_api.py → ...server/tests/services/kernels/test_api.py
diff --git a/tests/services/kernels/test_config.py → ...ver/tests/services/kernels/test_config.py b/tests/services/kernels/test_config.py → ...ver/tests/services/kernels/test_config.py
diff --git a/tests/services/kernels/test_cull.py → ...erver/tests/services/kernels/test_cull.py b/tests/services/kernels/test_cull.py → ...erver/tests/services/kernels/test_cull.py
diff --git a/tests/services/kernelspecs/__init__.py → ...er/tests/services/kernelspecs/__init__.py b/tests/services/kernelspecs/__init__.py → ...er/tests/services/kernelspecs/__init__.py
diff --git a/tests/services/kernelspecs/test_api.py → ...er/tests/services/kernelspecs/test_api.py b/tests/services/kernelspecs/test_api.py → ...er/tests/services/kernelspecs/test_api.py
diff --git a/tests/services/nbconvert/__init__.py → ...rver/tests/services/nbconvert/__init__.py b/tests/services/nbconvert/__init__.py → ...rver/tests/services/nbconvert/__init__.py
diff --git a/tests/services/nbconvert/test_api.py → ...rver/tests/services/nbconvert/test_api.py b/tests/services/nbconvert/test_api.py → ...rver/tests/services/nbconvert/test_api.py
diff --git a/tests/services/sessions/__init__.py → ...erver/tests/services/sessions/__init__.py b/tests/services/sessions/__init__.py → ...erver/tests/services/sessions/__init__.py
diff --git a/tests/services/sessions/test_api.py → ...erver/tests/services/sessions/test_api.py b/tests/services/sessions/test_api.py → ...erver/tests/services/sessions/test_api.py
diff --git a/tests/services/sessions/test_manager.py → ...r/tests/services/sessions/test_manager.py b/tests/services/sessions/test_manager.py → ...r/tests/services/sessions/test_manager.py
diff --git a/tests/test_config_manager.py → jupyter_server/tests/test_config_manager.py b/tests/test_config_manager.py → jupyter_server/tests/test_config_manager.py
diff --git a/tests/test_files.py → jupyter_server/tests/test_files.py b/tests/test_files.py → jupyter_server/tests/test_files.py
diff --git a/tests/test_gateway.py → jupyter_server/tests/test_gateway.py b/tests/test_gateway.py → jupyter_server/tests/test_gateway.py
diff --git a/tests/test_paths.py → jupyter_server/tests/test_paths.py b/tests/test_paths.py → jupyter_server/tests/test_paths.py
diff --git a/tests/test_serialize.py → jupyter_server/tests/test_serialize.py b/tests/test_serialize.py → jupyter_server/tests/test_serialize.py
diff --git a/tests/test_serverapp.py → jupyter_server/tests/test_serverapp.py b/tests/test_serverapp.py → jupyter_server/tests/test_serverapp.py
diff --git a/tests/test_terminal.py → jupyter_server/tests/test_terminal.py b/tests/test_terminal.py → jupyter_server/tests/test_terminal.py
diff --git a/tests/test_traittypes.py → jupyter_server/tests/test_traittypes.py b/tests/test_traittypes.py → jupyter_server/tests/test_traittypes.py
diff --git a/tests/test_utils.py → jupyter_server/tests/test_utils.py b/tests/test_utils.py → jupyter_server/tests/test_utils.py
diff --git a/tests/test_version.py → jupyter_server/tests/test_version.py b/tests/test_version.py → jupyter_server/tests/test_version.py
diff --git a/tests/test_view.py → jupyter_server/tests/test_view.py b/tests/test_view.py → jupyter_server/tests/test_view.py
diff --git a/tests/utils.py → jupyter_server/tests/utils.py b/tests/utils.py → jupyter_server/tests/utils.py
diff --git a/pytest.ini b/pytest.ini
@@ -1,3 +1,5 @@
 [pytest]
 # Exclude the example tests.
 norecursedirs = examples/*
+# Work around for https://github.com/pytest-dev/pytest/issues/4039
+addopts = --pyargs jupyter_server