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.

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

Feature/issue 809 Audio Source AM/FM/XM/DAB #822

Closed
wants to merge 4 commits into from

Conversation

mrapitis
Copy link
Contributor

@mrapitis mrapitis commented Jul 20, 2018

Fixes #809

This PR is ready for review.

Risk

This PR makes minor API changes.

Testing Plan

Added new unit tests and expanded existing ones

Summary

Added RPC's, structs, enums as defined in the Audio Source AM/FM/XM/DAB – proposal. Please note this should be merged after #624 as it has a dependency on it.

CLA

  • I have signed the CLA

@codecov-io
Copy link

codecov-io commented Jul 20, 2018

Codecov Report

Merging #822 into develop will increase coverage by 0.87%.
The diff coverage is 92.14%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop     #822      +/-   ##
=============================================
+ Coverage      43.47%   44.34%   +0.87%     
- Complexity      3029     3179     +150     
=============================================
  Files            382      399      +17     
  Lines          17454    17787     +333     
  Branches        1734     1734              
=============================================
+ Hits            7588     7888     +300     
- Misses          9547     9580      +33     
  Partials         319      319
Impacted Files Coverage Δ Complexity Δ
.../smartdevicelink/proxy/rpc/ClimateControlData.java 100% <100%> (ø) 30 <30> (+8) ⬆️
...om/smartdevicelink/proxy/rpc/AudioControlData.java 100% <100%> (ø) 10 <10> (?)
...java/com/smartdevicelink/proxy/rpc/ModuleData.java 100% <100%> (ø) 17 <6> (+6) ⬆️
...com/smartdevicelink/proxy/rpc/enums/LightName.java 100% <100%> (ø) 2 <2> (?)
...in/java/com/smartdevicelink/proxy/rpc/SisData.java 100% <100%> (ø) 12 <12> (?)
...com/smartdevicelink/proxy/rpc/StationIDNumber.java 100% <100%> (ø) 6 <6> (?)
...m/smartdevicelink/proxy/rpc/enums/DisplayMode.java 100% <100%> (ø) 2 <2> (?)
...devicelink/proxy/rpc/enums/PrimaryAudioSource.java 100% <100%> (ø) 2 <1> (ø) ⬇️
...vicelink/proxy/rpc/ClimateControlCapabilities.java 100% <100%> (ø) 35 <35> (+8) ⬆️
...evicelink/proxy/rpc/RemoteControlCapabilities.java 100% <100%> (ø) 16 <6> (+6) ⬆️
... and 32 more

Continue to review full report at Codecov.

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

@mrapitis
Copy link
Contributor Author

Closing in favor of #712

@mrapitis mrapitis closed this Aug 13, 2018
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.

3 participants