Add some "pragma: no cover" guards

Author: mara004

run

@@ -30,6 +30,11 @@ function packaging_pypi() {
     check-wheel-contents dist/*.whl --ignore W002 --toplevel "pypdfium2,pypdfium2_raw"
 }
 
+function coverage_impl() {
+    python3 -m coverage run --omit "$OMISSIONS" -m pytest tests/ $args
+    python3 -m coverage report
+}
+
 set -x
 
 case $1 in
@@ -38,11 +43,12 @@ test)
     python3 -m pytest tests/ $args;;
 
 coverage)
-    # you can add src/pypdfium2/__main__.py,src/pypdfium2/_cli/* to omission list to check only the core library
-    # also, you may want to add src/pypdfium2/_library_scope.py because coverage seems to be unreliable for that one
-    # BTW, you may get a minor increase in coverage by setting DEBUG_AUTOCLOSE=1
-    python3 -m coverage run --omit "src/pypdfium2_raw/bindings.py,tests/*,setupsrc/*" -m pytest tests/ $args
-    python3 -m coverage report;;
+    OMISSIONS="src/pypdfium2_raw/bindings.py,tests/*,setupsrc/*"
+    coverage_impl;;
+
+coverage-core)
+    OMISSIONS="src/pypdfium2_raw/bindings.py,src/pypdfium2/__main__.py,src/pypdfium2/_cli/*,tests/*,setupsrc/*"
+    coverage_impl;;
 
 docs-build)
     python3 -m sphinx -b html docs/source docs/build/html $args;;

src/pypdfium2/_deferred.py

@@ -12,7 +12,7 @@
 
 logger = logging.getLogger(__name__)
 
-if sys.version_info < (3, 8):
+if sys.version_info < (3, 8):  # pragma: no cover
     # NOTE alternatively, we could write our own cached property backport with python's descriptor protocol
     def cached_property(func):
         return property( functools.lru_cache(maxsize=1)(func) )

src/pypdfium2/_helpers/document.py

@@ -158,11 +158,11 @@ def init_forms(self, config=None):
 
         # safety check for older binaries to prevent a segfault (could be removed at some point)
         # https://github.com/bblanchon/pdfium-binaries/issues/105
-        if "V8" in PDFIUM_INFO.flags and "pdfium-binaries" in PDFIUM_INFO.origin and PDFIUM_INFO.build <= 5677:
+        if "V8" in PDFIUM_INFO.flags and "pdfium-binaries" in PDFIUM_INFO.origin and PDFIUM_INFO.build <= 5677:  # pragma: no cover
             raise RuntimeError("V8 enabled pdfium-binaries builds <= 5677 crash on init_forms().")
 
         if not config:
-            if "XFA" in PDFIUM_INFO.flags:
+            if "XFA" in PDFIUM_INFO.flags:  # pragma: no cover
                 js_platform = pdfium_c.IPDF_JSPLATFORM(version=3)
                 config = pdfium_c.FPDF_FORMFILLINFO(version=2, xfa_disabled=False, m_pJsPlatform=ctypes.pointer(js_platform))
             else:
@@ -175,7 +175,7 @@ def init_forms(self, config=None):
         self._add_kid(self.formenv)
 
         if formtype in (pdfium_c.FORMTYPE_XFA_FULL, pdfium_c.FORMTYPE_XFA_FOREGROUND):
-            if "XFA" in PDFIUM_INFO.flags:
+            if "XFA" in PDFIUM_INFO.flags:  # pragma: no cover
                 ok = pdfium_c.FPDF_LoadXFA(self)
                 if not ok:
                     # FIXME ability to propagate an optional exception with error code info?

src/pypdfium2/_helpers/textpage.py

@@ -128,7 +128,7 @@ def get_text_range(self, index=0, count=-1, errors="ignore"):
         # pdfium builds from fea01fa9e2 (>6167) to d6a4b27d80 (<6415) require assuming 4 bytes per character
         # https://github.com/pypdfium2-team/pypdfium2/issues/298
         # https://crbug.com/pdfium/2133
-        if 6167 < PDFIUM_INFO.build < 6415:
+        if 6167 < PDFIUM_INFO.build < 6415:  # pragma: no cover
             logger.warning(f"Due to API issues, pdfium builds between 6167 and 6415 (you have {PDFIUM_INFO.build}) require allocating twice the amount of memory in get_text_range().")
             in_count *= 2
         in_count += 1  # null terminator

src/pypdfium2/_library_scope.py

@@ -9,7 +9,7 @@
 
 def init_lib():
     assert not pdfium_i.LIBRARY_AVAILABLE
-    if pdfium_i.DEBUG_AUTOCLOSE:
+    if pdfium_i.DEBUG_AUTOCLOSE:  # pragma: no cover
         # FIXME not shown on auto-init, because DEBUG_AUTOCLOSE can only be set on the caller side after pypdfium2 has been imported...
         print("Initialize PDFium", file=sys.stderr)
 
@@ -28,7 +28,7 @@ def init_lib():
     pdfium_i.LIBRARY_AVAILABLE.value = True
 
 
-def destroy_lib():
+def destroy_lib():  # pragma: no cover
     assert pdfium_i.LIBRARY_AVAILABLE
     if pdfium_i.DEBUG_AUTOCLOSE:
         pdfium_i._safe_debug("Destroy PDFium")

src/pypdfium2/internal/bases.py

@@ -13,7 +13,7 @@
 logger = logging.getLogger(__name__)
 
 
-def _safe_debug(msg):
+def _safe_debug(msg):  # pragma: no cover
     # try to use os.write() rather than print() to avoid "reentrant call" exceptions on shutdown (see https://stackoverflow.com/q/75367828/15547292)
     try:
         os.write(sys.stderr.fileno(), (msg+"\n").encode())
@@ -56,10 +56,10 @@ def _as_parameter_(self):
 
 def _close_template(close_func, raw, obj_repr, state, parent, *args, **kwargs):
 
-    if DEBUG_AUTOCLOSE:
+    if DEBUG_AUTOCLOSE:  # pragma: no cover
         _safe_debug(f"Close ({state.value.name.lower()}) {obj_repr}")
 
-    if not LIBRARY_AVAILABLE:
+    if not LIBRARY_AVAILABLE:  # pragma: no cover
         _safe_debug(f"-> Cannot close object; pdfium library is destroyed. This may cause a memory leak!")
         return
 

src/pypdfium2/internal/consts.py

@@ -126,7 +126,7 @@ def get(self, key, default_prefix="Unhandled constant"):
 })
 
 
-if "XFA" in PDFIUM_INFO.flags:
+if "XFA" in PDFIUM_INFO.flags:  # pragma: no cover
     #: [XFA builds only] Convert a PDFium XFA error constant (:attr:`FPDF_ERR_XFA*`) to string.
     XFAErrorToStr = _fallback_dict({
         pdfium_c.FPDF_ERR_XFALOAD:   "Load error",