fix(manager/nuget): skip invalid xml files (#16758)

renovatebot · Jul 25, 2022 · 17387fc · 17387fc
1 parent 2bbbc95
commit 17387fc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/lib/modules/manager/nuget/package-tree.spec.ts b/lib/modules/manager/nuget/package-tree.spec.ts
@@ -155,12 +155,10 @@ describe('modules/manager/nuget/package-tree', () => {
       );
     });
 
-    it('throws error on invalid xml file', async () => {
+    it('skips on invalid xml file', async () => {
       git.getFileList.mockResolvedValue(['foo/bar.csproj']);
       Fixtures.mock({ '/tmp/repo/foo/bar.csproj': '<invalid' });
-      await expect(getDependentPackageFiles('foo/bar.csproj')).rejects.toThrow(
-        'Invalid xml file: foo/bar.csproj'
-      );
+      expect(await getDependentPackageFiles('foo/bar.csproj')).toEqual([]);
     });
   });
 });
diff --git a/lib/modules/manager/nuget/package-tree.ts b/lib/modules/manager/nuget/package-tree.ts
@@ -40,7 +40,7 @@ export async function getDependentPackageFiles(
   for (const f of packageFiles) {
     const doc = await readFileAsXmlDocument(f);
     if (!doc) {
-      throw new Error(`Invalid xml file: ${f}`);
+      continue;
     }
 
     const projectReferenceAttributes = doc