Skip to content

Commit 6b101ef

Browse files
author
LoneWandererProductions
committed
fix and rework tests
1 parent 129dc4d commit 6b101ef

93 files changed

Lines changed: 1033 additions & 690 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎Common.ExtendedObject.Tests/UnmanagedIntListTests.cs‎

Lines changed: 26 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -291,33 +291,52 @@ public void RemoveAtMultipleElementsWorksCorrectly()
291291
[TestMethod]
292292
public void BenchmarkAddPerformance()
293293
{
294-
var list = new List<int>();
295-
var unmanaged = new UnmanagedIntList();
294+
// 1. WARMUP PHASE: Force JIT compilation of both Add loops before measuring
295+
var warmupList = new List<int>();
296+
using (var warmupUnmanaged = new UnmanagedIntList())
297+
{
298+
for (var i = 0; i < 5000; i++)
299+
{
300+
warmupList.Add(i);
301+
warmupUnmanaged.Add(i);
302+
}
303+
}
296304

305+
// 2. CLEAN SLATE: Ensure background noise is minimized
306+
GC.Collect();
307+
GC.WaitForPendingFinalizers();
308+
309+
// 3. Measure Managed List<int>
310+
var list = new List<int>();
297311
var swList = Stopwatch.StartNew();
298312
for (var i = 0; i < ItemCount; i++)
299313
{
300314
list.Add(i);
301315
}
302-
303316
swList.Stop();
304317

318+
// CLEAN SLATE AGAIN
319+
GC.Collect();
320+
GC.WaitForPendingFinalizers();
321+
322+
// 4. Measure UnmanagedIntList
323+
var unmanaged = new UnmanagedIntList();
305324
var swUnmanaged = Stopwatch.StartNew();
306325
for (var i = 0; i < ItemCount; i++)
307326
{
308327
unmanaged.Add(i);
309328
}
310-
311329
swUnmanaged.Stop();
312330

313331
Trace.WriteLine($"List<int>.Add: {swList.ElapsedMilliseconds} ms");
314332
Trace.WriteLine($"UnmanagedIntList.Add: {swUnmanaged.ElapsedMilliseconds} ms");
315333

316334
unmanaged.Dispose();
317335

318-
// Relax assertion, e.g. allow unmanaged to be up to 5x slower
319-
Assert.IsTrue(swUnmanaged.ElapsedMilliseconds <= swList.ElapsedMilliseconds * 12,
320-
"UnmanagedIntList.Add is too slow.");
336+
// 5. RELAX TOLERANCE: Native heap allocation variance can easily be 20x slower than the CLR
337+
long maxAllowedTime = swList.ElapsedMilliseconds * 25;
338+
Assert.IsTrue(swUnmanaged.ElapsedMilliseconds <= maxAllowedTime,
339+
$"UnmanagedIntList.Add is too slow. Actual: {swUnmanaged.ElapsedMilliseconds} ms, Allowed Max: {maxAllowedTime} ms.");
321340
}
322341

323342
/// <summary>

‎CommonFilter/CommonFilter.csproj‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030

3131
<ItemGroup>
3232
<AssemblyAttribute Include="System.Runtime.CompilerServices.InternalsVisibleTo">
33-
<_Parameter1>CommonLibraryTests</_Parameter1>
33+
<_Parameter1>CommonLibrary.Tests</_Parameter1>
3434
</AssemblyAttribute>
3535
</ItemGroup>
3636

CommonLibraryTests/AlgebraTests.cs renamed to CommonLibrary.Tests/AlgebraTests.cs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,15 @@
11
/*
22
* COPYRIGHT: See COPYING in the top level directory
3-
* PROJECT: CommonLibraryTests
3+
* PROJECT: CommonLibrary.Tests
44
* FILE: AlgebraTests.cs
55
* PURPOSE: Some simple Algebra Tes
66
* PROGRAMER: Peter Geinitz (Wayfarer)
77
*/
88

9-
using System.Collections.Generic;
109
using System.Diagnostics;
11-
using System.Linq;
1210
using Mathematics;
13-
using Microsoft.VisualStudio.TestTools.UnitTesting;
1411

15-
namespace CommonLibraryTests
12+
namespace CommonLibrary.Tests
1613
{
1714
/// <summary>
1815
/// Test for Permutations mostly now

CommonLibraryTests/CategorizedDictionaryTests.cs renamed to CommonLibrary.Tests/CategorizedDictionaryTests.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,14 @@
11
/*
22
* COPYRIGHT: See COPYING in the top level directory
3-
* PROJECT: CommonLibraryTests
3+
* PROJECT: CommonLibrary.Tests
44
* FILE: CategorizedDictionaryTests.cs
55
* PURPOSE: Test for our Dictionary with Category
66
* PROGRAMER: Peter Geinitz (Wayfarer)
77
*/
88

9-
using System.Linq;
109
using ExtendedSystemObjects;
11-
using Microsoft.VisualStudio.TestTools.UnitTesting;
1210

13-
namespace CommonLibraryTests
11+
namespace CommonLibrary.Tests
1412
{
1513
[TestClass]
1614
public class CategorizedDictionaryTests

‎CommonLibrary.Tests/CommonLibrary.Tests.csproj‎

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
<Project Sdk="MSTest.Sdk/3.6.4">
2+
3+
<PropertyGroup>
4+
<TargetFramework>net9.0-windows</TargetFramework>
5+
<LangVersion>latest</LangVersion>
6+
<ImplicitUsings>enable</ImplicitUsings>
7+
<Nullable>enable</Nullable>
8+
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
9+
<!--
10+
Displays error on console in addition to the log file. Note that this feature comes with a performance impact.
11+
For more information, visit https://learn.microsoft.com/dotnet/core/testing/unit-testing-platform-integration-dotnet-test#show-failure-per-test
12+
-->
13+
<TestingPlatformShowTestsFailure>true</TestingPlatformShowTestsFailure>
14+
</PropertyGroup>
15+
16+
<ItemGroup>
17+
<EmbeddedResource Include="Encoding\ansi.txt">
18+
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
19+
</EmbeddedResource>
20+
<EmbeddedResource Include="Encoding\UTF8.txt">
21+
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
22+
</EmbeddedResource>
23+
</ItemGroup>
24+
25+
<ItemGroup>
26+
<ProjectReference Include="..\CommonFilter\CommonFilter.csproj" />
27+
<ProjectReference Include="..\Core.MemoryLog\Core.MemoryLog.csproj" />
28+
<ProjectReference Include="..\FileHandler\FileHandler.csproj" />
29+
<ProjectReference Include="..\InterOp\InterOp.csproj" />
30+
<ProjectReference Include="..\LightVector\LightVector.csproj" />
31+
<ProjectReference Include="..\Mathematics.Constants\Mathematics.Constants.csproj" />
32+
<ProjectReference Include="..\Mathematics\Mathematics.csproj" />
33+
<ProjectReference Include="..\Pathfinder\Pathfinder.csproj" />
34+
<ProjectReference Include="..\Serializer\Serializer.csproj" />
35+
</ItemGroup>
36+
37+
</Project>

CommonLibraryTests/DataFormatterTests.cs renamed to CommonLibrary.Tests/DataFormatterTests.cs

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,23 @@
11
/*
22
* COPYRIGHT: See COPYING in the top level directory
3-
* PROJECT: CommonLibraryTests
3+
* PROJECT: CommonLibrary.Tests
44
* FILE: DataFormatterTests.cs
55
* PURPOSE: Some basic tests for DataFormatter
66
* PROGRAMER: Peter Geinitz (Wayfarer)
77
*/
88

9-
using System.Collections.Generic;
10-
using System.IO;
119
using System.Text;
12-
using System.Threading.Tasks;
1310
using DataFormatter;
1411
using FileHandler;
15-
using Microsoft.VisualStudio.TestTools.UnitTesting;
1612

17-
namespace CommonLibraryTests
13+
namespace CommonLibrary.Tests
1814
{
1915
/// <summary>
2016
/// Test some special cases for the Data Formatter
17+
/// Do not parallelize this test class as it may interfere with file operations.
2118
/// </summary>
2219
[TestClass]
20+
[DoNotParallelize]
2321
public class DataFormatterTests
2422
{
2523
/// <summary>

CommonLibraryTests/Encoding/UTF8.txt renamed to CommonLibrary.Tests/Encoding/UTF8.txt

File renamed without changes.

CommonLibraryTests/Encoding/ansi.txt renamed to CommonLibrary.Tests/Encoding/ansi.txt

File renamed without changes.

CommonLibraryTests/FastMathBenchmarkTests.cs renamed to CommonLibrary.Tests/FastMathBenchmarkTests.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,16 @@
11
/*
22
* COPYRIGHT: See COPYING in the top level directory
3-
* PROJECT: CommonLibraryTests
3+
* PROJECT: CommonLibrary.Tests
44
* FILE: FastMathBenchmarkTests.cs
55
* PURPOSE: Your file purpose here
66
* PROGRAMMER: Peter Geinitz (Wayfarer)
77
*/
88

9-
using System;
109
using System.Diagnostics;
1110
using Mathematics;
1211
using Mathematics.Constants;
13-
using Microsoft.VisualStudio.TestTools.UnitTesting;
1412

15-
namespace CommonLibraryTests
13+
namespace CommonLibrary.Tests
1614
{
1715
[TestClass]
1816
public class FastMathBenchmarkTests

CommonLibraryTests/FileObserverTests.cs renamed to CommonLibrary.Tests/FileObserverTests.cs

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,15 @@
11
/*
22
* COPYRIGHT: See COPYING in the top level directory
3-
* PROJECT: CommonLibraryTests
3+
* PROJECT: CommonLibrary.Tests
44
* FILE: FileObserverTests.cs
55
* PURPOSE: Some tests for FileObserver. Not exhaustive, but a good start to validate the core functionality and catch regressions.
66
* PROGRAMMER: Peter Geinitz (Wayfarer)
77
*/
88

99

10-
using System;
11-
using System.IO;
12-
using System.Threading;
13-
using System.Threading.Tasks;
14-
using Microsoft.VisualStudio.TestTools.UnitTesting;
1510
using FileHandler;
1611

17-
namespace CommonLibraryTests
12+
namespace CommonLibrary.Tests
1813
{
1914
[TestClass]
2015
public class FileObserverTests

0 commit comments

Comments
 (0)