jellyfin / jellyfin-expo

Jellyfin Mobile for iOS
https://jellyfin.org
Mozilla Public License 2.0
304 stars 66 forks source link

Add even more component tests #325

Closed thornbill closed 2 years ago

thornbill commented 2 years ago

Adds tests for ServerInput, NativeShellWebView, and VideoPlayer.

I can divide this into multiple PRs if this is too much to review.

codecov[bot] commented 2 years ago

Codecov Report

Merging #325 (d2a9671) into master (5ce2c13) will increase coverage by 10.97%. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #325       +/-   ##
===========================================
+ Coverage   37.77%   48.75%   +10.97%     
===========================================
  Files          39       39               
  Lines         720      724        +4     
  Branches      144      145        +1     
===========================================
+ Hits          272      353       +81     
+ Misses        402      334       -68     
+ Partials       46       37        -9     
Impacted Files Coverage Δ
components/ServerInput.js 100.00% <100.00%> (+100.00%) :arrow_up:
components/NativeShellWebView.js 17.91% <0.00%> (+17.91%) :arrow_up:
components/VideoPlayer.js 25.39% <0.00%> (+25.39%) :arrow_up:

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 5ce2c13...d2a9671. Read the comment docs.

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
1.6% 1.6% Duplication