Skip to content

[MOB-10999] - JaCoCo Reporting and Codecov #892

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Ayyanchira
Copy link
Member

@Ayyanchira Ayyanchira commented Mar 27, 2025

🔹 Jira Ticket(s) if any

✏️ Description

  1. Adding codecov yml
  2. Uncommented test and coverage related tasks in build.yml
  3. Gradle on both api and api-ui now have task defined for jacoco reporting

1. Adding codecov yml
2. Uncommented test and coverage related tasks in build.yml
3. Gradle on both api and api-ui now have task defined for jacoco reporting
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds configuration for Codecov reporting and enables Jacoco reporting tasks for both the API and API-UI projects via Gradle and GitHub Actions. The key changes include:

  • Introducing a new .codecov.yml configuration file with coverage, parser, ignore, and comment settings.
  • Updating the GitHub Actions workflow to uncomment and extend Jacoco reporting and Codecov upload steps.
  • Adding a fallback Codecov uploader using bash.

Reviewed Changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
.github/workflows/build.yml Updated Gradle command to run Jacoco tasks and added Codecov integration.
.codecov.yml Added detailed Codecov configuration settings.
Files not reviewed (2)
  • iterableapi-ui/build.gradle: Language not supported
  • iterableapi/build.gradle: Language not supported
Comments suppressed due to low confidence (1)

.github/workflows/build.yml:49

  • [nitpick] The step name 'Test' may be misleading as it now runs Jacoco report generation tasks rather than executing tests. Consider renaming the step to 'Generate Jacoco Reports' to more clearly reflect its functionality.
run: ./gradlew :iterableapi:jacocoTestDebugUnitTestReport :iterableapi-ui:jacocoTestDebugUnitTestReport :app:jacocoDebugTestReport

Another way to reference token
Enabling gradle cache along with specifying the emulator with 4gb memory
Copy link

codecov bot commented Mar 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 22.47%. Comparing base (20a9821) to head (6ebecbe).
Report is 264 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             master     #892       +/-   ##
=============================================
- Coverage     61.40%   22.47%   -38.93%     
- Complexity        0      148      +148     
=============================================
  Files            75       74        -1     
  Lines          4705     4810      +105     
  Branches        535      549       +14     
=============================================
- Hits           2889     1081     -1808     
- Misses         1515     3594     +2079     
+ Partials        301      135      -166     

see 60 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9542c15...6ebecbe. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Ayyanchira Ayyanchira marked this pull request as draft April 15, 2025 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants