Skip to content

Commit 4fae0c7

Browse files
author
Rene Damm
committed
FIX: doctools failure in APIVerificationTests.
1 parent 4c52078 commit 4fae0c7

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

Assets/Tests/InputSystem/APIVerificationTests.cs

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,12 @@
1414
using UnityEngine.InputSystem;
1515
using UnityEngine.InputSystem.LowLevel;
1616
using HtmlAgilityPack;
17-
using UnityEditor;
1817
using UnityEngine.InputSystem.DualShock;
1918
using UnityEngine.InputSystem.Editor;
2019
using UnityEngine;
2120
using UnityEngine.InputSystem.iOS.LowLevel;
2221
using UnityEngine.InputSystem.Utilities;
22+
using UnityEngine.TestTools;
2323
using Object = System.Object;
2424
using TypeAttributes = Mono.Cecil.TypeAttributes;
2525
using PropertyAttribute = NUnit.Framework.PropertyAttribute;
@@ -448,6 +448,13 @@ private bool IgnoreTypeWithoutPreserveAttribute(Type type)
448448
////TODO: move this to a fixture setup so that it runs *once* for all API checks in a test run
449449
private static string GenerateDocsDirectory(out string log)
450450
{
451+
// The dependency on `com.unity.modules.uielements` we have triggers a 404 error in doctools as it
452+
// tries to retrieve information on the "package" from `packages.unity.com`. As it is a module and not a
453+
// package, there's no metadata on the server and PacmanUtils.GetVersions() in doctools will log an
454+
// error to the console. This doesn't impact the rest of the run so just ignore it.
455+
// This is a workaround. Remove when fixed in doctools.
456+
LogAssert.ignoreFailingMessages = true;
457+
451458
// DocumentationBuilder users C:/temp on Windows to avoid deeply nested paths that go
452459
// beyond the Windows path limit. However, on Yamato agent, C:/temp does not exist.
453460
// Create it manually here.

0 commit comments

Comments
 (0)