Skip to content

Commit 3868d08

Browse files
Update minimum type count threshold to 8500
Mono.Android has 8,516 types. Using 8,500 as the threshold catches any significant regression in type discovery. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent cdccb24 commit 3868d08

File tree

2 files changed

+19
-19
lines changed

2 files changed

+19
-19
lines changed

tests/Microsoft.Android.Sdk.TrimmableTypeMap.IntegrationTests/ScannerComparisonTests.Helpers.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -281,8 +281,8 @@ public void ExactCompatJniNames_MonoAndroid ()
281281
var (legacyData, _) = TypeDataBuilder.BuildLegacy (MonoAndroidAssemblyPath);
282282
var newData = TypeDataBuilder.BuildNew (AllAssemblyPaths);
283283

284-
Assert.True (legacyData.Count > 8000, $"Expected >8000 legacy type data entries, got {legacyData.Count}");
285-
Assert.True (newData.Count > 8000, $"Expected >8000 new type data entries, got {newData.Count}");
284+
Assert.True (legacyData.Count > 8500, $"Expected >8500 legacy type data entries, got {legacyData.Count}");
285+
Assert.True (newData.Count > 8500, $"Expected >8500 new type data entries, got {newData.Count}");
286286

287287
var mismatches = ComparisonDiffHelper.CompareCompatJniNames (legacyData, newData);
288288

@@ -337,8 +337,8 @@ public void ExactInvokerTypes_MonoAndroid ()
337337
var (legacyData, _) = TypeDataBuilder.BuildLegacy (MonoAndroidAssemblyPath);
338338
var newData = TypeDataBuilder.BuildNew (AllAssemblyPaths);
339339

340-
Assert.True (legacyData.Count > 8000, $"Expected >8000 legacy type data entries, got {legacyData.Count}");
341-
Assert.True (newData.Count > 8000, $"Expected >8000 new type data entries, got {newData.Count}");
340+
Assert.True (legacyData.Count > 8500, $"Expected >8500 legacy type data entries, got {legacyData.Count}");
341+
Assert.True (newData.Count > 8500, $"Expected >8500 new type data entries, got {newData.Count}");
342342

343343
var mismatches = ComparisonDiffHelper.CompareInvokerTypes (legacyData, newData);
344344

@@ -351,8 +351,8 @@ public void ExactCannotRegisterInStaticCtor_MonoAndroid ()
351351
var (legacyData, _) = TypeDataBuilder.BuildLegacy (MonoAndroidAssemblyPath);
352352
var newData = TypeDataBuilder.BuildNew (AllAssemblyPaths);
353353

354-
Assert.True (legacyData.Count > 8000, $"Expected >8000 legacy type data entries, got {legacyData.Count}");
355-
Assert.True (newData.Count > 8000, $"Expected >8000 new type data entries, got {newData.Count}");
354+
Assert.True (legacyData.Count > 8500, $"Expected >8500 legacy type data entries, got {legacyData.Count}");
355+
Assert.True (newData.Count > 8500, $"Expected >8500 new type data entries, got {newData.Count}");
356356

357357
var mismatches = ComparisonDiffHelper.CompareCannotRegisterInStaticCtor (legacyData, newData);
358358

tests/Microsoft.Android.Sdk.TrimmableTypeMap.IntegrationTests/ScannerComparisonTests.cs

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@ public void ExactTypeMap_MonoAndroid ()
1212
var (legacy, _) = ScannerRunner.RunLegacy (MonoAndroidAssemblyPath);
1313
var (newEntries, _) = ScannerRunner.RunNew (AllAssemblyPaths);
1414

15-
Assert.True (legacy.Count > 8000, $"Expected >8000 legacy type map entries, got {legacy.Count}");
16-
Assert.True (newEntries.Count > 8000, $"Expected >8000 new type map entries, got {newEntries.Count}");
15+
Assert.True (legacy.Count > 8500, $"Expected >8500 legacy type map entries, got {legacy.Count}");
16+
Assert.True (newEntries.Count > 8500, $"Expected >8500 new type map entries, got {newEntries.Count}");
1717

1818
AssertTypeMapMatch (legacy, newEntries);
1919
}
@@ -47,7 +47,7 @@ public void ScannerDiagnostics_MonoAndroid ()
4747

4848
var interfaces = peers.Count (p => p.IsInterface);
4949
var totalMethods = peers.Sum (p => p.MarshalMethods.Count);
50-
Assert.True (peers.Count > 8000, $"Expected >8000 types, got {peers.Count}");
50+
Assert.True (peers.Count > 8500, $"Expected >8500 types, got {peers.Count}");
5151
Assert.True (interfaces > 500, $"Expected >500 interfaces, got {interfaces}");
5252
Assert.True (totalMethods > 10000, $"Expected >10000 marshal methods, got {totalMethods}");
5353
}
@@ -58,8 +58,8 @@ public void ExactBaseJavaNames_MonoAndroid ()
5858
var (legacyData, _) = TypeDataBuilder.BuildLegacy (MonoAndroidAssemblyPath);
5959
var newData = TypeDataBuilder.BuildNew (AllAssemblyPaths);
6060

61-
Assert.True (legacyData.Count > 8000, $"Expected >8000 legacy type data entries, got {legacyData.Count}");
62-
Assert.True (newData.Count > 8000, $"Expected >8000 new type data entries, got {newData.Count}");
61+
Assert.True (legacyData.Count > 8500, $"Expected >8500 legacy type data entries, got {legacyData.Count}");
62+
Assert.True (newData.Count > 8500, $"Expected >8500 new type data entries, got {newData.Count}");
6363

6464
var mismatches = ComparisonDiffHelper.CompareBaseJavaNames (legacyData, newData);
6565

@@ -72,8 +72,8 @@ public void ExactImplementedInterfaces_MonoAndroid ()
7272
var (legacyData, _) = TypeDataBuilder.BuildLegacy (MonoAndroidAssemblyPath);
7373
var newData = TypeDataBuilder.BuildNew (AllAssemblyPaths);
7474

75-
Assert.True (legacyData.Count > 8000, $"Expected >8000 legacy type data entries, got {legacyData.Count}");
76-
Assert.True (newData.Count > 8000, $"Expected >8000 new type data entries, got {newData.Count}");
75+
Assert.True (legacyData.Count > 8500, $"Expected >8500 legacy type data entries, got {legacyData.Count}");
76+
Assert.True (newData.Count > 8500, $"Expected >8500 new type data entries, got {newData.Count}");
7777

7878
var (missingInterfaces, extraInterfaces) = ComparisonDiffHelper.CompareImplementedInterfaces (legacyData, newData);
7979

@@ -87,8 +87,8 @@ public void ExactActivationCtors_MonoAndroid ()
8787
var (legacyData, _) = TypeDataBuilder.BuildLegacy (MonoAndroidAssemblyPath);
8888
var newData = TypeDataBuilder.BuildNew (AllAssemblyPaths);
8989

90-
Assert.True (legacyData.Count > 8000, $"Expected >8000 legacy type data entries, got {legacyData.Count}");
91-
Assert.True (newData.Count > 8000, $"Expected >8000 new type data entries, got {newData.Count}");
90+
Assert.True (legacyData.Count > 8500, $"Expected >8500 legacy type data entries, got {legacyData.Count}");
91+
Assert.True (newData.Count > 8500, $"Expected >8500 new type data entries, got {newData.Count}");
9292

9393
var (presenceMismatches, declaringTypeMismatches, styleMismatches) = ComparisonDiffHelper.CompareActivationCtors (legacyData, newData);
9494

@@ -103,8 +103,8 @@ public void ExactJavaConstructors_MonoAndroid ()
103103
var (legacyData, _) = TypeDataBuilder.BuildLegacy (MonoAndroidAssemblyPath);
104104
var newData = TypeDataBuilder.BuildNew (AllAssemblyPaths);
105105

106-
Assert.True (legacyData.Count > 8000, $"Expected >8000 legacy type data entries, got {legacyData.Count}");
107-
Assert.True (newData.Count > 8000, $"Expected >8000 new type data entries, got {newData.Count}");
106+
Assert.True (legacyData.Count > 8500, $"Expected >8500 legacy type data entries, got {legacyData.Count}");
107+
Assert.True (newData.Count > 8500, $"Expected >8500 new type data entries, got {newData.Count}");
108108

109109
var (missingCtors, extraCtors) = ComparisonDiffHelper.CompareJavaConstructors (legacyData, newData);
110110

@@ -118,8 +118,8 @@ public void ExactTypeFlags_MonoAndroid ()
118118
var (legacyData, _) = TypeDataBuilder.BuildLegacy (MonoAndroidAssemblyPath);
119119
var newData = TypeDataBuilder.BuildNew (AllAssemblyPaths);
120120

121-
Assert.True (legacyData.Count > 8000, $"Expected >8000 legacy type data entries, got {legacyData.Count}");
122-
Assert.True (newData.Count > 8000, $"Expected >8000 new type data entries, got {newData.Count}");
121+
Assert.True (legacyData.Count > 8500, $"Expected >8500 legacy type data entries, got {legacyData.Count}");
122+
Assert.True (newData.Count > 8500, $"Expected >8500 new type data entries, got {newData.Count}");
123123

124124
var (interfaceMismatches, abstractMismatches, genericMismatches, acwMismatches) = ComparisonDiffHelper.CompareTypeFlags (legacyData, newData);
125125

0 commit comments

Comments
 (0)