-
Notifications
You must be signed in to change notification settings - Fork 462
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
RAB: Integrate staging tests for the .reduce method #4156
Conversation
…des, while adjusting usage of CollectValuesAndResize and applying review changes from PRs for previously tested methods.
ba5120c
to
2023900
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some of the files seem not quite right here.
test/built-ins/Array/prototype/reduceRight/resizable-buffer-grow-mid-iteration.js
Outdated
Show resolved
Hide resolved
test/built-ins/TypedArray/prototype/reduce/resizable-buffer-grow-mid-iteration.js
Show resolved
Hide resolved
test/built-ins/TypedArray/prototype/reduce/resizable-buffer-shrink-mid-iteration.js
Show resolved
Hide resolved
test/built-ins/TypedArray/prototype/reduceRight/resizable-buffer-grow-mid-iteration.js
Outdated
Show resolved
Hide resolved
test/built-ins/TypedArray/prototype/reduceRight/resizable-buffer-shrink-mid-iteration.js
Outdated
Show resolved
Hide resolved
test/built-ins/TypedArray/prototype/reduceRight/resizable-buffer.js
Outdated
Show resolved
Hide resolved
@ptomato oh oops! Thank you! |
Addresses review comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@ioannad Could you rebase this please? I think the removals of the reduceRight files are causing merge conflicts, we want to make sure they stay in-tree when it's merged. |
of Array.prototype and TypedArray.prototype
This is part of PR #3888 to make reviewing easier. Includes changes to use the helper ./harness/resizableArrayBufferUtils.js
This should be very similar to #4157 for
.reduceRight
.