Skip to content

Run local tests with run-test262#591

Merged
bnoordhuis merged 3 commits intoquickjs-ng:masterfrom
bnoordhuis:local-tests
Oct 10, 2024
Merged

Run local tests with run-test262#591
bnoordhuis merged 3 commits intoquickjs-ng:masterfrom
bnoordhuis:local-tests

Conversation

@bnoordhuis
Copy link
Copy Markdown
Contributor

First commit is #579.

@saghul
Copy link
Copy Markdown
Contributor

saghul commented Oct 10, 2024

Love this!

Copy link
Copy Markdown
Contributor

@saghul saghul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! How do you want to proceed? Land this and fix run-262 later so we can run the actual suite on Windows or the other way around?

@bnoordhuis
Copy link
Copy Markdown
Contributor Author

I'll go ahead and land this because this whole exercise ultimately is so I can add an expect-fail test for #567 :-)

The ulterior motive here is not that I want to increase CI times
further but that I want to repurpose run-test262 for running our
own tests.
We always run the test262 suite in new-style mode.
@bnoordhuis bnoordhuis merged commit b9be6d4 into quickjs-ng:master Oct 10, 2024
@bnoordhuis bnoordhuis deleted the local-tests branch October 10, 2024 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants