feat: Add joypad camera support

[emersonbottero]

Take a look at InputExtensions and see if it makes sense to be part of your library...
the intention is to keep the logic used in the mouse the same with the one used with joypad..
since is my first interaction with your libs and even Godot it can be completely wrong.. 😆

[jolexxa]

Hey, really good job implementing this! It's really solid.

I added a few whitespace things you can just click to commit right in (for this project I try to keep lines at 80 wide).

We'll also need to add unit tests for your new code. If you're not sure how to do this, I'm happy to help in Discord or give it a shot later.

I ran the coverage.sh script and it shows these areas needing coverage from your new code:

For InputExtensions, I would rename it and its file to InputUtilities and make an InputUtiltiesTest.cs file in test/src/utils.

Note that in the test, you could use Input.ActionPress to simulate the names of the actions you want to press, similar to how its done in this test:

GameDemo/test/src/game/GameTest.cs

Lines 218 to 227 in 6b593e9

	public void InputsPauseButtonPressed() {
	_logic.Setup(
	logic => logic.Input(It.IsAny<GameLogic.Input.PauseButtonPressed>())
	);
	Input.ActionPress("ui_cancel");
	_game._Input(default!);
	_logic.VerifyAll();
	}

For PlayerCamera.cs, these added lines now need coverage:

And lastly, you can probably copy/paste the PlayerCameraLogic.State.InputEnabled tests for mouse input and modify them to test your joypad input to get these lines covered:

Again, happy to answer questions in Discord :)) Thank you so much for doing this!! It's really good work.

[jolexxa]

(wrap long lines)

Suggested change

	CameraLogic.Input(new PlayerCameraLogic.Input.JoyPadInputOccurred(xMotion));
	CameraLogic.Input(
	new PlayerCameraLogic.Input.JoyPadInputOccurred(xMotion)
	);

[jolexxa]

(wrap long lines)

Suggested change

	CameraLogic.Input(new PlayerCameraLogic.Input.JoyPadInputOccurred(yMotion));
	CameraLogic.Input(
	new PlayerCameraLogic.Input.JoyPadInputOccurred(yMotion)
	);

[jolexxa]

Looks really good! Looks like you can get rid of some of the stuff that's copy/pasted from the other test for the input utilities since they are only interfacing with the engine.

[jolexxa]

Looks like two whitespace changes still need to be made (see previous comments) and we're not quite back at 100% test coverage yet. Super close!

(InputUtilities.cs)

(PlayerCamera.cs)

[jolexxa]

This is really awesome work. Thank you for doing the tests and dealing with all the Joypad input simulation to make sure it's good. I'll merge if you're ready!