no-licenses-on-some-platforms

src/NuGetUtility/ReferencedPackagesReader/ReferencedPackageReader.cs

@@ -119,6 +119,11 @@ private bool TryLoadAssetsFile(IProject project, [NotNullWhen(true)] out ILockFi
             }
             assetsFile = _lockFileFactory.GetFromFile(assetsPath);
 
+            if (assetsFile.TryGetErrors(out string[] errors))
+            {
+                throw new ReferencedPackageReaderException($"Project assets file for project {project.FullPath} contains errors:{Environment.NewLine}{string.Join(Environment.NewLine, errors)}");
+            }
+
             if (!assetsFile.PackageSpec.IsValid() || !(assetsFile.Targets?.Any() ?? false))
             {
                 throw new ReferencedPackageReaderException(

src/NuGetUtility/Wrapper/NuGetWrapper/ProjectModel/ILockFile.cs

@@ -5,6 +5,7 @@ namespace NuGetUtility.Wrapper.NuGetWrapper.ProjectModel
 {
     public interface ILockFile
     {
+        bool TryGetErrors(out string[] errors);
         IPackageSpec PackageSpec { get; }
         IEnumerable<ILockFileTarget>? Targets { get; }
     }

src/NuGetUtility/Wrapper/NuGetWrapper/ProjectModel/WrappedLockFile.cs

@@ -18,5 +18,12 @@ public WrappedLockFile(LockFile file)
 
         public IPackageSpec PackageSpec => new WrappedPackageSpec(_file.PackageSpec);
         public IEnumerable<ILockFileTarget>? Targets => _file.Targets?.Select(t => new WrappedLockFileTarget(t));
+
+        public bool TryGetErrors(out string[] errors)
+        {
+            IAssetsLogMessage[] fileErrors = _file.LogMessages.Where(l => l.Level == NuGet.Common.LogLevel.Error).ToArray();
+            errors = fileErrors.Select(e => $"[{e.Code}] {e.Message}").ToArray();
+            return errors.Length > 0;
+        }
     }
 }

tests/NuGetUtility.Test/ReferencedPackagesReader/ReferencedPackageReaderTest.cs

@@ -120,6 +120,24 @@ public void SetUp()
         private IDictionary<INuGetFramework, PackageIdentity[]> _referencedPackagesForFramework = null!;
         private IDictionary<INuGetFramework, PackageIdentity[]> _directlyReferencedPackagesForFramework = null!;
 
+        [Test]
+        public void GetInstalledPackages_Should_ThrowReferencedPackageReaderException_If_AssetsFileContainsErrors(
+            [Values] bool includeTransitive)
+        {
+            string[] errors = _fixture.CreateMany<string>().ToArray();
+            _lockFileMock.TryGetErrors(out Arg.Any<string[]>()).Returns(args =>
+            {
+                args[0] = errors;
+                return true;
+            });
+            _projectMock.FullPath.Returns(_projectPath);
+
+            ReferencedPackageReaderException? exception = Assert.Throws<ReferencedPackageReaderException>(() =>
+                _uut.GetInstalledPackages(_projectPath, includeTransitive));
+
+            Assert.That(exception!.Message, Is.EqualTo($"Project assets file for project {_projectPath} contains errors:{Environment.NewLine}{string.Join(Environment.NewLine, errors)}"));
+        }
+
         [Test]
         public void GetInstalledPackages_Should_ThrowReferencedPackageReaderException_If_PackageSpecificationIsInvalid(
             [Values] bool includeTransitive)