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
Normative: Add missing RequireInternalSlot #1693
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1693 +/- ##
===========================================
- Coverage 95.03% 72.89% -22.14%
===========================================
Files 19 18 -1
Lines 10785 4840 -5945
Branches 1725 1058 -667
===========================================
- Hits 10249 3528 -6721
- Misses 523 1030 +507
- Partials 13 282 +269
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Sorry, I should mark this as draft, pending my action in #1692. |
We'll present this one at the December TC39 plenary. |
This achieved consensus at the December 2021 TC39 meeting. |
References: + tc39#1693
tc39/proposal-temporal#1693 added checks for the receiver of certain Temporal.Calendar and Temporal.TimeZone methods. Add branding tests for these methods, similar to the already existing branding tests. This was a normative change that achieved consensus at the December 2021 TC39 meeting.
A follow up to the previous commit, this rewrites some tests that relied on the lack of brand checks for certain Temporal.TimeZone methods. tc39/proposal-temporal#1693 added brand checks to these methods. We can no longer use a plain object time zone or even a Proxy with a real branded Temporal.TimeZone object as its handler to do these tests, so we instead create an instance of Temporal.TimeZone and define own accessor properties on it in order to test the observable property accesses that we need to see according to the spec text. This requires an improvement to TemporalHelpers.observeProperty() in order to be able to log property accesses to Symbol-valued properties.
tc39/proposal-temporal#1693 added checks for the receiver of certain Temporal.Calendar and Temporal.TimeZone methods. Add branding tests for these methods, similar to the already existing branding tests. This was a normative change that achieved consensus at the December 2021 TC39 meeting.
A follow up to the previous commit, this rewrites some tests that relied on the lack of brand checks for certain Temporal.TimeZone methods. tc39/proposal-temporal#1693 added brand checks to these methods. We can no longer use a plain object time zone or even a Proxy with a real branded Temporal.TimeZone object as its handler to do these tests, so we instead create an instance of Temporal.TimeZone and define own accessor properties on it in order to test the observable property accesses that we need to see according to the spec text. This requires an improvement to TemporalHelpers.observeProperty() in order to be able to log property accesses to Symbol-valued properties.
tc39/proposal-temporal#1693 added checks for the receiver of certain Temporal.Calendar and Temporal.TimeZone methods. Add branding tests for these methods, similar to the already existing branding tests. This was a normative change that achieved consensus at the December 2021 TC39 meeting.
A follow up to the previous commit, this rewrites some tests that relied on the lack of brand checks for certain Temporal.TimeZone methods. tc39/proposal-temporal#1693 added brand checks to these methods. We can no longer use a plain object time zone or even a Proxy with a real branded Temporal.TimeZone object as its handler to do these tests, so we instead create an instance of Temporal.TimeZone and define own accessor properties on it in order to test the observable property accesses that we need to see according to the spec text. This requires an improvement to TemporalHelpers.observeProperty() in order to be able to log property accesses to Symbol-valued properties.
References: + tc39#1693
References: + tc39#1693
Fix #1692
I assume these missing RequireInternalSlot is mistake.