Skip to content

Comments

TEST ONLY#2350

Draft
josepmariapujol-unity wants to merge 68 commits intodevelopfrom
input/ci-check-with-ugui-package
Draft

TEST ONLY#2350
josepmariapujol-unity wants to merge 68 commits intodevelopfrom
input/ci-check-with-ugui-package

Conversation

@josepmariapujol-unity
Copy link
Collaborator

Description

Please fill this section with a description what the pull request is trying to address and what changes were made.

Testing status & QA

Please describe the testing already done by you and what testing you request/recommend QA to execute. If you used or created any testing project please link them here too for QA.

Overall Product Risks

Please rate the potential complexity and halo effect from low to high for the reviewers. Note down potential risks to specific Editor branches if any.

  • Complexity:
  • Halo Effect:

Comments to reviewers

Please describe any additional information such as what to focus on, or historical info for the reviewers.

Checklist

Before review:

  • Changelog entry added.
    • Explains the change in Changed, Fixed, Added sections.
    • For API change contains an example snippet and/or migration example.
    • JIRA ticket linked, example (case %%). If it is a private issue, just add the case ID without a link.
    • Jira port for the next release set as "Resolved".
  • Tests added/changed, if applicable.
    • Functional tests Area_CanDoX, Area_CanDoX_EvenIfYIsTheCase, Area_WhenIDoX_AndYHappens_ThisIsTheResult.
    • Performance tests.
    • Integration tests.
  • Docs for new/changed API's.
    • Xmldoc cross references are set correctly.
    • Added explanation how the API works.
    • Usage code examples added.
    • The manual is updated, if needed.

During merge:

  • Commit message for squash-merge is prefixed with one of the list:
    • NEW: ___.
    • FIX: ___.
    • DOCS: ___.
    • CHANGE: ___.
    • RELEASE: 1.1.0-preview.3.

jfreire-unity and others added 30 commits November 28, 2025 22:24
- Move InputSystem/* to Runtime/ (preserving history)
- Move InputSystem/Editor to Editor/ (preserving history)
- Add meta files for new folder structure
- Follows Unity package layout conventions
- All file history preserved via git mv
Also exposes the required internals between assemblies.
This is done to avoid calling into Edito specific code. Instead, it will
be called if Editor code has registered a callback.
Due to refactoring, a lot of paths still contained the InputSystem folder path that no longer exists. We only have Editor and Runtime folders.
Some files were moved to other folders, after merging conflicts arise but since they showed as removed changes didn't land on the moved files. This commit is fixing this
… event buffer init; checked casts; null when empty
josepmariapujol-unity and others added 28 commits February 9, 2026 13:21
…elnputDeviceBasedOnSteam|GAFile

Uncommenting code from InputSystem.cs -> s_RemoteConnection fixes 2 playmode tests
List of tests:
	1.	Editor_DomainReload_InputSystemInitializationCausesDevicesToBeRecreated
	2.	Editor_DomainReload_PreservesEnabledState
	3.	Editor_DomainReload_PreservesUsagesOnDevices
	4.	Editor_DomainReload_RetainsUnsupportedDevices
…lay mode tests

	1.	Editor_InitializeInEditor_EnablesProjectWideActions
	2.	Editor_InputEventsOccurringWhileGoingIntoPlayMode_AreDiscarded
	3.	Editor_LeavingPlayMode_DestroysAllActionStates
	4.	Editor_LeavingPlayMode_DiscardsInputActionAssetChanges
	5.	Editor_LeavingPlayMode_RemovesAllInputUsersAndStopsListeningForUnpairedDeviceActivity
… test

Delete the two LogAssert.Expect lines.
Were we intentionally validating that duplicate singleton construction still produces errors? If so, I will revert this commit
Fixing compilation errors for CI tests in InputSystem-StandaloneIl2CppPerformanceTests - 6000.0 - MacOS job
Remote_CanDisableUnityRemoteSupport test fixed
Instable test fix solution:
EditorInputControlLayoutCache.Refresh() could return early without holding
a cache reference, causing "Must hold an instance reference" asserts
after InputSystem.SaveAndReset() in tests.

Now, Refresh() ensures a cache ref is held even on the early-return path.
…ble(): ensure CustomDevice restored from saved state

Fix domain reload in InputSystem tests: ensure CustomDevice restored from saved state

- SimulateDomainReload() now calls SimulateDomainReloadRecoveryFromSerializedState()
- Clears InputSystem.devices and restores saved device states from serialized InputSystemObject
- Ensures Editor_DomainReload_CustomDevicesAreRestoredAsLayoutsBecomeAvailable test passes
…meAvailable(): ensure CustomDevice restored from saved state"

This reverts commit 24a6469.
…_SYSTEM_ENABLE_UI

Fix compilation errors in VirtualMouseInputEditor.cs with UNITY_INPUT_SYSTEM_ENABLE_UI so it only compiles when UI runtime types are present
@codecov-github-com
Copy link

codecov-github-com bot commented Feb 19, 2026

Codecov Report

Attention: Patch coverage is 22.63374% with 188 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
.../Actions/Interactions/MultiTapInteractionEditor.cs 0.00% 30 Missing ⚠️
...tor/Actions/Interactions/PressInteractionEditor.cs 0.00% 25 Missing ⚠️
...Controls/Processors/AxisDeadzoneProcessorEditor.cs 0.00% 18 Missing ⚠️
...ontrols/Processors/StickDeadzoneProcessorEditor.cs 0.00% 18 Missing ⚠️
...r/Actions/Interactions/SlowTapInteractionEditor.cs 0.00% 17 Missing ⚠️
...ditor/Actions/Interactions/TapInteractionEditor.cs 0.00% 17 Missing ⚠️
...itor/Actions/Interactions/HoldInteractionEditor.cs 0.00% 16 Missing ⚠️
...m/Editor/Actions/Composites/AxisCompositeEditor.cs 0.00% 15 Missing ⚠️
...ditor/Actions/Composites/Vector2CompositeEditor.cs 0.00% 15 Missing ⚠️
...ditor/Actions/Composites/Vector3CompositeEditor.cs 0.00% 15 Missing ⚠️
... and 2 more
@@             Coverage Diff             @@
##           develop    #2350      +/-   ##
===========================================
- Coverage    77.95%   76.73%   -1.23%     
===========================================
  Files          476      490      +14     
  Lines        97453    87317   -10136     
===========================================
- Hits         75971    67003    -8968     
+ Misses       21482    20314    -1168     
Flag Coverage Δ
inputsystem_MacOS_2021.3 ?
inputsystem_MacOS_2021.3_project ?
inputsystem_MacOS_2022.3 ?
inputsystem_MacOS_6000.0 ?
inputsystem_MacOS_6000.2 ?
inputsystem_MacOS_6000.2_project ?
inputsystem_MacOS_6000.3 ?
inputsystem_MacOS_6000.4 ?
inputsystem_MacOS_6000.5 ?
inputsystem_Ubuntu_2021.3 ?
inputsystem_Ubuntu_2021.3_project ?
inputsystem_Ubuntu_2022.3 ?
inputsystem_Ubuntu_6000.0 ?
inputsystem_Ubuntu_6000.2 ?
inputsystem_Ubuntu_6000.2_project ?
inputsystem_Ubuntu_6000.3 ?
inputsystem_Ubuntu_6000.4 ?
inputsystem_Ubuntu_6000.5 ?
inputsystem_Windows_2021.3 ?
inputsystem_Windows_2021.3_project ?
inputsystem_Windows_2022.3 ?
inputsystem_Windows_6000.0 ?
inputsystem_Windows_6000.2 ?
inputsystem_Windows_6000.2_project ?
inputsystem_Windows_6000.3 ?
inputsystem_Windows_6000.4 ?
inputsystem_Windows_6000.5 ?
linux_2021.3_pkg ?
linux_2021.3_project ?
linux_2022.3_pkg ?
linux_2022.3_project ?
linux_6000.0_pkg ?
linux_6000.0_project ?
linux_6000.1_pkg ?
linux_6000.1_project ?
linux_6000.2_pkg ?
linux_6000.2_project ?
linux_trunk_pkg ?
linux_trunk_project ?
mac_2021.3_pkg ?
mac_2021.3_project ?
mac_2022.3_pkg ?
mac_2022.3_project ?
mac_6000.0_pkg ?
mac_6000.0_project ?
mac_6000.1_pkg ?
mac_6000.1_project ?
mac_6000.2_pkg ?
mac_6000.2_project ?
mac_trunk_pkg ?
mac_trunk_project ?
win_2021.3_pkg ?
win_2021.3_project ?
win_2022.3_pkg ?
win_2022.3_project ?
win_6000.0_pkg ?
win_6000.0_project ?
win_6000.1_pkg ?
win_6000.1_project ?
win_6000.2_pkg ?
win_6000.2_project ?
win_trunk_pkg ?
win_trunk_project ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
Assets/Tests/InputSystem/APIVerificationTests.cs 89.41% <100.00%> (-0.40%) ⬇️
Assets/Tests/InputSystem/CoreTests_Analytics.cs 99.27% <ø> (-0.10%) ⬇️
Assets/Tests/InputSystem/CoreTests_Editor.cs 97.70% <100.00%> (-0.26%) ⬇️
.../Tests/InputSystem/CoreTests_ProjectWideActions.cs 72.05% <100.00%> (ø)
Assets/Tests/InputSystem/CoreTests_Remoting.cs 92.80% <100.00%> (-0.05%) ⬇️
Assets/Tests/InputSystem/Plugins/UITests.cs 94.33% <ø> (-0.35%) ⬇️
...sets/Tests/InputSystem/Plugins/UnityRemoteTests.cs 96.61% <100.00%> (-0.76%) ⬇️
...tor/Analytics/InputActionsEditorSessionAnalytic.cs 85.32% <ø> (ø)
...inputsystem/Editor/Analytics/InputBuildAnalytic.cs 77.77% <ø> (ø)
...m/Editor/Analytics/InputComponentEditorAnalytic.cs 100.00% <ø> (ø)
... and 49 more

... and 673 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants