Browse Source

Merge pull request #2852 from ElektroKill/enable-passing-ignored-tests

Enable tests which no longer fail
pull/2854/head
Siegfried Pammer 3 years ago committed by GitHub
parent
commit
efa71dd065
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 8
      ICSharpCode.Decompiler.Tests/PrettyTestRunner.cs
  2. 6
      ICSharpCode.Decompiler.Tests/Semantics/OverloadResolutionTests.cs

8
ICSharpCode.Decompiler.Tests/PrettyTestRunner.cs

@ -1,4 +1,4 @@
// Copyright (c) AlphaSierraPapa for the SharpDevelop Team // Copyright (c) AlphaSierraPapa for the SharpDevelop Team
// //
// Permission is hereby granted, free of charge, to any person obtaining a copy of this // Permission is hereby granted, free of charge, to any person obtaining a copy of this
// software and associated documentation files (the "Software"), to deal in the Software // software and associated documentation files (the "Software"), to deal in the Software
@ -560,12 +560,6 @@ namespace ICSharpCode.Decompiler.Tests
[Test] [Test]
public async Task TupleTests([ValueSource(nameof(roslyn2OrNewerOptions))] CompilerOptions cscOptions) public async Task TupleTests([ValueSource(nameof(roslyn2OrNewerOptions))] CompilerOptions cscOptions)
{ {
if (cscOptions.HasFlag(CompilerOptions.UseRoslynLatest))
{
Assert.Ignore("DefaultInterpolatedStringHandler is not yet supported!");
return;
}
await RunForLibrary(cscOptions: cscOptions); await RunForLibrary(cscOptions: cscOptions);
} }

6
ICSharpCode.Decompiler.Tests/Semantics/OverloadResolutionTests.cs

@ -1,4 +1,4 @@
// Copyright (c) 2010-2013 AlphaSierraPapa for the SharpDevelop Team // Copyright (c) 2010-2013 AlphaSierraPapa for the SharpDevelop Team
// //
// Permission is hereby granted, free of charge, to any person obtaining a copy of this // Permission is hereby granted, free of charge, to any person obtaining a copy of this
// software and associated documentation files (the "Software"), to deal in the Software // software and associated documentation files (the "Software"), to deal in the Software
@ -104,7 +104,7 @@ namespace ICSharpCode.Decompiler.Tests.Semantics
Assert.AreSame(c1, r.BestCandidate); Assert.AreSame(c1, r.BestCandidate);
} }
[Test, Ignore("Broken after migration to ICS.Decompiler")] [Test]
public void NullableIntAndNullableUIntIsAmbiguous() public void NullableIntAndNullableUIntIsAmbiguous()
{ {
OverloadResolution r = new OverloadResolution(compilation, MakeArgumentList(typeof(ushort?))); OverloadResolution r = new OverloadResolution(compilation, MakeArgumentList(typeof(ushort?)));
@ -300,7 +300,7 @@ namespace ICSharpCode.Decompiler.Tests.Semantics
Assert.AreEqual(OverloadResolutionErrors.None, r.BestCandidateErrors); Assert.AreEqual(OverloadResolutionErrors.None, r.BestCandidateErrors);
} }
[Test, Ignore("Broken on SRM branch???")] [Test]
public void Lambda_DelegateAndExpressionTreeOverloadsAreAmbiguous() public void Lambda_DelegateAndExpressionTreeOverloadsAreAmbiguous()
{ {
var m1 = MakeMethod(typeof(Func<int>)); var m1 = MakeMethod(typeof(Func<int>));

Loading…
Cancel
Save