Skip to content

refactor(overlay): convert fullscreen overlay container e2e tests into unit tests - #13657

Merged
josephperrott merged 1 commit into
angular:masterfrom
crisbeto:fullscreen-unit-tests
Oct 18, 2018
Merged

refactor(overlay): convert fullscreen overlay container e2e tests into unit tests#13657
josephperrott merged 1 commit into
angular:masterfrom
crisbeto:fullscreen-unit-tests

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Converts the e2e tests for the FullscreenOverlayContainer into unit tests since we can't trigger the fullscreen API in Chrome Headless.

@crisbetocrisbeto added pr: merge safe target: major This PR is targeted for the next major release labels Oct 17, 2018
@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Oct 17, 2018

@devversiondevversion left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

// Provide a (very limited) stub for the document. This is the most practical solution for
// now since we only hit a handful of Document APIs. If we end up having to add more
// stubs here, we should reconsider whether to use a Proxy instead. Avoiding a proxy for
// now since isn't supported on IE. See:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it isn't?

@devversiondevversion added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Oct 17, 2018

@jelbournjelbourn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

…o unit tests
Converts the e2e tests for the `FullscreenOverlayContainer` into unit tests since we can't trigger the fullscreen API in Chrome Headless.
@crisbeto
crisbetoforce-pushed the fullscreen-unit-tests branch from f494fb8 to 9ba0986CompareOctober 17, 2018 20:26
@josephperrott
josephperrott merged commit 0c00e70 into angular:masterOct 18, 2018
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 9, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakercla: yesPR author has agreed to Google's Contributor License Agreementtarget: majorThis PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@crisbeto@jelbourn@devversion@googlebot@josephperrott