Skip to content

Add tests that use the new input system#288

Draft
nowsprinting wants to merge 1 commit intomasterfrom
chore/input-system
Draft

Add tests that use the new input system#288
nowsprinting wants to merge 1 commit intomasterfrom
chore/input-system

Conversation

@nowsprinting
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings January 6, 2026 21:48
@github-actions github-actions bot added the chore label Jan 6, 2026
@nowsprinting nowsprinting changed the title Add tests using new input system Add tests that use the new input system Jan 6, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds test support for Unity's new Input System by introducing a dedicated test scene and accompanying test case. The changes enable testing of UI interactions using the new Input System API alongside the existing input handling tests.

  • Adds a comprehensive test scene (NewInputSystem.unity) with multiple test objects demonstrating various event handler patterns with the new Input System
  • Introduces Input Actions configuration (TestInputActions.inputactions) with Player and UI action maps
  • Adds a test case to verify click operations work correctly with the new Input System

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Tests/TestAssets/TestInputActions.inputactions Defines Input Actions with Player and UI maps for keyboard, mouse, gamepad, touch, joystick, and XR control schemes
Tests/TestAssets/TestInputActions.inputactions.meta Unity metadata for the Input Actions asset with InputSystem namespace configuration
Tests/TestAssets.meta Unity metadata for the TestAssets directory
Tests/Scenes/NewInputSystem.unity Unity scene containing test objects (UsingOnPointerClickHandler, DestroyItselfIfPointerDown, UsingPointerDownUpEventTrigger, UsingMultipleEventTriggers, UsingPointerClickEventTrigger, UsingOnPointerDownUpHandler) with EventSystem configured for new Input System
Tests/Scenes/NewInputSystem.unity.meta Unity metadata for the test scene
Tests/Runtime/Operators/UguiClickOperatorTest.cs Adds test case OperateAsync_NewInputSystem_InvokeOnClick to verify click operator functionality with new Input System

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Code Metrics Report

master (043f2af) #288 (b44a2b9) +/-
Coverage 88.9% 88.9% 0.0%
Code to Test Ratio 1:1.3 1:1.3 +0.0
Test Execution Time 12m41s 15m10s +2m29s
Details
  |                     | master (043f2af) | #288 (b44a2b9) |  +/-   |
  |---------------------|------------------|----------------|--------|
  | Coverage            |            88.9% |          88.9% |   0.0% |
  |   Files             |               70 |             70 |      0 |
  |   Lines             |             2557 |           2557 |      0 |
  |   Covered           |             2275 |           2275 |      0 |
+ | Code to Test Ratio  |            1:1.3 |          1:1.3 |   +0.0 |
  |   Code              |             4578 |           4578 |      0 |
+ |   Test              |             6103 |           6112 |     +9 |
- | Test Execution Time |           12m41s |         15m10s | +2m29s |

Reported by octocov

@nowsprinting nowsprinting marked this pull request as draft January 7, 2026 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants