Some extended attributes are supported but have no test coverage #207
Comments
This comment has been minimized.
This comment has been minimized.
domenic
pushed a commit
that referenced
this issue
Apr 26, 2020
For [LegacyLenientThis], this fixes an issue where it would still try to get the impl value, even when the thisArg did not pass the is() check, resulting in a TypeError. This also adds test coverage for [LegacyLenientThis] and [Replaceable], as part of #207.
domenic
pushed a commit
that referenced
this issue
Apr 26, 2020
For [LegacyLenientThis], this fixes an issue where it would still try to get the impl value, even when the thisArg did not pass the is() check, resulting in a TypeError. This also adds test coverage for [LegacyLenientThis] and [Replaceable], as part of #207.
domenic
pushed a commit
that referenced
this issue
Apr 26, 2020
For [LegacyLenientThis], this fixes an issue where it would still try to get the impl value, even when the thisArg did not pass the is() check, resulting in a TypeError. This also adds test coverage for [LegacyLenientThis] and [Replaceable], as part of #207.
domenic
changed the title
|
I've renamed this issue and updated the OP to cover all untested extended attributes. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In particular:
[Clamp][EnforceRange][LegacyNoInterfaceObject][LegacyOverrideBuiltins][PutForwards]The text was updated successfully, but these errors were encountered: