Make the // skip-codegen property apply to ui tests too.

pnkfelix · pnkfelix · commit 1eb8690cf308 · 2018-09-21T16:21:40.000+02:00
diff --git a/src/tools/compiletest/src/runtest.rs b/src/tools/compiletest/src/runtest.rs
@@ -262,11 +262,12 @@ impl<'test> TestCx<'test> {
     }
 
     fn should_run_successfully(&self) -> bool {
-        match self.config.mode {
-            RunPass => !self.props.skip_codegen,
+        let run_pass = match self.config.mode {
+            RunPass => true,
             Ui => self.props.run_pass,
             _ => unimplemented!(),
-        }
+        };
+        return run_pass && !self.props.skip_codegen;
     }
 
     fn should_compile_successfully(&self) -> bool {

Original file line number	Diff line number	Diff line change
`@@ -262,11 +262,12 @@ impl<'test> TestCx<'test> {`
`262`	`262`	`}`
`263`	`263`
`264`	`264`	`fn should_run_successfully(&self) -> bool {`
`265`		`- match self.config.mode {`
`266`		`- RunPass => !self.props.skip_codegen,`
	`265`	`+ let run_pass = match self.config.mode {`
	`266`	`+ RunPass => true,`
`267`	`267`	`Ui => self.props.run_pass,`
`268`	`268`	`_ => unimplemented!(),`
`269`		`- }`
	`269`	`+ };`
	`270`	`+ return run_pass && !self.props.skip_codegen;`
`270`	`271`	`}`
`271`	`272`
`272`	`273`	`fn should_compile_successfully(&self) -> bool {`