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

Support multiple revisions in compiletest #66524

Merged
merged 4 commits into from
Nov 22, 2019

Commits on Nov 21, 2019

  1. Allow multiple cfgs per comment in "revisions:" tests

    The `//[X]~` syntax filters errors for tests that are run across
    multiple cfgs with  `// revisions:`. This commit extends that syntax to
    accept `//[X,Y]~`, which will match multiple cfgs to the same error
    annotation. This is functionally the same as writing two comments,
    `//[X]~` and `//[Y]~`, but can fit on a single line.
    ecstatic-morse committed Nov 21, 2019
    Configuration menu
    Copy the full SHA
    54d51bc View commit details
    Browse the repository at this point in the history
  2. Fix broken incremental test

    This test does not actually emit any warnings, since
    `#![allow(warnings)]` was specified. `compiletest` was erroneously
    ignoring `//~` tests and looking only for `//[X]~` ones. As a result of
    the changes in the previous commit, we now look for `//~` comments in
    incremental tests and expect them to appear in *all* revisions.
    ecstatic-morse committed Nov 21, 2019
    Configuration menu
    Copy the full SHA
    701f6e5 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    ae22938 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    c537f22 View commit details
    Browse the repository at this point in the history