From 77dd0d4a92222ae8468ba2561b866f5a13b9c787 Mon Sep 17 00:00:00 2001 From: Will Godbe Date: Mon, 3 Jan 2022 15:07:21 -0800 Subject: [PATCH] Don't exclude System.IO.Pipelines from TargetingPack version check --- src/Framework/test/TargetingPackTests.cs | 23 +++++++++-------------- 1 file changed, 9 insertions(+), 14 deletions(-) diff --git a/src/Framework/test/TargetingPackTests.cs b/src/Framework/test/TargetingPackTests.cs index 8d9f02524e24..3c72b04c7f7c 100644 --- a/src/Framework/test/TargetingPackTests.cs +++ b/src/Framework/test/TargetingPackTests.cs @@ -109,22 +109,17 @@ public void RefAssemblyReferencesHaveExpectedAssemblyVersions() Assert.All(dlls, path => { - // Skip netstandard2.0 System.IO.Pipelines assembly. References have old versions. - var filename = Path.GetFileName(path); - if (!string.Equals("System.IO.Pipelines.dll", filename, StringComparison.OrdinalIgnoreCase)) - { - using var fileStream = File.OpenRead(path); - using var peReader = new PEReader(fileStream, PEStreamOptions.Default); - var reader = peReader.GetMetadataReader(MetadataReaderOptions.Default); + using var fileStream = File.OpenRead(path); + using var peReader = new PEReader(fileStream, PEStreamOptions.Default); + var reader = peReader.GetMetadataReader(MetadataReaderOptions.Default); - Assert.All(reader.AssemblyReferences, handle => - { - var reference = reader.GetAssemblyReference(handle); - var result = 0 == reference.Version.Revision; + Assert.All(reader.AssemblyReferences, handle => + { + var reference = reader.GetAssemblyReference(handle); + var result = (0 == reference.Version.Revision && 0 == reference.Version.Build); - Assert.True(result, $"In {filename}, {reference.GetAssemblyName()} has unexpected version {reference.Version}."); - }); - } + Assert.True(result, $"In {Path.GetFileName(path)}, {reference.GetAssemblyName()} has unexpected version {reference.Version}."); + }); }); }