-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Replace legacy describe
tests with one of describes.{realWin|sandboxed|fakeWin|integration}
#24144
Comments
Bumping this issue because we still have ~150 legacy amphtml/build-system/tasks/presubmit-checks.js Lines 756 to 894 in 5d94316
|
[Infra triage] Putting this on the radar of @rileyajones for comment / prioritization. |
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This is part of a larger effort to isolate AMP's tests and prevent global state from leaking across tests. See #24090 (comment) for the discussion that led to this issue being filed.
Related to #24090
The text was updated successfully, but these errors were encountered: