player-ui / player

https://player-ui.github.io
MIT License
73 stars 47 forks source link

iOS: move HeadlessPlayerImpl from PlayerUIInternalTestUtilities to PlayerUITestUtilitiesCore #500

Closed nancywu1 closed 2 months ago

nancywu1 commented 2 months ago

Move HeadlessPlayerImpl PlayerUITestUtilitiesCore so it can be exposed and used by test helpers in FTR

Change Type (required)

Indicate the type of change your pull request is:

Does your PR have any documentation updates?

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 91.91%. Comparing base (ae9a118) to head (a7bbe93). Report is 42 commits behind head on main.

Files with missing lines Patch % Lines
...ls-core/Sources/utilities/HeadlessPlayerImpl.swift 80.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #500 +/- ## ========================================== - Coverage 91.97% 91.91% -0.06% ========================================== Files 340 341 +1 Lines 26960 27002 +42 Branches 1945 1945 ========================================== + Hits 24796 24820 +24 - Misses 2150 2168 +18 Partials 14 14 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

nancywu1 commented 2 months ago

/canary

nancywu1 commented 2 months ago

/canary