Skip to content

Edge extension and public classes - coverage++#17

Merged
kevinlind merged 1 commit intoadobe:dev-v2.0.0from
emdobrin:cov++
Jan 10, 2023
Merged

Edge extension and public classes - coverage++#17
kevinlind merged 1 commit intoadobe:dev-v2.0.0from
emdobrin:cov++

Conversation

@emdobrin
Copy link
Copy Markdown
Contributor

@emdobrin emdobrin commented Jan 10, 2023 •
edited
Loading

Description

Bump code coverage

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 10, 2023 •
edited
Loading

Codecov Report

Merging #17 (b20a08a) into dev-v2.0.0 (192f27a) will increase coverage by 6.47%.
The diff coverage is 100.00%.

@@               Coverage Diff                @@
##             dev-v2.0.0      #17      +/-   ##
================================================
+ Coverage         76.03%   82.50%   +6.47%     
- Complexity          365      385      +20     
================================================
  Files                29       29              
  Lines              1577     1577              
  Branches            225      225              
================================================
+ Hits               1199     1301     +102     
+ Misses              266      167      -99     
+ Partials            112      109       -3
Flag Coverage Δ
functional-tests 66.20% <100.00%> (?)
unit-tests 78.82% <100.00%> (+2.79%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...java/com/adobe/marketing/mobile/EdgeExtension.java 84.97% <100.00%> (+25.49%) ⬆️
.../marketing/mobile/StoreResponsePayloadManager.java 77.27% <0.00%> (+3.03%) ⬆️
...ain/java/com/adobe/marketing/mobile/EdgeState.java 98.41% <0.00%> (+3.17%) ⬆️
...adobe/marketing/mobile/NetworkResponseHandler.java 82.81% <0.00%> (+4.07%) ⬆️
...a/com/adobe/marketing/mobile/EdgeHitProcessor.java 81.82% <0.00%> (+4.55%) ⬆️
...va/com/adobe/marketing/mobile/ExperienceEvent.java 80.49% <0.00%> (+4.88%) ⬆️
...in/java/com/adobe/marketing/mobile/EventUtils.java 87.50% <0.00%> (+6.25%) ⬆️
...java/com/adobe/marketing/mobile/ConsentStatus.java 91.30% <0.00%> (+8.70%) ⬆️
...com/adobe/marketing/mobile/EdgeNetworkService.java 73.40% <0.00%> (+9.04%) ⬆️
... and 1 more

@emdobrin emdobrin marked this pull request as ready for review January 10, 2023 18:15
@emdobrin emdobrin requested a review from kevinlind January 10, 2023 18:15
@kevinlind kevinlind merged commit c2a9a9d into adobe:dev-v2.0.0 Jan 10, 2023
@emdobrin emdobrin deleted the cov++ branch January 10, 2023 19:12
@emdobrin emdobrin linked an issue Jan 11, 2023 that may be closed by this pull request
Increase code coverage #22
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kevinlind kevinlind kevinlind approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Increase code coverage

2 participants

@emdobrin @kevinlind