Skip to content

Add test coverage workflow #8

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
Merged
granluo merged 16 commits into from
Mar 18, 2021
Merged

Add test coverage workflow #8

granluo merged 16 commits into from
Mar 18, 2021

Conversation

@granluo
Copy link
Contributor

@granluo granluo commented Mar 5, 2021

No description provided.

@google-oss-bot
Copy link

google-oss-bot commented Mar 12, 2021 •
edited
Loading

Coverage Report

Affected SDKs

  • GoogleDataTransport-ios-GoogleDataTransport.framework

    SDK overall coverage changed from ? (34bd15a) to 56.32% (0715501) by ?.

    Click to show coverage changes in 22 files.
    Filename Base (34bd15a) Head (0715501) Diff
    GDTCCTCompressionHelper.m ? 0.00% ?
    GDTCCTNanopbHelpers.m ? 0.00% ?
    GDTCCTUploadOperation.m ? 0.00% ?
    GDTCCTUploader.m ? 16.00% ?
    GDTCORAssert.m ? 88.89% ?
    GDTCORClock.m ? 85.26% ?
    GDTCORConsoleLogger.m ? 75.86% ?
    GDTCORDirectorySizeTracker.m ? 90.91% ?
    GDTCOREndpoints.m ? 100.00% ?
    GDTCOREvent+GDTCCTSupport.m ? 0.00% ?
    GDTCOREvent.m ? 97.70% ?
    GDTCORFlatFileStorage+Promises.m ? 0.00% ?
    GDTCORFlatFileStorage.m ? 91.22% ?
    GDTCORLifecycle.m ? 60.24% ?
    GDTCORPlatform.m ? 62.13% ?
    GDTCORReachability.m ? 78.05% ?
    GDTCORRegistrar.m ? 69.50% ?
    GDTCORStorageEventSelector.m ? 100.00% ?
    GDTCORTransformer.m ? 90.99% ?
    GDTCORTransport.m ? 86.05% ?
    GDTCORUploadBatch.m ? 0.00% ?
    GDTCORUploadCoordinator.m ? 79.53% ?

Test Logs

@granluo granluo marked this pull request as ready for review March 18, 2021 17:55
@granluo granluo requested a review from paulb777 March 18, 2021 17:56
paulb777
paulb777 approved these changes Mar 18, 2021
View reviewed changes
@granluo granluo merged commit bfc1839 into main Mar 18, 2021
@granluo granluo deleted the code_coverage branch March 18, 2021 20:06
ryihan
ryihan reviewed Apr 23, 2021
View reviewed changes
Copy link

@ryihan ryihan left a comment

Choose a reason for hiding this comment

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

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@paulb777 paulb777 paulb777 approved these changes

+2 more reviewers

@ryihan ryihan ryihan approved these changes

@jgtome39 jgtome39 jgtome39 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@granluo @google-oss-bot @paulb777 @ryihan @jgtome39