Test doubles

[ddreimane]

Testing using test doubles, in particular, test cases that make use of any mocks, spies, fakes, etc.

[nightm4re94]

Thank you for your PR! Please address the minor issues I mentioned in my review before I can merge.

[nightm4re94]

what's the reason for removing createAnimationController and createMovementController?

[nightm4re94]

Now, everything looks fine. Thank you!

[Gamebuster19901]

This PR causes build failures on Java 8.

@ddreimane @jluech @ddreimane @niels89 @nightm4re94

gamebuster@gamebuster-System-Product-Name:~/Desktop/Modding/LitiEngine$ ./gradlew build

> Task :compileTestJava FAILED
/home/gamebuster/Desktop/Modding/LitiEngine/tests/de/gurkenlabs/litiengine/abilities/effects/EffectTests.java:50: error: cannot find symbol
            List.of(new Creature(),
                ^
  symbol:   method of(Creature,Creature,Creature)
  location: interface List
/home/gamebuster/Desktop/Modding/LitiEngine/tests/de/gurkenlabs/litiengine/environment/EntitySpawnerTests.java:87: error: cannot find symbol
            super(List.of(spawnPoint), amount, SpawnMode.ONERANDOMSPAWNPOINT);
                      ^
  symbol:   method of(Spawnpoint)
  location: interface List
2 errors

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':compileTestJava'.
> Compilation failed; see the compiler error output for details.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1s
6 actionable tasks: 4 executed, 2 up-to-date

[nightm4re94]

This PR causes build failures on Java 8.

@ddreimane @jluech @ddreimane @niels89 @nightm4re94

...

Resolved by 44d2eb6.

[jluech]

Resolved by 44d2eb6.

Thanks @nightm4re94! Would not have had time to look at it properly until Thursday at least.

Your commit did introduce some wildcard imports on Mockito, though 😉 and it changed some test method visibility to package-private, I assume that was on purpose.

[nightm4re94]

Damn you, IntelliJ! 🗡️ The wildcard import was not on purpose, of course. To configure IntelliJ the right way, see this answer.

Since JUnit5, Test classes, test methods, and lifecycle methods are no longer required to be public, but they must not be private. (See JUnit 5 User Guide.)