test(e2e): drop POSIX regex workaround + use [test] extras

Upstream pve-openapi@9deff0a's pattern-sanitize.ts now expands POSIX
character classes (`[:cntrl:]` → `\x00-\x1f\x7f`, etc.) before emitting
the spec, so the Python field_validator regexes are valid in Python's
`re` module. The conftest re.match monkey-patch can be removed.

Workflow simplified to `pip install -e .[test]` now that pyproject.toml
carries `[project.optional-dependencies]` via the template.

Author: bencurio

.github/workflows/e2e.yml

@@ -19,13 +19,7 @@ jobs:
           cache: pip
 
       - name: Install package + test deps
-        # Install test deps explicitly rather than via `[test]` extras: the
-        # generator overwrites pyproject.toml on each regen and currently drops
-        # `[project.optional-dependencies]`. Until the upstream Mustache
-        # template carries the block, the workflow owns the deps list.
-        run: |
-          pip install -e .
-          pip install 'pytest>=8' 'pytest-timeout>=2.3' 'requests>=2.32'
+        run: pip install -e .[test]
 
       - name: Authenticate to GHCR
         uses: docker/login-action@v3

e2e/conftest.py

@@ -1,30 +1,9 @@
 """Shared pytest fixtures for the PBS E2E suite."""
 from __future__ import annotations
 
-import re as _re
 from typing import Iterator
 
-# Generator gap workaround: PBS regex patterns use POSIX character classes like
-# `[:cntrl:]` / `[:^cntrl:]` which Python's `re` doesn't support — `re.match`
-# silently returns None for any input, so EVERY field_validator on PBS models
-# raises ValueError, including for legitimate values like 'root@pam'. We patch
-# `re.match` for the test process to bypass POSIX-class patterns; this is a
-# workaround until the generator translates these to PCRE-compatible regexes
-# (or the spec switches to a Python-friendly syntax).
-_POSIX_CLASS_MARKERS = ("[:cntrl:]", "[:^cntrl:]", "[:alpha:]", "[:digit:]",
-                        "[:alnum:]", "[:upper:]", "[:lower:]", "[:space:]")
-_orig_re_match = _re.match
-
-
-def _patched_re_match(pattern, string, flags=0):  # type: ignore[no-untyped-def]
-    if isinstance(pattern, str) and any(m in pattern for m in _POSIX_CLASS_MARKERS):
-        return _orig_re_match(r"^.*$", string, flags) or _orig_re_match(r"", string, flags)
-    return _orig_re_match(pattern, string, flags)
-
-
-_re.match = _patched_re_match  # type: ignore[assignment]
-
-import pytest  # noqa: E402  (must come after the patch so any pytest imports also see it)
+import pytest
 
 from clientapi_pbs import Pbs
 from e2e.helpers.clients import token_client