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

Add more TrustedType tests for eval and function constructor. #49997

Merged
merged 2 commits into from
Jan 9, 2025

Conversation

fred-wang
Copy link
Contributor

We already have many tests for eval and function constructor but things are a bit messy and it's difficult to figure out what we are covering. Write a simple test to test blocking, compilationSink and execution in workers.

We already have many tests for eval and function constructor but things
are a bit messy and it's difficult to figure out what we are covering.
Write a simple test to test blocking, compilationSink and execution
in workers.
Copy link
Member

@lukewarlow lukewarlow left a comment

Choose a reason for hiding this comment

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

Might be worth also testing indirect eval eval?.()

@fred-wang
Copy link
Contributor Author

Might be worth also testing indirect eval eval?.()

Good point. I also added tests for other function constructor, for completeness

@fred-wang fred-wang requested a review from lukewarlow January 9, 2025 15:29
@fred-wang fred-wang merged commit 2585a7b into master Jan 9, 2025
19 checks passed
@fred-wang fred-wang deleted the block-eval-function-constructor branch January 9, 2025 16:10
sadym-chromium pushed a commit that referenced this pull request Jan 14, 2025
* Add more TrustedType tests for eval and function constructor.

We already have many tests for eval and function constructor but things
are a bit messy and it's difficult to figure out what we are covering.
Write a simple test to test blocking, compilationSink and execution
in workers.

* also test indirect eval and more function constructor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants