stubber: Docstubs - Fix/undo removal of positional only /) from par…

…ameters Signed-off-by: Jos Verlinde <Jos.Verlinde@microsoft.com>
Josverl · Aug 25, 2024 · 6833bf3 · 6833bf3
1 parent 4a5a79a
commit 6833bf3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/stubber/rst/lookup.py b/src/stubber/rst/lookup.py
@@ -269,7 +269,7 @@ class Fix:
     Fix(r"\**", "*"),  # change weirdly written wildcards \* --> *
     Fix(r"/*", "*"),  # change weirdly written wildcards /* --> *
     Fix(r"**", "*"),  # change weirdly written wildcards ** --> *
-    Fix(r"/)", ")"),  # strange terminator in machine.USBDevice `USBDevice.active(self, [value] /)`
+    # Fix(r"/)", ")"),  # do not fix , this is a positional only notation
     Fix("'param'", "param"),  # loose notation in documentation
     # illegal keywords
     Fix(

diff --git a/tests/rst/test_rst.py b/tests/rst/test_rst.py
@@ -202,8 +202,8 @@ def test_rst_parse_class_10(line: str):
         # ESPNow extra notation
         ("ESPNow.config('param')   (ESP32 only)", "ESPNow.config(param)"),
         # machine.USBDevice
-        # Extranous / `USBDevice.active(self, [value] /)`
-        ("USBDevice.active(self, [value]/)", "USBDevice.active(self, value: Optional[Any]=None)"),
+        # Positional Only indicator should not be removed / `USBDevice.active(self, [value] /)`
+        ("USBDevice.active(self, [value]/)", "USBDevice.active(self, value: Optional[Any]=None/)"),
     ],
 )
 def test_fix_param(param_in, expected):