Skip to content
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.

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

feat(material-experimental): add test harness for snack-bar #16697

Closed

Conversation

devversion
Copy link
Member

No description provided.

@devversion devversion added pr: merge safe target: patch This PR is targeted for the next patch release labels Aug 6, 2019
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Aug 6, 2019
@devversion devversion force-pushed the feat/test-harness-mdc-snack-bar branch from 6827a07 to b95e59d Compare August 6, 2019 10:35
@devversion devversion changed the title feat(material-experimental): add test harness for mdc-snack-bar feat(material-experimental): add test harness for snack-bar Aug 6, 2019
@devversion devversion force-pushed the feat/test-harness-mdc-snack-bar branch from b95e59d to 1ebb813 Compare August 6, 2019 11:05
crisbeto added a commit to crisbeto/material2 that referenced this pull request Aug 7, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Aug 7, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
devversion pushed a commit to devversion/material2 that referenced this pull request Aug 7, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Aug 7, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
devversion pushed a commit to devversion/material2 that referenced this pull request Aug 7, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
@devversion devversion force-pushed the feat/test-harness-mdc-snack-bar branch from 1ebb813 to 54e554b Compare August 7, 2019 07:37
devversion pushed a commit to devversion/material2 that referenced this pull request Aug 7, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
@googlebot

This comment has been minimized.

@googlebot googlebot added cla: no PR author must sign Google's Contributor License Agreement: https://opensource.google.com/docs/cla and removed cla: yes PR author has agreed to Google's Contributor License Agreement labels Aug 7, 2019
@devversion devversion force-pushed the feat/test-harness-mdc-snack-bar branch from 54e554b to fb0a398 Compare August 7, 2019 07:57
@googlebot

This comment has been minimized.

@googlebot googlebot added cla: yes PR author has agreed to Google's Contributor License Agreement and removed cla: no PR author must sign Google's Contributor License Agreement: https://opensource.google.com/docs/cla labels Aug 7, 2019
devversion pushed a commit to devversion/material2 that referenced this pull request Aug 15, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Aug 22, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Aug 22, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Aug 22, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Aug 22, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Aug 22, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Aug 22, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Aug 22, 2019
… add API for getting property value

Based on the discussion in angular#16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in angular#16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
mmalerba pushed a commit that referenced this pull request Aug 22, 2019
… add API for getting property value (#16706)

Based on the discussion in #16697 (comment), these changes fix a couple of things that I ran into while doing the `mat-autocomplete` test harness in #16620.

* Fixes querying for elements outside the harness not working, because the wrong root node was set.
* Adds an API to retrieve the value of a property on a DOM node.
@devversion devversion force-pushed the feat/test-harness-mdc-snack-bar branch from fb0a398 to 44d4f44 Compare August 28, 2019 06:54
@devversion devversion requested a review from a team as a code owner August 28, 2019 06:54
devversion added a commit to devversion/material2 that referenced this pull request Aug 28, 2019
…nment

Adds a new method that can be used by harness consumers to load a harness
through `HarnessLoader` from the document root. This is helpful for harnesses
which need to match elements outside of testbed fixtures (e.g. snack-bars, overlays etc.).

Needed for angular#16697 and angular#16709.
devversion added a commit to devversion/material2 that referenced this pull request Aug 29, 2019
…s environment

Adds a new method that can be used by harness consumers to load a harness
through `HarnessLoader` from the document root. This is helpful for harnesses
which need to match elements outside of testbed fixtures (e.g. snack-bars, overlays etc.).

Needed for angular#16697 and angular#16709.
devversion added a commit to devversion/material2 that referenced this pull request Aug 29, 2019
…s environment

Adds a new method that can be used by harness consumers to load a harness
through `HarnessLoader` from the document root. This is helpful for harnesses
which need to match elements outside of testbed fixtures (e.g. snack-bars, overlays etc.).

Needed for angular#16697 and angular#16709.
devversion added a commit to devversion/material2 that referenced this pull request Sep 5, 2019
…s environment

Adds a new method that can be used by harness consumers to load a harness
through `HarnessLoader` from the document root. This is helpful for harnesses
which need to match elements outside of testbed fixtures (e.g. snack-bars, overlays etc.).

Needed for angular#16697 and angular#16709.
devversion added a commit to devversion/material2 that referenced this pull request Sep 6, 2019
…s environment

Adds a new method that can be used by harness consumers to load a harness
through `HarnessLoader` from the document root. This is helpful for harnesses
which need to match elements outside of testbed fixtures (e.g. snack-bars, overlays etc.).

Needed for angular#16697 and angular#16709.
@devversion devversion force-pushed the feat/test-harness-mdc-snack-bar branch from 44d4f44 to 3e9db90 Compare September 6, 2019 09:49
devversion added a commit to devversion/material2 that referenced this pull request Sep 6, 2019
…s environment

Adds a new method that can be used by harness consumers to load a harness
through `HarnessLoader` from the document root. This is helpful for harnesses
which need to match elements outside of testbed fixtures (e.g. snack-bars, overlays etc.).

Needed for angular#16697 and angular#16709.
devversion added a commit to devversion/material2 that referenced this pull request Sep 17, 2019
…s environment

Adds a new method that can be used by harness consumers to load a harness
through `HarnessLoader` from the document root. This is helpful for harnesses
which need to match elements outside of testbed fixtures (e.g. snack-bars, overlays etc.).

Needed for angular#16697 and angular#16709.
devversion added a commit to devversion/material2 that referenced this pull request Sep 17, 2019
…s environment

Adds a new method that can be used by harness consumers to load a harness
through `HarnessLoader` from the document root. This is helpful for harnesses
which need to match elements outside of testbed fixtures (e.g. snack-bars, overlays etc.).

Needed for angular#16697 and angular#16709.
devversion added a commit to devversion/material2 that referenced this pull request Sep 17, 2019
…s environment

Adds a new method that can be used by harness consumers to load a harness
through `HarnessLoader` from the document root. This is helpful for harnesses
which need to match elements outside of testbed fixtures (e.g. snack-bars, overlays etc.).

Needed for angular#16697 and angular#16709.
jelbourn pushed a commit that referenced this pull request Sep 17, 2019
…s environment (#16903)

Adds a new method that can be used by harness consumers to load a harness
through `HarnessLoader` from the document root. This is helpful for harnesses
which need to match elements outside of testbed fixtures (e.g. snack-bars, overlays etc.).

Needed for #16697 and #16709.
@devversion devversion force-pushed the feat/test-harness-mdc-snack-bar branch from 3e9db90 to 3899191 Compare September 18, 2019 13:12
@devversion devversion closed this Sep 18, 2019
jelbourn pushed a commit to jelbourn/components that referenced this pull request Sep 24, 2019
…s environment (angular#16903)

Adds a new method that can be used by harness consumers to load a harness
through `HarnessLoader` from the document root. This is helpful for harnesses
which need to match elements outside of testbed fixtures (e.g. snack-bars, overlays etc.).

Needed for angular#16697 and angular#16709.

(cherry picked from commit 3765433)
@angular-automatic-lock-bot
Copy link

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-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Oct 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes PR author has agreed to Google's Contributor License Agreement target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants