diff options
| author | Alexander Köplinger <alex.koeplinger@outlook.com> | 2014-02-23 20:54:54 +0400 |
|---|---|---|
| committer | Alexander Köplinger <alex.koeplinger@outlook.com> | 2014-02-23 22:44:21 +0400 |
| commit | b0032294da2ddfc86f2ba04fd099331761737cad (patch) | |
| tree | 67533d56e57bdca965f7f15b3ff237ecaa8bb76f /mcs/class/Microsoft.Build/Test | |
| parent | 06502ec1e902d63bc61df3d0ff8f4b27059cb90f (diff) | |
[Test] Cleaned up how a bunch of tests were ignored
Previously, many tests just had a return statement when the test shouldn't
be executed (e.g. Unix-only). This made it look like the test was successful
when in fact it wasn't executed at all.
Assert.Ignore() is now used instead to correctly exclude those tests.
Diffstat (limited to 'mcs/class/Microsoft.Build/Test')
| -rw-r--r-- | mcs/class/Microsoft.Build/Test/Microsoft.Build.Execution/BuildManagerTest.cs | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/mcs/class/Microsoft.Build/Test/Microsoft.Build.Execution/BuildManagerTest.cs b/mcs/class/Microsoft.Build/Test/Microsoft.Build.Execution/BuildManagerTest.cs index 0fcbdf7560c..5b0c778cd8e 100644 --- a/mcs/class/Microsoft.Build/Test/Microsoft.Build.Execution/BuildManagerTest.cs +++ b/mcs/class/Microsoft.Build/Test/Microsoft.Build.Execution/BuildManagerTest.cs @@ -140,7 +140,8 @@ namespace MonoTests.Microsoft.Build.Execution case PlatformID.Unix: break; default: - return; // ignore, cannot run it + Assert.Ignore (); // ignore, cannot run it + break; } var xml = XmlReader.Create (new StringReader (project_xml)); |