Merge pull request #149 from rust-lang-nursery/mark-i-m-patch-1
clarify run-rustfix compiletest header
This commit is contained in:
commit
d3939fa152
|
|
@ -149,9 +149,14 @@ exhaustive. Header commands can generally be found by browsing the
|
||||||
source.
|
source.
|
||||||
|
|
||||||
* `run-rustfix` for UI tests, indicates that the test produces
|
* `run-rustfix` for UI tests, indicates that the test produces
|
||||||
structured suggestions, which are then applied and the final
|
structured suggestions. The test writer should create a `.fixed`
|
||||||
source is stored in a `.fixed` file and compiled again. The final
|
file, which contains the source with the suggestions applied.
|
||||||
compilation is required to succeed.
|
When the test is run, compiletest first checks that the correct
|
||||||
|
lint/warning is generated. Then, it applies the suggestion and
|
||||||
|
compares against `.fixed` (they must match). Finally, the fixed
|
||||||
|
source is compiled, and this compilation is required to succeed.
|
||||||
|
The `.fixed` file can also be generated automatically with the
|
||||||
|
`--bless` option, discussed [below](#bless).
|
||||||
* `min-{gdb,lldb}-version`
|
* `min-{gdb,lldb}-version`
|
||||||
* `min-llvm-version`
|
* `min-llvm-version`
|
||||||
* `compile-pass` for UI tests, indicates that the test is
|
* `compile-pass` for UI tests, indicates that the test is
|
||||||
|
|
@ -267,6 +272,8 @@ you can even run the resulting program. Just add one of the following
|
||||||
- `// run-pass` -- compilation should succeed and we should run the
|
- `// run-pass` -- compilation should succeed and we should run the
|
||||||
resulting binary
|
resulting binary
|
||||||
|
|
||||||
|
<a name="bless"></a>
|
||||||
|
|
||||||
### Editing and updating the reference files
|
### Editing and updating the reference files
|
||||||
|
|
||||||
If you have changed the compiler's output intentionally, or you are
|
If you have changed the compiler's output intentionally, or you are
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue