Skip to content

Commit

Permalink
Run tests that requires more than 2 Gb of Memory only on 64-bit env (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
asmirnov82 authored Jul 7, 2023
1 parent 69eca56 commit d9e1ee1
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions test/Microsoft.Data.Analysis.Tests/BufferTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
using System.Linq;
using System.Text;
using Apache.Arrow;
using Microsoft.ML.TestFramework.Attributes;
using Xunit;

namespace Microsoft.Data.Analysis.Tests
Expand Down Expand Up @@ -188,16 +189,15 @@ public void TestArrowStringColumnClone()
Assert.Null(clone[i]);
}

/* Don't run tests during build as they fail, because build if build machine doesn't have enought memory
[Fact]
[X64Fact("32-bit dosn't allow to allocate more than 2 Gb")]
public void TestAppend_SizeMoreThanMaxBufferCapacity()
{
//Check appending value, than can increase buffer size over MaxCapacity (default strategy is to double buffer capacity)
PrimitiveDataFrameColumn<byte> intColumn = new PrimitiveDataFrameColumn<byte>("Byte1", int.MaxValue / 2 - 1);
intColumn.Append(10);
}

[Fact]
[X64Fact("32-bit dosn't allow to allocate more than 2 Gb")]
public void TestAppendMany_SizeMoreThanMaxBufferCapacity()
{
const int MaxCapacityInBytes = 2147483591;
Expand All @@ -208,7 +208,6 @@ public void TestAppendMany_SizeMoreThanMaxBufferCapacity()

Assert.Equal(MaxCapacityInBytes + 5, intColumn.Length);
}
*/

//#if !NETFRAMEWORK // https://github.com/dotnet/corefxlab/issues/2796
// [Fact]
Expand Down

0 comments on commit d9e1ee1

Please sign in to comment.