Closed jonathanpeppers closed 2 weeks ago
It's been a while, since I went through and refactored these tests. After seeing a couple build-time regressions in .NET 9, I think it's overdue.
Removing test cases that likely won't catch an issue:
Build_Designer_Change
Build_CSProj_Change
Build_From_Clean_DontIncludeRestore would likely catch similar issues as these two, anyway.
Build_From_Clean_DontIncludeRestore
Build_XAML_Change_RefAssembly now uses a net8.0 class library where $(ProduceReferenceAssembly) defaults to true.
Build_XAML_Change_RefAssembly
net8.0
$(ProduceReferenceAssembly)
true
Removed a 500ms addition, that only made sense when we supported both "classic" Xamarin.Android and .NET 6+ projects.
Update the time thresholds to be a bit shorter, in general.
It's been a while, since I went through and refactored these tests. After seeing a couple build-time regressions in .NET 9, I think it's overdue.
Removing test cases that likely won't catch an issue:
Build_Designer_Change(less concerned about designer now)Build_CSProj_ChangeBuild_From_Clean_DontIncludeRestorewould likely catch similar issues as these two, anyway.Build_XAML_Change_RefAssemblynow uses anet8.0class library where$(ProduceReferenceAssembly)defaults totrue.Removed a 500ms addition, that only made sense when we supported both "classic" Xamarin.Android and .NET 6+ projects.
Update the time thresholds to be a bit shorter, in general.