fix: additional fixes

Author: ideepu

codecov/coverage/base.py

@@ -101,7 +101,7 @@ def get_coverage_info(self, coverage_path: pathlib.Path) -> Coverage:
 
     @abstractmethod
     def extract_info(self, data: dict) -> Coverage:
-        raise NotImplementedError
+        raise NotImplementedError  # pragma: no cover
 
     def get_diff_coverage_info(  # pylint: disable=too-many-locals
         self,

codecov/template_files/comment.md.j2

@@ -5,7 +5,7 @@
 {%- block coverage_evolution_badge -%}
 {%- if coverage %}
 {%- set text = "Coverage of the whole project for this PR is " ~ coverage.info.percent_covered_display ~ "%." -%}
-{%- set color = coverage.info.percent_covered | get_badge_color -%}
+{%- set color = coverage.info.percent_covered | x100 | get_badge_color -%}
 <img title="{{ text }}" src="{{ 'Coverage' | generate_badge(message=coverage.info.percent_covered_display ~ '%', color=color) }}">
 {%- endif -%}
 {%- endblock coverage_evolution_badge -%}

codecov/template_files/macros.md.j2

@@ -34,7 +34,7 @@
 {%- set text = "The coverage rate of " ~ path ~ " is " ~ percent_covered_display ~ "% (" ~ covered_statements_count ~ "/" ~ statements_count ~ ")." -%}
 {%- set label = percent_covered_display ~ "%" -%}
 {%- set message = "(" ~ covered_statements_count ~ "/" ~ statements_count ~ ")" -%}
-{%- set color = percent_covered | get_badge_color -%}
+{%- set color = percent_covered | x100 | get_badge_color -%}
 <td align="center"><a href="{{ path | file_url(base=base) }}"><img title="{{ text }}" src="{{ label | generate_badge(message=message, color=color) }}"></a></td>
 {%- endmacro -%}
 

codecov/template_files/pr.md.j2

@@ -1,6 +1,6 @@
 {%- if not files %}
 
-_This PR does not seem to contain any modification to coverable code._
+_This PR does not include changes to coverable code or code with missing coverage._
 {%- else -%}
 <details><summary>Click to see coverage of changed files</summary>
 <br>

tests/test_github.py

@@ -361,14 +361,19 @@ def test_post_comment_update(
         test_config,
         gh_client,
     ):
+        comment_with_no_marker = {
+            'user': {'login': 'foo'},
+            'body': 'Hey! Hi! How are you?',
+            'id': 123,
+        }
         comment = {
             'user': {'login': 'foo'},
             'body': 'Hey! Hi! How are you? marker',
             'id': 456,
         }
         session.register(
             'GET', f'/repos/{test_config.GITHUB_REPOSITORY}/issues/{test_config.GITHUB_PR_NUMBER}/comments'
-        )(json=[comment])
+        )(json=[comment_with_no_marker, comment])
         session.register(
             'PATCH',
             f'/repos/{test_config.GITHUB_REPOSITORY}/issues/comments/{comment["id"]}',

tests/test_template.py

@@ -236,7 +236,7 @@ def test_template_no_files(coverage_obj):
         marker='<!-- foo -->',
         subproject_id='foo',
     )
-    assert '_This PR does not seem to contain any modification to coverable code.' in result
+    assert '_This PR does not include changes to coverable code or code with missing coverage.' in result
     assert 'code.py' not in result
     assert 'other.py' not in result