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
Object.fromEntries: use verifyProperty; add specification details #1677
Conversation
|
||
/*--- | ||
esid: sec-object.fromentries | ||
description: Closes iterators when toString on a value throws. |
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.
This test isn't valid; only the keys get toString'd (or, to be precise, toPropertyKey'd). The values are passed through without being touched.
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.
Whoops, yes, that would make sense. I will drop this.
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 for the followup; sorry for my typos.
@@ -19,7 +36,7 @@ var iterable = { | |||
advanced = true; | |||
return { | |||
done: false, | |||
value: 'null', | |||
value: null, |
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.
👍
@@ -0,0 +1,64 @@ | |||
// Copyright (C) 2018 Kevin Gibbons. All rights reserved. |
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.
Presumably this (and other new tests) should be (C) someone else?
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.
Well, I didn't really write a majority of the test, it's your work with only a slight modification
203d874
to
ff475fc
Compare
You're very welcome and don't worry about the typos :) |
https://tc39.github.io/proposal-object-from-entries/