-
-
Notifications
You must be signed in to change notification settings - Fork 402
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
Add test comparison to Test262 result compare #1279
Conversation
I will be testing if this actually works now with a couple of commits making things fail/panic. |
Benchmark for 2a85a3cClick to view benchmark
|
Benchmark for f58d835Click to view benchmark
|
Benchmark for f41cc14Click to view benchmark
|
Benchmark for 328153fClick to view benchmark
|
Test262 conformance changes:
|
Benchmark for fb8db27Click to view benchmark
|
Example output:
Broken tests:
New panics:
|
Benchmark for 08cc474Click to view benchmark
|
Benchmark for e092ba4Click to view benchmark
|
Benchmark for 5b08782Click to view benchmark
|
Benchmark for d6c5a64Click to view benchmark
|
Can you put those tests lists inside a |
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.
@0x7D2B's idea sounds like a nice quality-of-life improvement, other than that it looks great.
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.
I think test names should be wrapped in <details>
tags like the benchmarks. Also I wanted to try leaving a review~
Should be ready now :) |
Benchmark for 806064dClick to view benchmark
|
Benchmark for fe18fd7Click to view benchmark
|
This adds some result comparison to Test262 results. It will add 4 lists (if not empty) to conformance results:
Note that this will only check for tests that already existed, so if we update the Test262 submodule, new tests will not appear in this list.