devops: separated linting into own job (#48)

Author: mxschmitt

.github/workflows/ci.yml

@@ -5,6 +5,26 @@ on:
   pull_request:
     branches: [ master ]
 jobs:
+  infra:
+    name: Lint
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - uses: microsoft/playwright-github-action@v1
+    - name: Set up Python
+      uses: actions/setup-python@v2
+      with:
+        python-version: 3.8
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install -r local-requirements.txt
+        pip install .
+    - name: Lint
+      run: |
+        black --check .
+        mypy .
+        flake8 playwright tests
   build:
     timeout-minutes: 30
     strategy:
@@ -40,16 +60,17 @@ jobs:
         python -m pip install --upgrade pip
         pip install -r local-requirements.txt
         pip install .
-    - name: Lint
-      run: |
-        black --check .
-        mypy .
-        flake8  playwright tests
     - name: Build driver
       run: python build_driver.py
       env:
         PKG_CACHE_PATH: ${{ steps.node-pkg-cache.outputs.dir }}
     - name: Build package
       run: python build_package.py
     - name: Test
+      if: ${{ matrix.os == 'windows-latest' }}
+      # pytest-xdist does not exit on Windows
+      # https://github.com/pytest-dev/pytest-xdist/issues/60
       run: pytest -vv --browser=${{ matrix.browser }}
+    - name: Test
+      if: ${{ matrix.os != 'windows-latest' }}
+      run: pytest -vv --browser=${{ matrix.browser }} -n auto

client.py

@@ -17,7 +17,7 @@
 from playwright import browser_types
 
 
-async def run():
+async def run() -> None:
     print("Launching browser...")
     browser = await browser_types["webkit"].launch(headless=False)
     print("Contexts in browser: %d" % len(browser.contexts))
@@ -59,6 +59,7 @@ async def run():
 
     print("\nQuerying body...")
     body1 = await page1.querySelector("body")
+    assert body1
     print("Body text %s" % await body1.textContent())
 
     print("Closing page1...")

playwright/browser.py

@@ -20,9 +20,9 @@
 from types import SimpleNamespace
 from typing import Dict, List, Union
 
-if sys.version_info >= (3, 8):
+if sys.version_info >= (3, 8):  # pragma: no cover
     from typing import Literal
-else:
+else:  # pragma: no cover
     from typing_extensions import Literal
 
 

playwright/browser_context.py

@@ -34,7 +34,7 @@
 from types import SimpleNamespace
 from typing import Any, Callable, Dict, List, Optional, Union, TYPE_CHECKING
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from playwright.browser import Browser
 
 
@@ -189,7 +189,7 @@ async def waitForEvent(
             self, self._timeout_settings, event, predicate=predicate, timeout=timeout
         )
 
-    def _on_close(self):
+    def _on_close(self) -> None:
         if self._browser:
             self._browser._contexts.remove(self)
 

playwright/connection.py

@@ -32,13 +32,14 @@ async def send(self, method: str, params: dict = None) -> Any:
             params = dict()
         return await self._scope.send_message_to_server(self._guid, method, params)
 
-    def _on_message(self, method: str, params: Dict):
-        self.emit(method, params)
-
 
 class ChannelOwner(BaseEventEmitter):
     def __init__(
-        self, scope: "ConnectionScope", guid: str, initializer: Dict, is_scope=False
+        self,
+        scope: "ConnectionScope",
+        guid: str,
+        initializer: Dict,
+        is_scope: bool = False,
     ) -> None:
         super().__init__()
         self._guid = guid
@@ -64,7 +65,7 @@ def create_child(self, guid: str) -> "ConnectionScope":
         self._children.append(scope)
         return scope
 
-    def dispose(self):
+    def dispose(self) -> None:
         # Take care of hierarchy.
         for child in self._children:
             child.dispose()
@@ -98,7 +99,7 @@ def create_remote_object(self, type: str, guid: str, initializer: Dict) -> Any:
 
 
 class ProtocolCallback:
-    def __init__(self, loop: asyncio.AbstractEventLoop):
+    def __init__(self, loop: asyncio.AbstractEventLoop) -> None:
         self.stack_trace = "".join(traceback.format_stack()[-10:])
         self.future = loop.create_future()
 
@@ -145,7 +146,7 @@ async def _send_message_to_server(
         self._callbacks[id] = callback
         return await callback.future
 
-    def _dispatch(self, msg: ParsedMessagePayload):
+    def _dispatch(self, msg: ParsedMessagePayload) -> None:
 
         id = msg.get("id")
         if id:

playwright/console_message.py

@@ -33,20 +33,10 @@ def type(self) -> str:
     def text(self) -> str:
         return self._initializer["text"]
 
-    @property
-    def defaultValue(self) -> str:
-        return self._initializer["defaultValue"]
-
     @property
     def args(self) -> List[JSHandle]:
         return list(map(from_channel, self._initializer["args"]))
 
     @property
     def location(self) -> ConsoleMessageLocation:
         return self._initializer["location"]
-
-    async def accept(self, prompt_text: str = None) -> None:
-        await self._channel.send("accept", dict(promptText=prompt_text))
-
-    async def dismiss(self) -> None:
-        await self._channel.send("dismiss")

playwright/element_handle.py

@@ -37,12 +37,12 @@
     cast,
 )
 
-if sys.version_info >= (3, 8):
+if sys.version_info >= (3, 8):  # pragma: no cover
     from typing import Literal
-else:
+else:  # pragma: no cover
     from typing_extensions import Literal
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from playwright.frame import Frame
 
 

playwright/file_chooser.py

@@ -16,7 +16,7 @@
 
 from typing import List, Union, TYPE_CHECKING
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from playwright.page import Page
     from playwright.element_handle import ElementHandle
 

playwright/frame.py

@@ -38,12 +38,12 @@
 from playwright.serializers import normalize_file_payloads
 from typing import Any, Awaitable, Dict, List, Optional, Union, TYPE_CHECKING, cast
 
-if sys.version_info >= (3, 8):
+if sys.version_info >= (3, 8):  # pragma: no cover
     from typing import Literal
-else:
+else:  # pragma: no cover
     from typing_extensions import Literal
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from playwright.page import Page
 
 

playwright/helper.py

@@ -16,6 +16,7 @@
 import fnmatch
 import re
 import traceback
+from types import TracebackType
 
 from typing import (
     Any,
@@ -31,13 +32,13 @@
 
 import sys
 
-if sys.version_info >= (3, 8):
+if sys.version_info >= (3, 8):  # pragma: no cover
     from typing import Literal, TypedDict
-else:
+else:  # pragma: no cover
     from typing_extensions import Literal, TypedDict
 
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from playwright.network import Route, Request
 
 Cookie = List[Dict[str, Union[str, int, bool]]]
@@ -98,7 +99,7 @@ class ParsedMessagePayload(TypedDict, total=False):
 
 
 class URLMatcher:
-    def __init__(self, match: URLMatch):
+    def __init__(self, match: URLMatch) -> None:
         self._callback: Optional[Callable[[str], bool]] = None
         self._regex_obj: Optional[Pattern] = None
         if isinstance(match, str):
@@ -124,7 +125,7 @@ def __init__(self, parent: Optional["TimeoutSettings"]) -> None:
         self._timeout = 30000
         self._navigation_timeout = 30000
 
-    def set_timeout(self, timeout: int):
+    def set_timeout(self, timeout: int) -> None:
         self._timeout = timeout
 
     def timeout(self) -> int:
@@ -134,7 +135,7 @@ def timeout(self) -> int:
             return self._parent.timeout()
         return 30000
 
-    def set_navigation_timeout(self, navigation_timeout: int):
+    def set_navigation_timeout(self, navigation_timeout: int) -> None:
         self._navigation_timeout = navigation_timeout
 
     def navigation_timeout(self) -> int:
@@ -155,7 +156,7 @@ class TimeoutError(Error):
     pass
 
 
-def serialize_error(ex: Exception, tb) -> ErrorPayload:
+def serialize_error(ex: Exception, tb: Optional[TracebackType]) -> ErrorPayload:
     return dict(message=str(ex), stack="".join(traceback.format_tb(tb)))
 
 
@@ -193,7 +194,7 @@ def __init__(
         self.future = future
         self.timeout_future = timeout_future
 
-    def reject(self, is_crash: bool, target: str):
+    def reject(self, is_crash: bool, target: str) -> None:
         self.timeout_future.cancel()
         if self.event == "close" and not is_crash:
             return