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

Create tooling for end-to-end testing #25

Merged
merged 5 commits into from
Oct 3, 2023

Conversation

celinval
Copy link
Contributor

@celinval celinval commented Sep 1, 2023

Create two different tools:

  • test-drive: A rustc_driver that compiles a crate and run a few sanity checks on StableMIR.
  • compiletest: A wrapper to run compiler tests using the test-drive tool.

I am also adding a script to run a few rustc tests and a nightly workflow. The files diff is not quite working yet so most tests that fail compilation don't succeed yet.

For our local tests, so far I created two suites (one with a test each): sanity-checks and fixme suite. The fixme suite is for issues that we find that needs fixing. I found an issue on the Trait API. I haven't debugged it yet.

Create two different tools:

  - `test-drive`: A rustc_driver that compiles a crate and run a few
  sanity checks on StableMIR.
  - `compiletest`: A wrapper to run compiler tests using the
    `test-drive` tool.

I am also adding a script to run a few rustc tests and a nightly
workflow. The files diff is not quite working yet so most tests
that fail compilation don't succeed yet.
@celinval celinval mentioned this pull request Sep 1, 2023
@celinval celinval marked this pull request as ready for review September 26, 2023 00:38
@celinval
Copy link
Contributor Author

@oli-obk this is ready for review

I had forgotten to enable the smir checks before. Unfortunately,
enabling them triggered a few issues.

For now, I set the rustc suite to not fail the CI and created issue
rust-lang#39 to track
the work needed to enable it.
@oli-obk
Copy link
Contributor

oli-obk commented Sep 27, 2023

hmm... do we need bors for merging now?

@bors r+

@oli-obk
Copy link
Contributor

oli-obk commented Sep 27, 2023

Nope... 😆 uh what now. I can't merge PRs anymore
image

@oli-obk
Copy link
Contributor

oli-obk commented Sep 27, 2023

Maybe after https://github.com/rust-lang/project-stable-mir/pull/19/files we need to do something to the teams repo to give us permissions again? 😕

edit: afaict we do have the right permissions...

@celinval
Copy link
Contributor Author

Nope... 😆 uh what now. I can't merge PRs anymore image

Oh no! 🤦‍♀️

@celinval celinval merged commit f0f892f into rust-lang:main Oct 3, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants