Skip to content

Conversation

@JulianKast
Copy link
Contributor

@JulianKast JulianKast commented Jun 21, 2022

5.5.0

This PR is [ready] for review.

Risk

This PR makes [minor] API changes.

Testing Plan

  • SoftButton manager test, Sdl 2.0 test
  • Run all unit tests
  • Additional smoke tests

Summary

See the 5.5.0 Project

CLA

jordynmackool and others added 14 commits April 21, 2022 10:43
* check isGraphicSupported in SoftButtonManager

* Remove log from testing

* fix formatting

* Unit test updates

* Add unit test to test when graphic is not supported. Align with iOS on uploading graphic when no capability is retrieved

* Fix logic to allow for static images to still be uploaded if dynamic are not supported

* Fix logic in uploading softbuttons, align with iOS

* Revert log back to a warning

* Add unit test

* Refactor isGraphicSupported to isDynamicGraphicSupported
@codecov
Copy link

codecov bot commented Jun 21, 2022

Codecov Report

Merging #1816 (a23b15a) into master (16b2ab6) will increase coverage by 0.04%.
The diff coverage is 68.51%.

❗ Current head a23b15a differs from pull request most recent head 527537c. Consider uploading reports for the commit 527537c to get more accurate results

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1816      +/-   ##
============================================
+ Coverage     54.02%   54.07%   +0.04%     
- Complexity     5522     5534      +12     
============================================
  Files           562      562              
  Lines         25757    25809      +52     
  Branches       3382     3395      +13     
============================================
+ Hits          13916    13955      +39     
- Misses        10577    10583       +6     
- Partials       1264     1271       +7     
Impacted Files Coverage Δ
...java/com/smartdevicelink/proxy/rpc/AddCommand.java 100.00% <ø> (ø)
...java/com/smartdevicelink/proxy/rpc/AddSubMenu.java 100.00% <ø> (ø)
...com/smartdevicelink/proxy/rpc/BodyInformation.java 87.17% <ø> (ø)
.../com/smartdevicelink/proxy/rpc/GetVehicleData.java 97.39% <ø> (ø)
.../smartdevicelink/proxy/rpc/KeyboardProperties.java 83.78% <ø> (ø)
...java/com/smartdevicelink/proxy/rpc/MenuParams.java 100.00% <ø> (ø)
...a/com/smartdevicelink/proxy/rpc/OnVehicleData.java 97.56% <ø> (ø)
.../smartdevicelink/proxy/rpc/SetMediaClockTimer.java 91.48% <ø> (ø)
.../main/java/com/smartdevicelink/proxy/rpc/Show.java 94.23% <ø> (ø)
...martdevicelink/proxy/rpc/SubscribeVehicleData.java 97.32% <ø> (ø)
... and 5 more

@joeygrover joeygrover merged commit c1afd4f into master Jun 29, 2022
@joeygrover joeygrover deleted the release/5.5.0_RC branch June 29, 2022 15:35
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.

5 participants