Skip to content

Commit 9512dbb

Browse files
authored
Merge pull request dotnet/corefx#15050 from tarekgh/FixNumericTests
Fix System.Runtime.Numerics Tests to run against desktop Commit migrated from dotnet/corefx@b434d42
2 parents 4102cf8 + 1ad4ece commit 9512dbb

File tree

6 files changed

+48
-23
lines changed

6 files changed

+48
-23
lines changed

src/libraries/System.Runtime.Numerics/tests/BigInteger/Comparison.cs

+16-3
Original file line numberDiff line numberDiff line change
@@ -359,12 +359,25 @@ private static void RunNegativeTests(Random random)
359359
Assert.Equal(false, BigInteger.Zero.Equals((Object)"0"));
360360
}
361361

362-
[Fact]
363-
public static void IComparable_Invalid()
362+
public static void IComparable_Invalid(string paramName)
364363
{
365364
IComparable comparable = new BigInteger();
366365
Assert.Equal(1, comparable.CompareTo(null));
367-
Assert.Throws<ArgumentException>("obj", () => comparable.CompareTo(0)); // Obj is not a BigInteger
366+
Assert.Throws<ArgumentException>(paramName, () => comparable.CompareTo(0)); // Obj is not a BigInteger
367+
}
368+
369+
[Fact]
370+
[SkipOnTargetFramework(TargetFrameworkMonikers.Netcoreapp | TargetFrameworkMonikers.NetcoreUwp)]
371+
public static void IComparable_Invalid_net46()
372+
{
373+
IComparable_Invalid(null);
374+
}
375+
376+
[Fact]
377+
[SkipOnTargetFramework(TargetFrameworkMonikers.NetFramework)]
378+
public static void IComparable_Invalid_netcore()
379+
{
380+
IComparable_Invalid("obj");
368381
}
369382

370383
private static void VerifyComparison(BigInteger x, BigInteger y, int expectedResult)

src/libraries/System.Runtime.Numerics/tests/BigInteger/IsEven.cs

+1
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ public class IsEvenTest
1313
private static int s_seed = 0;
1414

1515
[Fact]
16+
[SkipOnTargetFramework(TargetFrameworkMonikers.NetFramework)]
1617
public static void RunIsEvenTests()
1718
{
1819
Random random = new Random(s_seed);

src/libraries/System.Runtime.Numerics/tests/BigInteger/modpow.cs

+2
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,7 @@ public static void ModPow1Large2SmallInt()
140140
}
141141

142142
[Fact]
143+
[SkipOnTargetFramework(TargetFrameworkMonikers.NetFramework)]
143144
public static void ModPow1Large2SmallInt_Threshold()
144145
{
145146
// Again, with lower threshold
@@ -164,6 +165,7 @@ public static void ModPow2Large1SmallInt()
164165
}
165166

166167
[Fact]
168+
[SkipOnTargetFramework(TargetFrameworkMonikers.NetFramework)]
167169
public static void ModPow2Large1SmallInt_Threshold()
168170
{
169171
// Again, with lower threshold

src/libraries/System.Runtime.Numerics/tests/BigInteger/multiply.cs

+1
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ public static void RunMultiply_TwoLargeBigIntegers()
3535
}
3636

3737
[Fact]
38+
[SkipOnTargetFramework(TargetFrameworkMonikers.NetFramework)]
3839
public static void RunMultiply_TwoLargeBigIntegers_Threshold()
3940
{
4041
// Again, with lower threshold

src/libraries/System.Runtime.Numerics/tests/BigInteger/pow.cs

+1
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,7 @@ public static void RunPowNegative()
169169
}
170170

171171
[Fact]
172+
[SkipOnTargetFramework(TargetFrameworkMonikers.NetFramework)]
172173
public static void RunOverflow()
173174
{
174175
var bytes = new byte[1000];

src/libraries/System.Runtime.Numerics/tests/ComplexTests.cs

+27-20
Original file line numberDiff line numberDiff line change
@@ -292,6 +292,7 @@ public static IEnumerable<object[]> ACos_Advanced_TestData()
292292
}
293293

294294
[Theory, MemberData("ACos_Advanced_TestData")]
295+
[SkipOnTargetFramework(TargetFrameworkMonikers.NetFramework)]
295296
public static void ACos_Advanced(double real, double imaginary, double expectedReal, double expectedImaginary)
296297
{
297298
var complex = new Complex(real, imaginary);
@@ -373,6 +374,7 @@ public static IEnumerable<object[]> ASin_Advanced_TestData()
373374
}
374375

375376
[Theory, MemberData("ASin_Advanced_TestData")]
377+
[SkipOnTargetFramework(TargetFrameworkMonikers.NetFramework)]
376378
public static void ASin_Advanced(double real, double imaginary, double expectedReal, double expectedImaginary)
377379
{
378380
var complex = new Complex(real, imaginary);
@@ -596,6 +598,31 @@ public static void Divide(double realLeft, double imaginaryLeft, double realRigh
596598
VerifyRealImaginaryProperties(result, expectedReal, expectedImaginary);
597599
}
598600

601+
[Fact]
602+
[SkipOnTargetFramework(TargetFrameworkMonikers.NetFramework)]
603+
public static void Equals_netcore()
604+
{
605+
// Invalid values
606+
Complex invalidComplex;
607+
var complex = new Complex(2, 3);
608+
foreach (double invalidReal in s_invalidDoubleValues)
609+
{
610+
invalidComplex = new Complex(invalidReal, 1);
611+
Equals(invalidComplex, complex, false, false);
612+
Equals(invalidComplex, invalidComplex, !double.IsNaN(invalidReal), true); // Handle double.NaN != double.NaN
613+
foreach (double invalidImaginary in s_invalidDoubleValues)
614+
{
615+
invalidComplex = new Complex(1, invalidImaginary);
616+
Equals(invalidComplex, complex, false, false);
617+
Equals(invalidComplex, invalidComplex, !double.IsNaN(invalidImaginary), true); // Handle double.NaN != double.NaN
618+
619+
invalidComplex = new Complex(invalidReal, invalidImaginary);
620+
Equals(invalidComplex, complex, false, false);
621+
Equals(invalidComplex, invalidComplex, !double.IsNaN(invalidReal) && !double.IsNaN(invalidImaginary), true); // Handle double.NaN != double.NaN
622+
}
623+
}
624+
}
625+
599626
[Fact]
600627
public static void Equals()
601628
{
@@ -676,26 +703,6 @@ public static void Equals()
676703
Equals(new Complex(0, 100.5), 0, false, false);
677704
Equals(new Complex(0, 100.5), "0", false, false);
678705
Equals(new Complex(0, 100.5), null, false, false);
679-
680-
// Invalid values
681-
Complex invalidComplex;
682-
var complex = new Complex(2, 3);
683-
foreach (double invalidReal in s_invalidDoubleValues)
684-
{
685-
invalidComplex = new Complex(invalidReal, 1);
686-
Equals(invalidComplex, complex, false, false);
687-
Equals(invalidComplex, invalidComplex, !double.IsNaN(invalidReal), true); // Handle double.NaN != double.NaN
688-
foreach (double invalidImaginary in s_invalidDoubleValues)
689-
{
690-
invalidComplex = new Complex(1, invalidImaginary);
691-
Equals(invalidComplex, complex, false, false);
692-
Equals(invalidComplex, invalidComplex, !double.IsNaN(invalidImaginary), true); // Handle double.NaN != double.NaN
693-
694-
invalidComplex = new Complex(invalidReal, invalidImaginary);
695-
Equals(invalidComplex, complex, false, false);
696-
Equals(invalidComplex, invalidComplex, !double.IsNaN(invalidReal) && !double.IsNaN(invalidImaginary), true); // Handle double.NaN != double.NaN
697-
}
698-
}
699706
}
700707

701708
private static void Equals(Complex complex1, object obj, bool expected, bool expectedEquals)

0 commit comments

Comments
 (0)