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.

Sign up for GitHub

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

Jump to bottom

✅ [`amp-redbull-player`] Add unit tests to increase coverage. #32904

Merged

alanorozco merged 2 commits into ampproject:master from rbeckthomas:fixit-redbull-player

Feb 25, 2021

Contributor

rbeckthomas commented Feb 25, 2021

Fixes #32902

This change increases code coverage in amp-redbull-player from 41.67% to 63.33%.
(Measured using the --coverage flag with gulp unit)

rbeckthomas added 2 commits

February 24, 2021 23:29


          [amp-redbull-player] Add unit tests to increase coverage.

7cf450b


          [amp-redbull-player] Remove duplicate supportsPlatform test.

f4c8e3f

alanorozco approved these changes

View reviewed changes

alanorozco merged commit c88fbbb into ampproject:master

rbeckthomas deleted the fixit-redbull-player branch

February 25, 2021 22:49

ampprojectbot added PR use: In Beta / Experimental PR use: In Stable labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment