Skip to content
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

Bump test262 from 0bccacd to 99b2a70 #1982

Closed
wants to merge 1 commit into from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Mar 28, 2022

Bumps test262 from 0bccacd to 99b2a70.

Commits
  • 99b2a70 Use ECMAScript version 11 (#3448)
  • f964584 Expand overflow-invalid-string.js tests.
  • 7b78d4b Port some basic Temporal.PlainDateTime tests from Demitasse to test262 (#3430)
  • f71d5e2 Add test for throwing WrappedFunctionCreate
  • 81895b1 use all values max and min fraction and significant digits
  • 7d5b0ec Add test for Temporal.Instant.p*.add
  • 11c2559 Add tests for Temporal.Instant.p*.subtract
  • See full diff in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [test262](https://github.com/tc39/test262) from `0bccacd` to `99b2a70`.
- [Release notes](https://github.com/tc39/test262/releases)
- [Commits](tc39/test262@0bccacd...99b2a70)

---
updated-dependencies:
- dependency-name: test262
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file submodules Pull requests that update Submodules code labels Mar 28, 2022
@codecov
Copy link

codecov bot commented Mar 28, 2022

Codecov Report

Merging #1982 (5d0a4c1) into main (5d2420e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1982   +/-   ##
=======================================
  Coverage   45.93%   45.93%           
=======================================
  Files         206      206           
  Lines       17138    17138           
=======================================
  Hits         7873     7873           
  Misses       9265     9265           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d2420e...5d0a4c1. Read the comment docs.

@Razican Razican added the blocked Waiting for another code change label Mar 28, 2022
@Razican
Copy link
Member

Razican commented Mar 28, 2022

This is blocked on tc39/test262#3451, and waiting for tc39/test262#3452 to be merged.

bors bot pushed a commit that referenced this pull request Mar 29, 2022
Trying to fix the issue in #1982, I noticed that we didn't have a proper error handling for the boa tester.

This adds the `anyhow` dependency to the tester, which makes it much easier to handle errors and bubble them up with attached context. Thanks to this I was able to easily find out the issue, and I think it could be useful to have it. It gives errors such as this one:

```
Error: could not read the suite test
    caused by: error reading sub-suite ./test262/test/built-ins
    caused by: error reading sub-suite ./test262/test/built-ins/ShadowRealm
    caused by: error reading sub-suite ./test262/test/built-ins/ShadowRealm/WrappedFunction
    caused by: error reading test ./test262/test/built-ins/ShadowRealm/WrappedFunction/throws-typeerror-on-revoked-proxy.js
    caused by: while scanning a block scalar, found a tab character where an indentation space is expected at line 4 column 3
    caused by: while scanning a block scalar, found a tab character where an indentation space is expected at line 4 column 3
```
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Apr 4, 2022

Superseded by #2007.

@dependabot dependabot bot closed this Apr 4, 2022
@dependabot dependabot bot deleted the dependabot/submodules/test262-99b2a70 branch April 4, 2022 08:22
Razican added a commit that referenced this pull request Jun 8, 2022
Trying to fix the issue in #1982, I noticed that we didn't have a proper error handling for the boa tester.

This adds the `anyhow` dependency to the tester, which makes it much easier to handle errors and bubble them up with attached context. Thanks to this I was able to easily find out the issue, and I think it could be useful to have it. It gives errors such as this one:

```
Error: could not read the suite test
    caused by: error reading sub-suite ./test262/test/built-ins
    caused by: error reading sub-suite ./test262/test/built-ins/ShadowRealm
    caused by: error reading sub-suite ./test262/test/built-ins/ShadowRealm/WrappedFunction
    caused by: error reading test ./test262/test/built-ins/ShadowRealm/WrappedFunction/throws-typeerror-on-revoked-proxy.js
    caused by: while scanning a block scalar, found a tab character where an indentation space is expected at line 4 column 3
    caused by: while scanning a block scalar, found a tab character where an indentation space is expected at line 4 column 3
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Waiting for another code change dependencies Pull requests that update a dependency file submodules Pull requests that update Submodules code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant