Skip to content

Excluding "immutable" from TA tests with mutable expectations.#5063

Open
iamstolis wants to merge 1 commit into
tc39:mainfrom
iamstolis:exclude-immutable
Open

Excluding "immutable" from TA tests with mutable expectations.#5063
iamstolis wants to merge 1 commit into
tc39:mainfrom
iamstolis:exclude-immutable

Conversation

@iamstolis
Copy link
Copy Markdown
Contributor

Disables the usage of immutable ArrayBuffers in TypedArray-related tests that have "mutable" expectations:

  • TypedArray/prototype/slice/speciesctor-return-same-buffer-with-offset.js: expects slice to copy into a species-created result backed by the same buffer. With an immutable buffer, the species-created result is not a valid write target, so slice throws instead of producing the mutable-copy result.

  • TypedArrayConstructors/internals/GetOwnProperty/index-prop-desc.js, TypedArrayConstructors/internals/GetOwnProperty/BigInt/index-prop-desc.js: expect indexed property descriptors to be writable and configurable. For immutable-backed TypedArrays, indexed properties are not writable/configurable.

  • TypedArrayConstructors/internals/Set/bigint-tonumber.js,
    TypedArrayConstructors/internals/Set/BigInt/null-tobigint.js,
    TypedArrayConstructors/internals/Set/BigInt/number-tobigint.js,
    TypedArrayConstructors/internals/Set/BigInt/string-nan-tobigint.js, TypedArrayConstructors/internals/Set/BigInt/symbol-tobigint.js,
    TypedArrayConstructors/internals/Set/BigInt/tonumber-value-throws.js,
    TypedArrayConstructors/internals/Set/BigInt/undefined-tobigint.js,
    TypedArrayConstructors/internals/Set/tonumber-value-throws.js: validate value-conversion abrupt completions during indexed assignment, such as ToNumber, ToBigInt, or user-defined coercion hooks throwing. For immutable-backed TypedArrays, indexed [[Set]] short-circuits before value conversion, so those mutable-buffer conversion expectations do not apply.

@iamstolis iamstolis requested a review from a team as a code owner May 26, 2026 19:57
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.

2 participants