Move test back to in Editor only tests

Author: jfreire-unity

Assets/Tests/InputSystem/CoreTests_Editor.cs

@@ -2905,6 +2905,39 @@ private static void DisableProjectWideActions()
 #endif
     }
 
+    [Test]
+    [Category("Editor")]
+    public void Editor_InitializeInEditor_EnablesProjectWideActions()
+    {
+        #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
+        if (InputSystem.actions != null)
+        {
+            // Asserts that project wide actions are enabled by default.
+            // Before the test is run, InputSystem.Reset() is called which will enable them.
+            // It can be interpreted as a mock of the behavior that happens when `InitializeInEditor()` is called.
+            Assert.That(InputSystem.actions.enabled, Is.True);
+
+            // Calling exit play mode callbacks will disable them
+            InputSystem.OnPlayModeChange(PlayModeStateChange.ExitingPlayMode);
+            InputSystem.OnPlayModeChange(PlayModeStateChange.EnteredEditMode);
+
+            Assert.That(InputSystem.actions.enabled, Is.False);
+
+            // Calling enter play mode callbacks will not re-enable them per default. They are only
+            // enabled when `InputSystem.InitializeInEditor()` is called, which happens before these callbacks.
+            // Note: Project-wide actions are disabled at this point. These next lines are added to make sure we
+            // establish behavior that project-wide actions should be enabled only once
+            // `InputSystem.InitializeInEditor()` is called. Before this test was introduced, project-wide actions were
+            // enabled after entering play mode again which would lead to a different behavior than Player
+            // builds.
+            InputSystem.OnPlayModeChange(PlayModeStateChange.ExitingEditMode);
+            InputSystem.OnPlayModeChange(PlayModeStateChange.EnteredPlayMode);
+
+            Assert.That(InputSystem.actions.enabled, Is.False);
+        }
+        #endif
+    }
+
     [Test]
     [Category("Editor")]
     public void Editor_LeavingPlayMode_DestroysAllActionStates()
@@ -2920,6 +2953,7 @@ public void Editor_LeavingPlayMode_DestroysAllActionStates()
         InputSystem.OnPlayModeChange(PlayModeStateChange.ExitingEditMode);
 
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
+
         // This simulates enabling project-wide actions, which is done before just before entering play mode,
         // called from InputSystem.InitializeInEditor().
         if (InputSystem.actions)

Assets/Tests/InputSystem/CoreTests_ProjectWideActions.cs

@@ -196,37 +196,6 @@ public void ProjectWideActions_CanEnableCurrentActionMapOfPlayerInput(string act
         //NOTE: Asset actions are considered enabled even if a single action map is enabled
         Assert.That(playerInput.actions.enabled, Is.EqualTo(expectedResult));
     }
-
-    [Test]
-    [Category("Editor")]
-    public void ProjectWideActions_InitializeInEditorEnablesProjectWideActions()
-    {
-        if (InputSystem.actions != null)
-        {
-            // Asserts that project wide actions are enabled by default.
-            // Before the test is run, InputSystem.Reset() is called which will enable them.
-            // It can be interpreted as a mock of the behavior that happens when `InitializeInEditor()` is called.
-            Assert.That(InputSystem.actions.enabled, Is.True);
-
-            // Calling exit play mode callbacks will disable them
-            InputSystem.OnPlayModeChange(PlayModeStateChange.ExitingPlayMode);
-            InputSystem.OnPlayModeChange(PlayModeStateChange.EnteredEditMode);
-
-            Assert.That(InputSystem.actions.enabled, Is.False);
-
-            // Calling enter play mode callbacks will not re-enable them per default. They are only
-            // enabled when `InputSystem.InitializeInEditor()` is called, which happens before these callbacks.
-            // Note: Project-wide actions are disabled at this point. These next lines are added to make sure we
-            // establish behavior that project-wide actions should be enabled only once
-            // `InputSystem.InitializeInEditor()` is called. Before this test was introduced, project-wide actions were
-            // enabled after entering play mode again which would lead to a different behavior than Player
-            // builds.
-            InputSystem.OnPlayModeChange(PlayModeStateChange.ExitingEditMode);
-            InputSystem.OnPlayModeChange(PlayModeStateChange.EnteredPlayMode);
-
-            Assert.That(InputSystem.actions.enabled, Is.False);
-        }
-    }
 }
 
 #endif