[ISV-1097] Remove "entrypoints" module from ignored coverage config (#188)

* [ISV-1097] Add missing unit tests

* [ISV-1097] Add missing unit tests

* [ISV-1097] Add missing unit tests

* [ISV-1097] Add missing unit tests

* [ISV-1097] Add missing unit tests

* Add json return value to the test data for ocp_version_info tests

Co-authored-by: haripate <>

Author: haripate

.coveragerc

@@ -2,5 +2,3 @@
 source = operator-pipeline-images/operatorcert
 omit = operator-pipeline-images/operatorcert/webhook/*
 
-[report]
-omit = operator-pipeline-images/operatorcert/entrypoints/*

operator-pipeline-images/operatorcert/entrypoints/bundle_dockerfile.py

@@ -13,7 +13,7 @@ class AnnotationFileNotFound(Exception):
     """
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     """
     Setup argument parser
 
@@ -66,7 +66,7 @@ def generate_dockerfile_content(args: Any) -> str:
     return dockerfile_content
 
 
-def main() -> None:
+def main() -> None:  # pragma: no cover
     """
     Main func
     """
@@ -84,5 +84,5 @@ def main() -> None:
     store_results(results)
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/create_container_image.py

@@ -12,7 +12,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> Any:
+def setup_argparser() -> Any:  # pragma: no cover
     """
     Setup argument parser
 
@@ -200,7 +200,7 @@ def remove_latest_from_previous_image(pyxis_url: str, isv_pid: str):
         pyxis.put(put_image_url, prev_image)
 
 
-def main():
+def main():  # pragma: no cover
     """
     Main func
     """
@@ -224,5 +224,5 @@ def main():
         create_container_image(args, skopeo_result, podman_result)
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/download_test_results.py

@@ -5,7 +5,7 @@
 from operatorcert.utils import store_results
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     parser = argparse.ArgumentParser(
         description="Get the test results and logs from the CI pipeline. Test results can be found in the file test_results.json"
     )
@@ -47,5 +47,5 @@ def main() -> None:
     store_results({"test_result_id": test_results_id})
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/get_cert_project_related_data.py

@@ -7,7 +7,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     """
     Setup argument parser
 
@@ -46,7 +46,7 @@ def get_cert_project_related_data(pyxis_url: str, cert_project_id: str) -> None:
     store_results({"cert_project": cert_project})
 
 
-def main() -> None:
+def main() -> None:  # pragma: no cover
     parser = setup_argparser()
     args = parser.parse_args()
 
@@ -58,5 +58,5 @@ def main() -> None:
     get_cert_project_related_data(args.pyxis_url, args.cert_project_id)
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/get_vendor_related_data.py

@@ -7,7 +7,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     """
     Setup argument parser
 
@@ -45,7 +45,7 @@ def get_vendor_related_data(pyxis_url: str, org_id: str) -> None:
     store_results({"vendor": vendor})
 
 
-def main() -> None:
+def main() -> None:  # pragma: no cover
     parser = setup_argparser()
     args = parser.parse_args()
 
@@ -57,5 +57,5 @@ def main() -> None:
     get_vendor_related_data(args.pyxis_url, args.org_id)
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/github_pr.py

@@ -13,7 +13,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> Any:
+def setup_argparser() -> Any:  # pragma: no cover
     """
     Setup argument parser
 
@@ -141,7 +141,7 @@ def get_pr_body(args: Any) -> str:
     return body
 
 
-def main() -> None:
+def main() -> None:  # pragma: no cover
     """
     Main function
     """
@@ -167,5 +167,5 @@ def main() -> None:
     LOGGER.info(f"Pull request URL: {url}")
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/hydra_checklist.py

@@ -8,7 +8,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> Any:
+def setup_argparser() -> Any:  # pragma: no cover
     """
     Setup argument parser
 
@@ -77,5 +77,5 @@ def main() -> None:
     check_hydra_checklist_status(args.cert_project_id, args.hydra_url)
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/index.py

@@ -11,7 +11,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     """
     Setup argument parser
 

operator-pipeline-images/operatorcert/entrypoints/link_pull_request.py

@@ -10,7 +10,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> Any:
+def setup_argparser() -> Any:  # pragma: no cover
     """
     Setup argument parser
 
@@ -76,7 +76,7 @@ def link_pr_to_test_results(
     return pyxis.patch(test_result_url, data)
 
 
-def main():
+def main():  # pragma: no cover
     """
     Main func
     """
@@ -94,5 +94,5 @@ def main():
     )
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/marketplace_replication.py

@@ -20,7 +20,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> Any:
+def setup_argparser() -> Any:  # pragma: no cover
     """
     Setup argument parser
 
@@ -111,12 +111,12 @@ def call_ibm_webhook(args: Any) -> None:
             verify=args.verify,
         )
         LOGGER.debug("Webhook response: %s", response)
-    except TwirpServerException as e:
+    except TwirpServerException as e:  # pragma: no cover
         LOGGER.exception(str(e))
         sys.exit(1)
 
 
-def main():
+def main():  # pragma: no cover
     """
     Main func
     """
@@ -129,5 +129,5 @@ def main():
     call_ibm_webhook(args)
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/ocp_version_info.py

@@ -7,7 +7,7 @@
 from operatorcert import ocp_version_info
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     parser = argparse.ArgumentParser(
         description="Determines the OCP version under test."
     )
@@ -37,5 +37,5 @@ def main() -> None:
     logging.info(json.dumps(version_info))
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/pipelinerun_summary.py

@@ -5,7 +5,7 @@
 from operatorcert.tekton import PipelineRun
 
 
-def parse_args() -> argparse.ArgumentParser:
+def parse_args() -> argparse.ArgumentParser:  # pragma: no cover
     parser = argparse.ArgumentParser(
         description="Construct a markdown summary for a Tekton PipelineRun."
     )

operator-pipeline-images/operatorcert/entrypoints/reserve_operator_name.py

@@ -8,7 +8,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     """
     Setup argument parser
 
@@ -79,7 +79,7 @@ def reserve_operator_name(args) -> None:
     )
 
 
-def main() -> None:
+def main() -> None:  # pragma: no cover
     parser = setup_argparser()
     args = parser.parse_args()
 
@@ -92,5 +92,5 @@ def main() -> None:
     reserve_operator_name(args)
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/set_github_status.py

@@ -7,7 +7,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     """
     Setup argument parser
 
@@ -75,5 +75,5 @@ def main() -> None:
     set_github_status(args)
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/update_cert_project_status.py

@@ -13,7 +13,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     """
     Setup argument parser
 
@@ -61,7 +61,7 @@ def update_cert_project_status(args: Any) -> None:
     store_results({"cert_project": cert_project})
 
 
-def main() -> None:
+def main() -> None:  # pragma: no cover
     parser = setup_argparser()
     args = parser.parse_args()
 
@@ -73,5 +73,5 @@ def main() -> None:
     update_cert_project_status(args)
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/upload_artifacts.py

@@ -13,7 +13,7 @@
 LOGGER = logging.getLogger("operator-cert")
 
 
-def setup_argparser() -> Any:
+def setup_argparser() -> Any:  # pragma: no cover
     """
     Setup argument parser
 
@@ -194,7 +194,7 @@ def upload_results_and_artifacts(args: Any) -> Dict[str, Any]:
     return response
 
 
-def main():
+def main():  # pragma: no cover
     """
     Main func
     """
@@ -210,5 +210,5 @@ def main():
     LOGGER.info(f"Output stored in: {args.output}")
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/verify_changed_dirs.py

@@ -8,7 +8,7 @@
 )
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     """
     Setup argument parser
 
@@ -59,5 +59,5 @@ def main() -> None:
     )
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/verify_pr_title.py

@@ -5,7 +5,7 @@
 from operatorcert.utils import store_results
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     parser = argparse.ArgumentParser(
         description="Verify, if the pull request title complies to regex"
     )
@@ -34,5 +34,5 @@ def main() -> None:
     store_results(results)
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()

operator-pipeline-images/operatorcert/entrypoints/verify_pr_uniqueness.py

@@ -4,7 +4,7 @@
 from operatorcert import verify_pr_uniqueness
 
 
-def setup_argparser() -> argparse.ArgumentParser:
+def setup_argparser() -> argparse.ArgumentParser:  # pragma: no cover
     parser = argparse.ArgumentParser(
         description="Verify if submission is unique in the repository"
     )
@@ -38,5 +38,5 @@ def main() -> None:
     verify_pr_uniqueness(repos, args.pr_url, args.bundle_name)
 
 
-if __name__ == "__main__":
+if __name__ == "__main__":  # pragma: no cover
     main()