chore: expose --verbosity through standards.py (#1085)

Author: 0xFA11

com.unity.netcode.gameobjects/Tests/Editor/Metrics/NetworkMetricsRegistrationTests.cs

@@ -1,4 +1,4 @@
-#if MULTIPLAYER_TOOLS
+#if MULTIPLAYER_TOOLS
 
 using System;
 using System.Linq;
@@ -40,4 +40,4 @@ public void ValidateThatAllMetricTypesAreRegistered(Type metricType)
     }
 }
 
-#endif
+#endif

com.unity.netcode.gameobjects/Tests/Runtime/NetworkPrefabHandlerTests.cs

@@ -16,7 +16,7 @@ namespace Unity.Netcode.RuntimeTests
     public class NetworkPrefabHandlerTests
     {
 
-        private const string  k_TestPrefabObjectName = "NetworkPrefabTestObject";
+        private const string k_TestPrefabObjectName = "NetworkPrefabTestObject";
         private uint m_GlobalObjectIdHashBase = 123456;
         private GameObject MakeValidNetworkPrefab()
         {
@@ -43,7 +43,7 @@ public void NetworkConfigInvalidNetworkPrefabTest()
 
             // Add a NetworkPrefab override with an invalid hash
             NetworkManagerHelper.NetworkManagerObject.NetworkConfig.NetworkPrefabs.Add(new NetworkPrefab() { Override = NetworkPrefabOverride.Hash, SourceHashToOverride = 0 });
-            
+
             // Add a NetworkPrefab override with a valid hash but an invalid target prefab
             NetworkManagerHelper.NetworkManagerObject.NetworkConfig.NetworkPrefabs.Add(new NetworkPrefab() { Override = NetworkPrefabOverride.Hash, SourceHashToOverride = 654321, OverridingTargetPrefab = null });
 

standards.py

@@ -16,6 +16,7 @@
 parser.add_argument("--fix", action="store_true")
 parser.add_argument("--yamato", action="store_true")
 
+parser.add_argument("--verbosity", default="minimal")
 parser.add_argument("--tool-path", default="dotnet-format")
 parser.add_argument("--project-path", default="testproject")
 parser.add_argument("--project-glob", default="*.sln")
@@ -102,7 +103,7 @@
     any_error = False
     for project_file in glob_files:
         print(f"check: project -> {project_file}")
-        any_error = 0 != os.system(f"{args.tool_path} {project_file} --fix-whitespace --fix-style error --check") or any_error
+        any_error = 0 != os.system(f"{args.tool_path} {project_file} --fix-whitespace --fix-style error --check --verbosity {args.verbosity}") or any_error
 
     if any_error:
         exit("check: failed")
@@ -116,7 +117,7 @@
     any_error = False
     for project_file in glob_files:
         print(f"fix: project -> {project_file}")
-        any_error = 0 != os.system(f"{args.tool_path} {project_file} --fix-whitespace --fix-style error") or any_error
+        any_error = 0 != os.system(f"{args.tool_path} {project_file} --fix-whitespace --fix-style error --verbosity {args.verbosity}") or any_error
 
     if any_error:
         exit("fix: failed")
@@ -128,7 +129,7 @@
 
     for project_file in glob_files:
         print(f"yamato: project -> {project_file}")
-        yamato_exec = os.system(f"{args.tool_path} {project_file} --fix-style error --check")
+        yamato_exec = os.system(f"{args.tool_path} {project_file} --fix-style error --check --verbosity {args.verbosity}")
         if yamato_exec != 0:
             exit(f"yamato: failed, exit code -> {yamato_exec}")
 

testproject/Assets/Samples/EnableDisableNetworkObject/EnableDisableSceneNetworkObjectComponent.cs

@@ -19,7 +19,7 @@ public class EnableDisableSceneNetworkObjectComponent : NetworkBehaviour
 
     private void Start()
     {
-        if(m_ActivateObjectButton != null)
+        if (m_ActivateObjectButton != null)
         {
             m_ButtonText = m_ActivateObjectButton.GetComponentInChildren<Text>();
             if (m_ButtonText != null)

testproject/Assets/Tests/Runtime/NetworkSceneManagerTests.cs

@@ -11,7 +11,7 @@
 
 namespace TestProject.RuntimeTests
 {
-    public class NetworkSceneManagerTests: BaseMultiInstanceTest
+    public class NetworkSceneManagerTests : BaseMultiInstanceTest
     {
         protected override int NbClients => 9;
 
@@ -224,15 +224,15 @@ private void SetClientWaitDone(List<ulong> clients)
         private bool ContainsAllClients(List<ulong> clients)
         {
             // First, make sure we have the expected client count
-            if(clients.Count != m_ShouldWaitList.Count)
+            if (clients.Count != m_ShouldWaitList.Count)
             {
                 return false;
             }
 
             // Next, make sure we have all client identifiers
-            foreach(var sceneTestInfo in m_ShouldWaitList)
+            foreach (var sceneTestInfo in m_ShouldWaitList)
             {
-                if(!clients.Contains(sceneTestInfo.ClientId))
+                if (!clients.Contains(sceneTestInfo.ClientId))
                 {
                     return false;
                 }
@@ -248,12 +248,12 @@ private bool ContainsAllClients(List<ulong> clients)
         /// <param name="sceneEvent"></param>
         private void SceneManager_OnSceneEvent(SceneEvent sceneEvent)
         {
-            switch(sceneEvent.SceneEventType)
+            switch (sceneEvent.SceneEventType)
             {
                 case SceneEventData.SceneEventTypes.S2C_Load:
                 case SceneEventData.SceneEventTypes.S2C_Unload:
                     {
-                        Assert.AreEqual(sceneEvent.SceneName,m_CurrentSceneName);
+                        Assert.AreEqual(sceneEvent.SceneName, m_CurrentSceneName);
                         Assert.IsTrue(ContainsClient(sceneEvent.ClientId));
                         Assert.IsNotNull(sceneEvent.AsyncOperation);
                         break;