Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

test(engine-server): test scoped slot forwarding #4669

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

nolanlawson
Copy link
Contributor

Details

This is merely copied from the Karma tests.

Unlike normal light DOM slots, which throw an error (#4668), scoped light DOM slots do not throw an error when forwarded. So we can just add a test here and increase our test coverage in engine-server.

Does this pull request introduce a breaking change?

  • 😮‍💨 No, it does not introduce a breaking change.

Does this pull request introduce an observable change?

  • 🤞 No, it does not introduce an observable change.

@nolanlawson nolanlawson requested a review from a team as a code owner October 18, 2024 21:51
@nolanlawson nolanlawson merged commit 5391bc3 into master Oct 21, 2024
11 checks passed
@nolanlawson nolanlawson deleted the nolan/scoped-slots-slot-forwarding branch October 21, 2024 15:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants