some missing impl/score unit tests
(cherry picked from commit b99cb0b626f22ae2687fc9cd0dd19828f64a69b2)
cleaner code
(cherry picked from commit c814d511273f23ebeadddae32a3da170bc96600d)
even cleaner
(cherry picked from commit 495b5eb3fdfca996659318ceb9bf8fe0d4371622)
EasyScoreDirectorFactory test
(cherry picked from commit 3d0628fee2c3fb4d854128fa6777d342349371f2)
PR 58 review: refactor createInitializingScoreTrendLevelArray() to InitializingScoreTrend.buildUniformTrend() (can be useful for a user using programmatic solver config too) + remove stating-the-obvious comment "for unit tests only" (it's in src/test/resources) + rename ...TrendLevel trend to ...TrendLevel trendLevel to avoid confusing Trend and TrendLevel
(cherry picked from commit 1db56c09537e55bfd6da81fac61023981e064595)
PR 58 review: refactor EasyScoreDirectorFactoryTest to use mocks + to use the Testdata* classes + to use assertEquals instead of assertTrue + to not assert that easy calc is too weak to fulfill the matchPreference
(cherry picked from commit 5dc1beb2b5e958151657ba34eade5507f2a958f2)
PR 58 review (last one): rename score2 to optimisticBound/pessimisticBound + hardcode level sizes in tests (in tests this is fine and even encouraged to make them more readable) + refactor Score.parseScore to Score.valueOf + use exact primitive types (-7L for longs, -7.0 for doubles instead of just -7) to avoid casting surprises
(cherry picked from commit 9a0e96783d874640ae60f0bbbbff924b7bbb2a8f)