Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Code coverage generate report #5513

Closed
wants to merge 125 commits into from
Closed

Code coverage generate report #5513

wants to merge 125 commits into from

Conversation

adhiamboperes
Copy link
Collaborator

Explanation

Essential Checklist

  • The PR title and explanation each start with "Fix #bugnum: " (If this PR fixes part of an issue, prefix the title with "Fix part of #bugnum: ...".)
  • Any changes to scripts/assets files have their rationale included in the PR explanation.
  • The PR follows the style guide.
  • The PR does not contain any unnecessary code changes from Android Studio (reference).
  • The PR is made from a branch that's not called "develop" and is up-to-date with "develop".
  • The PR is assigned to the appropriate reviewers (reference).

For UI-specific PRs only

If your PR includes UI-related changes, then:

  • Add screenshots for portrait/landscape for both a tablet & phone of the before & after UI changes
  • For the screenshots above, include both English and pseudo-localized (RTL) screenshots (see RTL guide)
  • Add a video showing the full UX flow with a screen reader enabled (see accessibility guide)
  • For PRs introducing new UI elements or color changes, both light and dark mode screenshots must be included
  • Add a screenshot demonstrating that you ran affected Espresso tests locally & that they're passing

Rd4dev added 30 commits June 13, 2024 20:00
@Rd4dev
Fix lint new line and kdoc checks
6ec35a7
@Rd4dev
Execute Bazel Command to run coverage on the test target
1ec224a
@Rd4dev
Parse the coverage execution result to acquire the generated coverage…
ee92168 
… data file path
@Rd4dev
Implemented functionality to TestBazelWorkspace utility to add source…
075c77c 
… file content and its related test content in the provided subpackage
@Rd4dev
Sample Test data and Listing all contents
d0519d6
@Rd4dev
Added Test for BazelClient - for sample target and non target executions
b15b7bb
@Rd4dev
Added Coverage Runner functionality tests
f441d11
@Rd4dev
Added tests for the RunCoverageForTestTarget utility
cc86309
@Rd4dev
Replaced Java Test Sources with Kotlin Sources
4a6073b
@Rd4dev
Fix Static Checks / Lint warnings
13534a1
@Rd4dev
Fix Buildifier Lint checks
fc52d82
@Rd4dev
Added tests for TestBazelWorkspaceTest to check addition of source an…
0226baa 
…d test files with corresponding build configurations and content
@Rd4dev
Fix Lint warnings with TestBazelWorkspaceTest
2fe7485
@Rd4dev
Fix Regex Pattern checks
bd683b5
@Rd4dev
Prototyped parsing and reading the extracted file data as byte array
d410d17
@Rd4dev
Updated the runCoverageForTestTarget to parse the data file and read …
ece147a 
…the contents as byte array
@Rd4dev
Updated Bazel Client Tests with updated execution to return byte array
b101ec8
@Rd4dev
Updated BazelClientTest for ParseCoverageDataFile and readDatFileAsBi…
5db0650 
…nary
@Rd4dev
Updated CoverageRunnerTest to check with updated getCoverage result r…
87be114 
…eturning coverage data
@Rd4dev
Updated RunCoverageForTestTargetTest to check the returned coverage data
456e81b
@Rd4dev
Update kdoc to match the function paramaters
2c65524
@Rd4dev
Removing suspend while keeping the async operations
62f9e1d
@Rd4dev
Updated RunCoverageForTestTarget to take in custom process timeout as…
5e6d2ef 
… arguments and use them
@Rd4dev
Initialized bazelClient using lazy field
31b6401
@Rd4dev
Adjusted visibility and removed unnecessary documentation for private…
81d6855 
… methods
@Rd4dev
Revoking visibility of bazelClient functions as that is called outsid…
6cb1da7 
…e in testing the utilities
@Rd4dev
Fix Lint Check issues
b41fb01
@Rd4dev
Merge branch 'develop' of https://github.com/Rd4dev/oppia-android int…
e66ac12 
…o code_coverage_bazel_command_execution
@Rd4dev
Merge branch 'develop' of https://github.com/Rd4dev/oppia-android int…
c646840 
…o code_coverage_bazel_command_execution
@Rd4dev
Introduce new script RunCoverage.kt to later take in source filename …
1ffc83d 
…instead of test target
Rd4dev added 25 commits June 26, 2024 12:07
@Rd4dev
Fixed improper dep assignment
d88edf6
@Rd4dev
Updated with upstream code_coverage_using_filename
4e89da1
@Rd4dev
Refactor runCoverageForTarget to return non-nullable CoverageReport
7a837ea
@Rd4dev
Fix tests in CoverageRunner and Lint checks
cba0815
@Rd4dev
Fix Lint checks
f6b4620
@Rd4dev
Added tests for local and sharedTests separately
cf405c6
@Rd4dev
Fix Lint check, missed separator
88f286b
@Rd4dev
Updated with upstream code_coverage_store_proto
c4c7ead
@Rd4dev
Fix Coverage Runner Test
32d9ae4
@Rd4dev
Merge branch 'code_coverage_store_as_proto' of https://github.com/opp…
402f33f 
…ia/oppia-android into code_coverage_generate_report
@Rd4dev
Cleaned the md report generation to just hold line coverage data
a17377a
@Rd4dev
Cleaned the html report generation to just hold line coverage data
12eecfe
@Rd4dev
Code cleanup - refactored redundant code
9b28dca
@Rd4dev
Resolved conflicts with develop branch
266d0f4
@Rd4dev
Resolved conflicting overloads
a0bc41c
@Rd4dev
Resolve merge conflicts with upstream code_coverage_store_proto
098a057
@Rd4dev
Added tests for CoverageReporter utility
01e0ad9
@Rd4dev
Renamed test case names to be more crisp and clear
6416569
@Rd4dev
Added tests for RunCoverage to check md and html reports
3bf9919
@Rd4dev
Fix Lint checks and added kdocs
e1e2db3
@Rd4dev
Fixed failing tests and fixed kdoc checks
821725f
@Rd4dev
Fix test cases that failed due to wrong output path
2b97b51
@Rd4dev
Fix test case that had mis-matched outputs
d6fb599
@Rd4dev
Resolved conflicts with develop branch
036ab34
@Rd4dev
Fix Lint checks on unused imports
032abcd
@adhiamboperes adhiamboperes requested review from a team as code owners August 24, 2024 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BenHenning BenHenning Awaiting requested review from BenHenning BenHenning is a code owner automatically assigned from oppia/android-app-infrastructure-reviewers

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@adhiamboperes @Rd4dev @BenHenning