aprf-reader 0.1.0__tar.gz → 0.1.1__tar.gz

{aprf_reader-0.1.0/src/aprf_reader.egg-info → aprf_reader-0.1.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: aprf-reader
-Version: 0.1.0
+Version: 0.1.1
 Summary: Summarise APRF (Aggregate Performance Reporting) inbox placement reports.
 License: MIT
 Requires-Python: >=3.11
@@ -8,6 +8,7 @@ Description-Content-Type: text/markdown
 License-File: LICENSE
 Provides-Extra: dev
 Requires-Dist: pytest>=7; extra == "dev"
+Requires-Dist: pytest-cov>=5; extra == "dev"
 Dynamic: license-file
 
 # aprf-reader

{aprf_reader-0.1.0 → aprf_reader-0.1.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "aprf-reader"
-version = "0.1.0"
+version = "0.1.1"
 description = "Summarise APRF (Aggregate Performance Reporting) inbox placement reports."
 readme = "README.md"
 license = { text = "MIT" }
@@ -12,7 +12,7 @@ requires-python = ">=3.11"
 dependencies = []
 
 [project.optional-dependencies]
-dev = ["pytest>=7"]
+dev = ["pytest>=7", "pytest-cov>=5"]
 
 [project.scripts]
 aprf-reader = "aprf_reader.cli:main"

{aprf_reader-0.1.0 → aprf_reader-0.1.1}/src/aprf_reader/__init__.py

@@ -1,3 +1,3 @@
 """Read APRF aggregate performance reports."""
 
-__version__ = "0.1.0"
+__version__ = "0.1.1"

{aprf_reader-0.1.0 → aprf_reader-0.1.1}/src/aprf_reader/render.py

@@ -34,7 +34,8 @@ def _counts(loaded: Loaded, ss: list[Stream]) -> str:
 
 def _span(s: Stream) -> str:
     gap = f", {_plural(s.missing, 'day')} with no report" if s.missing else ""
-    return f"{s.first} to {s.last}: {_plural(len(s.by_day), 'report')}{gap}"
+    days = str(s.first) if s.first == s.last else f"{s.first} to {s.last}"
+    return f"{days}: {_plural(len(s.by_day), 'report')}{gap}"
 
 
 def _day(row: Row) -> str:

{aprf_reader-0.1.0 → aprf_reader-0.1.1}/src/aprf_reader/report.py

@@ -15,7 +15,7 @@ DRAFT = "draft-ietf-mailmaint-aprf-00"
 # Names the draft defines, in display order; providers may add others
 PLACEMENT_NAMES = ("inbox", "unwanted")
 ENGAGEMENT_NAMES = ("positive", "neutral", "negative")
-# sdi_used values that mean no SDI was applied (section 5.2.1)
+# sdi_used values (first word) that mean no SDI was applied (section 5.2.1)
 NO_SDI = ("", "N/A", "N/F")
 
 
@@ -120,7 +120,8 @@ def _report(obj: object) -> Report:
         report_start=h["report_start"],
         report_end=h["report_end"],
         version=_text(h.get("version")),
-        sdi_used=_text(h.get("sdi_used")),
+        # Comcast's beta sends " - N/A - Beta - Not in use"
+        sdi_used=_text(h.get("sdi_used")).strip(" -"),
         contact_info=_text(h.get("contact_info")),
         extra_info=_text(h.get("extra_info")),
         segments=tuple(_segment(s, i) for i, s in enumerate(body)),

{aprf_reader-0.1.0 → aprf_reader-0.1.1}/src/aprf_reader/summary.py

@@ -80,7 +80,8 @@ class Stream:
             prev = r.version
         # "TBD" is the draft samples' placeholder
         out += [f"Provider note: {t}" for t in sorted({r.extra_info for r in self.by_day.values()} - {"", "TBD"})]
-        sdi = sorted({r.sdi_used for r in self.by_day.values()} - set(NO_SDI))
+        sdi = sorted({r.sdi_used for r in self.by_day.values()
+                      if r.sdi_used.split(" ", 1)[0] not in NO_SDI})
         if sdi:
             out.append("SDI used: " + ", ".join(sdi))
         return out

{aprf_reader-0.1.0 → aprf_reader-0.1.1/src/aprf_reader.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: aprf-reader
-Version: 0.1.0
+Version: 0.1.1
 Summary: Summarise APRF (Aggregate Performance Reporting) inbox placement reports.
 License: MIT
 Requires-Python: >=3.11
@@ -8,6 +8,7 @@ Description-Content-Type: text/markdown
 License-File: LICENSE
 Provides-Extra: dev
 Requires-Dist: pytest>=7; extra == "dev"
+Requires-Dist: pytest-cov>=5; extra == "dev"
 Dynamic: license-file
 
 # aprf-reader

{aprf_reader-0.1.0 → aprf_reader-0.1.1}/src/aprf_reader.egg-info/requires.txt

@@ -1,3 +1,4 @@
 
 [dev]
 pytest>=7
+pytest-cov>=5

{aprf_reader-0.1.0 → aprf_reader-0.1.1}/tests/test_reader.py

@@ -39,6 +39,16 @@ def test_draft_samples_parse():
     assert r2.segments[0].path == () and r2.sdi_used == "N/F"
 
 
+def test_comcast_beta_report(tmp_path, capsys):
+    # Shape of a real Comcast beta report as published by Spam Resource, July 2026; domain replaced
+    (r,) = parse((FIX / "comcast_beta.json").read_bytes())
+    assert r.version == "1" and r.sdi_used == "N/A - Beta - Not in use"
+    assert main([str(FIX / "comcast_beta.json")]) == 0
+    out = capsys.readouterr().out
+    assert "2026-07-13: 1 report" in out and " to 2026-07-13" not in out
+    assert "SDI used" not in out
+
+
 def test_bare_object_and_string_segment():
     obj = report(0)
     obj["body"][0]["segment"] = "CustomerA"