add some small tests (#5)

* only weakly match the filepath to capture test groups in the test name

* add a hypothesis-based test

* ignore hypothesis cache and python bytecode files

* use the module name and fix the mistaken use of `starmap`

* add a test for truncate

* add CI

Author: keewis

.github/workflows/ci.yaml

@@ -0,0 +1,31 @@
+name: CI
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+jobs:
+  ci:
+    name: tests
+    runs-on: [ubuntu-latest]
+    strategy:
+      matrix:
+        python-version: ["3.10", "3.11"]
+
+    steps:
+      - name: clone the repository
+        uses: actions/checkout@v3
+      - name: setup python
+        uses: actions/setup-python@v4
+        with:
+          python-version: ${{ matrix.python-version }}
+      - name: upgrade pip
+        run: |
+          python -m pip install --upgrade pip
+      - name: install dependencies
+        run: |
+          python -m pip install pytest hypothesis more-itertools
+      - name: run tests
+        run: |
+          python -m pytest -rf

.gitignore

@@ -0,0 +1,2 @@
+/.hypothesis/
+__pycache__/

parse_logs.py

@@ -57,7 +57,7 @@ def parse_record(record):
     return cls._from_json(record)
 
 
-nodeid_re = re.compile(r"(?P<filepath>.+)::(?P<name>.+?)(?:\[(?P<variant>.+)\])?")
+nodeid_re = re.compile(r"(?P<filepath>.+?)::(?P<name>.+?)(?:\[(?P<variant>.+)\])?")
 
 
 def parse_nodeid(nodeid):

test_parse_log.py

@@ -0,0 +1,49 @@
+import re
+import sys
+
+import hypothesis.strategies as st
+from hypothesis import given, note
+
+import parse_logs
+
+directory_re = r"(\w|-)+"
+path_re = re.compile(rf"/?({directory_re}(/{directory_re})*/)?test_[A-Za-z0-9_]+\.py")
+filepaths = st.from_regex(path_re, fullmatch=True)
+
+group_re = r"Test[A-Za-z0-9_]+"
+name_re = re.compile(rf"({group_re}::)*test_[A-Za-z0-9_]+")
+names = st.from_regex(name_re, fullmatch=True)
+
+variants = st.from_regex(re.compile(r"(\w+-)*\w+"), fullmatch=True)
+
+messages = st.text()
+
+
+def preformatted_reports():
+    return st.tuples(filepaths, names, variants | st.none(), messages).map(
+        lambda x: parse_logs.PreformattedReport(*x)
+    )
+
+
+@given(filepaths, names, variants)
+def test_parse_nodeid(path, name, variant):
+    if variant is not None:
+        nodeid = f"{path}::{name}[{variant}]"
+    else:
+        nodeid = f"{path}::{name}"
+
+    note(f"nodeid: {nodeid}")
+
+    expected = {"filepath": path, "name": name, "variant": variant}
+    actual = parse_logs.parse_nodeid(nodeid)
+
+    assert actual == expected
+
+
+@given(st.lists(preformatted_reports()), st.integers(min_value=0))
+def test_truncate(reports, max_chars):
+    py_version = ".".join(str(part) for part in sys.version_info[:3])
+
+    formatted = parse_logs.truncate(reports, max_chars=max_chars, py_version=py_version)
+
+    assert formatted is None or len(formatted) <= max_chars