test: guard the adopted context options and video sizes before unifying them

atest/test/01_Browser_Management/tracing_groups.py → atest/library/tracing_groups.py

@@ -1,25 +1,27 @@
 import json
 from pathlib import Path
-from typing import Union
 
 
 def get_trace_lines(path: Path) -> list[dict]:
+    """Returns the ``before`` entries of a Playwright trace file."""
     with path.open(encoding="utf-8") as file:
         return [
             json.loads(line) for line in file.readlines() if '"type":"before"' in line
         ]
 
 
 def get_trace_group_titles(path: Path) -> list[str]:
+    """Returns the titles of the trace groups in a Playwright trace file, in order."""
     return [
         line["title"]
         for line in get_trace_lines(path)
         if line.get("method") == "tracingGroup"
     ]
 
 
-def get_file_line(file: Union[Path, None], line: int, column: int) -> str:
+def get_file_line(file: Path | None, line: int, column: int) -> str | None:
+    """Returns line ``line`` of ``file``, or ``None`` without a file."""
     if file is None:
         return None
-    with file.open(encoding="utf-8") as file:
-        return file.readlines()[line - 1]
+    with file.open(encoding="utf-8") as source:
+        return source.readlines()[line - 1]

atest/test/01_Browser_Management/tracing_groups.robot

@@ -3,7 +3,7 @@ Resource    ../variables.resource
 Library     Browser    auto_delete_passed_tracing=True    tracing_group_mode=Full
 Library     OperatingSystem
 Library     ArchiveLibrary
-Library     tracing_groups.py
+Library     ../../library/tracing_groups.py
 Library     Collections
 
 *** Variables ***

atest/test/05_JS_Tests/adopted_context_recording.robot

@@ -3,7 +3,7 @@ Library      ../../library/AdoptingLibrary.py    jsextension=${CURDIR}/adopt.js
 Library      ArchiveLibrary
 Library      Collections
 Library      OperatingSystem
-Library      ../01_Browser_Management/tracing_groups.py
+Library      ../../library/tracing_groups.py
 Resource     ../variables.resource
 
 Test Tags    no-iframe