search

jellyfin / jellyfin-expo

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

Add basic render tests for screen components #356

Closed thornbill closed 2 years ago

thornbill commented 2 years ago

Just adds more tests to cover the basic rendering of screens

(The snapshot files are generated and can largely be ignored :wink:)

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
0.0% 0.0% Duplication

codecov[bot] commented 2 years ago

Codecov Report

Merging #356 (6da56e4) into master (4986e0b) will increase coverage by 14.81%. The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #356       +/-   ##
===========================================
+ Coverage   48.75%   63.57%   +14.81%     
===========================================
  Files          40       40               
  Lines         722      722               
  Branches      143      143               
===========================================
+ Hits          352      459      +107     
+ Misses        333      240       -93     
+ Partials       37       23       -14
Impacted Files Coverage Δ
screens/SettingsScreen.js 54.92% <0.00%> (+54.92%) :arrow_up:
screens/HomeScreen.js 55.55% <0.00%> (+55.55%) :arrow_up:
screens/ServerHelpScreen.js 63.63% <0.00%> (+63.63%) :arrow_up:
screens/ErrorScreen.js 70.00% <0.00%> (+70.00%) :arrow_up:
screens/LoadingScreen.js 75.00% <0.00%> (+75.00%) :arrow_up:
screens/AddServerScreen.js 75.00% <0.00%> (+75.00%) :arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more