Skip to content

Commit 259960e

Browse files
committed
test: fix [Ignore] → [Skip] for TUnit compatibility
MSTest's [Ignore] attribute was not migrated to TUnit's [Skip] for StringArraySample1 test. TUnit doesn't recognize [Ignore], causing the test to run instead of being skipped. Fixes CI test execution where this test was unexpectedly running.
1 parent a5aedcb commit 259960e

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

test/NumSharp.UnitTest/Random/np.random.choice.Test.cs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
using Microsoft.VisualStudio.TestTools.UnitTesting;
2+
using NumSharp.UnitTest;
23
using System;
34
using System.Collections.Generic;
45
using System.Diagnostics;
@@ -74,15 +75,15 @@ public void NonUniformSample()
7475
}
7576

7677
[Test]
77-
[Ignore("Choice without replacement not implemented yet")]
78+
[OpenBugs] // Choice without replacement not implemented yet
7879
public void UniformSampleWithoutReplace()
7980
{
8081
NDArray actual = np.random.choice(5, (Shape)3, replace: false);
8182
Assert.Fail("Not implemented");
8283
}
8384

8485
[Test]
85-
[Ignore("Choice without replacement not implemented yet")]
86+
[OpenBugs] // Choice without replacement not implemented yet
8687
public void NonUniformSampleWithoutReplace()
8788
{
8889
double[] probabilities = new double[] {0.1, 0, 0.3, 0.6, 0};
@@ -91,7 +92,7 @@ public void NonUniformSampleWithoutReplace()
9192
}
9293

9394
[Test]
94-
[Ignore("Choice with string arrays not implemented yet")]
95+
[Skip("Choice with string arrays not implemented yet")]
9596
public void StringArraySample1()
9697
{
9798
//int nrSamples = 5;

0 commit comments

Comments
 (0)